Sorry to miss to add the Ori's ack to this patch and sorry to add to wrong location. I re-add ack this time.
> -----Original Message----- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Su, Simei > Sent: Sunday, September 29, 2019 4:56 PM > To: Zhang, Qi Z <qi.z.zh...@intel.com>; Wu, Jingjing <jingjing...@intel.com>; > Ye, Xiaolong <xiaolong...@intel.com>; Yigit, Ferruh <ferruh.yi...@intel.com> > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v7 2/3] ethdev: extend RSS offload types > > > > > -----Original Message----- > > From: Su, Simei > > Sent: Sunday, September 29, 2019 3:09 PM > > To: Zhang, Qi Z <qi.z.zh...@intel.com>; Wu, Jingjing > > <jingjing...@intel.com>; Ye, Xiaolong <xiaolong...@intel.com>; Yigit, > > Ferruh <ferruh.yi...@intel.com> > > Cc: dev@dpdk.org; Su, Simei <simei...@intel.com> > > Subject: [PATCH v7 2/3] ethdev: extend RSS offload types > > > > This patch reserves several bits as input set selection from the high > > end of the 64 bits. It is combined with exisiting ETH_RSS_* to represent rss > types. > > > > for example: > > ETH_RSS_IPV4 | ETH_RSS_L3_SRC_ONLY: hash on src ip address only > > ETH_RSS_IPV4_UDP | ETH_RSS_L4_DST_ONLY: hash on src/dst IP and > > dst UDP port > > > > Signed-off-by: Simei Su <simei...@intel.com> > Acked-by: Ori Kam <or...@mellanox.com> > > --- Acked-by: Ori Kam <or...@mellanox.com> > > lib/librte_ethdev/rte_ethdev.h | 13 +++++++++++++ > > 1 file changed, 13 insertions(+) > > > > diff --git a/lib/librte_ethdev/rte_ethdev.h > > b/lib/librte_ethdev/rte_ethdev.h index > > 7722f70..e68bca8 100644 > > --- a/lib/librte_ethdev/rte_ethdev.h > > +++ b/lib/librte_ethdev/rte_ethdev.h > > @@ -505,6 +505,19 @@ struct rte_eth_rss_conf { > > #define ETH_RSS_GENEVE (1ULL << 20) > > #define ETH_RSS_NVGRE (1ULL << 21) > > > > +/* > > + * We use the following macros to combine with above ETH_RSS_* for > > + * more specific input set selection. These bits are defined starting > > + * from the high end of the 64 bits. > > + * Note: If we use above ETH_RSS_* without SRC/DST_ONLY, it > > +represents > > + * both SRC and DST are taken into account. SRC_ONLY and DST_ONLY of > > + * the same level can't be used simultaneously. > > + */ > > +#define ETH_RSS_L3_SRC_ONLY (1ULL << 63) > > +#define ETH_RSS_L3_DST_ONLY (1ULL << 62) > > +#define ETH_RSS_L4_SRC_ONLY (1ULL << 61) > > +#define ETH_RSS_L4_DST_ONLY (1ULL << 60) > > + > > #define ETH_RSS_IP ( \ > > ETH_RSS_IPV4 | \ > > ETH_RSS_FRAG_IPV4 | \ > > -- > > 1.8.3.1
