Thanks for the comments. Old version patches' states are changed.

-----Original Message-----
From: Gavin Hu (Arm Technology China) [mailto:gavin...@arm.com] 
Sent: Wednesday, September 25, 2019 11:20 AM
To: Cao, Yahui <yahui....@intel.com>; Yang, Qiming <qiming.y...@intel.com>; Lu, 
Wenzhuo <wenzhuo...@intel.com>
Cc: dev@dpdk.org; Zhang, Qi Z <qi.z.zh...@intel.com>; Ye, Xiaolong 
<xiaolong...@intel.com>; Xing, Beilei <beilei.x...@intel.com>; nd <n...@arm.com>
Subject: RE: [dpdk-dev] [PATCH v3 1/8] net/ice: enable flow director engine

Hi Yahui,

two general comments: 
1.Please make old versions of patches Superseded to facilitate review.
2. the two occurrences of rte_wmb can be removed, a similar patch has been 
reviewed by Qi. http://patches.dpdk.org/patch/59278/ 

/Gavin

> -----Original Message-----
> From: dev <dev-boun...@dpdk.org> On Behalf Of Yahui Cao
> Sent: Wednesday, September 25, 2019 5:53 AM
> To: Qiming Yang <qiming.y...@intel.com>; Wenzhuo Lu 
> <wenzhuo...@intel.com>
> Cc: dev@dpdk.org; Qi Zhang <qi.z.zh...@intel.com>; Xiaolong Ye 
> <xiaolong...@intel.com>; Beilei Xing <beilei.x...@intel.com>; Yahui 
> Cao <yahui....@intel.com>
> Subject: [dpdk-dev] [PATCH v3 1/8] net/ice: enable flow director 
> engine
> 
> From: Beilei Xing <beilei.x...@intel.com>
> 
> Enable flow director engine, including initialization and teardown.
>  - Control VSI create and release.
>  - Queue pair allocated, set up and release.
>  - Programming packet create and release.
>  - FDIR profile create and release.
> 
> Signed-off-by: Beilei Xing <beilei.x...@intel.com>
> ---

Reply via email to