Re: [PATCH v2 2/7] net/iavf: defer device start when PF VSI not ready
Bruce Richardson <[email protected]>
| Newsgroups | org.dpdk.dev |
|---|---|
| Message-ID | <[email protected]> |
On Thu, Aug 13, 2026 at 02:12:54PM +0100, Loftus, Ciara wrote:
> > Subject: [PATCH v2 2/7] net/iavf: defer device start when PF VSI not ready
> >
> > During reset recovery iavf_dev_start() might fail (typically -EIO
> > from VIRTCHNL_OP_CONFIG_VSI_QUEUES) because the PF VSI is not yet
> > active, leaving the VF down and requiring manual intervention
> > to recover.
> >
> > Added a start_pending flag: when device start fails during recovery,
> > defer it instead of erroring out and resume it from newly added
> > iavf_resume_pending_start() on the next link-up event, so the
> > VF comes back automatically.
> >
> > Fixes: 105d9fcefbee ("net/iavf: fix race condition on link status change")
>
> I don't think this is the correct Fixes tag. It seems like an issue that has
> been present since the recovery path was introduced. Maybe:
> 3e6a5d2d310a ("net/iavf: add devargs to enable VF auto-reset") ?
>
> >
> > Signed-off-by: Anurag Mandal <[email protected]>
> > ---
>
If the rest of the series is ok, I can fix up things like fixes tags and
log message changes on commit, so no need to do up a new version just for
this alone
/Bruce