On Thu, Jun 11, 2026 at 09:52:39AM +0100, Loftus, Ciara wrote:
> > Subject: [PATCH v6] net/iavf: fix duplicate VF reset during PF reset
> > recovery
> >
> > During PF initiated reset recovery, iavf_dev_close() sends an
> > extra 'VIRTCHNL_OP_RESET_VF' while recovery is already in progress.
> > That second reset can leave PF/VF virtchnl state inconsistent and
> > cause 'VIRTCHNL_OP_CONFIG_VSI_QUEUES' to fail with 'ERR_PARAM' after
> > ToR link flap/power-cycle, leaving the VF unable to recover.
> > This results in connection loss.
> >
> > This patch introduces a new flag 'pf_reset_in_progress', which
> > is set only when iavf_handle_hw_reset() is entered for a
> > PF-initiated reset (vf_initiated_reset is false), and
> > it is cleared on exit.
> > The aforesaid flag is used to prevent sending close-time VF
> > reset and related close-time virtchnl operation messages to the
> > AdminQ when PF triggered reset recovery is set.
> > This is done to avoid duplicate VF reset requests while preserving
> > normal behavior for application-driven close or VF-initiated reinit.
> >
> > Fixes: 675a104e2e94 ("net/iavf: fix abnormal disable HW interrupt")
> > Fixes: b34fe66ea893 ("net/iavf: delay VF reset command")
> > Fixes: 5e03e316c753 ("net/iavf: handle virtchnl event message without
> > interrupt")
> > Cc: [email protected]
> >
> > Signed-off-by: Anurag Mandal <[email protected]>
>
> Thanks Anurag.
>
> Acked-by: Ciara Loftus <[email protected]>
>
Applied to dpdk-next-net-intel.
Thanks,
/Bruce
