[Buildroot] [PATCH 1/1] package/liburing: needs MMU

[Buildroot] [PATCH 1/1] package/liburing: needs MMU

[Fabrice Fontaine]

liburing needs MMU since its addition in commit
03ca6f4e39874583060317e7e15e9e360220877e:

setup.c: In function 'io_uring_ring_dontfork':
setup.c:119:8: warning: implicit declaration of function 'madvise'; did you mean 'raise'? [-Wimplicit-function-declaration]
  119 |  ret = madvise(ring->sq.sqes, len, MADV_DONTFORK);
      |        ^~~~~~~
      |        raise

Fixes:
 - http://autobuild.buildroot.org/results/33f3c58e98daab07139b4f400b85f87c0e314240

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/liburing/Config.in | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/package/liburing/Config.in b/package/liburing/Config.in
index d65a3f1322..fc13f1c17c 100644
--- a/package/liburing/Config.in
+++ b/package/liburing/Config.in
@@ -1,5 +1,6 @@
 config BR2_PACKAGE_LIBURING
 	bool "liburing"
+	depends on BR2_USE_MMU # madvise()
 	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 # C11/stdatomic.h
 	depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_5_1
 	help
@@ -11,5 +12,6 @@ config BR2_PACKAGE_LIBURING
 	  https://git.kernel.dk/cgit/liburing
 
 comment "liburing needs a toolchain w/ gcc >= 4.9, headers >= 5.1"
+	depends on BR2_USE_MMU
 	depends on !BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 || \
 		!BR2_TOOLCHAIN_HEADERS_AT_LEAST_5_1
-- 
2.35.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 1/1] package/liburing: needs MMU

[Peter Korsgaard]

>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > liburing needs MMU since its addition in commit
 > 03ca6f4e39874583060317e7e15e9e360220877e:

 > setup.c: In function 'io_uring_ring_dontfork':
 > setup.c:119:8: warning: implicit declaration of function 'madvise'; did you mean 'raise'? [-Wimplicit-function-declaration]
 >   119 |  ret = madvise(ring->sq.sqes, len, MADV_DONTFORK);
 >       |        ^~~~~~~
 >       |        raise

 > Fixes:
 >  - http://autobuild.buildroot.org/results/33f3c58e98daab07139b4f400b85f87c0e314240

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Committed, thanks.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 1/1] package/liburing: needs MMU

[Peter Korsgaard]

>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > liburing needs MMU since its addition in commit
 > 03ca6f4e39874583060317e7e15e9e360220877e:

 > setup.c: In function 'io_uring_ring_dontfork':
 > setup.c:119:8: warning: implicit declaration of function 'madvise';
 > did you mean 'raise'? [-Wimplicit-function-declaration]
 >   119 |  ret = madvise(ring->sq.sqes, len, MADV_DONTFORK);
 >       |        ^~~~~~~
 >       |        raise

 > Fixes:
 >  - http://autobuild.buildroot.org/results/33f3c58e98daab07139b4f400b85f87c0e314240

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Committed to 2022.02.x, thanks.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot