[dpdk-dev] [PATCH v3 4/5] net/fm10k: add flow interface and switch management

Xiaojun Liu xiaojun.liu at silicom.co.il
Thu Apr 9 08:24:55 CEST 2020


Hi Xiao,

OK. I will check the two empty lines carefully and check the patch by "git am". 

Best regards,
Xiaojun

-----Original Message-----
From: Wang, Xiao W [mailto:xiao.w.wang at intel.com] 
Sent: Thursday, April 02, 2020 4:56 PM
To: Xiaojun Liu; Zhang, Qi Z; Kwan, Ngai-mint; Keller, Jacob E
Cc: dev at dpdk.org
Subject: RE: [PATCH v3 4/5] net/fm10k: add flow interface and switch management

Hi,

Applying: net/fm10k: add flow interface and switch management
.git/rebase-apply/patch:915: new blank line at EOF.
+
warning: 1 line adds whitespace errors.

Besides, I see that you use two empty lines to delimit two functions in some places, please align to 1 empty line.

Please understand upstreaming may mean burden of maintenance, so let's make the big patch in better shape. 😊

Best Regards,
Xiao

> -----Original Message-----
> From: Xiaojun Liu <xiaojun.liu at silicom.co.il>
> Sent: Friday, March 20, 2020 2:59 PM
> To: Wang, Xiao W <xiao.w.wang at intel.com>; Zhang, Qi Z
> <qi.z.zhang at intel.com>; Kwan, Ngai-mint <ngai-mint.kwan at intel.com>; Keller,
> Jacob E <jacob.e.keller at intel.com>
> Cc: dev at dpdk.org; Xiaojun Liu <xiaojun.liu at silicom.co.il>
> Subject: [PATCH v3 4/5] net/fm10k: add flow interface and switch management
> 
> Add flow interface to support offload flow into HW.
> It supports parse vlan and parse mpls, all these
> data will be transferred to ffu data.
> Add switch management, includes initialization,
> port mapping, epl port link, LED controller, interrupt handler.
> It create 3 threads. One for interrupt handler, one for
> LED controller, one for statistics.
> 
> Signed-off-by: Xiaojun Liu <xiaojun.liu at silicom.co.il>
> ---
>  drivers/net/fm10k/Makefile              |    2 +
>  drivers/net/fm10k/base/fm10k_type.h     |    1 +
>  drivers/net/fm10k/switch/fm10k_flow.c   |  872 +++++++++++
>  drivers/net/fm10k/switch/fm10k_flow.h   |   26 +
>  drivers/net/fm10k/switch/fm10k_stats.c  |  128 ++
>  drivers/net/fm10k/switch/fm10k_switch.c | 2382
> +++++++++++++++++++++++++++++++
[...]


More information about the dev mailing list