On Mon, 14 Sept 2026 at 12:19, Loftus, Ciara <[email protected]> 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 <[email protected]>
Yes, true. I'll update in a new revision if needed, otherwise, could it be updated when applying? -- David Marchand
