package/ltp-testsuite: add upstream patches
PATCH 1/1 package/ltp-testsuite: add upstream patches
Add patches from the upstream ltp-testsuite github repository to compile correctly.
--- package/ltp-testsuite/0002-execve05.patch | 44 +++++++++++++++++++++++ package/ltp-testsuite/0003-statx.patch | 32 +++++++++++++++++ package/ltp-testsuite/0004-statx.patch | 26 ++++++++++++++ 3 files changed, 102 insertions(+) create mode 100644 package/ltp-testsuite/0002-execve05.patch create mode 100644 package/ltp-testsuite/0003-statx.patch create mode 100644 package/ltp-testsuite/0004-statx.patch
package/ltp-testsuite/0002-execve05.patch new file +44 -0
@@ -0,0 +1,44 @@ +From 6f61cad559df8791add50aa8d053ca77cd97c594 Mon Sep 17 00:00:00 2001 +From: "Allen, John" <[email protected]> +Date: Wed, 26 Sep 2018 14:31:37 +0000 +Subject: [PATCH] runtest/{ltplite, stress.part3}: Update execve05 calling + convention + +The calling convention for execve05 was recently changed and was updated +in runtest/syscalls when the change was introduced, but not for other +runtest files where execve05 is called. Update all calls to execve05 to +use the new calling convention. + +Signed-off-by: John Allen <[email protected]> +Acked-by: Cyril Hrubis <[email protected]> +--- + runtest/ltplite | 2 +- + runtest/stress.part3 | 2 +- + 2 files changed, 2 insertions(+), 2 deletions(-) + +diff --git a/runtest/ltplite b/runtest/ltplite +index 61ab275d8e..f0102e7e63 100644 +--- a/runtest/ltplite ++++ b/runtest/ltplite +@@ -174,7 +174,7 @@ execve01 execve01 + execve02 execve02 + execve03 execve03 + execve04 execve04 +-execve05 execve05 20 $LTPROOT/testcases/bin/execve05 $LTPROOT/testcases/bin/execve05 4 ++execve05 execve05 -i 5 -n 32 + execvp01 execvp01 + + exit01 exit01 +diff --git a/runtest/stress.part3 b/runtest/stress.part3 +index 332daa9ece..513d2ea39d 100644 +--- a/runtest/stress.part3 ++++ b/runtest/stress.part3 +@@ -115,7 +115,7 @@ execve01 execve01 + execve02 execve02 + execve03 execve03 + execve04 execve04 +-execve05 execve05 20 $LTPROOT/testcases/bin/execve05 $LTPROOT/testcases/bin/execve05 4 ++execve05 execve05 -i 5 -n 32 + execvp01 execvp01 + + exit01 exit01
package/ltp-testsuite/0003-statx.patch new file +32 -0
@@ -0,0 +1,32 @@ +From a945304b6a2286d0665c70250cc5475eb07fd21e Mon Sep 17 00:00:00 2001 +From: Thadeu Lima de Souza Cascardo <[email protected]> +Date: Wed, 26 Sep 2018 11:15:45 -0300 +Subject: [PATCH] statx: fix compile errors + +struct statx is only defined in sys/stat.h when _GNU_SOURCE is defined. +The testcases do define _GNU_SOURCE, but not the m4 struct check. + +Signed-off-by: Thadeu Lima de Souza Cascardo <[email protected]> +Acked-by: Cyril Hrubis <[email protected]> +--- + m4/ltp-statx.m4 | 9 +++++++-- + 1 file changed, 7 insertions(+), 2 deletions(-) + +diff --git a/m4/ltp-statx.m4 b/m4/ltp-statx.m4 +index 44303d30c2..5f79a94cdf 100644 +--- a/m4/ltp-statx.m4 ++++ b/m4/ltp-statx.m4 +@@ -23,6 +23,11 @@ dnl + AC_DEFUN([LTP_CHECK_STATX],[ + AC_CHECK_FUNCS(statx,,,[[#include <sys/stat.h>]]) + AC_CHECK_HEADER(linux/fs.h,,,) +-AC_CHECK_TYPES([struct statx],,,[[#include <sys/stat.h>]]) +-AC_CHECK_TYPES([struct statx_timestamp],,,[[#include <sys/stat.h>]]) ++AC_CHECK_TYPES([struct statx],,,[[ ++ #define _GNU_SOURCE ++ #include <sys/stat.h> ++]]) ++AC_CHECK_TYPES([struct statx_timestamp],,,[[ ++ #define _GNU_SOURCE ++ #include <sys/stat.h>]]) + ])
package/ltp-testsuite/0004-statx.patch new file +26 -0
@@ -0,0 +1,26 @@ +This is based on 0003-statx.patch so that autoconf does not have to run again. + +--- ltp-testsuite-20180926/configure ++++ ltp-testsuite-20180926/configure +@@ -6502,7 +6502,9 @@ + fi + + +-ac_fn_c_check_type "$LINENO" "struct statx" "ac_cv_type_struct_statx" "#include <sys/stat.h> ++ac_fn_c_check_type "$LINENO" "struct statx" "ac_cv_type_struct_statx" " ++#define _GNU_SOURCE ++#include <sys/stat.h> + " + if test "x$ac_cv_type_struct_statx" = xyes; then : + +@@ -6513,7 +6515,9 @@ + + fi + +-ac_fn_c_check_type "$LINENO" "struct statx_timestamp" "ac_cv_type_struct_statx_timestamp" "#include <sys/stat.h> ++ac_fn_c_check_type "$LINENO" "struct statx_timestamp" "ac_cv_type_struct_statx_timestamp" " ++#define _GNU_SOURCE ++#include <sys/stat.h> + " + if test "x$ac_cv_type_struct_statx_timestamp" = xyes; then : +
Signature · 9 lines
2.17.1 Gesch?ftsf?hrung: Susanne Kunschert, Thomas Pilz Pilz GmbH & Co. KG, Sitz: Ostfildern, HRA 210 893, Amtsgericht Stuttgart Kompl. Ges. Peter Pilz GmbH, Sitz: Ostfildern, HRB 210 612, Amtsgericht Stuttgart Umsatzsteuer: ID-Nr. DE 145 355 773, WEEE-Reg.-Nr. DE 71636849 This email is intended solely for the use of the named address(es). Any unauthorised disclosure, copying or distribution of these confidential information contained therein, or the taking of any action based on it, is prohibited. The sender disclaims any liability for the integrity of this email. Legally binding declarations must be in written form. Umweltschutz liegt uns am Herzen! - Bitte denken Sie an unsere Umwelt, bevor Sie diese E-Mail drucken. We do care about the environment! - Please consider the environment before printing this e-mail.
PATCH 1/1 package/ltp-testsuite: add upstream patches
Hi Florian,
Quoted 4 lines
Add patches from the upstream ltp-testsuite github repository to compile correctly.
Signed-off-by Florian La Roche <[email protected]>
Quoted 4 lines
--- package/ltp-testsuite/0002-execve05.patch | 44 +++++++++++++++++++++++ package/ltp-testsuite/0003-statx.patch | 32 +++++++++++++++++ package/ltp-testsuite/0004-statx.patch | 26 ++++++++++++++
This is a duplicate patch of patch sent in
http://lists.busybox.net/pipermail/buildroot/2018-October/232173.html
https://patchwork.ozlabs.org/patch/979320/
Kind regards,
Petr
PATCH 1/1 package/ltp-testsuite: add upstream patches
In reply to Florian La RocheHello Florian,
Thanks for working on ltp!
Quoted 2 lines · Florian La Roche
Add patches from the upstream ltp-testsuite github repository to compile correctly.
"compile correctly" is very vague. Which issues is this fixing ?
It seems like the statx patch is fixing a build issue with glibc 2.28. If that is the case, it should be explained in the commit log, and a reference to the autobuilder failure should be added (see
http://autobuild.buildroot.net/?reason=ltp% for all ltp-testsuite build failures).
The commit title should then be fixed to be:
package/ltp-testsuite: fix build with glibc 2.28
Quoted 1 line
package/ltp-testsuite/0002-execve05.patch | 44 +++++++++++++++++++++++
This is a runtime fix, it should be added a separate patch, explaining why it should be added.
Also, please use "git format-patch" to generate the patches, and keep the file name generated by git format-patch for the patches.
Quoted 9 lines
package/ltp-testsuite/0002-execve05.patch new file +3
-0@@ -0,0 +1,44 @@ +From 6f61cad559df8791add50aa8d053ca77cd97c594 Mon Sep 17 00:00:00 2001 +From: "Allen, John" <[email protected]> +Date: Wed, 26 Sep 2018 14:31:37 +0000
package/ltp-testsuite/0002-execve05.patch line 13
Quoted 10 lines
package/ltp-testsuite/0002-execve05.patch new file +10
-0+Subject: [PATCH] runtest/{ltplite, stress.part3}: Update execve05 calling + convention + +The calling convention for execve05 was recently changed and was updated +in runtest/syscalls when the change was introduced, but not for other +runtest files where execve05 is called. Update all calls to execve05 to +use the new calling convention. + +Signed-off-by: John Allen <[email protected]> +Acked-by: Cyril Hrubis <[email protected]>
Please add your Signed-off-by to patches added in Buildroot.
package/ltp-testsuite/0004-statx.patch line 2
Quoted 8 lines
package/ltp-testsuite/0004-statx.patch new file +2
-0@@ -0,0 +1,26 @@ +This is based on 0003-statx.patch so that autoconf does not have to run again. +
As Petr suggested, we prefer to add AUTORECONF = YES in ltp-testsuite.mk.
Just add something like this:
# 0004-blablalba.patchLTP_TESTSUITE_AUTORECONF = YES
Could you fix those minor details, and send an updated version of your patch ?
Thanks,
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com