Re: [PATCH 08/15] ftrace: fix event alignment: mce:mce_record

[Frederic Weisbecker]

On Fri, Dec 03, 2010 at 08:50:22PM -0500, Steven Rostedt wrote:
> [ Ccing Hidetoshi and Frederic for Acked-by's ]

No problem for me.

Thanks.

> -- Steve
> 
> 
> On Fri, 2010-12-03 at 16:13 -0800, David Sharp wrote:
> > Signed-off-by: David Sharp <dhsharp@xxxxxxxxxx>
> > ---
> >  include/trace/events/mce.h |    8 ++++----
> >  1 files changed, 4 insertions(+), 4 deletions(-)
> > 
> > diff --git a/include/trace/events/mce.h b/include/trace/events/mce.h
> > index 7eee778..4cbbcef 100644
> > --- a/include/trace/events/mce.h
> > +++ b/include/trace/events/mce.h
> > @@ -17,36 +17,36 @@ TRACE_EVENT(mce_record,
> >  	TP_STRUCT__entry(
> >  		__field(	u64,		mcgcap		)
> >  		__field(	u64,		mcgstatus	)
> > -		__field(	u8,		bank		)
> >  		__field(	u64,		status		)
> >  		__field(	u64,		addr		)
> >  		__field(	u64,		misc		)
> >  		__field(	u64,		ip		)
> > -		__field(	u8,		cs		)
> >  		__field(	u64,		tsc		)
> >  		__field(	u64,		walltime	)
> >  		__field(	u32,		cpu		)
> >  		__field(	u32,		cpuid		)
> >  		__field(	u32,		apicid		)
> >  		__field(	u32,		socketid	)
> > +		__field(	u8,		cs		)
> > +		__field(	u8,		bank		)
> >  		__field(	u8,		cpuvendor	)
> >  	),
> >  
> >  	TP_fast_assign(
> >  		__entry->mcgcap		= m->mcgcap;
> >  		__entry->mcgstatus	= m->mcgstatus;
> > -		__entry->bank		= m->bank;
> >  		__entry->status		= m->status;
> >  		__entry->addr		= m->addr;
> >  		__entry->misc		= m->misc;
> >  		__entry->ip		= m->ip;
> > -		__entry->cs		= m->cs;
> >  		__entry->tsc		= m->tsc;
> >  		__entry->walltime	= m->time;
> >  		__entry->cpu		= m->extcpu;
> >  		__entry->cpuid		= m->cpuid;
> >  		__entry->apicid		= m->apicid;
> >  		__entry->socketid	= m->socketid;
> > +		__entry->cs		= m->cs;
> > +		__entry->bank		= m->bank;
> >  		__entry->cpuvendor	= m->cpuvendor;
> >  	),
> >  
> 
> 
--
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/