* [PATCH v4 1/3] net/xen-netfront: Correct printf format in xennet_get_responses
[not found] <1434481848-26387-1-git-send-email-julien.grall@citrix.com>
@ 2015-06-16 19:10 ` Julien Grall
2015-06-16 19:10 ` [PATCH v4 2/3] net/xen-netback: Remove unused code in xenvif_rx_action Julien Grall
` (4 subsequent siblings)
5 siblings, 0 replies; 15+ messages in thread
From: Julien Grall @ 2015-06-16 19:10 UTC (permalink / raw)
To: xen-devel
Cc: netdev, linux-kernel, Julien Grall, David Vrabel, Boris Ostrovsky
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
---
Changes in v4:
- Use %u for the rx->offset because offset is unsigned
Changes in v3:
- Use %d for the rx->offset too.
Changes in v2:
- Add David's Reviewed-by
---
drivers/net/xen-netfront.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/xen-netfront.c b/drivers/net/xen-netfront.c
index e031c94..281720f 100644
--- a/drivers/net/xen-netfront.c
+++ b/drivers/net/xen-netfront.c
@@ -733,7 +733,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;
--
2.1.4
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v4 2/3] net/xen-netback: Remove unused code in xenvif_rx_action
[not found] <1434481848-26387-1-git-send-email-julien.grall@citrix.com>
2015-06-16 19:10 ` [PATCH v4 1/3] net/xen-netfront: Correct printf format in xennet_get_responses Julien Grall
@ 2015-06-16 19:10 ` Julien Grall
2015-06-16 19:10 ` [PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format Julien Grall
` (3 subsequent siblings)
5 siblings, 0 replies; 15+ messages in thread
From: Julien Grall @ 2015-06-16 19:10 UTC (permalink / raw)
To: xen-devel; +Cc: Julien Grall, netdev, Wei Liu, linux-kernel, Ian Campbell
The variables old_req_cons and ring_slots_used are assigned but never
used since commit 1650d5455bd2dc6b5ee134bd6fc1a3236c266b5b "xen-netback:
always fully coalesce guest Rx packets".
Signed-off-by: Julien Grall <julien.grall@citrix.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
Cc: Ian Campbell <ian.campbell@citrix.com>
Cc: netdev@vger.kernel.org
---
Changes in v2:
- Add Wei's Acked-by
---
drivers/net/xen-netback/netback.c | 5 -----
1 file changed, 5 deletions(-)
diff --git a/drivers/net/xen-netback/netback.c b/drivers/net/xen-netback/netback.c
index 0d25943..ba3ae30 100644
--- a/drivers/net/xen-netback/netback.c
+++ b/drivers/net/xen-netback/netback.c
@@ -515,14 +515,9 @@ static void xenvif_rx_action(struct xenvif_queue *queue)
while (xenvif_rx_ring_slots_available(queue, XEN_NETBK_RX_SLOTS_MAX)
&& (skb = xenvif_rx_dequeue(queue)) != NULL) {
- RING_IDX old_req_cons;
- RING_IDX ring_slots_used;
-
queue->last_rx_time = jiffies;
- old_req_cons = queue->rx.req_cons;
XENVIF_RX_CB(skb)->meta_slots_used = xenvif_gop_skb(skb, &npo, queue);
- ring_slots_used = queue->rx.req_cons - old_req_cons;
__skb_queue_tail(&rxq, skb);
}
--
2.1.4
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format
[not found] <1434481848-26387-1-git-send-email-julien.grall@citrix.com>
2015-06-16 19:10 ` [PATCH v4 1/3] net/xen-netfront: Correct printf format in xennet_get_responses Julien Grall
2015-06-16 19:10 ` [PATCH v4 2/3] net/xen-netback: Remove unused code in xenvif_rx_action Julien Grall
@ 2015-06-16 19:10 ` Julien Grall
[not found] ` <1434481848-26387-4-git-send-email-julien.grall@citrix.com>
` (2 subsequent siblings)
5 siblings, 0 replies; 15+ messages in thread
From: Julien Grall @ 2015-06-16 19:10 UTC (permalink / raw)
To: xen-devel; +Cc: Julien Grall, netdev, Wei Liu, linux-kernel, Ian Campbell
Append 0x to all %x in order to avoid while reading when there is other
decimal value in the log.
Also replace some of the hexadecimal print to decimal to uniformize the
format with netfront.
Signed-off-by: Julien Grall <julien.grall@citrix.com>
Cc: Wei Liu <wei.liu2@citrix.com>
Cc: Ian Campbell <ian.campbell@citrix.com>
Cc: netdev@vger.kernel.org
---
Changes in v4:
- Patch added
---
drivers/net/xen-netback/netback.c | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/net/xen-netback/netback.c b/drivers/net/xen-netback/netback.c
index ba3ae30..11bd9d8 100644
--- a/drivers/net/xen-netback/netback.c
+++ b/drivers/net/xen-netback/netback.c
@@ -748,7 +748,7 @@ static int xenvif_count_requests(struct xenvif_queue *queue,
slots++;
if (unlikely((txp->offset + txp->size) > PAGE_SIZE)) {
- netdev_err(queue->vif->dev, "Cross page boundary, txp->offset: %x, size: %u\n",
+ netdev_err(queue->vif->dev, "Cross page boundary, txp->offset: %u, size: %u\n",
txp->offset, txp->size);
xenvif_fatal_tx_err(queue->vif);
return -EINVAL;
@@ -874,7 +874,7 @@ static inline void xenvif_grant_handle_set(struct xenvif_queue *queue,
if (unlikely(queue->grant_tx_handle[pending_idx] !=
NETBACK_INVALID_HANDLE)) {
netdev_err(queue->vif->dev,
- "Trying to overwrite active handle! pending_idx: %x\n",
+ "Trying to overwrite active handle! pending_idx: 0x%x\n",
pending_idx);
BUG();
}
@@ -887,7 +887,7 @@ static inline void xenvif_grant_handle_reset(struct xenvif_queue *queue,
if (unlikely(queue->grant_tx_handle[pending_idx] ==
NETBACK_INVALID_HANDLE)) {
netdev_err(queue->vif->dev,
- "Trying to unmap invalid handle! pending_idx: %x\n",
+ "Trying to unmap invalid handle! pending_idx: 0x%x\n",
pending_idx);
BUG();
}
@@ -1243,7 +1243,7 @@ static void xenvif_tx_build_gops(struct xenvif_queue *queue,
/* No crossing a page as the payload mustn't fragment. */
if (unlikely((txreq.offset + txreq.size) > PAGE_SIZE)) {
netdev_err(queue->vif->dev,
- "txreq.offset: %x, size: %u, end: %lu\n",
+ "txreq.offset: %u, size: %u, end: %lu\n",
txreq.offset, txreq.size,
(unsigned long)(txreq.offset&~PAGE_MASK) + txreq.size);
xenvif_fatal_tx_err(queue->vif);
@@ -1593,12 +1593,12 @@ static inline void xenvif_tx_dealloc_action(struct xenvif_queue *queue)
queue->pages_to_unmap,
gop - queue->tx_unmap_ops);
if (ret) {
- netdev_err(queue->vif->dev, "Unmap fail: nr_ops %tx ret %d\n",
+ netdev_err(queue->vif->dev, "Unmap fail: nr_ops %tu ret %d\n",
gop - queue->tx_unmap_ops, ret);
for (i = 0; i < gop - queue->tx_unmap_ops; ++i) {
if (gop[i].status != GNTST_okay)
netdev_err(queue->vif->dev,
- " host_addr: %llx handle: %x status: %d\n",
+ " host_addr: 0x%llx handle: 0x%x status: %d\n",
gop[i].host_addr,
gop[i].handle,
gop[i].status);
@@ -1731,7 +1731,7 @@ void xenvif_idx_unmap(struct xenvif_queue *queue, u16 pending_idx)
&queue->mmap_pages[pending_idx], 1);
if (ret) {
netdev_err(queue->vif->dev,
- "Unmap fail: ret: %d pending_idx: %d host_addr: %llx handle: %x status: %d\n",
+ "Unmap fail: ret: %d pending_idx: %d host_addr: %llx handle: 0x%x status: %d\n",
ret,
pending_idx,
tx_unmap_op.host_addr,
--
2.1.4
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format
[not found] ` <1434481848-26387-4-git-send-email-julien.grall@citrix.com>
@ 2015-06-16 20:07 ` Sergei Shtylyov
[not found] ` <5580821D.4080408@cogentembedded.com>
` (4 subsequent siblings)
5 siblings, 0 replies; 15+ messages in thread
From: Sergei Shtylyov @ 2015-06-16 20:07 UTC (permalink / raw)
To: Julien Grall, xen-devel; +Cc: netdev, Wei Liu, linux-kernel, Ian Campbell
Hello.
On 06/16/2015 10:10 PM, Julien Grall wrote:
> Append 0x to all %x in order to avoid while reading when there is other
> decimal value in the log.
> Also replace some of the hexadecimal print to decimal to uniformize the
> format with netfront.
> Signed-off-by: Julien Grall <julien.grall@citrix.com>
> Cc: Wei Liu <wei.liu2@citrix.com>
> Cc: Ian Campbell <ian.campbell@citrix.com>
> Cc: netdev@vger.kernel.org
> ---
> Changes in v4:
> - Patch added
> ---
> drivers/net/xen-netback/netback.c | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
> diff --git a/drivers/net/xen-netback/netback.c b/drivers/net/xen-netback/netback.c
> index ba3ae30..11bd9d8 100644
> --- a/drivers/net/xen-netback/netback.c
> +++ b/drivers/net/xen-netback/netback.c
[...]
> @@ -874,7 +874,7 @@ static inline void xenvif_grant_handle_set(struct xenvif_queue *queue,
> if (unlikely(queue->grant_tx_handle[pending_idx] !=
> NETBACK_INVALID_HANDLE)) {
> netdev_err(queue->vif->dev,
> - "Trying to overwrite active handle! pending_idx: %x\n",
> + "Trying to overwrite active handle! pending_idx: 0x%x\n",
Using "%#x" is shorter ind does the same.
> pending_idx);
> BUG();
> }
> @@ -887,7 +887,7 @@ static inline void xenvif_grant_handle_reset(struct xenvif_queue *queue,
> if (unlikely(queue->grant_tx_handle[pending_idx] ==
> NETBACK_INVALID_HANDLE)) {
> netdev_err(queue->vif->dev,
> - "Trying to unmap invalid handle! pending_idx: %x\n",
> + "Trying to unmap invalid handle! pending_idx: 0x%x\n",
Same here.
[...]
> @@ -1731,7 +1731,7 @@ void xenvif_idx_unmap(struct xenvif_queue *queue, u16 pending_idx)
> &queue->mmap_pages[pending_idx], 1);
> if (ret) {
> netdev_err(queue->vif->dev,
> - "Unmap fail: ret: %d pending_idx: %d host_addr: %llx handle: %x status: %d\n",
> + "Unmap fail: ret: %d pending_idx: %d host_addr: %llx handle: 0x%x status: %d\n",
And here.
[...]
WBR, Sergei
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format
[not found] ` <5580821D.4080408@cogentembedded.com>
@ 2015-06-16 22:09 ` Joe Perches
[not found] ` <1434492550.2689.8.camel@perches.com>
1 sibling, 0 replies; 15+ messages in thread
From: Joe Perches @ 2015-06-16 22:09 UTC (permalink / raw)
To: Sergei Shtylyov
Cc: Wei Liu, Ian Campbell, netdev, linux-kernel, Julien Grall, xen-devel
On Tue, 2015-06-16 at 23:07 +0300, Sergei Shtylyov wrote:
> On 06/16/2015 10:10 PM, Julien Grall wrote:
> > Append 0x to all %x in order to avoid while reading when there is other
> > decimal value in the log.
[]
> > @@ -874,7 +874,7 @@ static inline void xenvif_grant_handle_set(struct xenvif_queue *queue,
> > if (unlikely(queue->grant_tx_handle[pending_idx] !=
> > NETBACK_INVALID_HANDLE)) {
> > netdev_err(queue->vif->dev,
> > - "Trying to overwrite active handle! pending_idx: %x\n",
> > + "Trying to overwrite active handle! pending_idx: 0x%x\n",
>
> Using "%#x" is shorter ind does the same.
That's true, but it's also far less common.
$ git grep -E "%#[\*\d\.]*x" | wc -l
1419
$ git grep "0x%" | wc -l
29844
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format
[not found] ` <1434492550.2689.8.camel@perches.com>
@ 2015-06-16 22:29 ` Sergei Shtylyov
[not found] ` <5580A33C.4040007@cogentembedded.com>
1 sibling, 0 replies; 15+ messages in thread
From: Sergei Shtylyov @ 2015-06-16 22:29 UTC (permalink / raw)
To: Joe Perches
Cc: Wei Liu, Ian Campbell, netdev, linux-kernel, Julien Grall, xen-devel
Hello.
On 06/17/2015 01:09 AM, Joe Perches wrote:
>>> Append 0x to all %x in order to avoid while reading when there is other
>>> decimal value in the log.
> []
>>> @@ -874,7 +874,7 @@ static inline void xenvif_grant_handle_set(struct xenvif_queue *queue,
>>> if (unlikely(queue->grant_tx_handle[pending_idx] !=
>>> NETBACK_INVALID_HANDLE)) {
>>> netdev_err(queue->vif->dev,
>>> - "Trying to overwrite active handle! pending_idx: %x\n",
>>> + "Trying to overwrite active handle! pending_idx: 0x%x\n",
>> Using "%#x" is shorter ind does the same.
> That's true, but it's also far less common.
Which is a pity... People just don't know the format specifiers well
enough. :-(
> $ git grep -E "%#[\*\d\.]*x" | wc -l
> 1419
> $ git grep "0x%" | wc -l
> 29844
Which means 29 KB could theoretically be saved on allyesconfig build. :-)
(Actually less since the width specifiers will likely need to be fixed where
present.)
WBR, Sergei
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format
[not found] ` <5580A33C.4040007@cogentembedded.com>
@ 2015-06-16 23:55 ` Joe Perches
[not found] ` <1434498919.2689.35.camel@perches.com>
1 sibling, 0 replies; 15+ messages in thread
From: Joe Perches @ 2015-06-16 23:55 UTC (permalink / raw)
To: Sergei Shtylyov
Cc: Wei Liu, Ian Campbell, netdev, linux-kernel, Julien Grall, xen-devel
On Wed, 2015-06-17 at 01:29 +0300, Sergei Shtylyov wrote:
> Hello.
>
> On 06/17/2015 01:09 AM, Joe Perches wrote:
>
> >>> Append 0x to all %x in order to avoid while reading when there is other
> >>> decimal value in the log.
>
> > []
>
> >>> @@ -874,7 +874,7 @@ static inline void xenvif_grant_handle_set(struct xenvif_queue *queue,
> >>> if (unlikely(queue->grant_tx_handle[pending_idx] !=
> >>> NETBACK_INVALID_HANDLE)) {
> >>> netdev_err(queue->vif->dev,
> >>> - "Trying to overwrite active handle! pending_idx: %x\n",
> >>> + "Trying to overwrite active handle! pending_idx: 0x%x\n",
>
> >> Using "%#x" is shorter ind does the same.
>
> > That's true, but it's also far less common.
>
> Which is a pity... People just don't know the format specifiers well
> enough. :-(
>
> > $ git grep -E "%#[\*\d\.]*x" | wc -l
> > 1419
> > $ git grep "0x%" | wc -l
> > 29844
>
> Which means 29 KB could theoretically be saved on allyesconfig build. :-)
> (Actually less since the width specifiers will likely need to be fixed where
> present.)
And less than that because a lot of these are in
arch specific code.
0x%x is easier and simpler to visualize than %#x.
But you are welcome to try to make the kernel smaller.
One byte at a time.
There are ~14.5k uses of 0x%x in ~10.5k lines and
~2600 files that would be changed.
That's a lot of lines and a lot of patches.
$ git grep --name-only "0x%x" | xargs sed -i -e 's/0x%x/%#x/g'
$ git diff | wc
96250 415388 3949872
Only a 4M patch.
The pretty common (~5k) "0x%08x" would be "%#010x"
so that doesn't save any space.
but this one's a ~3.5M patch.
$ git grep --name-only -P "0x%\d+\w*x" | xargs perl -p -i -e 's/0x%0(\d+)(\w*)x/"\%#0" . eval($1 + 2) . "$2x"/eg'
$ git diff | wc
80857 344565 3306990
enjoy...
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format
[not found] ` <1434498919.2689.35.camel@perches.com>
@ 2015-06-17 8:54 ` Ian Campbell
0 siblings, 0 replies; 15+ messages in thread
From: Ian Campbell @ 2015-06-17 8:54 UTC (permalink / raw)
To: Joe Perches
Cc: Wei Liu, Sergei Shtylyov, netdev, linux-kernel, Julien Grall, xen-devel
On Tue, 2015-06-16 at 16:55 -0700, Joe Perches wrote:
> 0x%x is easier and simpler to visualize than %#x.
They also don't produce the same output if the value is 0 (0x%x=>0x0, %
#x=>0), which can matter if you are trying to line up a column or
whatever.
Ian.
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format
[not found] ` <1434481848-26387-4-git-send-email-julien.grall@citrix.com>
2015-06-16 20:07 ` Sergei Shtylyov
[not found] ` <5580821D.4080408@cogentembedded.com>
@ 2015-06-17 9:13 ` Jan Beulich
2015-06-17 9:25 ` Ian Campbell
` (2 subsequent siblings)
5 siblings, 0 replies; 15+ messages in thread
From: Jan Beulich @ 2015-06-17 9:13 UTC (permalink / raw)
To: Julien Grall; +Cc: xen-devel, netdev, Wei Liu, Ian Campbell, linux-kernel
>>> On 16.06.15 at 21:10, <julien.grall@citrix.com> wrote:
> Append 0x to all %x in order to avoid while reading when there is other
> decimal value in the log.
To save on the space taken by the format strings you should prefer
%#x over 0x%x (like we do in the hypervisor).
Jan
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format
[not found] ` <1434481848-26387-4-git-send-email-julien.grall@citrix.com>
` (2 preceding siblings ...)
2015-06-17 9:13 ` Jan Beulich
@ 2015-06-17 9:25 ` Ian Campbell
[not found] ` <1434533143.13744.273.camel@citrix.com>
2015-06-21 16:41 ` David Miller
5 siblings, 0 replies; 15+ messages in thread
From: Ian Campbell @ 2015-06-17 9:25 UTC (permalink / raw)
To: Julien Grall; +Cc: xen-devel, Wei Liu, linux-kernel, netdev
On Tue, 2015-06-16 at 20:10 +0100, Julien Grall wrote:
> Append 0x to all %x in order to avoid while reading when there is other
> decimal value in the log.
>
> Also replace some of the hexadecimal print to decimal to uniformize the
> format with netfront.
>
> Signed-off-by: Julien Grall <julien.grall@citrix.com>
> Cc: Wei Liu <wei.liu2@citrix.com>
> Cc: Ian Campbell <ian.campbell@citrix.com>
> Cc: netdev@vger.kernel.org
You meant s/Append/Prepend/, nonetheless:
Acked-by: Ian Campbell <ian.campbell@citrix.com>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format
[not found] ` <1434533143.13744.273.camel@citrix.com>
@ 2015-06-17 9:30 ` Julien Grall
[not found] ` <55813E3F.8030602@citrix.com>
1 sibling, 0 replies; 15+ messages in thread
From: Julien Grall @ 2015-06-17 9:30 UTC (permalink / raw)
To: Ian Campbell; +Cc: xen-devel, Wei Liu, linux-kernel, netdev
Hi Ian,
On 17/06/2015 10:25, Ian Campbell wrote:
> On Tue, 2015-06-16 at 20:10 +0100, Julien Grall wrote:
>> Append 0x to all %x in order to avoid while reading when there is other
>> decimal value in the log.
>>
>> Also replace some of the hexadecimal print to decimal to uniformize the
>> format with netfront.
>>
>> Signed-off-by: Julien Grall <julien.grall@citrix.com>
>> Cc: Wei Liu <wei.liu2@citrix.com>
>> Cc: Ian Campbell <ian.campbell@citrix.com>
>> Cc: netdev@vger.kernel.org
>
> You meant s/Append/Prepend/, nonetheless:
I noticed a missing word after "avoid" in the commit message too. I will
update to:
"Prepend 0x to all %x in order to avoid confusion while reading when
there is other decimal value in the log.
[...]".
>
> Acked-by: Ian Campbell <ian.campbell@citrix.com>
I see different opinion on whether using 0x% or %#. As I plan to resend
a version with the commit message update, shall I use %#?
Regards,
--
Julien Grall
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format
[not found] ` <55813E3F.8030602@citrix.com>
@ 2015-06-17 9:37 ` Ian Campbell
0 siblings, 0 replies; 15+ messages in thread
From: Ian Campbell @ 2015-06-17 9:37 UTC (permalink / raw)
To: Julien Grall; +Cc: xen-devel, Wei Liu, linux-kernel, netdev
On Wed, 2015-06-17 at 10:30 +0100, Julien Grall wrote:
> I see different opinion on whether using 0x% or %#. As I plan to resend
> a version with the commit message update, shall I use %#?
I think it's mostly pointless bike-shedding over a saving measured in
single digit bytes, use whichever you like.
Ian.
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/3] net/xen-netfront: Correct printf format in xennet_get_responses
[not found] ` <1434481848-26387-2-git-send-email-julien.grall@citrix.com>
@ 2015-06-21 16:41 ` David Miller
0 siblings, 0 replies; 15+ messages in thread
From: David Miller @ 2015-06-21 16:41 UTC (permalink / raw)
To: julien.grall
Cc: netdev, linux-kernel, david.vrabel, xen-devel, boris.ostrovsky
From: Julien Grall <julien.grall@citrix.com>
Date: Tue, 16 Jun 2015 20:10:46 +0100
> 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>
Applied.
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 2/3] net/xen-netback: Remove unused code in xenvif_rx_action
[not found] ` <1434481848-26387-3-git-send-email-julien.grall@citrix.com>
@ 2015-06-21 16:41 ` David Miller
0 siblings, 0 replies; 15+ messages in thread
From: David Miller @ 2015-06-21 16:41 UTC (permalink / raw)
To: julien.grall; +Cc: xen-devel, netdev, wei.liu2, linux-kernel, ian.campbell
From: Julien Grall <julien.grall@citrix.com>
Date: Tue, 16 Jun 2015 20:10:47 +0100
> The variables old_req_cons and ring_slots_used are assigned but never
> used since commit 1650d5455bd2dc6b5ee134bd6fc1a3236c266b5b "xen-netback:
> always fully coalesce guest Rx packets".
>
> Signed-off-by: Julien Grall <julien.grall@citrix.com>
> Acked-by: Wei Liu <wei.liu2@citrix.com>
Applied.
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format
[not found] ` <1434481848-26387-4-git-send-email-julien.grall@citrix.com>
` (4 preceding siblings ...)
[not found] ` <1434533143.13744.273.camel@citrix.com>
@ 2015-06-21 16:41 ` David Miller
5 siblings, 0 replies; 15+ messages in thread
From: David Miller @ 2015-06-21 16:41 UTC (permalink / raw)
To: julien.grall; +Cc: xen-devel, netdev, wei.liu2, linux-kernel, ian.campbell
From: Julien Grall <julien.grall@citrix.com>
Date: Tue, 16 Jun 2015 20:10:48 +0100
> Append 0x to all %x in order to avoid while reading when there is other
> decimal value in the log.
>
> Also replace some of the hexadecimal print to decimal to uniformize the
> format with netfront.
>
> Signed-off-by: Julien Grall <julien.grall@citrix.com>
Applied.
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2015-06-21 16:30 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1434481848-26387-1-git-send-email-julien.grall@citrix.com>
2015-06-16 19:10 ` [PATCH v4 1/3] net/xen-netfront: Correct printf format in xennet_get_responses Julien Grall
2015-06-16 19:10 ` [PATCH v4 2/3] net/xen-netback: Remove unused code in xenvif_rx_action Julien Grall
2015-06-16 19:10 ` [PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format Julien Grall
[not found] ` <1434481848-26387-4-git-send-email-julien.grall@citrix.com>
2015-06-16 20:07 ` Sergei Shtylyov
[not found] ` <5580821D.4080408@cogentembedded.com>
2015-06-16 22:09 ` Joe Perches
[not found] ` <1434492550.2689.8.camel@perches.com>
2015-06-16 22:29 ` Sergei Shtylyov
[not found] ` <5580A33C.4040007@cogentembedded.com>
2015-06-16 23:55 ` Joe Perches
[not found] ` <1434498919.2689.35.camel@perches.com>
2015-06-17 8:54 ` Ian Campbell
2015-06-17 9:13 ` Jan Beulich
2015-06-17 9:25 ` Ian Campbell
[not found] ` <1434533143.13744.273.camel@citrix.com>
2015-06-17 9:30 ` Julien Grall
[not found] ` <55813E3F.8030602@citrix.com>
2015-06-17 9:37 ` Ian Campbell
2015-06-21 16:41 ` David Miller
[not found] ` <1434481848-26387-2-git-send-email-julien.grall@citrix.com>
2015-06-21 16:41 ` [PATCH v4 1/3] net/xen-netfront: Correct printf format in xennet_get_responses David Miller
[not found] ` <1434481848-26387-3-git-send-email-julien.grall@citrix.com>
2015-06-21 16:41 ` [PATCH v4 2/3] net/xen-netback: Remove unused code in xenvif_rx_action David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).