Re: [Intel-wired-lan] [PATCH net] i40e: fix netdev leak in i40e_vsi_setup() error paths

From: Dawei Feng

Date: Fri May 29 2026 - 22:38:05 EST


On Fri, May 29, 2026 at 1:57 AM, Shannon Nelson wrote:
>
> >> err_netdev:
> >> + if (vsi->netdev) {
> >> + free_netdev(vsi->netdev);
> >> + vsi->netdev = NULL;
> >> + }
> >> i40e_aq_delete_element(&pf->hw, vsi->seid, NULL);
>
> Would it make sense to put these 4 lines into i40e_vsi_clear()? Then you
> can also clean up i40e_vsi_release() and i40e_vsi_reinit_setup() in a
> similar way.
>
> sln

Hi sln,

Thanks for the suggestion. I will adjust this part in the upcoming v2.

Best regards,
Dawei