Page MenuHomeFreeBSD

iflib: hold the CTX lock in iflib_pseudo_register
ClosedPublic

Authored by jacob.e.keller_intel.com on Mar 15 2019, 11:53 PM.
Tags
None
Referenced Files
F132889094: D19604.diff
Mon, Oct 20, 9:29 PM
Unknown Object (File)
Mon, Oct 20, 1:11 AM
Unknown Object (File)
Mon, Oct 20, 12:09 AM
Unknown Object (File)
Sat, Oct 18, 8:47 AM
Unknown Object (File)
Thu, Oct 16, 11:23 PM
Unknown Object (File)
Sat, Oct 11, 3:19 PM
Unknown Object (File)
Fri, Oct 10, 9:06 PM
Unknown Object (File)
Thu, Sep 25, 12:15 PM
Subscribers

Details

Summary

The iflib_device_register function takes the CTX lock before calling
IFDI_ATTACH_PRE, and releases it upon finishing the registration.

Mirror this process in iflib_pseudo_register, so that we always hold the
CTX lock during the attach process when registering a pseudo interface
or a regular interface.

This was caught by code inspection while attempting to analyze where the
CTX lock was held.

Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

erj added a subscriber: erj.

Anyone else have any feedback? It makes sense to me that the attach process for pseudo devices should use the context lock like the attach process for regular devices.

This revision is now accepted and ready to land.Mar 27 2019, 5:31 PM
This revision was automatically updated to reflect the committed changes.