[OE-core] [PATCH] ltp: fix knob to control numa support

Yi Zhao yi.zhao at windriver.com
Sun Feb 11 00:22:13 UTC 2018



在 2018年02月09日 18:26, Burton, Ross 写道:
> There's now two patches which do this on the list, can you decide 
> which one should be merged?

Sorry I missed Anders's patch. His patch is better than mine.

Regards,
Yi

>
> Cheers,
> Ross
>
> On 8 February 2018 at 17:34, Anders Roxell <anders.roxell at linaro.org 
> <mailto:anders.roxell at linaro.org>> wrote:
>
>     Reported-by: Yi Zhao <yi.zhao at windriver.com
>     <mailto:yi.zhao at windriver.com>>
>     Signed-off-by: Anders Roxell <anders.roxell at linaro.org
>     <mailto:anders.roxell at linaro.org>>
>     ---
>      ...ix-default-value-of-without-numa-switch-i.patch | 31
>     ++++++++++++++
>      ...onfigure-add-knob-to-control-numa-support.patch | 50
>     ++++++++++++++++++++++
>      ...-control-whether-numa-support-should-be-c.patch | 45
>     -------------------
>      meta/recipes-extended/ltp/ltp_20180118.bb
>     <http://ltp_20180118.bb>         |  3 +-
>      4 files changed, 83 insertions(+), 46 deletions(-)
>      create mode 100644
>     meta/recipes-extended/ltp/ltp/0001-configure-Fix-default-value-of-without-numa-switch-i.patch
>      create mode 100644
>     meta/recipes-extended/ltp/ltp/0001-configure-add-knob-to-control-numa-support.patch
>      delete mode 100644
>     meta/recipes-extended/ltp/ltp/0002-Add-knob-to-control-whether-numa-support-should-be-c.patch
>
>     diff --git
>     a/meta/recipes-extended/ltp/ltp/0001-configure-Fix-default-value-of-without-numa-switch-i.patch
>     b/meta/recipes-extended/ltp/ltp/0001-configure-Fix-default-value-of-without-numa-switch-i.patch
>     new file mode 100644
>     index 0000000..da62687
>     --- /dev/null
>     +++
>     b/meta/recipes-extended/ltp/ltp/0001-configure-Fix-default-value-of-without-numa-switch-i.patch
>     @@ -0,0 +1,31 @@
>     +From 4ace68a6ba501907a0741dba5f5f155e06a74c35 Mon Sep 17 00:00:00
>     2001
>     +From: Petr Vorel <pvorel at suse.cz <mailto:pvorel at suse.cz>>
>     +Date: Wed, 7 Feb 2018 11:09:49 +0100
>     +Subject: [PATCH] configure: Fix default value of --without-numa
>     switch in help
>     +
>     +The default value is no.
>     +
>     +Fixes: 39a85a1f1 ("configure: add knob to control numa support")
>     +
>     +Signed-off-by: Petr Vorel <pvorel at suse.cz <mailto:pvorel at suse.cz>>
>     +Upstream-Status: Accepted
>     +---
>     + configure.ac <http://configure.ac> | 2 +-
>     + 1 file changed, 1 insertion(+), 1 deletion(-)
>     +
>     +diff --git a/configure.ac <http://configure.ac> b/configure.ac
>     <http://configure.ac>
>     +index b39a31d..d66ea00 100644
>     +--- a/configure.ac <http://configure.ac>
>     ++++ b/configure.ac <http://configure.ac>
>     +@@ -86,7 +86,7 @@ fi
>     + # Numa
>     + AC_ARG_WITH([numa],
>     +   AC_HELP_STRING([--without-numa],
>     +-    [without numa support (default=yes)]),
>     ++    [without numa support (default=no)]),
>     +   [with_numa=no],
>     +   [with_numa=yes]
>     + )
>     +--
>     +2.7.4
>     +
>     diff --git
>     a/meta/recipes-extended/ltp/ltp/0001-configure-add-knob-to-control-numa-support.patch
>     b/meta/recipes-extended/ltp/ltp/0001-configure-add-knob-to-control-numa-support.patch
>     new file mode 100644
>     index 0000000..8aaa287
>     --- /dev/null
>     +++
>     b/meta/recipes-extended/ltp/ltp/0001-configure-add-knob-to-control-numa-support.patch
>     @@ -0,0 +1,50 @@
>     +From 39a85a1f1e6624f554215ba4b6843d3689a8415e Mon Sep 17 00:00:00
>     2001
>     +From: Anders Roxell <anders.roxell at linaro.org
>     <mailto:anders.roxell at linaro.org>>
>     +Date: Mon, 5 Feb 2018 11:56:04 +0100
>     +Subject: [PATCH] configure: add knob to control numa support
>     +
>     +Allow to disable numa from the top level.
>     +
>     +Based on patch:
>     +http://cgit.openembedded.org/openembedded-core/commit/meta/recipes-extended/ltp?id=4c7873552e13dfdba96afca7562c398d2966ca71
>     <http://cgit.openembedded.org/openembedded-core/commit/meta/recipes-extended/ltp?id=4c7873552e13dfdba96afca7562c398d2966ca71>
>     +
>     +Reported-by: Roy Li <rongqing.li at windriver.com
>     <mailto:rongqing.li at windriver.com>>
>     +Signed-off-by: Anders Roxell <anders.roxell at linaro.org
>     <mailto:anders.roxell at linaro.org>>
>     +Signed-off-by: Petr Vorel <pvorel at suse.cz <mailto:pvorel at suse.cz>>
>     +Upstream-Status: Accepted
>     +---
>     + configure.ac <http://configure.ac> | 10 ++++++++++
>     + 1 file changed, 10 insertions(+)
>     +
>     +diff --git a/configure.ac <http://configure.ac> b/configure.ac
>     <http://configure.ac>
>     +index 5c8d4ea..b39a31d 100644
>     +--- a/configure.ac <http://configure.ac>
>     ++++ b/configure.ac <http://configure.ac>
>     +@@ -83,6 +83,14 @@ else
>     +     AC_SUBST([WITH_EXPECT],["no"])
>     + fi
>     +
>     ++# Numa
>     ++AC_ARG_WITH([numa],
>     ++  AC_HELP_STRING([--without-numa],
>     ++    [without numa support (default=yes)]),
>     ++  [with_numa=no],
>     ++  [with_numa=yes]
>     ++)
>     ++
>     + # Perl
>     + AC_ARG_WITH([perl],
>     +   [AC_HELP_STRING([--with-perl],
>     +@@ -157,7 +165,9 @@ LTP_CHECK_SELINUX
>     + LTP_CHECK_SIGNAL
>     + LTP_CHECK_SYSCALL_EVENTFD
>     + LTP_CHECK_SYSCALL_MODIFY_LDT
>     ++if test "x$with_numa" = xyes; then
>     + LTP_CHECK_SYSCALL_NUMA
>     ++fi
>     + LTP_CHECK_SYSCALL_QUOTACTL
>     + LTP_CHECK_SYSCALL_SIGNALFD
>     + LTP_CHECK_SYSCALL_UNSHARE
>     +--
>     +2.7.4
>     +
>     diff --git
>     a/meta/recipes-extended/ltp/ltp/0002-Add-knob-to-control-whether-numa-support-should-be-c.patch
>     b/meta/recipes-extended/ltp/ltp/0002-Add-knob-to-control-whether-numa-support-should-be-c.patch
>     deleted file mode 100644
>     index 31460b9..0000000
>     ---
>     a/meta/recipes-extended/ltp/ltp/0002-Add-knob-to-control-whether-numa-support-should-be-c.patch
>     +++ /dev/null
>     @@ -1,45 +0,0 @@
>     -From 867ad5d5d64b9b27ee32148027532db0a00f6433 Mon Sep 17 00:00:00
>     2001
>     -From: "Roy.Li" <rongqing.li at windriver.com
>     <mailto:rongqing.li at windriver.com>>
>     -Date: Thu, 7 Jan 2016 17:33:26 +0000
>     -Subject: [PATCH 02/32] Add knob to control whether numa support
>     should be
>     - checked
>     -
>     -otherwise undeterministic dependency will be generated
>     -during build depending upong numa being staged or not
>     -
>     -Signed-off-by: Roy.Li <rongqing.li at windriver.com
>     <mailto:rongqing.li at windriver.com>>
>     -Signed-off-by: Khem Raj <raj.khem at gmail.com
>     <mailto:raj.khem at gmail.com>>
>     -Signed-off-by: Anders Roxell <anders.roxell at linaro.org
>     <mailto:anders.roxell at linaro.org>>
>     -Upstream-Status: Pending
>     ----
>     - m4/ltp-numa.m4 | 10 +++++++++-
>     - 1 file changed, 9 insertions(+), 1 deletion(-)
>     -
>     -diff --git a/m4/ltp-numa.m4 b/m4/ltp-numa.m4
>     -index fc3383e..44cb559 100644
>     ---- a/m4/ltp-numa.m4
>     -+++ b/m4/ltp-numa.m4
>     -@@ -24,7 +24,13 @@ dnl LTP_CHECK_SYSCALL_NUMA
>     - dnl ----------------------------
>     - dnl
>     - AC_DEFUN([LTP_CHECK_SYSCALL_NUMA], [
>     --      AC_CHECK_LIB(numa, numa_available, [have_libnuma=yes])
>     -+      AC_MSG_CHECKING([for numa])
>     -+      AC_ARG_WITH(
>     -+              [numa],
>     -+              AC_HELP_STRING([--without-numa],
>     -+              [without numa support]),
>     -+              [],
>     -+      [AC_CHECK_LIB(numa, numa_available, [have_libnuma=yes])
>     -       AC_CHECK_HEADERS([numa.h numaif.h], [],
>     [have_numa_headers=no])
>     -
>     -       if test "x$have_numa_headers" != "xno"; then
>     -@@ -42,3 +48,5 @@ exit(1);
>     -               AC_DEFINE(HAVE_NUMA_V2, 1, [Define to 1 if you
>     have libnuma and it's headers version >= 2 installed.])
>     -       fi
>     - ])
>     -+AC_MSG_RESULT([$with_numa])
>     -+])
>     ---
>     -2.7.0
>     -
>     diff --git a/meta/recipes-extended/ltp/ltp_20180118.bb
>     <http://ltp_20180118.bb>
>     b/meta/recipes-extended/ltp/ltp_20180118.bb <http://ltp_20180118.bb>
>     index 369b98d..a0dd9aa 100644
>     --- a/meta/recipes-extended/ltp/ltp_20180118.bb
>     <http://ltp_20180118.bb>
>     +++ b/meta/recipes-extended/ltp/ltp_20180118.bb
>     <http://ltp_20180118.bb>
>     @@ -24,7 +24,8 @@ CFLAGS_append_mipsarchn64 = "
>     -D__SANE_USERSPACE_TYPES__"
>      SRCREV = "731cd34e682d297b207668be8b1d15320a9ac1b1"
>
>      SRC_URI = "git://github.com/linux-test-project/ltp.git
>     <http://github.com/linux-test-project/ltp.git> \
>     -         
>      file://0002-Add-knob-to-control-whether-numa-support-should-be-c.patch
>     \
>     +         
>      file://0001-configure-add-knob-to-control-numa-support.patch \
>     +         
>      file://0001-configure-Fix-default-value-of-without-numa-switch-i.patch
>     \
>                 file://0003-Add-knob-to-control-tirpc-support.patch \
>                
>     file://0004-build-Add-option-to-select-libc-implementation.patch \
>                
>     file://0005-kernel-controllers-Link-with-libfts-explicitly-on-mu.patch
>     \
>     --
>     2.7.4
>
>     --
>     _______________________________________________
>     Openembedded-core mailing list
>     Openembedded-core at lists.openembedded.org
>     <mailto:Openembedded-core at lists.openembedded.org>
>     http://lists.openembedded.org/mailman/listinfo/openembedded-core
>     <http://lists.openembedded.org/mailman/listinfo/openembedded-core>
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20180211/066b243a/attachment-0002.html>


More information about the Openembedded-core mailing list