From: Julien Grall <julien.grall@citrix.com>
Date: Tue, 16 Jun 2015 19:10:46 +0000 (+0100)
Subject: net/xen-netfront: Correct printf format in xennet_get_responses
X-Git-Tag: v4.1.12-92~304^2^2^2~1
X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=932bec4491b1c498b24aeabcf1c8f9889e296f72;p=users%2Fjedix%2Flinux-maple.git

net/xen-netfront: Correct printf format in xennet_get_responses

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.

Also use %u rather than %x for rx->offset.

Signed-off-by: Julien Grall <julien.grall@citrix.com>
Reviewed-by: David Vrabel <david.vrabel@citrix.com>
Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Cc: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Cc: netdev@vger.kernel.org
Signed-off-by: David S. Miller <davem@davemloft.net>
(cherry picked from commit 6c10127d91bdc80e02938085d03c232ae3118ad5)
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
---

diff --git a/drivers/net/xen-netfront.c b/drivers/net/xen-netfront.c
index e2c23ab0cf65..b3e9491bc8ec 100644
--- a/drivers/net/xen-netfront.c
+++ b/drivers/net/xen-netfront.c
@@ -732,7 +732,7 @@ static int xennet_get_responses(struct netfront_queue *queue,
 		if (unlikely(rx->status < 0 ||
 			     rx->offset + rx->status > PAGE_SIZE)) {
 			if (net_ratelimit())
-				dev_warn(dev, "rx->offset: %x, size: %u\n",
+				dev_warn(dev, "rx->offset: %u, size: %d\n",
 					 rx->offset, rx->status);
 			xennet_move_rx_slot(queue, skb, ref);
 			err = -EINVAL;