RE: [PATCH v2 2/7] net/iavf: defer device start when PF VSI not ready
"Loftus, Ciara" <[email protected]>
| Newsgroups | org.dpdk.dev |
|---|---|
| Message-ID | <IA4PR11MB92787BA8C6BB190A0EE0F31C8EDB2@IA4PR11MB9278.namprd11.prod.outlook.com> |
> 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]>
> ---