Re: [PATCH 0/3] [GIT PULL][v2.6.38] tracing: fix unaligned eventarrays

From: Ingo Molnar
Date: Wed Jan 26 2011 - 15:59:18 EST



* Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> On Wed, 2011-01-26 at 07:47 +0100, Ingo Molnar wrote:
>
> > > Mathieu Desnoyers (3):
> > > Introduce __u64_aligned and U64_ALIGN() for structure alignment in custom sections
> > > tracing: Fix sparc64 alignment crash with __u64_aligned/U64_ALIGN()
> > > tracepoints: Use __u64_aligned/U64_ALIGN()
> > >
>
> > Pulled, thanks a lot Steve!
> >
> > Ingo
>
>
> Hi Ingo,
>
> We (well Peter really) just discovered that this can cause alignment
> issues with gcc 4.4.4 and 4.4.5 (I unfortunately only tested it with gcc
> 4.5.1). We are currently looking into fixing it.
>
> Letting you know so that you don't push this off to Linus ;-)

Well, i found and reported that 'perf stat -e irq:irq_handler_entry -a sleep 1'
crash to Peter (i first thought it's due to his patch) so yes i know about it :-)

Thanks,

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/