On Tue, 14 Apr 2020 09:48:31 +0000
Ori Kam <or...@mellanox.com> wrote:

> > @@ -2299,6 +2299,22 @@ cmd_config_rss_parsed(void *parsed_result,
> >             rss_conf.rss_hf = ETH_RSS_L4_SRC_ONLY;
> >     else if (!strcmp(res->value, "l4-dst-only"))
> >             rss_conf.rss_hf = ETH_RSS_L4_DST_ONLY;
> > +   else if (!strcmp(res->value, "eth-src-only"))
> > +           rss_conf.rss_hf = ETH_RSS_ETH_SRC_ONLY;
> > +   else if (!strcmp(res->value, "eth-dst-only"))
> > +           rss_conf.rss_hf = ETH_RSS_ETH_DST_ONLY;
> > +   else if (!strcmp(res->value, "s-vlan"))
> > +           rss_conf.rss_hf = ETH_RSS_S_VLAN;
> > +   else if (!strcmp(res->value, "c-vlan"))
> > +           rss_conf.rss_hf = ETH_RSS_C_VLAN;
> > +   else if (!strcmp(res->value, "l2tpv3"))
> > +           rss_conf.rss_hf = ETH_RSS_L2TPV3;
> > +   else if (!strcmp(res->value, "esp"))
> > +           rss_conf.rss_hf = ETH_RSS_ESP;
> > +   else if (!strcmp(res->value, "ah"))
> > +           rss_conf.rss_hf = ETH_RSS_AH;
> > +   else if (!strcmp(res->value, "pfcp"))
> > +           rss_conf.rss_hf = ETH_RSS_PFCP;  

Hasn't this grown big enough that it should be a table with lookup function,
versus a cascade of strcmp's.

Reply via email to