Page MenuHomeFreeBSD

dummynet: don't use per-vnet locks to protect global data.
ClosedPublic

Authored by kp on Jan 27 2022, 10:48 AM.
Tags
None
Referenced Files
F166655323: D34059.id102479.diff
Sat, Aug 15, 6:32 AM
Unknown Object (File)
Fri, Aug 14, 6:06 AM
Unknown Object (File)
Fri, Aug 14, 12:48 AM
Unknown Object (File)
Thu, Aug 13, 12:34 PM
Unknown Object (File)
Thu, Aug 13, 12:34 PM
Unknown Object (File)
Thu, Aug 13, 12:32 PM
Unknown Object (File)
Tue, Aug 11, 10:40 PM
Unknown Object (File)
Sun, Aug 9, 5:54 PM

Details

Summary

The ref_count counter is global (i.e. not per-vnet) so we can't use a
per-vnet lock to protect it. Moreover, in callouts curvnet is not set,
so we'd end up panicing when trying to use DN_BH_WLOCK().

Instead we use the global sched_lock, which is already used when
evaluating ref_count (in unload_dn_aqm()).

Sponsored by: Rubicon Communications, LLC ("Netgate")

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable