Message ID | 20171005174751.30098-1-fathi.boudra@linaro.org |
---|---|
State | New |
Headers | show |
Series | ltp: update to new upstream release 20170929 | expand |
On Thu, Oct 5, 2017 at 10:47 AM, Fathi Boudra <fathi.boudra@linaro.org> wrote: > * Update SRCREV to match 20170929 release > * Drop patches available in 20170929 release: > - 0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch > - 0038-commands-gdb01-replace-stdin-with-dev-null.patch > A build time test for musl target would be appreciated. > Signed-off-by: Fathi Boudra <fathi.boudra@linaro.org> > --- > ...E_SIZE-redefinition-and-O_CREAT-undeclear.patch | 113 --------------------- > ...ommands-gdb01-replace-stdin-with-dev-null.patch | 34 ------- > .../ltp/{ltp_20170516.bb => ltp_20170929.bb} | 4 +- > 3 files changed, 1 insertion(+), 150 deletions(-) > delete mode 100644 meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch > delete mode 100644 meta/recipes-extended/ltp/ltp/0038-commands-gdb01-replace-stdin-with-dev-null.patch > rename meta/recipes-extended/ltp/{ltp_20170516.bb => ltp_20170929.bb} (96%) > > diff --git a/meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch b/meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch > deleted file mode 100644 > index c8738ae0d4..0000000000 > --- a/meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch > +++ /dev/null > @@ -1,113 +0,0 @@ > -From a9d5595d2fa2ab252f1cabf63f4b65c3efbafeb9 Mon Sep 17 00:00:00 2001 > -From: Dengke Du <dengke.du@windriver.com> > -Date: Thu, 10 Aug 2017 15:27:03 +0800 > -Subject: [PATCH] ltp: fix PAGE_SIZE redefinition and O_CREAT undeclear when > - build with musl > - > -error 1: > - > -|stack_clash.c:50:22: error: expected identifier or '(' before numeric constant > -| static unsigned long PAGE_SIZE; > - > -This is because the musl libc already contain PAGE_SIZE definition in limits.c, > -we can check it here: > - > - https://git.musl-libc.org/cgit/musl/tree/include/limits.h#n43 > - > -error 2: > - > -|ck01.c:157:22: error: 'O_CREAT' undeclared (first use in this function); did you mean 'S_IREAD'? > -| fd = open(filename, O_CREAT | O_TRUNC | O_RDWR, 0644); > -| ^~~~~~~ > -| S_IREAD > - > -This is because the musl libc put those in fcntl.h, so we should include that > -file. > - > -Upstream-Status: Submitted [ https://github.com/linux-test-project/ltp/pull/194 ] > - > -Signed-off-by: Dengke Du <dengke.du@windriver.com> > ---- > - testcases/cve/stack_clash.c | 12 ++++++------ > - testcases/kernel/syscalls/flock/flock01.c | 1 + > - testcases/kernel/syscalls/flock/flock02.c | 1 + > - 3 files changed, 8 insertions(+), 6 deletions(-) > - > -diff --git a/testcases/cve/stack_clash.c b/testcases/cve/stack_clash.c > -index 2ef1a82..7c45991 100644 > ---- a/testcases/cve/stack_clash.c > -+++ b/testcases/cve/stack_clash.c > -@@ -47,7 +47,7 @@ > - #include "tst_test.h" > - #include "tst_safe_stdio.h" > - > --static unsigned long PAGE_SIZE; > -+static unsigned long PAGE_SIZE_tst; > - static unsigned long PAGE_MASK; > - static unsigned long GAP_PAGES = 256; > - static unsigned long THRESHOLD; > -@@ -66,7 +66,7 @@ void exhaust_stack_into_sigsegv(void) > - exhaust_stack_into_sigsegv(); > - } > - > --#define MAPPED_LEN PAGE_SIZE > -+#define MAPPED_LEN PAGE_SIZE_tst > - static unsigned long mapped_addr; > - > - void segv_handler(int sig, siginfo_t *info, void *data LTP_ATTRIBUTE_UNUSED) > -@@ -150,7 +150,7 @@ void do_child(void) > - stack_t signal_stack; > - struct sigaction segv_sig = {.sa_sigaction = segv_handler, .sa_flags = SA_ONSTACK|SA_SIGINFO}; > - void *map; > -- unsigned long gap = GAP_PAGES * PAGE_SIZE; > -+ unsigned long gap = GAP_PAGES * PAGE_SIZE_tst; > - struct rlimit rlimit; > - > - rlimit.rlim_cur = rlimit.rlim_max = RLIM_INFINITY; > -@@ -200,8 +200,8 @@ void setup(void) > - { > - char buf[4096], *p; > - > -- PAGE_SIZE = sysconf(_SC_PAGESIZE); > -- PAGE_MASK = ~(PAGE_SIZE - 1); > -+ PAGE_SIZE_tst = sysconf(_SC_PAGESIZE); > -+ PAGE_MASK = ~(PAGE_SIZE_tst - 1); > - > - buf[4095] = '\0'; > - SAFE_FILE_SCANF("/proc/cmdline", "%4095[^\n]", buf); > -@@ -214,7 +214,7 @@ void setup(void) > - tst_res(TINFO, "stack_guard_gap = %ld", GAP_PAGES); > - } > - > -- THRESHOLD = (GAP_PAGES - 1) * PAGE_SIZE; > -+ THRESHOLD = (GAP_PAGES - 1) * PAGE_SIZE_tst; > - > - { > - volatile int *a = alloca(128); > -diff --git a/testcases/kernel/syscalls/flock/flock01.c b/testcases/kernel/syscalls/flock/flock01.c > -index 3e17be4..06d89e3 100644 > ---- a/testcases/kernel/syscalls/flock/flock01.c > -+++ b/testcases/kernel/syscalls/flock/flock01.c > -@@ -69,6 +69,7 @@ > - #include <stdio.h> > - #include <sys/wait.h> > - #include <sys/file.h> > -+#include <fcntl.h> > - #include "test.h" > - > - void setup(void); > -diff --git a/testcases/kernel/syscalls/flock/flock02.c b/testcases/kernel/syscalls/flock/flock02.c > -index 414df68..9ddf729 100644 > ---- a/testcases/kernel/syscalls/flock/flock02.c > -+++ b/testcases/kernel/syscalls/flock/flock02.c > -@@ -75,6 +75,7 @@ > - #include <sys/types.h> > - #include <sys/file.h> > - #include <sys/wait.h> > -+#include <fcntl.h> > - #include <errno.h> > - #include <stdio.h> > - #include "test.h" > --- > -2.7.4 > - > diff --git a/meta/recipes-extended/ltp/ltp/0038-commands-gdb01-replace-stdin-with-dev-null.patch b/meta/recipes-extended/ltp/ltp/0038-commands-gdb01-replace-stdin-with-dev-null.patch > deleted file mode 100644 > index f7c0a4bf17..0000000000 > --- a/meta/recipes-extended/ltp/ltp/0038-commands-gdb01-replace-stdin-with-dev-null.patch > +++ /dev/null > @@ -1,34 +0,0 @@ > -From 2f6ab8f694b26b7f2566624f6d1f23788d6ab8a0 Mon Sep 17 00:00:00 2001 > -From: Jan Stancek <jstancek@redhat.com> > -Date: Mon, 11 Sep 2017 12:57:58 +0200 > -Subject: [PATCH] commands/gdb01: replace stdin with /dev/null > - > -If this testcase runs as background process, gdb can receive > -SIGTTOU and then testcase gets stuck. > - > -Signed-off-by: Jan Stancek <jstancek@redhat.com> > - > -Upstream-Status: Backport > -[https://github.com/linux-test-project/ltp/commit/2f6ab8f694b26b7f2566624f6d1f23788d6ab8a0] > - > -Signed-off-by: Yi Zhao <yi.zhao@windriver.com> > ---- > - testcases/commands/gdb/gdb01.sh | 2 +- > - 1 file changed, 1 insertion(+), 1 deletion(-) > - > -diff --git a/testcases/commands/gdb/gdb01.sh b/testcases/commands/gdb/gdb01.sh > -index 07ae36f..e3a5b51 100755 > ---- a/testcases/commands/gdb/gdb01.sh > -+++ b/testcases/commands/gdb/gdb01.sh > -@@ -29,7 +29,7 @@ TST_NEEDS_CMDS="gdb /bin/cat" > - > - simple_test() > - { > -- gdb /bin/cat -ex "run /etc/passwd" -ex quit > -+ gdb /bin/cat -ex "run /etc/passwd" -ex quit < /dev/null > - RC=$? > - if [ $RC -eq 0 ] ; then > - tst_res TPASS "gdb attached to process and completed run" > --- > -2.7.4 > - > diff --git a/meta/recipes-extended/ltp/ltp_20170516.bb b/meta/recipes-extended/ltp/ltp_20170929.bb > similarity index 96% > rename from meta/recipes-extended/ltp/ltp_20170516.bb > rename to meta/recipes-extended/ltp/ltp_20170929.bb > index 653cbfd4ed..925dd91256 100644 > --- a/meta/recipes-extended/ltp/ltp_20170516.bb > +++ b/meta/recipes-extended/ltp/ltp_20170929.bb > @@ -21,7 +21,7 @@ DEPENDS_append_libc-musl = " fts " > EXTRA_OEMAKE_append_libc-musl = " LIBC=musl " > CFLAGS_append_powerpc64 = " -D__SANE_USERSPACE_TYPES__" > CFLAGS_append_mipsarchn64 = " -D__SANE_USERSPACE_TYPES__" > -SRCREV = "18916a2e6d8c997b7b29dcfa9550d5a15b22ed22" > +SRCREV = "e671f2a13c695bbd87f7dfec2954ca7e3c43f377" > > SRC_URI = "git://github.com/linux-test-project/ltp.git \ > file://0001-add-_GNU_SOURCE-to-pec_listener.c.patch \ > @@ -48,8 +48,6 @@ SRC_URI = "git://github.com/linux-test-project/ltp.git \ > file://0034-periodic_output.patch \ > file://0035-fix-test_proc_kill-hang.patch \ > file://0036-testcases-network-nfsv4-acl-acl1.c-Security-fix-on-s.patch \ > - file://0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch \ > - file://0038-commands-gdb01-replace-stdin-with-dev-null.patch \ > " > > S = "${WORKDIR}/git" > -- > 2.14.2 > > -- > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Sorry: | fanotify08.c:56:3: error: format not a string literal and no format arguments [-Werror=format-security] | tst_res(TPASS, msg); | ^~~~~~~ | fanotify08.c:58:3: error: format not a string literal and no format arguments [-Werror=format-security] | tst_res(TFAIL, msg); | ^~~~~~~ Ross On 5 October 2017 at 18:47, Fathi Boudra <fathi.boudra@linaro.org> wrote: > * Update SRCREV to match 20170929 release > * Drop patches available in 20170929 release: > - 0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch > - 0038-commands-gdb01-replace-stdin-with-dev-null.patch > > Signed-off-by: Fathi Boudra <fathi.boudra@linaro.org> > --- > ...E_SIZE-redefinition-and-O_CREAT-undeclear.patch | 113 > --------------------- > ...ommands-gdb01-replace-stdin-with-dev-null.patch | 34 ------- > .../ltp/{ltp_20170516.bb => ltp_20170929.bb} | 4 +- > 3 files changed, 1 insertion(+), 150 deletions(-) > delete mode 100644 meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE- > redefinition-and-O_CREAT-undeclear.patch > delete mode 100644 meta/recipes-extended/ltp/ltp/ > 0038-commands-gdb01-replace-stdin-with-dev-null.patch > rename meta/recipes-extended/ltp/{ltp_20170516.bb => ltp_20170929.bb} > (96%) > > diff --git a/meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE- > redefinition-and-O_CREAT-undeclear.patch b/meta/recipes-extended/ltp/ > ltp/0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch > deleted file mode 100644 > index c8738ae0d4..0000000000 > --- a/meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE- > redefinition-and-O_CREAT-undeclear.patch > +++ /dev/null > @@ -1,113 +0,0 @@ > -From a9d5595d2fa2ab252f1cabf63f4b65c3efbafeb9 Mon Sep 17 00:00:00 2001 > -From: Dengke Du <dengke.du@windriver.com> > -Date: Thu, 10 Aug 2017 15:27:03 +0800 > -Subject: [PATCH] ltp: fix PAGE_SIZE redefinition and O_CREAT undeclear > when > - build with musl > - > -error 1: > - > -|stack_clash.c:50:22: error: expected identifier or '(' before numeric > constant > -| static unsigned long PAGE_SIZE; > - > -This is because the musl libc already contain PAGE_SIZE definition in > limits.c, > -we can check it here: > - > - https://git.musl-libc.org/cgit/musl/tree/include/limits.h#n43 > - > -error 2: > - > -|ck01.c:157:22: error: 'O_CREAT' undeclared (first use in this function); > did you mean 'S_IREAD'? > -| fd = open(filename, O_CREAT | O_TRUNC | O_RDWR, 0644); > -| ^~~~~~~ > -| S_IREAD > - > -This is because the musl libc put those in fcntl.h, so we should include > that > -file. > - > -Upstream-Status: Submitted [ https://github.com/linux-test- > project/ltp/pull/194 ] > - > -Signed-off-by: Dengke Du <dengke.du@windriver.com> > ---- > - testcases/cve/stack_clash.c | 12 ++++++------ > - testcases/kernel/syscalls/flock/flock01.c | 1 + > - testcases/kernel/syscalls/flock/flock02.c | 1 + > - 3 files changed, 8 insertions(+), 6 deletions(-) > - > -diff --git a/testcases/cve/stack_clash.c b/testcases/cve/stack_clash.c > -index 2ef1a82..7c45991 100644 > ---- a/testcases/cve/stack_clash.c > -+++ b/testcases/cve/stack_clash.c > -@@ -47,7 +47,7 @@ > - #include "tst_test.h" > - #include "tst_safe_stdio.h" > - > --static unsigned long PAGE_SIZE; > -+static unsigned long PAGE_SIZE_tst; > - static unsigned long PAGE_MASK; > - static unsigned long GAP_PAGES = 256; > - static unsigned long THRESHOLD; > -@@ -66,7 +66,7 @@ void exhaust_stack_into_sigsegv(void) > - exhaust_stack_into_sigsegv(); > - } > - > --#define MAPPED_LEN PAGE_SIZE > -+#define MAPPED_LEN PAGE_SIZE_tst > - static unsigned long mapped_addr; > - > - void segv_handler(int sig, siginfo_t *info, void *data > LTP_ATTRIBUTE_UNUSED) > -@@ -150,7 +150,7 @@ void do_child(void) > - stack_t signal_stack; > - struct sigaction segv_sig = {.sa_sigaction = segv_handler, > .sa_flags = SA_ONSTACK|SA_SIGINFO}; > - void *map; > -- unsigned long gap = GAP_PAGES * PAGE_SIZE; > -+ unsigned long gap = GAP_PAGES * PAGE_SIZE_tst; > - struct rlimit rlimit; > - > - rlimit.rlim_cur = rlimit.rlim_max = RLIM_INFINITY; > -@@ -200,8 +200,8 @@ void setup(void) > - { > - char buf[4096], *p; > - > -- PAGE_SIZE = sysconf(_SC_PAGESIZE); > -- PAGE_MASK = ~(PAGE_SIZE - 1); > -+ PAGE_SIZE_tst = sysconf(_SC_PAGESIZE); > -+ PAGE_MASK = ~(PAGE_SIZE_tst - 1); > - > - buf[4095] = '\0'; > - SAFE_FILE_SCANF("/proc/cmdline", "%4095[^\n]", buf); > -@@ -214,7 +214,7 @@ void setup(void) > - tst_res(TINFO, "stack_guard_gap = %ld", GAP_PAGES); > - } > - > -- THRESHOLD = (GAP_PAGES - 1) * PAGE_SIZE; > -+ THRESHOLD = (GAP_PAGES - 1) * PAGE_SIZE_tst; > - > - { > - volatile int *a = alloca(128); > -diff --git a/testcases/kernel/syscalls/flock/flock01.c > b/testcases/kernel/syscalls/flock/flock01.c > -index 3e17be4..06d89e3 100644 > ---- a/testcases/kernel/syscalls/flock/flock01.c > -+++ b/testcases/kernel/syscalls/flock/flock01.c > -@@ -69,6 +69,7 @@ > - #include <stdio.h> > - #include <sys/wait.h> > - #include <sys/file.h> > -+#include <fcntl.h> > - #include "test.h" > - > - void setup(void); > -diff --git a/testcases/kernel/syscalls/flock/flock02.c > b/testcases/kernel/syscalls/flock/flock02.c > -index 414df68..9ddf729 100644 > ---- a/testcases/kernel/syscalls/flock/flock02.c > -+++ b/testcases/kernel/syscalls/flock/flock02.c > -@@ -75,6 +75,7 @@ > - #include <sys/types.h> > - #include <sys/file.h> > - #include <sys/wait.h> > -+#include <fcntl.h> > - #include <errno.h> > - #include <stdio.h> > - #include "test.h" > --- > -2.7.4 > - > diff --git a/meta/recipes-extended/ltp/ltp/0038-commands-gdb01- > replace-stdin-with-dev-null.patch b/meta/recipes-extended/ltp/ > ltp/0038-commands-gdb01-replace-stdin-with-dev-null.patch > deleted file mode 100644 > index f7c0a4bf17..0000000000 > --- a/meta/recipes-extended/ltp/ltp/0038-commands-gdb01- > replace-stdin-with-dev-null.patch > +++ /dev/null > @@ -1,34 +0,0 @@ > -From 2f6ab8f694b26b7f2566624f6d1f23788d6ab8a0 Mon Sep 17 00:00:00 2001 > -From: Jan Stancek <jstancek@redhat.com> > -Date: Mon, 11 Sep 2017 12:57:58 +0200 > -Subject: [PATCH] commands/gdb01: replace stdin with /dev/null > - > -If this testcase runs as background process, gdb can receive > -SIGTTOU and then testcase gets stuck. > - > -Signed-off-by: Jan Stancek <jstancek@redhat.com> > - > -Upstream-Status: Backport > -[https://github.com/linux-test-project/ltp/commit/ > 2f6ab8f694b26b7f2566624f6d1f23788d6ab8a0] > - > -Signed-off-by: Yi Zhao <yi.zhao@windriver.com> > ---- > - testcases/commands/gdb/gdb01.sh | 2 +- > - 1 file changed, 1 insertion(+), 1 deletion(-) > - > -diff --git a/testcases/commands/gdb/gdb01.sh b/testcases/commands/gdb/ > gdb01.sh > -index 07ae36f..e3a5b51 100755 > ---- a/testcases/commands/gdb/gdb01.sh > -+++ b/testcases/commands/gdb/gdb01.sh > -@@ -29,7 +29,7 @@ TST_NEEDS_CMDS="gdb /bin/cat" > - > - simple_test() > - { > -- gdb /bin/cat -ex "run /etc/passwd" -ex quit > -+ gdb /bin/cat -ex "run /etc/passwd" -ex quit < /dev/null > - RC=$? > - if [ $RC -eq 0 ] ; then > - tst_res TPASS "gdb attached to process and completed run" > --- > -2.7.4 > - > diff --git a/meta/recipes-extended/ltp/ltp_20170516.bb > b/meta/recipes-extended/ltp/ltp_20170929.bb > similarity index 96% > rename from meta/recipes-extended/ltp/ltp_20170516.bb > rename to meta/recipes-extended/ltp/ltp_20170929.bb > index 653cbfd4ed..925dd91256 100644 > --- a/meta/recipes-extended/ltp/ltp_20170516.bb > +++ b/meta/recipes-extended/ltp/ltp_20170929.bb > @@ -21,7 +21,7 @@ DEPENDS_append_libc-musl = " fts " > EXTRA_OEMAKE_append_libc-musl = " LIBC=musl " > CFLAGS_append_powerpc64 = " -D__SANE_USERSPACE_TYPES__" > CFLAGS_append_mipsarchn64 = " -D__SANE_USERSPACE_TYPES__" > -SRCREV = "18916a2e6d8c997b7b29dcfa9550d5a15b22ed22" > +SRCREV = "e671f2a13c695bbd87f7dfec2954ca7e3c43f377" > > SRC_URI = "git://github.com/linux-test-project/ltp.git \ > file://0001-add-_GNU_SOURCE-to-pec_listener.c.patch \ > @@ -48,8 +48,6 @@ SRC_URI = "git://github.com/linux-test-project/ltp.git \ > file://0034-periodic_output.patch \ > file://0035-fix-test_proc_kill-hang.patch \ > file://0036-testcases-network-nfsv4-acl-acl1.c-Security-fix-on-s.patch > \ > - file://0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch > \ > - file://0038-commands-gdb01-replace-stdin-with-dev-null.patch \ > " > > S = "${WORKDIR}/git" > -- > 2.14.2 > > -- > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core > <div dir="ltr">Sorry:<div><br></div><div><div>| fanotify08.c:56:3: error: format not a string literal and no format arguments [-Werror=format-security]</div><div>| tst_res(TPASS, msg);</div><div>| ^~~~~~~</div><div>| fanotify08.c:58:3: error: format not a string literal and no format arguments [-Werror=format-security]</div><div>| tst_res(TFAIL, msg);</div><div>| ^~~~~~~</div></div><div><br></div><div>Ross</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 5 October 2017 at 18:47, Fathi Boudra <span dir="ltr"><<a href="mailto:fathi.boudra@linaro.org" target="_blank">fathi.boudra@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">* Update SRCREV to match 20170929 release<br> * Drop patches available in 20170929 release:<br> - 0037-ltp-fix-PAGE_SIZE-<wbr>redefinition-and-O_CREAT-<wbr>undeclear.patch<br> - 0038-commands-gdb01-replace-<wbr>stdin-with-dev-null.patch<br> <br> Signed-off-by: Fathi Boudra <<a href="mailto:fathi.boudra@linaro.org">fathi.boudra@linaro.org</a>><br> ---<br> ...E_SIZE-redefinition-and-O_<wbr>CREAT-undeclear.patch | 113 ---------------------<br> ...ommands-gdb01-replace-<wbr>stdin-with-dev-null.patch | 34 -------<br> .../ltp/{<a href="http://ltp_20170516.bb" rel="noreferrer" target="_blank">ltp_20170516.bb</a> => <a href="http://ltp_20170929.bb" rel="noreferrer" target="_blank">ltp_20170929.bb</a>} | 4 +-<br> 3 files changed, 1 insertion(+), 150 deletions(-)<br> delete mode 100644 meta/recipes-extended/ltp/ltp/<wbr>0037-ltp-fix-PAGE_SIZE-<wbr>redefinition-and-O_CREAT-<wbr>undeclear.patch<br> delete mode 100644 meta/recipes-extended/ltp/ltp/<wbr>0038-commands-gdb01-replace-<wbr>stdin-with-dev-null.patch<br> rename meta/recipes-extended/ltp/{<a href="http://ltp_20170516.bb" rel="noreferrer" target="_blank">ltp<wbr>_20170516.bb</a> => <a href="http://ltp_20170929.bb" rel="noreferrer" target="_blank">ltp_20170929.bb</a>} (96%)<br> <br> diff --git a/meta/recipes-extended/ltp/<wbr>ltp/0037-ltp-fix-PAGE_SIZE-<wbr>redefinition-and-O_CREAT-<wbr>undeclear.patch b/meta/recipes-extended/ltp/<wbr>ltp/0037-ltp-fix-PAGE_SIZE-<wbr>redefinition-and-O_CREAT-<wbr>undeclear.patch<br> deleted file mode 100644<br> index c8738ae0d4..0000000000<br> --- a/meta/recipes-extended/ltp/<wbr>ltp/0037-ltp-fix-PAGE_SIZE-<wbr>redefinition-and-O_CREAT-<wbr>undeclear.patch<br> +++ /dev/null<br> @@ -1,113 +0,0 @@<br> -From a9d5595d2fa2ab252f1cabf63f4b65<wbr>c3efbafeb9 Mon Sep 17 00:00:00 2001<br> -From: Dengke Du <<a href="mailto:dengke.du@windriver.com">dengke.du@windriver.com</a>><br> -Date: Thu, 10 Aug 2017 15:27:03 +0800<br> -Subject: [PATCH] ltp: fix PAGE_SIZE redefinition and O_CREAT undeclear when<br> - build with musl<br> -<br> -error 1:<br> -<br> -|stack_clash.c:50:22: error: expected identifier or '(' before numeric constant<br> -| static unsigned long PAGE_SIZE;<br> -<br> -This is because the musl libc already contain PAGE_SIZE definition in limits.c,<br> -we can check it here:<br> -<br> - <a href="https://git.musl-libc.org/cgit/musl/tree/include/limits.h#n43" rel="noreferrer" target="_blank">https://git.musl-libc.org/<wbr>cgit/musl/tree/include/limits.<wbr>h#n43</a><br> -<br> -error 2:<br> -<br> -|ck01.c:157:22: error: 'O_CREAT' undeclared (first use in this function); did you mean 'S_IREAD'?<br> -| fd = open(filename, O_CREAT | O_TRUNC | O_RDWR, 0644);<br> -| ^~~~~~~<br> -| S_IREAD<br> -<br> -This is because the musl libc put those in fcntl.h, so we should include that<br> -file.<br> -<br> -Upstream-Status: Submitted [ <a href="https://github.com/linux-test-project/ltp/pull/194" rel="noreferrer" target="_blank">https://github.com/linux-test-<wbr>project/ltp/pull/194</a> ]<br> -<br> -Signed-off-by: Dengke Du <<a href="mailto:dengke.du@windriver.com">dengke.du@windriver.com</a>><br> ----<br> - testcases/cve/stack_clash.c | 12 ++++++------<br> - testcases/kernel/syscalls/<wbr>flock/flock01.c | 1 +<br> - testcases/kernel/syscalls/<wbr>flock/flock02.c | 1 +<br> - 3 files changed, 8 insertions(+), 6 deletions(-)<br> -<br> -diff --git a/testcases/cve/stack_clash.c b/testcases/cve/stack_clash.c<br> -index 2ef1a82..7c45991 100644<br> ---- a/testcases/cve/stack_clash.c<br> -+++ b/testcases/cve/stack_clash.c<br> -@@ -47,7 +47,7 @@<br> - #include "tst_test.h"<br> - #include "tst_safe_stdio.h"<br> -<br> --static unsigned long PAGE_SIZE;<br> -+static unsigned long PAGE_SIZE_tst;<br> - static unsigned long PAGE_MASK;<br> - static unsigned long GAP_PAGES = 256;<br> - static unsigned long THRESHOLD;<br> -@@ -66,7 +66,7 @@ void exhaust_stack_into_sigsegv(<wbr>void)<br> - exhaust_stack_into_sigsegv();<br> - }<br> -<br> --#define MAPPED_LEN PAGE_SIZE<br> -+#define MAPPED_LEN PAGE_SIZE_tst<br> - static unsigned long mapped_addr;<br> -<br> - void segv_handler(int sig, siginfo_t *info, void *data LTP_ATTRIBUTE_UNUSED)<br> -@@ -150,7 +150,7 @@ void do_child(void)<br> - stack_t signal_stack;<br> - struct sigaction segv_sig = {.sa_sigaction = segv_handler, .sa_flags = SA_ONSTACK|SA_SIGINFO};<br> - void *map;<br> -- unsigned long gap = GAP_PAGES * PAGE_SIZE;<br> -+ unsigned long gap = GAP_PAGES * PAGE_SIZE_tst;<br> - struct rlimit rlimit;<br> -<br> - rlimit.rlim_cur = rlimit.rlim_max = RLIM_INFINITY;<br> -@@ -200,8 +200,8 @@ void setup(void)<br> - {<br> - char buf[4096], *p;<br> -<br> -- PAGE_SIZE = sysconf(_SC_PAGESIZE);<br> -- PAGE_MASK = ~(PAGE_SIZE - 1);<br> -+ PAGE_SIZE_tst = sysconf(_SC_PAGESIZE);<br> -+ PAGE_MASK = ~(PAGE_SIZE_tst - 1);<br> -<br> - buf[4095] = '\0';<br> - SAFE_FILE_SCANF("/proc/<wbr>cmdline", "%4095[^\n]", buf);<br> -@@ -214,7 +214,7 @@ void setup(void)<br> - tst_res(TINFO, "stack_guard_gap = %ld", GAP_PAGES);<br> - }<br> -<br> -- THRESHOLD = (GAP_PAGES - 1) * PAGE_SIZE;<br> -+ THRESHOLD = (GAP_PAGES - 1) * PAGE_SIZE_tst;<br> -<br> - {<br> - volatile int *a = alloca(128);<br> -diff --git a/testcases/kernel/syscalls/<wbr>flock/flock01.c b/testcases/kernel/syscalls/<wbr>flock/flock01.c<br> -index 3e17be4..06d89e3 100644<br> ---- a/testcases/kernel/syscalls/<wbr>flock/flock01.c<br> -+++ b/testcases/kernel/syscalls/<wbr>flock/flock01.c<br> -@@ -69,6 +69,7 @@<br> - #include <stdio.h><br> - #include <sys/wait.h><br> - #include <sys/file.h><br> -+#include <fcntl.h><br> - #include "test.h"<br> -<br> - void setup(void);<br> -diff --git a/testcases/kernel/syscalls/<wbr>flock/flock02.c b/testcases/kernel/syscalls/<wbr>flock/flock02.c<br> -index 414df68..9ddf729 100644<br> ---- a/testcases/kernel/syscalls/<wbr>flock/flock02.c<br> -+++ b/testcases/kernel/syscalls/<wbr>flock/flock02.c<br> -@@ -75,6 +75,7 @@<br> - #include <sys/types.h><br> - #include <sys/file.h><br> - #include <sys/wait.h><br> -+#include <fcntl.h><br> - #include <errno.h><br> - #include <stdio.h><br> - #include "test.h"<br> ---<br> -2.7.4<br> -<br> diff --git a/meta/recipes-extended/ltp/<wbr>ltp/0038-commands-gdb01-<wbr>replace-stdin-with-dev-null.<wbr>patch b/meta/recipes-extended/ltp/<wbr>ltp/0038-commands-gdb01-<wbr>replace-stdin-with-dev-null.<wbr>patch<br> deleted file mode 100644<br> index f7c0a4bf17..0000000000<br> --- a/meta/recipes-extended/ltp/<wbr>ltp/0038-commands-gdb01-<wbr>replace-stdin-with-dev-null.<wbr>patch<br> +++ /dev/null<br> @@ -1,34 +0,0 @@<br> -From 2f6ab8f694b26b7f2566624f6d1f23<wbr>788d6ab8a0 Mon Sep 17 00:00:00 2001<br> -From: Jan Stancek <<a href="mailto:jstancek@redhat.com">jstancek@redhat.com</a>><br> -Date: Mon, 11 Sep 2017 12:57:58 +0200<br> -Subject: [PATCH] commands/gdb01: replace stdin with /dev/null<br> -<br> -If this testcase runs as background process, gdb can receive<br> -SIGTTOU and then testcase gets stuck.<br> -<br> -Signed-off-by: Jan Stancek <<a href="mailto:jstancek@redhat.com">jstancek@redhat.com</a>><br> -<br> -Upstream-Status: Backport<br> -[<a href="https://github.com/linux-test-project/ltp/commit/2f6ab8f694b26b7f2566624f6d1f23788d6ab8a0" rel="noreferrer" target="_blank">https://github.com/linux-<wbr>test-project/ltp/commit/<wbr>2f6ab8f694b26b7f2566624f6d1f23<wbr>788d6ab8a0</a>]<br> -<br> -Signed-off-by: Yi Zhao <<a href="mailto:yi.zhao@windriver.com">yi.zhao@windriver.com</a>><br> ----<br> - testcases/commands/gdb/gdb01.<wbr>sh | 2 +-<br> - 1 file changed, 1 insertion(+), 1 deletion(-)<br> -<br> -diff --git a/testcases/commands/gdb/<wbr>gdb01.sh b/testcases/commands/gdb/<wbr>gdb01.sh<br> -index 07ae36f..e3a5b51 100755<br> ---- a/testcases/commands/gdb/<wbr>gdb01.sh<br> -+++ b/testcases/commands/gdb/<wbr>gdb01.sh<br> -@@ -29,7 +29,7 @@ TST_NEEDS_CMDS="gdb /bin/cat"<br> -<br> - simple_test()<br> - {<br> -- gdb /bin/cat -ex "run /etc/passwd" -ex quit<br> -+ gdb /bin/cat -ex "run /etc/passwd" -ex quit < /dev/null<br> - RC=$?<br> - if [ $RC -eq 0 ] ; then<br> - tst_res TPASS "gdb attached to process and completed run"<br> ---<br> -2.7.4<br> -<br> diff --git a/meta/recipes-extended/ltp/<a href="http://ltp_20170516.bb" rel="noreferrer" target="_blank">lt<wbr>p_20170516.bb</a> b/meta/recipes-extended/ltp/<a href="http://ltp_20170929.bb" rel="noreferrer" target="_blank">lt<wbr>p_20170929.bb</a><br> similarity index 96%<br> rename from meta/recipes-extended/ltp/<a href="http://ltp_20170516.bb" rel="noreferrer" target="_blank">ltp_<wbr>20170516.bb</a><br> rename to meta/recipes-extended/ltp/<a href="http://ltp_20170929.bb" rel="noreferrer" target="_blank">ltp_<wbr>20170929.bb</a><br> index 653cbfd4ed..925dd91256 100644<br> --- a/meta/recipes-extended/ltp/<a href="http://ltp_20170516.bb" rel="noreferrer" target="_blank">lt<wbr>p_20170516.bb</a><br> +++ b/meta/recipes-extended/ltp/<a href="http://ltp_20170929.bb" rel="noreferrer" target="_blank">lt<wbr>p_20170929.bb</a><br> @@ -21,7 +21,7 @@ DEPENDS_append_libc-musl = " fts "<br> EXTRA_OEMAKE_append_libc-musl = " LIBC=musl "<br> CFLAGS_append_powerpc64 = " -D__SANE_USERSPACE_TYPES__"<br> CFLAGS_append_mipsarchn64 = " -D__SANE_USERSPACE_TYPES__"<br> -SRCREV = "<wbr>18916a2e6d8c997b7b29dcfa9550d5<wbr>a15b22ed22"<br> +SRCREV = "<wbr>e671f2a13c695bbd87f7dfec2954ca<wbr>7e3c43f377"<br> <br> SRC_URI = "git://<a href="http://github.com/linux-test-project/ltp.git" rel="noreferrer" target="_blank">github.com/linux-test-<wbr>project/ltp.git</a> \<br> file://0001-add-_GNU_SOURCE-<wbr>to-pec_listener.c.patch \<br> @@ -48,8 +48,6 @@ SRC_URI = "git://<a href="http://github.com/linux-test-project/ltp.git" rel="noreferrer" target="_blank">github.com/linux-test-<wbr>project/ltp.git</a> \<br> file://0034-periodic_output.<wbr>patch \<br> file://0035-fix-test_proc_<wbr>kill-hang.patch \<br> file://0036-testcases-network-<wbr>nfsv4-acl-acl1.c-Security-fix-<wbr>on-s.patch \<br> - file://0037-ltp-fix-PAGE_SIZE-<wbr>redefinition-and-O_CREAT-<wbr>undeclear.patch \<br> - file://0038-commands-gdb01-<wbr>replace-stdin-with-dev-null.<wbr>patch \<br> "<br> <br> S = "${WORKDIR}/git"<br> <span class="HOEnZb"><font color="#888888">--<br> 2.14.2<br> <br> --<br> ______________________________<wbr>_________________<br> Openembedded-core mailing list<br> <a href="mailto:Openembedded-core@lists.openembedded.org">Openembedded-core@lists.<wbr>openembedded.org</a><br> <a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core" rel="noreferrer" target="_blank">http://lists.openembedded.org/<wbr>mailman/listinfo/openembedded-<wbr>core</a><br> </font></span></blockquote></div><br></div> -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
On Wed, Oct 11, 2017 at 1:30 PM, Burton, Ross <ross.burton@intel.com> wrote: > Sorry: > > | fanotify08.c:56:3: error: format not a string literal and no format > arguments [-Werror=format-security] > | tst_res(TPASS, msg); > | ^~~~~~~ > | fanotify08.c:58:3: error: format not a string literal and no format > arguments [-Werror=format-security] > | tst_res(TFAIL, msg); > | ^~~~~~~ > tst_res(TFAIL, "%s", msg) might fix it. > Ross > > On 5 October 2017 at 18:47, Fathi Boudra <fathi.boudra@linaro.org> wrote: >> >> * Update SRCREV to match 20170929 release >> * Drop patches available in 20170929 release: >> - 0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch >> - 0038-commands-gdb01-replace-stdin-with-dev-null.patch >> >> Signed-off-by: Fathi Boudra <fathi.boudra@linaro.org> >> --- >> ...E_SIZE-redefinition-and-O_CREAT-undeclear.patch | 113 >> --------------------- >> ...ommands-gdb01-replace-stdin-with-dev-null.patch | 34 ------- >> .../ltp/{ltp_20170516.bb => ltp_20170929.bb} | 4 +- >> 3 files changed, 1 insertion(+), 150 deletions(-) >> delete mode 100644 >> meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch >> delete mode 100644 >> meta/recipes-extended/ltp/ltp/0038-commands-gdb01-replace-stdin-with-dev-null.patch >> rename meta/recipes-extended/ltp/{ltp_20170516.bb => ltp_20170929.bb} >> (96%) >> >> diff --git >> a/meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch >> b/meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch >> deleted file mode 100644 >> index c8738ae0d4..0000000000 >> --- >> a/meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch >> +++ /dev/null >> @@ -1,113 +0,0 @@ >> -From a9d5595d2fa2ab252f1cabf63f4b65c3efbafeb9 Mon Sep 17 00:00:00 2001 >> -From: Dengke Du <dengke.du@windriver.com> >> -Date: Thu, 10 Aug 2017 15:27:03 +0800 >> -Subject: [PATCH] ltp: fix PAGE_SIZE redefinition and O_CREAT undeclear >> when >> - build with musl >> - >> -error 1: >> - >> -|stack_clash.c:50:22: error: expected identifier or '(' before numeric >> constant >> -| static unsigned long PAGE_SIZE; >> - >> -This is because the musl libc already contain PAGE_SIZE definition in >> limits.c, >> -we can check it here: >> - >> - https://git.musl-libc.org/cgit/musl/tree/include/limits.h#n43 >> - >> -error 2: >> - >> -|ck01.c:157:22: error: 'O_CREAT' undeclared (first use in this function); >> did you mean 'S_IREAD'? >> -| fd = open(filename, O_CREAT | O_TRUNC | O_RDWR, 0644); >> -| ^~~~~~~ >> -| S_IREAD >> - >> -This is because the musl libc put those in fcntl.h, so we should include >> that >> -file. >> - >> -Upstream-Status: Submitted [ >> https://github.com/linux-test-project/ltp/pull/194 ] >> - >> -Signed-off-by: Dengke Du <dengke.du@windriver.com> >> ---- >> - testcases/cve/stack_clash.c | 12 ++++++------ >> - testcases/kernel/syscalls/flock/flock01.c | 1 + >> - testcases/kernel/syscalls/flock/flock02.c | 1 + >> - 3 files changed, 8 insertions(+), 6 deletions(-) >> - >> -diff --git a/testcases/cve/stack_clash.c b/testcases/cve/stack_clash.c >> -index 2ef1a82..7c45991 100644 >> ---- a/testcases/cve/stack_clash.c >> -+++ b/testcases/cve/stack_clash.c >> -@@ -47,7 +47,7 @@ >> - #include "tst_test.h" >> - #include "tst_safe_stdio.h" >> - >> --static unsigned long PAGE_SIZE; >> -+static unsigned long PAGE_SIZE_tst; >> - static unsigned long PAGE_MASK; >> - static unsigned long GAP_PAGES = 256; >> - static unsigned long THRESHOLD; >> -@@ -66,7 +66,7 @@ void exhaust_stack_into_sigsegv(void) >> - exhaust_stack_into_sigsegv(); >> - } >> - >> --#define MAPPED_LEN PAGE_SIZE >> -+#define MAPPED_LEN PAGE_SIZE_tst >> - static unsigned long mapped_addr; >> - >> - void segv_handler(int sig, siginfo_t *info, void *data >> LTP_ATTRIBUTE_UNUSED) >> -@@ -150,7 +150,7 @@ void do_child(void) >> - stack_t signal_stack; >> - struct sigaction segv_sig = {.sa_sigaction = segv_handler, >> .sa_flags = SA_ONSTACK|SA_SIGINFO}; >> - void *map; >> -- unsigned long gap = GAP_PAGES * PAGE_SIZE; >> -+ unsigned long gap = GAP_PAGES * PAGE_SIZE_tst; >> - struct rlimit rlimit; >> - >> - rlimit.rlim_cur = rlimit.rlim_max = RLIM_INFINITY; >> -@@ -200,8 +200,8 @@ void setup(void) >> - { >> - char buf[4096], *p; >> - >> -- PAGE_SIZE = sysconf(_SC_PAGESIZE); >> -- PAGE_MASK = ~(PAGE_SIZE - 1); >> -+ PAGE_SIZE_tst = sysconf(_SC_PAGESIZE); >> -+ PAGE_MASK = ~(PAGE_SIZE_tst - 1); >> - >> - buf[4095] = '\0'; >> - SAFE_FILE_SCANF("/proc/cmdline", "%4095[^\n]", buf); >> -@@ -214,7 +214,7 @@ void setup(void) >> - tst_res(TINFO, "stack_guard_gap = %ld", GAP_PAGES); >> - } >> - >> -- THRESHOLD = (GAP_PAGES - 1) * PAGE_SIZE; >> -+ THRESHOLD = (GAP_PAGES - 1) * PAGE_SIZE_tst; >> - >> - { >> - volatile int *a = alloca(128); >> -diff --git a/testcases/kernel/syscalls/flock/flock01.c >> b/testcases/kernel/syscalls/flock/flock01.c >> -index 3e17be4..06d89e3 100644 >> ---- a/testcases/kernel/syscalls/flock/flock01.c >> -+++ b/testcases/kernel/syscalls/flock/flock01.c >> -@@ -69,6 +69,7 @@ >> - #include <stdio.h> >> - #include <sys/wait.h> >> - #include <sys/file.h> >> -+#include <fcntl.h> >> - #include "test.h" >> - >> - void setup(void); >> -diff --git a/testcases/kernel/syscalls/flock/flock02.c >> b/testcases/kernel/syscalls/flock/flock02.c >> -index 414df68..9ddf729 100644 >> ---- a/testcases/kernel/syscalls/flock/flock02.c >> -+++ b/testcases/kernel/syscalls/flock/flock02.c >> -@@ -75,6 +75,7 @@ >> - #include <sys/types.h> >> - #include <sys/file.h> >> - #include <sys/wait.h> >> -+#include <fcntl.h> >> - #include <errno.h> >> - #include <stdio.h> >> - #include "test.h" >> --- >> -2.7.4 >> - >> diff --git >> a/meta/recipes-extended/ltp/ltp/0038-commands-gdb01-replace-stdin-with-dev-null.patch >> b/meta/recipes-extended/ltp/ltp/0038-commands-gdb01-replace-stdin-with-dev-null.patch >> deleted file mode 100644 >> index f7c0a4bf17..0000000000 >> --- >> a/meta/recipes-extended/ltp/ltp/0038-commands-gdb01-replace-stdin-with-dev-null.patch >> +++ /dev/null >> @@ -1,34 +0,0 @@ >> -From 2f6ab8f694b26b7f2566624f6d1f23788d6ab8a0 Mon Sep 17 00:00:00 2001 >> -From: Jan Stancek <jstancek@redhat.com> >> -Date: Mon, 11 Sep 2017 12:57:58 +0200 >> -Subject: [PATCH] commands/gdb01: replace stdin with /dev/null >> - >> -If this testcase runs as background process, gdb can receive >> -SIGTTOU and then testcase gets stuck. >> - >> -Signed-off-by: Jan Stancek <jstancek@redhat.com> >> - >> -Upstream-Status: Backport >> >> -[https://github.com/linux-test-project/ltp/commit/2f6ab8f694b26b7f2566624f6d1f23788d6ab8a0] >> - >> -Signed-off-by: Yi Zhao <yi.zhao@windriver.com> >> ---- >> - testcases/commands/gdb/gdb01.sh | 2 +- >> - 1 file changed, 1 insertion(+), 1 deletion(-) >> - >> -diff --git a/testcases/commands/gdb/gdb01.sh >> b/testcases/commands/gdb/gdb01.sh >> -index 07ae36f..e3a5b51 100755 >> ---- a/testcases/commands/gdb/gdb01.sh >> -+++ b/testcases/commands/gdb/gdb01.sh >> -@@ -29,7 +29,7 @@ TST_NEEDS_CMDS="gdb /bin/cat" >> - >> - simple_test() >> - { >> -- gdb /bin/cat -ex "run /etc/passwd" -ex quit >> -+ gdb /bin/cat -ex "run /etc/passwd" -ex quit < /dev/null >> - RC=$? >> - if [ $RC -eq 0 ] ; then >> - tst_res TPASS "gdb attached to process and completed run" >> --- >> -2.7.4 >> - >> diff --git a/meta/recipes-extended/ltp/ltp_20170516.bb >> b/meta/recipes-extended/ltp/ltp_20170929.bb >> similarity index 96% >> rename from meta/recipes-extended/ltp/ltp_20170516.bb >> rename to meta/recipes-extended/ltp/ltp_20170929.bb >> index 653cbfd4ed..925dd91256 100644 >> --- a/meta/recipes-extended/ltp/ltp_20170516.bb >> +++ b/meta/recipes-extended/ltp/ltp_20170929.bb >> @@ -21,7 +21,7 @@ DEPENDS_append_libc-musl = " fts " >> EXTRA_OEMAKE_append_libc-musl = " LIBC=musl " >> CFLAGS_append_powerpc64 = " -D__SANE_USERSPACE_TYPES__" >> CFLAGS_append_mipsarchn64 = " -D__SANE_USERSPACE_TYPES__" >> -SRCREV = "18916a2e6d8c997b7b29dcfa9550d5a15b22ed22" >> +SRCREV = "e671f2a13c695bbd87f7dfec2954ca7e3c43f377" >> >> SRC_URI = "git://github.com/linux-test-project/ltp.git \ >> file://0001-add-_GNU_SOURCE-to-pec_listener.c.patch \ >> @@ -48,8 +48,6 @@ SRC_URI = "git://github.com/linux-test-project/ltp.git \ >> file://0034-periodic_output.patch \ >> file://0035-fix-test_proc_kill-hang.patch \ >> >> file://0036-testcases-network-nfsv4-acl-acl1.c-Security-fix-on-s.patch \ >> - >> file://0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch \ >> - file://0038-commands-gdb01-replace-stdin-with-dev-null.patch \ >> " >> >> S = "${WORKDIR}/git" >> -- >> 2.14.2 >> >> -- >> _______________________________________________ >> Openembedded-core mailing list >> Openembedded-core@lists.openembedded.org >> http://lists.openembedded.org/mailman/listinfo/openembedded-core > > > > -- > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core > -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
diff --git a/meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch b/meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch deleted file mode 100644 index c8738ae0d4..0000000000 --- a/meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch +++ /dev/null @@ -1,113 +0,0 @@ -From a9d5595d2fa2ab252f1cabf63f4b65c3efbafeb9 Mon Sep 17 00:00:00 2001 -From: Dengke Du <dengke.du@windriver.com> -Date: Thu, 10 Aug 2017 15:27:03 +0800 -Subject: [PATCH] ltp: fix PAGE_SIZE redefinition and O_CREAT undeclear when - build with musl - -error 1: - -|stack_clash.c:50:22: error: expected identifier or '(' before numeric constant -| static unsigned long PAGE_SIZE; - -This is because the musl libc already contain PAGE_SIZE definition in limits.c, -we can check it here: - - https://git.musl-libc.org/cgit/musl/tree/include/limits.h#n43 - -error 2: - -|ck01.c:157:22: error: 'O_CREAT' undeclared (first use in this function); did you mean 'S_IREAD'? -| fd = open(filename, O_CREAT | O_TRUNC | O_RDWR, 0644); -| ^~~~~~~ -| S_IREAD - -This is because the musl libc put those in fcntl.h, so we should include that -file. - -Upstream-Status: Submitted [ https://github.com/linux-test-project/ltp/pull/194 ] - -Signed-off-by: Dengke Du <dengke.du@windriver.com> ---- - testcases/cve/stack_clash.c | 12 ++++++------ - testcases/kernel/syscalls/flock/flock01.c | 1 + - testcases/kernel/syscalls/flock/flock02.c | 1 + - 3 files changed, 8 insertions(+), 6 deletions(-) - -diff --git a/testcases/cve/stack_clash.c b/testcases/cve/stack_clash.c -index 2ef1a82..7c45991 100644 ---- a/testcases/cve/stack_clash.c -+++ b/testcases/cve/stack_clash.c -@@ -47,7 +47,7 @@ - #include "tst_test.h" - #include "tst_safe_stdio.h" - --static unsigned long PAGE_SIZE; -+static unsigned long PAGE_SIZE_tst; - static unsigned long PAGE_MASK; - static unsigned long GAP_PAGES = 256; - static unsigned long THRESHOLD; -@@ -66,7 +66,7 @@ void exhaust_stack_into_sigsegv(void) - exhaust_stack_into_sigsegv(); - } - --#define MAPPED_LEN PAGE_SIZE -+#define MAPPED_LEN PAGE_SIZE_tst - static unsigned long mapped_addr; - - void segv_handler(int sig, siginfo_t *info, void *data LTP_ATTRIBUTE_UNUSED) -@@ -150,7 +150,7 @@ void do_child(void) - stack_t signal_stack; - struct sigaction segv_sig = {.sa_sigaction = segv_handler, .sa_flags = SA_ONSTACK|SA_SIGINFO}; - void *map; -- unsigned long gap = GAP_PAGES * PAGE_SIZE; -+ unsigned long gap = GAP_PAGES * PAGE_SIZE_tst; - struct rlimit rlimit; - - rlimit.rlim_cur = rlimit.rlim_max = RLIM_INFINITY; -@@ -200,8 +200,8 @@ void setup(void) - { - char buf[4096], *p; - -- PAGE_SIZE = sysconf(_SC_PAGESIZE); -- PAGE_MASK = ~(PAGE_SIZE - 1); -+ PAGE_SIZE_tst = sysconf(_SC_PAGESIZE); -+ PAGE_MASK = ~(PAGE_SIZE_tst - 1); - - buf[4095] = '\0'; - SAFE_FILE_SCANF("/proc/cmdline", "%4095[^\n]", buf); -@@ -214,7 +214,7 @@ void setup(void) - tst_res(TINFO, "stack_guard_gap = %ld", GAP_PAGES); - } - -- THRESHOLD = (GAP_PAGES - 1) * PAGE_SIZE; -+ THRESHOLD = (GAP_PAGES - 1) * PAGE_SIZE_tst; - - { - volatile int *a = alloca(128); -diff --git a/testcases/kernel/syscalls/flock/flock01.c b/testcases/kernel/syscalls/flock/flock01.c -index 3e17be4..06d89e3 100644 ---- a/testcases/kernel/syscalls/flock/flock01.c -+++ b/testcases/kernel/syscalls/flock/flock01.c -@@ -69,6 +69,7 @@ - #include <stdio.h> - #include <sys/wait.h> - #include <sys/file.h> -+#include <fcntl.h> - #include "test.h" - - void setup(void); -diff --git a/testcases/kernel/syscalls/flock/flock02.c b/testcases/kernel/syscalls/flock/flock02.c -index 414df68..9ddf729 100644 ---- a/testcases/kernel/syscalls/flock/flock02.c -+++ b/testcases/kernel/syscalls/flock/flock02.c -@@ -75,6 +75,7 @@ - #include <sys/types.h> - #include <sys/file.h> - #include <sys/wait.h> -+#include <fcntl.h> - #include <errno.h> - #include <stdio.h> - #include "test.h" --- -2.7.4 - diff --git a/meta/recipes-extended/ltp/ltp/0038-commands-gdb01-replace-stdin-with-dev-null.patch b/meta/recipes-extended/ltp/ltp/0038-commands-gdb01-replace-stdin-with-dev-null.patch deleted file mode 100644 index f7c0a4bf17..0000000000 --- a/meta/recipes-extended/ltp/ltp/0038-commands-gdb01-replace-stdin-with-dev-null.patch +++ /dev/null @@ -1,34 +0,0 @@ -From 2f6ab8f694b26b7f2566624f6d1f23788d6ab8a0 Mon Sep 17 00:00:00 2001 -From: Jan Stancek <jstancek@redhat.com> -Date: Mon, 11 Sep 2017 12:57:58 +0200 -Subject: [PATCH] commands/gdb01: replace stdin with /dev/null - -If this testcase runs as background process, gdb can receive -SIGTTOU and then testcase gets stuck. - -Signed-off-by: Jan Stancek <jstancek@redhat.com> - -Upstream-Status: Backport -[https://github.com/linux-test-project/ltp/commit/2f6ab8f694b26b7f2566624f6d1f23788d6ab8a0] - -Signed-off-by: Yi Zhao <yi.zhao@windriver.com> ---- - testcases/commands/gdb/gdb01.sh | 2 +- - 1 file changed, 1 insertion(+), 1 deletion(-) - -diff --git a/testcases/commands/gdb/gdb01.sh b/testcases/commands/gdb/gdb01.sh -index 07ae36f..e3a5b51 100755 ---- a/testcases/commands/gdb/gdb01.sh -+++ b/testcases/commands/gdb/gdb01.sh -@@ -29,7 +29,7 @@ TST_NEEDS_CMDS="gdb /bin/cat" - - simple_test() - { -- gdb /bin/cat -ex "run /etc/passwd" -ex quit -+ gdb /bin/cat -ex "run /etc/passwd" -ex quit < /dev/null - RC=$? - if [ $RC -eq 0 ] ; then - tst_res TPASS "gdb attached to process and completed run" --- -2.7.4 - diff --git a/meta/recipes-extended/ltp/ltp_20170516.bb b/meta/recipes-extended/ltp/ltp_20170929.bb similarity index 96% rename from meta/recipes-extended/ltp/ltp_20170516.bb rename to meta/recipes-extended/ltp/ltp_20170929.bb index 653cbfd4ed..925dd91256 100644 --- a/meta/recipes-extended/ltp/ltp_20170516.bb +++ b/meta/recipes-extended/ltp/ltp_20170929.bb @@ -21,7 +21,7 @@ DEPENDS_append_libc-musl = " fts " EXTRA_OEMAKE_append_libc-musl = " LIBC=musl " CFLAGS_append_powerpc64 = " -D__SANE_USERSPACE_TYPES__" CFLAGS_append_mipsarchn64 = " -D__SANE_USERSPACE_TYPES__" -SRCREV = "18916a2e6d8c997b7b29dcfa9550d5a15b22ed22" +SRCREV = "e671f2a13c695bbd87f7dfec2954ca7e3c43f377" SRC_URI = "git://github.com/linux-test-project/ltp.git \ file://0001-add-_GNU_SOURCE-to-pec_listener.c.patch \ @@ -48,8 +48,6 @@ SRC_URI = "git://github.com/linux-test-project/ltp.git \ file://0034-periodic_output.patch \ file://0035-fix-test_proc_kill-hang.patch \ file://0036-testcases-network-nfsv4-acl-acl1.c-Security-fix-on-s.patch \ - file://0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch \ - file://0038-commands-gdb01-replace-stdin-with-dev-null.patch \ " S = "${WORKDIR}/git"
* Update SRCREV to match 20170929 release * Drop patches available in 20170929 release: - 0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch - 0038-commands-gdb01-replace-stdin-with-dev-null.patch Signed-off-by: Fathi Boudra <fathi.boudra@linaro.org> --- ...E_SIZE-redefinition-and-O_CREAT-undeclear.patch | 113 --------------------- ...ommands-gdb01-replace-stdin-with-dev-null.patch | 34 ------- .../ltp/{ltp_20170516.bb => ltp_20170929.bb} | 4 +- 3 files changed, 1 insertion(+), 150 deletions(-) delete mode 100644 meta/recipes-extended/ltp/ltp/0037-ltp-fix-PAGE_SIZE-redefinition-and-O_CREAT-undeclear.patch delete mode 100644 meta/recipes-extended/ltp/ltp/0038-commands-gdb01-replace-stdin-with-dev-null.patch rename meta/recipes-extended/ltp/{ltp_20170516.bb => ltp_20170929.bb} (96%) -- 2.14.2 -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core