Subject: Re: [PATCH 09/15] ftrace: fix event alignment: skb:kfree_skb
From: Steven Rostedt <rostedt@goodmis.org>
To: David Sharp <dhsharp@google.com>
Cc: linux-kernel@vger.kernel.org, mrubin@google.com,
        Neil Horman <nhorman@tuxdriver.com>
In-Reply-To: <1291421609-14665-10-git-send-email-dhsharp@google.com>
References: <1291421609-14665-1-git-send-email-dhsharp@google.com>
	 <1291421609-14665-10-git-send-email-dhsharp@google.com>
Content-Type: text/plain; charset="ISO-8859-15"
Date: Fri, 03 Dec 2010 20:52:05 -0500
Message-ID: <1291427525.16223.11.camel@gandalf.stny.rr.com>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1152
Lines: 41

[ Ccing Neil for Acked-by ]

-- Steve


On Fri, 2010-12-03 at 16:13 -0800, David Sharp wrote:
> Signed-off-by: David Sharp <dhsharp@google.com>
> ---
>  include/trace/events/skb.h |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/trace/events/skb.h b/include/trace/events/skb.h
> index 75ce9d5..aa4a56a 100644
> --- a/include/trace/events/skb.h
> +++ b/include/trace/events/skb.h
> @@ -19,16 +19,16 @@ TRACE_EVENT(kfree_skb,
>  
>  	TP_STRUCT__entry(
>  		__field(	void *,		skbaddr		)
> -		__field(	unsigned short,	protocol	)
>  		__field(	void *,		location	)
> +		__field(	unsigned short,	protocol	)
>  	),
>  
>  	TP_fast_assign(
>  		__entry->skbaddr = skb;
> +		__entry->location = location;
>  		if (skb) {
>  			__entry->protocol = ntohs(skb->protocol);
>  		}
> -		__entry->location = location;
>  	),
>  
>  	TP_printk("skbaddr=%p protocol=%u location=%p",


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