* [PATCH V2] net: i825xx: replace dev_kfree_skb_irq by dev_consume_skb_irq @ 2019-01-27 15:53 Yang Wei 2019-01-27 16:34 ` Andrew Lunn 0 siblings, 1 reply; 3+ messages in thread From: Yang Wei @ 2019-01-27 15:53 UTC (permalink / raw) To: netdev; +Cc: davem, yang.wei9, Yang Wei dev_consume_skb_irq() should be called in i596_interrupt() when xmit done. It makes drop profiles more friendly. Signed-off-by: Yang Wei <yang.wei9@zte.com.cn> --- drivers/net/ethernet/i825xx/82596.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/i825xx/82596.c b/drivers/net/ethernet/i825xx/82596.c index d719668..855900a 100644 --- a/drivers/net/ethernet/i825xx/82596.c +++ b/drivers/net/ethernet/i825xx/82596.c @@ -1310,7 +1310,7 @@ static irqreturn_t i596_interrupt(int irq, void *dev_id) dev->stats.tx_aborted_errors++; } - dev_kfree_skb_irq(skb); + dev_consume_skb_irq(skb); tx_cmd->cmd.command = 0; /* Mark free */ break; -- 2.7.4 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH V2] net: i825xx: replace dev_kfree_skb_irq by dev_consume_skb_irq 2019-01-27 15:53 [PATCH V2] net: i825xx: replace dev_kfree_skb_irq by dev_consume_skb_irq Yang Wei @ 2019-01-27 16:34 ` Andrew Lunn [not found] ` <201901280950423897221@zte.com.cn> 0 siblings, 1 reply; 3+ messages in thread From: Andrew Lunn @ 2019-01-27 16:34 UTC (permalink / raw) To: Yang Wei; +Cc: netdev, davem, yang.wei9 On Sun, Jan 27, 2019 at 11:53:08PM +0800, Yang Wei wrote: > dev_consume_skb_irq() should be called in i596_interrupt() when > xmit done. It makes drop profiles more friendly. > > Signed-off-by: Yang Wei <yang.wei9@zte.com.cn> > --- Hi Yang It is normal to include here, under the ---, what has changed since the previous version of the patch. Please also read https://www.kernel.org/doc/html/latest/networking/netdev-FAQ.html Your patches don't have the correct subject. Thanks Andrew ^ permalink raw reply [flat|nested] 3+ messages in thread
[parent not found: <201901280950423897221@zte.com.cn>]
* Re: [PATCH V2] net: i825xx: replace dev_kfree_skb_irq bydev_consume_skb_irq [not found] ` <201901280950423897221@zte.com.cn> @ 2019-01-28 2:03 ` Andrew Lunn 0 siblings, 0 replies; 3+ messages in thread From: Andrew Lunn @ 2019-01-28 2:03 UTC (permalink / raw) To: yang.wei9; +Cc: albin_yang, netdev, davem On Mon, Jan 28, 2019 at 09:50:42AM +0800, yang.wei9@zte.com.cn wrote: > > It is normal to include here, under the ---, what has changed since > > the previous version of the patch. > > > > Please also read > > > > https://www.kernel.org/doc/html/latest/networking/netdev-FAQ.html > > > > Your patches don't have the correct subject. > > Hi Andrew > > The previous patch (as follow) is wrong, so I submit a new and named V2. > Please reject the previous patch. > > net: i825xx: replace dev_kfree_skb_irq by dev_consume_skb_irq > http://patchwork.ozlabs.org/patch/1031590/ > > The original suject in patch is as follow, but it be truncated due to return character. > I have changed it, make the suject in one line. Do I need to resubmit it? https://www.kernel.org/doc/html/latest/networking/netdev-FAQ.html#q-how-do-i-indicate-which-tree-net-vs-net-next-my-patch-should-be-in Andrew ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-01-28 2:04 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-01-27 15:53 [PATCH V2] net: i825xx: replace dev_kfree_skb_irq by dev_consume_skb_irq Yang Wei
2019-01-27 16:34 ` Andrew Lunn
[not found] ` <201901280950423897221@zte.com.cn>
2019-01-28 2:03 ` [PATCH V2] net: i825xx: replace dev_kfree_skb_irq bydev_consume_skb_irq Andrew Lunn
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).