Patchwork [1/3] strace: add patch allowing compilation with kernel headers >= 2.6.32

login
register
mail settings
Submitter Eric BENARD
Date Jan. 12, 2011, 9:20 p.m.
Message ID <1294867230-7827-1-git-send-email-eric@eukrea.com>
Download mbox | patch
Permalink /patch/262/
State Superseded
Headers show

Comments

Eric BENARD - Jan. 12, 2011, 9:20 p.m.
tested with kernel headers 2.6.31 & 2.6.37 : both compile fine.

Signed-off-by: Eric Bénard <eric@eukrea.com>
---
 recipes/strace/strace_4.5.14.bb |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
Tom Rini - Jan. 13, 2011, 12:30 a.m.
On 01/12/2011 02:20 PM, Eric Bénard wrote:
> tested with kernel headers 2.6.31&  2.6.37 : both compile fine.

Is there a reason to not switch to 4.5.20, or even newer instead?

>
> Signed-off-by: Eric Bénard<eric@eukrea.com>
> ---
>   recipes/strace/strace_4.5.14.bb |    3 ++-
>   1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/recipes/strace/strace_4.5.14.bb b/recipes/strace/strace_4.5.14.bb
> index 33a44ad..395e4ee 100644
> --- a/recipes/strace/strace_4.5.14.bb
> +++ b/recipes/strace/strace_4.5.14.bb
> @@ -1,7 +1,7 @@
>   DESCRIPTION = "strace is a system call tracing tool."
>   SECTION = "console/utils"
>   LICENSE = "BSD"
> -PR = "r9"
> +PR = "r10"
>
>   SRC_URI = "${SOURCEFORGE_MIRROR}/strace/strace-${PV}.tar.bz2 \
>              file://glibc-2.5.patch \
> @@ -13,6 +13,7 @@ SRC_URI = "${SOURCEFORGE_MIRROR}/strace/strace-${PV}.tar.bz2 \
>              file://strace-arm-no-cachectl.patch \
>              file://strace-dont-include-linux-dirent-h.patch \
>              file://mips-sigcontext.patch \
> +           file://strace-linux-2.6.32.patch \
>             "
>
>   # The strace-4.5.14-avr32.patch conflicts with some other patches.
Khem Raj - Jan. 13, 2011, 5:49 a.m.
On 1/12/2011 1:20 PM, Eric Bénard wrote:
> tested with kernel headers 2.6.31&  2.6.37 : both compile fine.
>
> Signed-off-by: Eric Bénard<eric@eukrea.com>

I think you forgot to add the patch itself.

> ---
>   recipes/strace/strace_4.5.14.bb |    3 ++-
>   1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/recipes/strace/strace_4.5.14.bb b/recipes/strace/strace_4.5.14.bb
> index 33a44ad..395e4ee 100644
> --- a/recipes/strace/strace_4.5.14.bb
> +++ b/recipes/strace/strace_4.5.14.bb
> @@ -1,7 +1,7 @@
>   DESCRIPTION = "strace is a system call tracing tool."
>   SECTION = "console/utils"
>   LICENSE = "BSD"
> -PR = "r9"
> +PR = "r10"
>
>   SRC_URI = "${SOURCEFORGE_MIRROR}/strace/strace-${PV}.tar.bz2 \
>              file://glibc-2.5.patch \
> @@ -13,6 +13,7 @@ SRC_URI = "${SOURCEFORGE_MIRROR}/strace/strace-${PV}.tar.bz2 \
>              file://strace-arm-no-cachectl.patch \
>              file://strace-dont-include-linux-dirent-h.patch \
>              file://mips-sigcontext.patch \
> +           file://strace-linux-2.6.32.patch \
>             "
>
>   # The strace-4.5.14-avr32.patch conflicts with some other patches.
Eric BENARD - Jan. 13, 2011, 7:39 a.m.
On 13/01/2011 01:30, Tom Rini wrote:
> On 01/12/2011 02:20 PM, Eric Bénard wrote:
>> tested with kernel headers 2.6.31& 2.6.37 : both compile fine.
>
> Is there a reason to not switch to 4.5.20, or even newer instead?
>
I would like to do that but I don't know if 4.5.20 needs patches.

Eric
Eric BENARD - Jan. 13, 2011, 7:39 a.m.
On 13/01/2011 06:49, Khem Raj wrote:
> On 1/12/2011 1:20 PM, Eric Bénard wrote:
>> tested with kernel headers 2.6.31& 2.6.37 : both compile fine.
>>
>> Signed-off-by: Eric Bénard<eric@eukrea.com>
>
> I think you forgot to add the patch itself.
>
oops

Eric
Tom Rini - Jan. 13, 2011, 3:13 p.m.
On 01/13/2011 12:39 AM, Eric Bénard wrote:
> On 13/01/2011 01:30, Tom Rini wrote:
>> On 01/12/2011 02:20 PM, Eric Bénard wrote:
>>> tested with kernel headers 2.6.31& 2.6.37 : both compile fine.
>>
>> Is there a reason to not switch to 4.5.20, or even newer instead?
>>
> I would like to do that but I don't know if 4.5.20 needs patches.

I haven't tried .37, but it doesn't for .34 (when .14 does).  To be 
clear, I'm suggesting we look at removing the D_P = -1 from .20.  My 
assumption is that since .20 is the latest it probably doesn't need ARM 
EABI patches (and probably not the SH patches either).
Eric BENARD - Jan. 13, 2011, 3:17 p.m.
On 13/01/2011 16:13, Tom Rini wrote:
> On 01/13/2011 12:39 AM, Eric Bénard wrote:
>> On 13/01/2011 01:30, Tom Rini wrote:
>>> On 01/12/2011 02:20 PM, Eric Bénard wrote:
>>>> tested with kernel headers 2.6.31& 2.6.37 : both compile fine.
>>>
>>> Is there a reason to not switch to 4.5.20, or even newer instead?
>>>
>> I would like to do that but I don't know if 4.5.20 needs patches.
>
> I haven't tried .37, but it doesn't for .34 (when .14 does). To be clear, I'm
> suggesting we look at removing the D_P = -1 from .20. My assumption is that
> since .20 is the latest it probably doesn't need ARM EABI patches (and
> probably not the SH patches either).
>
OK lets go

Eric

Patch

diff --git a/recipes/strace/strace_4.5.14.bb b/recipes/strace/strace_4.5.14.bb
index 33a44ad..395e4ee 100644
--- a/recipes/strace/strace_4.5.14.bb
+++ b/recipes/strace/strace_4.5.14.bb
@@ -1,7 +1,7 @@ 
 DESCRIPTION = "strace is a system call tracing tool."
 SECTION = "console/utils"
 LICENSE = "BSD"
-PR = "r9"
+PR = "r10"
 
 SRC_URI = "${SOURCEFORGE_MIRROR}/strace/strace-${PV}.tar.bz2 \
            file://glibc-2.5.patch \
@@ -13,6 +13,7 @@  SRC_URI = "${SOURCEFORGE_MIRROR}/strace/strace-${PV}.tar.bz2 \
            file://strace-arm-no-cachectl.patch \
            file://strace-dont-include-linux-dirent-h.patch \
            file://mips-sigcontext.patch \
+           file://strace-linux-2.6.32.patch \
           "
 
 # The strace-4.5.14-avr32.patch conflicts with some other patches.