[dpdk-dev] [PATCH] net/i40e: move RSS to flow API

Zhang, Helin helin.zhang at intel.com
Sun Jan 7 16:43:25 CET 2018



> -----Original Message-----
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zhang, Qi Z
> Sent: Friday, December 22, 2017 12:36 PM
> To: Zhao1, Wei; dev at dpdk.org
> Cc: Zhao1, Wei
> Subject: Re: [dpdk-dev] [PATCH] net/i40e: move RSS to flow API
> 
> Hi Wei:
> 
> 	Please check my comment below.
> 	Besides, there some line over 80 character warning need to fix
> 
> Regards
> Qi
> 
> > -----Original Message-----
> > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Wei Zhao
> > Sent: Friday, November 24, 2017 4:43 PM
> > To: dev at dpdk.org
> > Cc: Zhao1, Wei <wei.zhao1 at intel.com>
> > Subject: [dpdk-dev] [PATCH] net/i40e: move RSS to flow API
> >
> > Rte_flow actually defined to include RSS, but till now, RSS is out of rte_flow.
> > This patch is to move i40e existing RSS to rte_flow.
> > This patch also enable queue region configuration using flow API for i40e.
> >
> > Signed-off-by: Wei Zhao <wei.zhao1 at intel.com>
> > ---
> >  drivers/net/i40e/i40e_ethdev.c |  91 +++++++++++
> > drivers/net/i40e/i40e_ethdev.h |  11 ++
> >  drivers/net/i40e/i40e_flow.c   | 336
> > +++++++++++++++++++++++++++++++++++++++++
> >  3 files changed, 438 insertions(+)

Have you addressed all the comments? Did I miss anything?
In addition, I think doc update is needed. e.g. doc/nics/i40e.rst or similar?

Thanks,
Helin


More information about the dev mailing list