[dpdk-dev] [PATCH] net/netvsc: disable before changing RSS parameters

Donald_Lee at trendmicro.com Donald_Lee at trendmicro.com
Tue Dec 10 03:31:31 CET 2019



On 10/12/2019, 5:29 AM, "Dexuan Cui" <decui at microsoft.com> wrote:

    
    This message was sent from outside of Trend Micro. Please do not click links or open attachments unless you recognise the source of this email and know the content is safe.
    
    
    > From: Dexuan Cui
    > Sent: Monday, November 25, 2019 7:29 PM
    > To: Stephen Hemminger <stephen at networkplumber.org>; dev at dpdk.org;
    > Donald_Lee at trendmicro.com
    > Cc: stable at dpdk.org
    > Subject: RE: [PATCH] net/netvsc: disable before changing RSS parameters
    >
    > > From: Stephen Hemminger <stephen at networkplumber.org>
    > > Sent: Monday, November 25, 2019 9:13 AM
    > > To: dev at dpdk.org
    > > Cc: Stephen Hemminger <stephen at networkplumber.org>;
    > stable at dpdk.org;
    > > Dexuan Cui <decui at microsoft.com>
    > >
    > > In order to change RSS parameters, the host requires that RSS
    > > is disable first.
    > >
    > > Fixes: 92d23a57cafe ("net/netvsc: support configuring RSS parameters")
    > > Cc: stable at dpdk.org
    > > Signed-off-by: Dexuan Cui <decui at microsoft.com>
    > > Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>
    > > ---
    > >  drivers/net/netvsc/hn_ethdev.c | 14 ++++++++++++++
    > >  1 file changed, 14 insertions(+)
    > >
    > > diff --git a/drivers/net/netvsc/hn_ethdev.c
    > b/drivers/net/netvsc/hn_ethdev.c
    > > index 164e9ad174a7..575780ee5f26 100644
    > > --- a/drivers/net/netvsc/hn_ethdev.c
    > > +++ b/drivers/net/netvsc/hn_ethdev.c
    > > @@ -291,6 +291,13 @@ static int hn_rss_reta_update(struct rte_eth_dev
    > > *dev,
    > >                     hv->rss_ind[i] = reta_conf[idx].reta[shift];
    > >     }
    > >
    > > +   err = hn_rndis_conf_rss(hv, NDIS_RSS_FLAG_DISABLE);
    > > +   if (err) {
    > > +           PMD_DRV_LOG(NOTICE,
    > > +                   "rss disable failed");
    > > +           return err;
    > > +   }
    > > +
    > >     err = hn_rndis_conf_rss(hv, 0);
    > >     if (err) {
    > >             PMD_DRV_LOG(NOTICE,
    > > @@ -578,6 +585,13 @@ static int hn_dev_configure(struct rte_eth_dev
    > > *dev)
    > >                     return err;
    > >             }
    > >
    > > +           err = hn_rndis_conf_rss(hv, NDIS_RSS_FLAG_DISABLE);
    > > +           if (err) {
    > > +                   PMD_DRV_LOG(NOTICE,
    > > +                           "rss disable failed");
    > > +                   return err;
    > > +           }
    > > +
    > >             err = hn_rndis_conf_rss(hv, 0);
    > >             if (err) {
    > >                     PMD_DRV_LOG(NOTICE,
    > > --
    > > 2.20.1
    >
    > Hi, Stephen,
    > I think the patch originally came from Donald Lee (I added him to "To").
    >
    > IMO Donald's Signed-off-by should be added.
    >
    > And, can you please change my Signed-off-by to Tested-by?
    >
    > Thanks,
    > -- Dexuan
    
    Hi Donald,
    Since you originally made the patch, can you please confirm that your
    Signed-off-by
    "Signed-off-by: Donald Lee <Donald_Lee at trendmicro.com>"
    can be added into the patch's changelog?
    
    Thanks,
    -- Dexuan


Hi Dexuan, 

Yes! Can you please add my signed-off on it? Our mailing system will add extra space on the patch last time I did. Thank you.

Regards,
Donald
    



More information about the dev mailing list