Re: [PATCH v7 2/4] net/iavf: fix duplicate MAC addresses install

[Bruce Richardson <bruce.richardson@intel.com>]

On Mon, Sep 14, 2026 at 01:56:36PM +0200, David Marchand wrote:
> On Mon, 14 Sept 2026 at 12:19, Loftus, Ciara <ciara.loftus@intel.com> wrote:
> > > @@ -3428,6 +3428,14 @@ iavf_post_reset_reconfig(struct rte_eth_dev
> > > *dev)
> > >       int ret = 0;
> > >       bool allmulti = false, allunicast = false;
> > >       struct iavf_adapter *adapter = IAVF_DEV_PRIVATE_TO_ADAPTER(dev-
> > > >data->dev_private);
> > > +     struct iavf_info *vf = IAVF_DEV_PRIVATE_TO_VF(dev->data-
> > > >dev_private);
> > > +
> > > +     /*
> > > +      * After a VF reset, all MAC addresses got flushed.
> > > +      * The primary MAC should have been set in iavf_dev_start, restore
> >
> > iavf_dev_start is not guaranteed to have been executed before this handler.
> > So I would change this comment to something like:
> > "The primary MAC has been or will be restored by iavf_dev_start".
> >
> > Other than that:
> >
> > Acked-by: Ciara Loftus <ciara.loftus@intel.com>
> 
> Yes, true.
> I'll update in a new revision if needed, otherwise, could it be
> updated when applying?
> 
Single line text changes I can handle on apply, no problem, so long as the
rest of the set is good to go.

/Bruce