Re: [OE-core] [PATCH 4/7] eglibc: ptrace: protect ptrace_peeksiginfo_args from redefintion
On Aug 25, 2013, at 7:46 PM, Bruce Ashfield bruce.ashfi...@gmail.com wrote: A final (maybe .. hopefully!?) thought on this. Do drop this change, but give me a few minutes to remove the offending structure from linux-libc-headers_3.10. That will also protect all users of yocto, regardless of the kernel unless they provide their own libc-headers .. and then they are on their own :) That leaves the c library alone, and should buy us time to fix the offending apps. this is not again the correct place to fix it. since you will back this out say later when all apps are fixed anyway. If we need to keep it I would rather have the app builds failing and keep it enabled and it should not be the guy who is doing the kernel headers upgrade to fix the whole ecosystem either. So add the headers by all means and leave the old headers in there until the apps work correctly with new headers. ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 4/7] eglibc: ptrace: protect ptrace_peeksiginfo_args from redefintion
On Mon, Aug 26, 2013 at 1:28 PM, Khem Raj raj.k...@gmail.com wrote: On Aug 25, 2013, at 7:46 PM, Bruce Ashfield bruce.ashfi...@gmail.com wrote: A final (maybe .. hopefully!?) thought on this. Do drop this change, but give me a few minutes to remove the offending structure from linux-libc-headers_3.10. That will also protect all users of yocto, regardless of the kernel unless they provide their own libc-headers .. and then they are on their own :) That leaves the c library alone, and should buy us time to fix the offending apps. this is not again the correct place to fix it. since you will back this out say later when all apps are fixed anyway. If we need to keep it I would rather have the app builds failing and keep it enabled and it should not be the guy who is doing the kernel headers upgrade to fix the whole ecosystem either. So add the headers by all means and leave the old headers in there until the apps work correctly with new headers. Whether or not to allow the apps to break, that's a call for Saul and Richard. For now, this fix works in all cases, and I'm able to build and boot with the 3.10 kernel and matching headers. Given that we are about to hit the M4 freeze, I'd prefer this in now, and the apps fixed later. We need to get mileage with the matching headers and kernel. I think this is a good compromise for the time being, we've had far nastier hacks in the libc-headers in the past. Bruce -- Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 4/7] eglibc: ptrace: protect ptrace_peeksiginfo_args from redefintion
On Aug 26, 2013, at 10:59 AM, Bruce Ashfield bruce.ashfi...@gmail.com wrote: On Mon, Aug 26, 2013 at 1:28 PM, Khem Raj raj.k...@gmail.com wrote: On Aug 25, 2013, at 7:46 PM, Bruce Ashfield bruce.ashfi...@gmail.com wrote: A final (maybe .. hopefully!?) thought on this. Do drop this change, but give me a few minutes to remove the offending structure from linux-libc-headers_3.10. That will also protect all users of yocto, regardless of the kernel unless they provide their own libc-headers .. and then they are on their own :) That leaves the c library alone, and should buy us time to fix the offending apps. this is not again the correct place to fix it. since you will back this out say later when all apps are fixed anyway. If we need to keep it I would rather have the app builds failing and keep it enabled and it should not be the guy who is doing the kernel headers upgrade to fix the whole ecosystem either. So add the headers by all means and leave the old headers in there until the apps work correctly with new headers. Whether or not to allow the apps to break, that's a call for Saul and Richard. For now, this fix works in all cases, and I'm able to build and boot with the 3.10 kernel and matching headers. Given that we are about to hit the M4 freeze, I'd prefer this in now, and the apps fixed later. We need to get mileage with the matching headers and kernel. I think this is a good compromise for the time being, we've had far nastier hacks in the libc-headers in the past. It would make it harder to fix this problem in the app if we work is around this way :) while I see the your urgency of making 3.10 defaults. I don't think we should rush it but thats just me Bruce -- Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 4/7] eglibc: ptrace: protect ptrace_peeksiginfo_args from redefintion
On Mon, Aug 26, 2013 at 5:16 PM, Khem Raj raj.k...@gmail.com wrote: On Aug 26, 2013, at 10:59 AM, Bruce Ashfield bruce.ashfi...@gmail.com wrote: On Mon, Aug 26, 2013 at 1:28 PM, Khem Raj raj.k...@gmail.com wrote: On Aug 25, 2013, at 7:46 PM, Bruce Ashfield bruce.ashfi...@gmail.com wrote: A final (maybe .. hopefully!?) thought on this. Do drop this change, but give me a few minutes to remove the offending structure from linux-libc-headers_3.10. That will also protect all users of yocto, regardless of the kernel unless they provide their own libc-headers .. and then they are on their own :) That leaves the c library alone, and should buy us time to fix the offending apps. this is not again the correct place to fix it. since you will back this out say later when all apps are fixed anyway. If we need to keep it I would rather have the app builds failing and keep it enabled and it should not be the guy who is doing the kernel headers upgrade to fix the whole ecosystem either. So add the headers by all means and leave the old headers in there until the apps work correctly with new headers. Whether or not to allow the apps to break, that's a call for Saul and Richard. For now, this fix works in all cases, and I'm able to build and boot with the 3.10 kernel and matching headers. Given that we are about to hit the M4 freeze, I'd prefer this in now, and the apps fixed later. We need to get mileage with the matching headers and kernel. I think this is a good compromise for the time being, we've had far nastier hacks in the libc-headers in the past. It would make it harder to fix this problem in the app if we work is around this way :) while I see the your urgency of making 3.10 defaults. I don't think we should rush it but thats just me Typically I just ask for bugzilla entries to track the fixes in a situation like this. At a high enough priority that they block a release. That gets everyone's attention :) Bruce Bruce -- Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end -- Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 4/7] eglibc: ptrace: protect ptrace_peeksiginfo_args from redefintion
On 25/08/2013 3:16 PM, Bruce Ashfield wrote: linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] added ptrace_peeksiginfo_args to the exported kernel user api. This leads to build errors such as: | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: | /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' | struct ptrace_peeksiginfo_args { | ^ | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, | from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: | /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here | struct ptrace_peeksiginfo_args | ^ | make[2]: *** [process.o] Error 1 In files that include both the eglibc sys/ptrace.h and the linux kernel exported one. We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO has not been defined. Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com --- ...ect-ptrace_peeksiginfo_args-from-redefint.patch | 55 meta/recipes-core/eglibc/eglibc_2.18.bb|1 + 2 files changed, 56 insertions(+) create mode 100644 meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch diff --git a/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch new file mode 100644 index 000..edf10fb --- /dev/null +++ b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch @@ -0,0 +1,55 @@ +From b5fe604678ffc7e0d6c2704a73d22f99b24a6d25 Mon Sep 17 00:00:00 2001 +From: Bruce Ashfield bruce.ashfi...@windriver.com +Date: Sat, 24 Aug 2013 23:51:06 -0400 +Subject: [PATCH] ptrace: protect ptrace_peeksiginfo_args from redefintion + +linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] +added ptrace_peeksiginfo_args to the exported kernel user api. + +This leads to build errors such as: + +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' +| struct ptrace_peeksiginfo_args { +| ^ +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, +| from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here +| struct ptrace_peeksiginfo_args +| ^ +| make[2]: *** [process.o] Error 1 + +In files that include both the eglibc sys/ptrace.h and the linux kernel exported +one. + +We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO +has not been defined. Missing Upstream-Status. Regards, Jonathan + +Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com +--- + libc/sysdeps/unix/sysv/linux/sys/ptrace.h |2 ++ + 1 file changed, 2 insertions(+) + +diff --git a/sysdeps/unix/sysv/linux/sys/ptrace.h b/sysdeps/unix/sysv/linux/sys/ptrace.h +index 2b78565..7800d9a 100644 +--- a/sysdeps/unix/sysv/linux/sys/ptrace.h b/sysdeps/unix/sysv/linux/sys/ptrace.h +@@ -187,6 +187,7 @@ enum __ptrace_eventcodes + PTRAVE_EVENT_SECCOMP = 7 + }; + ++#ifndef PTRACE_PEEKSIGINFO + /* Arguments for PTRACE_PEEKSIGINFO. */ + struct ptrace_peeksiginfo_args + { +@@ -194,6 +195,7 @@ struct ptrace_peeksiginfo_args + __uint32_t flags; /* Flags for peeksiginfo. */ + __int32_t nr; /* How many siginfos to take. */ + }; ++#endif + + enum __ptrace_peeksiginfo_flags + { +-- +1.7.10.4 + diff --git a/meta/recipes-core/eglibc/eglibc_2.18.bb b/meta/recipes-core/eglibc/eglibc_2.18.bb index 17b651f..2257c21 100644 --- a/meta/recipes-core/eglibc/eglibc_2.18.bb +++ b/meta/recipes-core/eglibc/eglibc_2.18.bb @@ -26,6 +26,7 @@ SRC_URI = http://downloads.yoctoproject.org/releases/eglibc/eglibc-${PV}-svnr23 file://tzselect-awk.patch \ file://0001-eglibc-run-libm-err-tab.pl-with-specific-dirs-in-S.patch \ file://fix-tibetian-locales.patch \ + file://0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch \ SRC_URI[md5sum] = b395b021422a027d89884992e91734fc SRC_URI[sha256sum] = 15f564b45dc5dd65faf0875579e3447961ae61e876933384ae05d19328539ad4 ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org
Re: [OE-core] [PATCH 4/7] eglibc: ptrace: protect ptrace_peeksiginfo_args from redefintion
On Sun, Aug 25, 2013 at 2:35 AM, Jonathan Liu net...@gmail.com wrote: On 25/08/2013 3:16 PM, Bruce Ashfield wrote: linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] added ptrace_peeksiginfo_args to the exported kernel user api. This leads to build errors such as: | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: | /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' | struct ptrace_peeksiginfo_args { | ^ | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, | from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: | /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here | struct ptrace_peeksiginfo_args | ^ | make[2]: *** [process.o] Error 1 In files that include both the eglibc sys/ptrace.h and the linux kernel exported one. We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO has not been defined. Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com --- ...ect-ptrace_peeksiginfo_args-from-redefint.patch | 55 meta/recipes-core/eglibc/eglibc_2.18.bb|1 + 2 files changed, 56 insertions(+) create mode 100644 meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch diff --git a/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch new file mode 100644 index 000..edf10fb --- /dev/null +++ b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch @@ -0,0 +1,55 @@ +From b5fe604678ffc7e0d6c2704a73d22f99b24a6d25 Mon Sep 17 00:00:00 2001 +From: Bruce Ashfield bruce.ashfi...@windriver.com +Date: Sat, 24 Aug 2013 23:51:06 -0400 +Subject: [PATCH] ptrace: protect ptrace_peeksiginfo_args from redefintion + +linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] +added ptrace_peeksiginfo_args to the exported kernel user api. + +This leads to build errors such as: + +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' +| struct ptrace_peeksiginfo_args { +| ^ +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, +| from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here +| struct ptrace_peeksiginfo_args +| ^ +| make[2]: *** [process.o] Error 1 + +In files that include both the eglibc sys/ptrace.h and the linux kernel exported +one. + +We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO +has not been defined. Missing Upstream-Status. Thanks for the double check, but I'm not the one to tag this with an upstream status. My opinion on the applicability of this to eglibc is largely irrelevant. Cheers, Bruce Regards, Jonathan + +Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com +--- + libc/sysdeps/unix/sysv/linux/sys/ptrace.h |2 ++ + 1 file changed, 2 insertions(+) + +diff --git a/sysdeps/unix/sysv/linux/sys/ptrace.h b/sysdeps/unix/sysv/linux/sys/ptrace.h +index 2b78565..7800d9a 100644 +--- a/sysdeps/unix/sysv/linux/sys/ptrace.h b/sysdeps/unix/sysv/linux/sys/ptrace.h +@@ -187,6 +187,7 @@ enum __ptrace_eventcodes + PTRAVE_EVENT_SECCOMP = 7 + }; + ++#ifndef PTRACE_PEEKSIGINFO + /* Arguments for PTRACE_PEEKSIGINFO. */ + struct ptrace_peeksiginfo_args + { +@@ -194,6 +195,7 @@ struct ptrace_peeksiginfo_args + __uint32_t flags; /* Flags for peeksiginfo. */ + __int32_t nr; /* How many siginfos to take. */ + }; ++#endif + + enum __ptrace_peeksiginfo_flags + { +-- +1.7.10.4 + diff --git a/meta/recipes-core/eglibc/eglibc_2.18.bb b/meta/recipes-core/eglibc/eglibc_2.18.bb index 17b651f..2257c21 100644 --- a/meta/recipes-core/eglibc/eglibc_2.18.bb +++ b/meta/recipes-core/eglibc/eglibc_2.18.bb @@ -26,6 +26,7 @@ SRC_URI = http://downloads.yoctoproject.org/releases/eglibc/eglibc-${PV}-svnr23 file://tzselect-awk.patch \ file://0001-eglibc-run-libm-err-tab.pl-with-specific-dirs-in-S.patch \ file://fix-tibetian-locales.patch \ +
Re: [OE-core] [PATCH 4/7] eglibc: ptrace: protect ptrace_peeksiginfo_args from redefintion
Hi Bruce On Sat, Aug 24, 2013 at 10:16 PM, Bruce Ashfield bruce.ashfi...@windriver.com wrote: linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] added ptrace_peeksiginfo_args to the exported kernel user api. This leads to build errors such as: | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: | /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' | struct ptrace_peeksiginfo_args { | ^ | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, | from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: | /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here | struct ptrace_peeksiginfo_args | ^ | make[2]: *** [process.o] Error 1 In files that include both the eglibc sys/ptrace.h and the linux kernel exported one. We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO has not been defined. The problem is there however we need not fix it in eglibc thats wrong place to fix it. The right place to fix it is affected application which seems so mix glibc and kernel interfaces. It should try to not mix kernel and glibc interfaces. This patch as such is not appropriate. Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com --- ...ect-ptrace_peeksiginfo_args-from-redefint.patch | 55 meta/recipes-core/eglibc/eglibc_2.18.bb|1 + 2 files changed, 56 insertions(+) create mode 100644 meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch diff --git a/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch new file mode 100644 index 000..edf10fb --- /dev/null +++ b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch @@ -0,0 +1,55 @@ +From b5fe604678ffc7e0d6c2704a73d22f99b24a6d25 Mon Sep 17 00:00:00 2001 +From: Bruce Ashfield bruce.ashfi...@windriver.com +Date: Sat, 24 Aug 2013 23:51:06 -0400 +Subject: [PATCH] ptrace: protect ptrace_peeksiginfo_args from redefintion + +linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] +added ptrace_peeksiginfo_args to the exported kernel user api. + +This leads to build errors such as: + +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' +| struct ptrace_peeksiginfo_args { +| ^ +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, +| from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here +| struct ptrace_peeksiginfo_args +| ^ +| make[2]: *** [process.o] Error 1 + +In files that include both the eglibc sys/ptrace.h and the linux kernel exported +one. + +We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO +has not been defined. + +Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com +--- + libc/sysdeps/unix/sysv/linux/sys/ptrace.h |2 ++ + 1 file changed, 2 insertions(+) + +diff --git a/sysdeps/unix/sysv/linux/sys/ptrace.h b/sysdeps/unix/sysv/linux/sys/ptrace.h +index 2b78565..7800d9a 100644 +--- a/sysdeps/unix/sysv/linux/sys/ptrace.h b/sysdeps/unix/sysv/linux/sys/ptrace.h +@@ -187,6 +187,7 @@ enum __ptrace_eventcodes + PTRAVE_EVENT_SECCOMP = 7 + }; + ++#ifndef PTRACE_PEEKSIGINFO + /* Arguments for PTRACE_PEEKSIGINFO. */ + struct ptrace_peeksiginfo_args + { +@@ -194,6 +195,7 @@ struct ptrace_peeksiginfo_args + __uint32_t flags; /* Flags for peeksiginfo. */ + __int32_t nr; /* How many siginfos to take. */ + }; ++#endif + + enum __ptrace_peeksiginfo_flags + { +-- +1.7.10.4 + diff --git a/meta/recipes-core/eglibc/eglibc_2.18.bb b/meta/recipes-core/eglibc/eglibc_2.18.bb index 17b651f..2257c21 100644 --- a/meta/recipes-core/eglibc/eglibc_2.18.bb +++ b/meta/recipes-core/eglibc/eglibc_2.18.bb @@ -26,6 +26,7 @@ SRC_URI = http://downloads.yoctoproject.org/releases/eglibc/eglibc-${PV}-svnr23 file://tzselect-awk.patch \ file://0001-eglibc-run-libm-err-tab.pl-with-specific-dirs-in-S.patch \
Re: [OE-core] [PATCH 4/7] eglibc: ptrace: protect ptrace_peeksiginfo_args from redefintion
On 13-08-25 8:46 PM, Khem Raj wrote: Hi Bruce On Sat, Aug 24, 2013 at 10:16 PM, Bruce Ashfield bruce.ashfi...@windriver.com wrote: linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] added ptrace_peeksiginfo_args to the exported kernel user api. This leads to build errors such as: | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: | /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' | struct ptrace_peeksiginfo_args { | ^ | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, | from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: | /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here | struct ptrace_peeksiginfo_args | ^ | make[2]: *** [process.o] Error 1 In files that include both the eglibc sys/ptrace.h and the linux kernel exported one. We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO has not been defined. The problem is there however we need not fix it in eglibc thats wrong place to fix it. The right place to fix it is affected application which seems so mix glibc and kernel interfaces. It should try to not mix kernel and glibc interfaces. This patch as such is not appropriate. My insistence is that this go in in the mean time, otherwise the 3.10 kernel headers miss the M4 cutoff and basically the whole release. I'm certainly not going to have the time and cycles to fix individual applications, so our choices boil down to taking this change, rolling back to the 3.8 headers, or waiting for individual package maintainers to update their packages to work with the 3.10 headers. I personally don't see the harm in this change, since these applications have always had this mix of both sys/ptrace.h and linux/ptrace.h in the past, now doesn't seem like the time to fix them. Cheers, Bruce Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com --- ...ect-ptrace_peeksiginfo_args-from-redefint.patch | 55 meta/recipes-core/eglibc/eglibc_2.18.bb|1 + 2 files changed, 56 insertions(+) create mode 100644 meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch diff --git a/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch new file mode 100644 index 000..edf10fb --- /dev/null +++ b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch @@ -0,0 +1,55 @@ +From b5fe604678ffc7e0d6c2704a73d22f99b24a6d25 Mon Sep 17 00:00:00 2001 +From: Bruce Ashfield bruce.ashfi...@windriver.com +Date: Sat, 24 Aug 2013 23:51:06 -0400 +Subject: [PATCH] ptrace: protect ptrace_peeksiginfo_args from redefintion + +linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] +added ptrace_peeksiginfo_args to the exported kernel user api. + +This leads to build errors such as: + +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' +| struct ptrace_peeksiginfo_args { +| ^ +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, +| from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here +| struct ptrace_peeksiginfo_args +| ^ +| make[2]: *** [process.o] Error 1 + +In files that include both the eglibc sys/ptrace.h and the linux kernel exported +one. + +We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO +has not been defined. + +Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com +--- + libc/sysdeps/unix/sysv/linux/sys/ptrace.h |2 ++ + 1 file changed, 2 insertions(+) + +diff --git a/sysdeps/unix/sysv/linux/sys/ptrace.h b/sysdeps/unix/sysv/linux/sys/ptrace.h +index 2b78565..7800d9a 100644 +--- a/sysdeps/unix/sysv/linux/sys/ptrace.h b/sysdeps/unix/sysv/linux/sys/ptrace.h +@@ -187,6 +187,7 @@ enum __ptrace_eventcodes + PTRAVE_EVENT_SECCOMP = 7 + }; + ++#ifndef PTRACE_PEEKSIGINFO + /* Arguments for PTRACE_PEEKSIGINFO. */ + struct ptrace_peeksiginfo_args + { +@@ -194,6 +195,7 @@ struct ptrace_peeksiginfo_args + __uint32_t flags; /* Flags for peeksiginfo. */ + __int32_t nr; /* How many
Re: [OE-core] [PATCH 4/7] eglibc: ptrace: protect ptrace_peeksiginfo_args from redefintion
On 13-08-25 8:54 PM, Bruce Ashfield wrote: On 13-08-25 8:46 PM, Khem Raj wrote: Hi Bruce On Sat, Aug 24, 2013 at 10:16 PM, Bruce Ashfield bruce.ashfi...@windriver.com wrote: linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] added ptrace_peeksiginfo_args to the exported kernel user api. This leads to build errors such as: | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: | /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' | struct ptrace_peeksiginfo_args { | ^ | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, | from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: | /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here | struct ptrace_peeksiginfo_args | ^ | make[2]: *** [process.o] Error 1 In files that include both the eglibc sys/ptrace.h and the linux kernel exported one. We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO has not been defined. The problem is there however we need not fix it in eglibc thats wrong place to fix it. The right place to fix it is affected application which seems so mix glibc and kernel interfaces. It should try to not mix kernel and glibc interfaces. This patch as such is not appropriate. My insistence is that this go in in the mean time, otherwise the 3.10 s/insistence/suggestion/ That came out a bit to strong! :) Bruce kernel headers miss the M4 cutoff and basically the whole release. I'm certainly not going to have the time and cycles to fix individual applications, so our choices boil down to taking this change, rolling back to the 3.8 headers, or waiting for individual package maintainers to update their packages to work with the 3.10 headers. I personally don't see the harm in this change, since these applications have always had this mix of both sys/ptrace.h and linux/ptrace.h in the past, now doesn't seem like the time to fix them. Cheers, Bruce Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com --- ...ect-ptrace_peeksiginfo_args-from-redefint.patch | 55 meta/recipes-core/eglibc/eglibc_2.18.bb|1 + 2 files changed, 56 insertions(+) create mode 100644 meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch diff --git a/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch new file mode 100644 index 000..edf10fb --- /dev/null +++ b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch @@ -0,0 +1,55 @@ +From b5fe604678ffc7e0d6c2704a73d22f99b24a6d25 Mon Sep 17 00:00:00 2001 +From: Bruce Ashfield bruce.ashfi...@windriver.com +Date: Sat, 24 Aug 2013 23:51:06 -0400 +Subject: [PATCH] ptrace: protect ptrace_peeksiginfo_args from redefintion + +linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] +added ptrace_peeksiginfo_args to the exported kernel user api. + +This leads to build errors such as: + +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' +| struct ptrace_peeksiginfo_args { +| ^ +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, +| from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here +| struct ptrace_peeksiginfo_args +| ^ +| make[2]: *** [process.o] Error 1 + +In files that include both the eglibc sys/ptrace.h and the linux kernel exported +one. + +We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO +has not been defined. + +Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com +--- + libc/sysdeps/unix/sysv/linux/sys/ptrace.h |2 ++ + 1 file changed, 2 insertions(+) + +diff --git a/sysdeps/unix/sysv/linux/sys/ptrace.h b/sysdeps/unix/sysv/linux/sys/ptrace.h +index 2b78565..7800d9a 100644 +--- a/sysdeps/unix/sysv/linux/sys/ptrace.h b/sysdeps/unix/sysv/linux/sys/ptrace.h +@@ -187,6 +187,7 @@ enum __ptrace_eventcodes + PTRAVE_EVENT_SECCOMP = 7 + }; + ++#ifndef PTRACE_PEEKSIGINFO + /* Arguments for PTRACE_PEEKSIGINFO. */ + struct ptrace_peeksiginfo_args + { +@@ -194,6 +195,7 @@ struct ptrace_peeksiginfo_args
Re: [OE-core] [PATCH 4/7] eglibc: ptrace: protect ptrace_peeksiginfo_args from redefintion
On Sun, Aug 25, 2013 at 10:14 PM, Bruce Ashfield bruce.ashfi...@windriver.com wrote: On 13-08-25 8:54 PM, Bruce Ashfield wrote: On 13-08-25 8:46 PM, Khem Raj wrote: Hi Bruce On Sat, Aug 24, 2013 at 10:16 PM, Bruce Ashfield bruce.ashfi...@windriver.com wrote: linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] added ptrace_peeksiginfo_args to the exported kernel user api. This leads to build errors such as: | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: | /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' | struct ptrace_peeksiginfo_args { | ^ | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, | from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: | /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here | struct ptrace_peeksiginfo_args | ^ | make[2]: *** [process.o] Error 1 In files that include both the eglibc sys/ptrace.h and the linux kernel exported one. We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO has not been defined. The problem is there however we need not fix it in eglibc thats wrong place to fix it. The right place to fix it is affected application which seems so mix glibc and kernel interfaces. It should try to not mix kernel and glibc interfaces. This patch as such is not appropriate. My insistence is that this go in in the mean time, otherwise the 3.10 s/insistence/suggestion/ That came out a bit to strong! :) ... and I should point out that the fix I have in this patch also requires a particular include order. So by all definitions .. it is temporary! Bruce Bruce kernel headers miss the M4 cutoff and basically the whole release. I'm certainly not going to have the time and cycles to fix individual applications, so our choices boil down to taking this change, rolling back to the 3.8 headers, or waiting for individual package maintainers to update their packages to work with the 3.10 headers. I personally don't see the harm in this change, since these applications have always had this mix of both sys/ptrace.h and linux/ptrace.h in the past, now doesn't seem like the time to fix them. Cheers, Bruce Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com --- ...ect-ptrace_peeksiginfo_args-from-redefint.patch | 55 meta/recipes-core/eglibc/eglibc_2.18.bb|1 + 2 files changed, 56 insertions(+) create mode 100644 meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch diff --git a/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch new file mode 100644 index 000..edf10fb --- /dev/null +++ b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch @@ -0,0 +1,55 @@ +From b5fe604678ffc7e0d6c2704a73d22f99b24a6d25 Mon Sep 17 00:00:00 2001 +From: Bruce Ashfield bruce.ashfi...@windriver.com +Date: Sat, 24 Aug 2013 23:51:06 -0400 +Subject: [PATCH] ptrace: protect ptrace_peeksiginfo_args from redefintion + +linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] +added ptrace_peeksiginfo_args to the exported kernel user api. + +This leads to build errors such as: + +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' +| struct ptrace_peeksiginfo_args { +| ^ +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, +| from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here +| struct ptrace_peeksiginfo_args +| ^ +| make[2]: *** [process.o] Error 1 + +In files that include both the eglibc sys/ptrace.h and the linux kernel exported +one. + +We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO +has not been defined. + +Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com +--- + libc/sysdeps/unix/sysv/linux/sys/ptrace.h |2 ++ + 1 file changed, 2 insertions(+) + +diff --git a/sysdeps/unix/sysv/linux/sys/ptrace.h b/sysdeps/unix/sysv/linux/sys/ptrace.h
Re: [OE-core] [PATCH 4/7] eglibc: ptrace: protect ptrace_peeksiginfo_args from redefintion
On Sun, Aug 25, 2013 at 10:39 PM, Bruce Ashfield bruce.ashfi...@gmail.com wrote: On Sun, Aug 25, 2013 at 10:14 PM, Bruce Ashfield bruce.ashfi...@windriver.com wrote: On 13-08-25 8:54 PM, Bruce Ashfield wrote: On 13-08-25 8:46 PM, Khem Raj wrote: Hi Bruce On Sat, Aug 24, 2013 at 10:16 PM, Bruce Ashfield bruce.ashfi...@windriver.com wrote: linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] added ptrace_peeksiginfo_args to the exported kernel user api. This leads to build errors such as: | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: | /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' | struct ptrace_peeksiginfo_args { | ^ | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, | from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: | /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here | struct ptrace_peeksiginfo_args | ^ | make[2]: *** [process.o] Error 1 In files that include both the eglibc sys/ptrace.h and the linux kernel exported one. We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO has not been defined. The problem is there however we need not fix it in eglibc thats wrong place to fix it. The right place to fix it is affected application which seems so mix glibc and kernel interfaces. It should try to not mix kernel and glibc interfaces. This patch as such is not appropriate. My insistence is that this go in in the mean time, otherwise the 3.10 s/insistence/suggestion/ That came out a bit to strong! :) ... and I should point out that the fix I have in this patch also requires a particular include order. So by all definitions .. it is temporary! A final (maybe .. hopefully!?) thought on this. Do drop this change, but give me a few minutes to remove the offending structure from linux-libc-headers_3.10. That will also protect all users of yocto, regardless of the kernel unless they provide their own libc-headers .. and then they are on their own :) That leaves the c library alone, and should buy us time to fix the offending apps. Bruce Bruce Bruce kernel headers miss the M4 cutoff and basically the whole release. I'm certainly not going to have the time and cycles to fix individual applications, so our choices boil down to taking this change, rolling back to the 3.8 headers, or waiting for individual package maintainers to update their packages to work with the 3.10 headers. I personally don't see the harm in this change, since these applications have always had this mix of both sys/ptrace.h and linux/ptrace.h in the past, now doesn't seem like the time to fix them. Cheers, Bruce Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com --- ...ect-ptrace_peeksiginfo_args-from-redefint.patch | 55 meta/recipes-core/eglibc/eglibc_2.18.bb|1 + 2 files changed, 56 insertions(+) create mode 100644 meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch diff --git a/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch new file mode 100644 index 000..edf10fb --- /dev/null +++ b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch @@ -0,0 +1,55 @@ +From b5fe604678ffc7e0d6c2704a73d22f99b24a6d25 Mon Sep 17 00:00:00 2001 +From: Bruce Ashfield bruce.ashfi...@windriver.com +Date: Sat, 24 Aug 2013 23:51:06 -0400 +Subject: [PATCH] ptrace: protect ptrace_peeksiginfo_args from redefintion + +linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] +added ptrace_peeksiginfo_args to the exported kernel user api. + +This leads to build errors such as: + +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' +| struct ptrace_peeksiginfo_args { +| ^ +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, +| from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here +| struct ptrace_peeksiginfo_args +| ^ +| make[2]: ***
[OE-core] [PATCH 4/7] eglibc: ptrace: protect ptrace_peeksiginfo_args from redefintion
linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] added ptrace_peeksiginfo_args to the exported kernel user api. This leads to build errors such as: | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: | /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' | struct ptrace_peeksiginfo_args { | ^ | In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, | from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: | /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here | struct ptrace_peeksiginfo_args | ^ | make[2]: *** [process.o] Error 1 In files that include both the eglibc sys/ptrace.h and the linux kernel exported one. We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO has not been defined. Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com --- ...ect-ptrace_peeksiginfo_args-from-redefint.patch | 55 meta/recipes-core/eglibc/eglibc_2.18.bb|1 + 2 files changed, 56 insertions(+) create mode 100644 meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch diff --git a/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch new file mode 100644 index 000..edf10fb --- /dev/null +++ b/meta/recipes-core/eglibc/eglibc-2.18/0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch @@ -0,0 +1,55 @@ +From b5fe604678ffc7e0d6c2704a73d22f99b24a6d25 Mon Sep 17 00:00:00 2001 +From: Bruce Ashfield bruce.ashfi...@windriver.com +Date: Sat, 24 Aug 2013 23:51:06 -0400 +Subject: [PATCH] ptrace: protect ptrace_peeksiginfo_args from redefintion + +linux kernel commit 84c751bd [ptrace: add ability to retrieve signals without removing from a queue (v4)] +added ptrace_peeksiginfo_args to the exported kernel user api. + +This leads to build errors such as: + +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:66:0: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/linux/ptrace.h:58:8: error: redefinition of 'struct ptrace_peeksiginfo_args' +| struct ptrace_peeksiginfo_args { +| ^ +| In file included from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/defs.h:159:0, +| from /poky-master/build/tmp/work/i586-poky-linux/strace/4.8-r0/strace-4.8/process.c:37: +| /poky-master/build/tmp/sysroots/qemux86/usr/include/sys/ptrace.h:191:8: note: originally defined here +| struct ptrace_peeksiginfo_args +| ^ +| make[2]: *** [process.o] Error 1 + +In files that include both the eglibc sys/ptrace.h and the linux kernel exported +one. + +We can avoid this by making the eglibc variant only declare its structure if PTRACE_PEEKSIGINFO +has not been defined. + +Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com +--- + libc/sysdeps/unix/sysv/linux/sys/ptrace.h |2 ++ + 1 file changed, 2 insertions(+) + +diff --git a/sysdeps/unix/sysv/linux/sys/ptrace.h b/sysdeps/unix/sysv/linux/sys/ptrace.h +index 2b78565..7800d9a 100644 +--- a/sysdeps/unix/sysv/linux/sys/ptrace.h b/sysdeps/unix/sysv/linux/sys/ptrace.h +@@ -187,6 +187,7 @@ enum __ptrace_eventcodes + PTRAVE_EVENT_SECCOMP = 7 + }; + ++#ifndef PTRACE_PEEKSIGINFO + /* Arguments for PTRACE_PEEKSIGINFO. */ + struct ptrace_peeksiginfo_args + { +@@ -194,6 +195,7 @@ struct ptrace_peeksiginfo_args + __uint32_t flags; /* Flags for peeksiginfo. */ + __int32_t nr; /* How many siginfos to take. */ + }; ++#endif + + enum __ptrace_peeksiginfo_flags + { +-- +1.7.10.4 + diff --git a/meta/recipes-core/eglibc/eglibc_2.18.bb b/meta/recipes-core/eglibc/eglibc_2.18.bb index 17b651f..2257c21 100644 --- a/meta/recipes-core/eglibc/eglibc_2.18.bb +++ b/meta/recipes-core/eglibc/eglibc_2.18.bb @@ -26,6 +26,7 @@ SRC_URI = http://downloads.yoctoproject.org/releases/eglibc/eglibc-${PV}-svnr23 file://tzselect-awk.patch \ file://0001-eglibc-run-libm-err-tab.pl-with-specific-dirs-in-S.patch \ file://fix-tibetian-locales.patch \ + file://0001-ptrace-protect-ptrace_peeksiginfo_args-from-redefint.patch \ SRC_URI[md5sum] = b395b021422a027d89884992e91734fc SRC_URI[sha256sum] = 15f564b45dc5dd65faf0875579e3447961ae61e876933384ae05d19328539ad4 -- 1.7.10.4 ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core