Commit ab832b8d authored by Jakub Kicinski's avatar Jakub Kicinski Committed by David S. Miller

nfp: make sure to cancel port refresh on the error path

If very last stages of netdev registering and init fail some
other netdevs and devlink ports may have been visible to user
space before we torn them back down.  In this case there is a
slight chance user may have triggered port refresh.  We need
to make sure the async work is cancelled.

We have to cancel after releasing pf->lock, so we will always
try to cancel, regardless of which part of probe has failed.
Signed-off-by: default avatarJakub Kicinski <jakub.kicinski@netronome.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 7fa13653
...@@ -813,6 +813,7 @@ int nfp_net_pci_probe(struct nfp_pf *pf) ...@@ -813,6 +813,7 @@ int nfp_net_pci_probe(struct nfp_pf *pf)
nfp_cpp_area_release_free(pf->data_vnic_bar); nfp_cpp_area_release_free(pf->data_vnic_bar);
err_unlock: err_unlock:
mutex_unlock(&pf->lock); mutex_unlock(&pf->lock);
cancel_work_sync(&pf->port_refresh_work);
return err; return err;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment