Page MenuHomeFreeBSD

lagg: remove use of net epoch in the ioctl paths
Needs ReviewPublic

Authored by glebius on Apr 3 2024, 5:41 AM.
Tags
None
Referenced Files
F154151160: D44607.id136464.diff
Sun, Apr 26, 3:07 PM
Unknown Object (File)
Mon, Apr 13, 6:42 PM
Unknown Object (File)
Sun, Apr 12, 4:49 AM
Unknown Object (File)
Thu, Apr 9, 2:12 AM
Unknown Object (File)
Sat, Apr 4, 4:15 PM
Unknown Object (File)
Sat, Apr 4, 9:26 AM
Unknown Object (File)
Tue, Mar 31, 3:56 AM
Unknown Object (File)
Mar 20 2026, 7:28 AM
Subscribers

Details

Reviewers
None
Group Reviewers
network
Summary

Rely on LAGG_SLOCK() instead. The use of network epoch(9) here was added
in 6573d7580b851 (later tidied by 87bf9b9cbeebc) as a large sweep that
blindly substituted blocking kernel primitives with epoch(9). In these
particular code paths use of epoch(9) is incorrect and doesn't provide any
protection against a stale pointer. Recent fix 48698ead6ff0, which should
actually have removed the epoch use, created a potential sleeping in epoch
problem.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 56887
Build 53775: arc lint + arc unit