Re: [Xen-devel] [PATCH v2 1/2] net/xen-netfront: Correct printf format in xennet_get_responses

2015-06-05 Thread Joe Perches
On Fri, 2015-06-05 at 13:34 +0100, Julien Grall wrote: > On 04/06/15 17:25, Joe Perches wrote: > > On Thu, 2015-06-04 at 13:52 +0100, Julien Grall wrote: > >> On 04/06/15 13:46, David Vrabel wrote: > >>> On 04/06/15 13:45, Julien Grall wrote: > On 03/06/15 18:06, Joe Perches wrote: > > On

Re: [Xen-devel] [PATCH v2 1/2] net/xen-netfront: Correct printf format in xennet_get_responses

2015-06-05 Thread Julien Grall
On 04/06/15 17:25, Joe Perches wrote: > On Thu, 2015-06-04 at 13:52 +0100, Julien Grall wrote: >> On 04/06/15 13:46, David Vrabel wrote: >>> On 04/06/15 13:45, Julien Grall wrote: On 03/06/15 18:06, Joe Perches wrote: > On Wed, 2015-06-03 at 17:55 +0100, Julien Grall wrote: >> rx->stat

Re: [PATCH v2 1/2] net/xen-netfront: Correct printf format in xennet_get_responses

2015-06-04 Thread David Miller
From: Julien Grall Date: Thu, 4 Jun 2015 13:45:00 +0100 > On 03/06/15 18:06, Joe Perches wrote: >> On Wed, 2015-06-03 at 17:55 +0100, Julien Grall wrote: >>> rx->status is an int16_t, print it using %d rather than %u in order to >>> have a meaningful value when the field is negative. >> [] >>> di

Re: [Xen-devel] [PATCH v2 1/2] net/xen-netfront: Correct printf format in xennet_get_responses

2015-06-04 Thread Joe Perches
On Thu, 2015-06-04 at 13:52 +0100, Julien Grall wrote: > On 04/06/15 13:46, David Vrabel wrote: > > On 04/06/15 13:45, Julien Grall wrote: > >> On 03/06/15 18:06, Joe Perches wrote: > >>> On Wed, 2015-06-03 at 17:55 +0100, Julien Grall wrote: > rx->status is an int16_t, print it using %d rathe

Re: [Xen-devel] [PATCH v2 1/2] net/xen-netfront: Correct printf format in xennet_get_responses

2015-06-04 Thread Julien Grall
On 04/06/15 13:46, David Vrabel wrote: > On 04/06/15 13:45, Julien Grall wrote: >> On 03/06/15 18:06, Joe Perches wrote: >>> On Wed, 2015-06-03 at 17:55 +0100, Julien Grall wrote: rx->status is an int16_t, print it using %d rather than %u in order to have a meaningful value when the field

Re: [PATCH v2 1/2] net/xen-netfront: Correct printf format in xennet_get_responses

2015-06-04 Thread David Vrabel
On 04/06/15 13:45, Julien Grall wrote: > On 03/06/15 18:06, Joe Perches wrote: >> On Wed, 2015-06-03 at 17:55 +0100, Julien Grall wrote: >>> rx->status is an int16_t, print it using %d rather than %u in order to >>> have a meaningful value when the field is negative. >> [] >>> diff --git a/drivers/

Re: [PATCH v2 1/2] net/xen-netfront: Correct printf format in xennet_get_responses

2015-06-04 Thread Julien Grall
On 03/06/15 18:06, Joe Perches wrote: > On Wed, 2015-06-03 at 17:55 +0100, Julien Grall wrote: >> rx->status is an int16_t, print it using %d rather than %u in order to >> have a meaningful value when the field is negative. > [] >> diff --git a/drivers/net/xen-netfront.c b/drivers/net/xen-netfront.

Re: [PATCH v2 1/2] net/xen-netfront: Correct printf format in xennet_get_responses

2015-06-03 Thread Joe Perches
On Wed, 2015-06-03 at 17:55 +0100, Julien Grall wrote: > rx->status is an int16_t, print it using %d rather than %u in order to > have a meaningful value when the field is negative. [] > diff --git a/drivers/net/xen-netfront.c b/drivers/net/xen-netfront.c [] > @@ -733,7 +733,7 @@ static int xennet_

[PATCH v2 1/2] net/xen-netfront: Correct printf format in xennet_get_responses

2015-06-03 Thread Julien Grall
rx->status is an int16_t, print it using %d rather than %u in order to have a meaningful value when the field is negative. Signed-off-by: Julien Grall Reviewed-by: David Vrabel Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky Cc: netdev@vger.kernel.org --- Changes in v2: - Add David'