> -----Original Message----- > From: Ferruh Yigit <ferruh.yi...@amd.com> > Sent: Monday, October 24, 2022 18:49 > To: Guo, Junfeng <junfeng....@intel.com>; Zhang, Qi Z > <qi.z.zh...@intel.com>; Wu, Jingjing <jingjing...@intel.com>; Xing, > Beilei <beilei.x...@intel.com> > Cc: dev@dpdk.org; Li, Xiaoyun <xiaoyun...@intel.com>; > awogbem...@google.com; Richardson, Bruce > <bruce.richard...@intel.com>; hemant.agra...@nxp.com; > step...@networkplumber.org; Xia, Chenbo <chenbo....@intel.com>; > Zhang, Helin <helin.zh...@intel.com> > Subject: Re: [PATCH v7 6/8] net/gve: add support for dev info get and dev > configure > > On 10/24/2022 6:04 AM, Guo, Junfeng wrote: > > > > > >> -----Original Message----- > >> From: Ferruh Yigit <ferruh.yi...@amd.com> > >> Sent: Friday, October 21, 2022 17:51 > >> To: Guo, Junfeng <junfeng....@intel.com>; Zhang, Qi Z > >> <qi.z.zh...@intel.com>; Wu, Jingjing <jingjing...@intel.com>; > >> ferruh.yi...@xilinx.com; Xing, Beilei <beilei.x...@intel.com> > >> Cc: dev@dpdk.org; Li, Xiaoyun <xiaoyun...@intel.com>; > >> awogbem...@google.com; Richardson, Bruce > >> <bruce.richard...@intel.com>; hemant.agra...@nxp.com; > >> step...@networkplumber.org; Xia, Chenbo <chenbo....@intel.com>; > >> Zhang, Helin <helin.zh...@intel.com> > >> Subject: Re: [PATCH v7 6/8] net/gve: add support for dev info get and > dev > >> configure > >> > >> On 10/21/2022 10:19 AM, Junfeng Guo wrote: > >> > >>> > >>> Add dev_ops dev_infos_get. > >>> Complete dev_configure with RX offloads force enabling. > >>> > >>> Signed-off-by: Xiaoyun Li <xiaoyun...@intel.com> > >>> Signed-off-by: Junfeng Guo <junfeng....@intel.com> > >> > >> <...> > >> > >>> --- a/doc/guides/nics/gve.rst > >>> +++ b/doc/guides/nics/gve.rst > >>> @@ -62,6 +62,7 @@ In this release, the GVE PMD provides the basic > >> functionality of packet > >>> reception and transmission. > >>> Supported features of the GVE PMD are: > >>> > >>> +- Receiver Side Scaling (RSS) > >> > >> [copy/paste from previous version] > >> > >> I am not sure if driver can claim this, I can see a RSS hash is provided > >> but is it possible to update which hash function to use or update key or > >> RETA table to configure which queue packets goes? > >> > >> Right now what is RSS calculated on? > >> > >> Perpaps RSS support can be documented as limited? > >> > >> And not sure if this update belongs this patch, it should be to the one > >> that has the datapath. > > > > Looks that the RSS is enabled by default, and there is no RSS init API. > > So I just added back the force-enabled RSS offloading code with the > > corresponding commit message. So the feature list remains unchaged. > > There is difference between RSS and RSS hash, what force enabled is "RSS > hash" where device calculated hash value is shared with application in > case application wants to reuse this value for some reasons. > > But for RSS support, there is a set of configuration required by DPDK > seems missing, as mentioned above, config RSS hash function (based on > which part of the packet is the hash calculated), or RETA table update > so app can select which packets goes to which queue, etc... > > Is it at least possible to document what existing configuration is? > > Because of missing configuration support, I don't think it is correct to > document RSS as supported, can you please update it to say something > like limited support exist with default config.
Thanks for the detailed explanations! Yes, you are right. The RSS cannot be configured by this user at this time. So it should be commented as an limitation here. Will update this in the coming version. Really thanks for your time and efforts to review!
