diff --git a/pkgs/development/libraries/liburing/default.nix b/pkgs/development/libraries/liburing/default.nix index ff88bdcaafd..d7b997437d5 100644 --- a/pkgs/development/libraries/liburing/default.nix +++ b/pkgs/development/libraries/liburing/default.nix @@ -1,16 +1,54 @@ { stdenv, fetchgit +, fetchpatch }: stdenv.mkDerivation rec { name = "liburing-${version}"; - version = "1.0.0pre156_${builtins.substring 0 7 src.rev}"; + version = "0.1"; src = fetchgit { url = "http://git.kernel.dk/liburing"; - rev = "c31c7ec4bcd7bb0d7b28897d730431c02b9d4ea1"; - sha256 = "17d6s03fyajcrijca9d2d6llbf8cl8dyalpxnl39az3452p04s11"; + rev = "refs/tags/liburing-${version}"; + sha256 = "038iqsbm9bdmlwvmb899bc6g1rw5dalr990azynbvgn8qs5adysh"; }; + patches = [ + + # This patch re-introduces support for aarch64-linux, by adding the + # necessary memory barrier primitives for it to work. + # + # Already upstream: remove when moving to the next version + (fetchpatch { + url = "http://git.kernel.dk/cgit/liburing/patch/?id=0520db454c29f1d96cda6cf6cedeb93df65301e8"; + sha256 = "1i8133sb1imzxpplmhlhnaxkffgplhj40vanivc6clbibvhgwpq6"; + }) + + # This patch shuffles the name of the io_uring memory barrier primitives. + # They were using extremely common names by accident, which caused + # namespace conflicts with many other projects using the same names. Note: + # this does not change the user-visible API of liburing (liburing is + # designed exactly to hide the necessary memory barriers when using the + # io_uring syscall directly). It only changes the names of some internals. + # The only reason this caused problems at all is because memory barrier + # primitives are written as preprocessor defines, in a common header file, + # which get included unilaterally. + # + # Already upstream: remove when moving to the next version + (fetchpatch { + url = "http://git.kernel.dk/cgit/liburing/patch/?id=552c6a08d04c74d20eeaa86f535bfd553b352370"; + sha256 = "123d6jdqfy7b8aq9f6ax767n48hhbx6pln3nlrp623595i8zz3wf"; + }) + + # Finally, this patch fixes the aarch64-linux support introduced by the + # first patch, but which was _broken_ by the second patch, in a horrid + # twist of fate: it neglected to change the names of the aarch64 barriers + # appropriately. (I assume the author did not attempt to compile for + # aarch64, hence this regression) + # + # Not yet upstream: TBD. + ./fix-aarch64-barrier-names.patch + ]; + separateDebugInfo = true; enableParallelBuilding = true; diff --git a/pkgs/development/libraries/liburing/fix-aarch64-barrier-names.patch b/pkgs/development/libraries/liburing/fix-aarch64-barrier-names.patch new file mode 100644 index 00000000000..6fcad5a4fad --- /dev/null +++ b/pkgs/development/libraries/liburing/fix-aarch64-barrier-names.patch @@ -0,0 +1,29 @@ +diff --git a/src/include/liburing/barrier.h b/src/include/liburing/barrier.h +index 8efa6dd..051b20f 100644 +--- a/src/include/liburing/barrier.h ++++ b/src/include/liburing/barrier.h +@@ -61,15 +61,15 @@ do { \ + + #elif defined(__aarch64__) + /* Adapted from arch/arm64/include/asm/barrier.h */ +-#define dmb(opt) asm volatile("dmb " #opt : : : "memory") +-#define dsb(opt) asm volatile("dsb " #opt : : : "memory") +- +-#define mb() dsb(sy) +-#define rmb() dsb(ld) +-#define wmb() dsb(st) +-#define smp_mb() dmb(ish) +-#define smp_rmb() dmb(ishld) +-#define smp_wmb() dmb(ishst) ++#define io_uring_dmb(opt) asm volatile("dmb " #opt : : : "memory") ++#define io_uring_dsb(opt) asm volatile("dsb " #opt : : : "memory") ++ ++#define io_uring_mb() io_uring_dsb(sy) ++#define io_uring_rmb() io_uring_dsb(ld) ++#define io_uring_wmb() io_uring_dsb(st) ++#define io_uring_smp_mb() io_uring_dmb(ish) ++#define io_uring_smp_rmb() io_uring_dmb(ishld) ++#define io_uring_smp_wmb() io_uring_dmb(ishst) + + #else /* defined(__x86_64__) || defined(__i386__) || defined(__aarch64__) */ + /*