lkml.org 
[lkml]   [2020]   [Oct]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH net-next] drivers/net/wan/hdlc_fr: Improvements to the code of pvc_xmit
From
From: Xie He <xie.he.0141@gmail.com>
Date: Sat, 3 Oct 2020 15:41:05 -0700

> 1. Keep the code for the normal (non-error) flow at the lowest
> indentation level. And use "goto drop" for all error handling.
>
> 2. Replace code that pads short Ethernet frames with a "__skb_pad" call.
>
> 3. Change "dev_kfree_skb" to "kfree_skb" in error handling code.
> "kfree_skb" is the correct function to call when dropping an skb due to
> an error. "dev_kfree_skb", which is an alias of "consume_skb", is for
> dropping skbs normally (not due to an error).
>
> Cc: Krzysztof Halasa <khc@pm.waw.pl>
> Cc: Stephen Hemminger <stephen@networkplumber.org>
> Signed-off-by: Xie He <xie.he.0141@gmail.com>

Applied, thank you.

\
 
 \ /
  Last update: 2020-10-05 00:16    [W:0.052 / U:0.048 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site