Page MenuHomeFreeBSD

Fix an msleep() that can use different mutexes to avoid assert panic
ClosedPublic

Authored by rmacklem on Oct 13 2022, 3:02 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 21, 2:42 AM
Unknown Object (File)
Mon, Apr 20, 7:49 PM
Unknown Object (File)
Mon, Apr 20, 7:47 PM
Unknown Object (File)
Sun, Apr 19, 9:44 PM
Unknown Object (File)
Fri, Apr 17, 11:51 PM
Unknown Object (File)
Mon, Apr 13, 10:43 AM
Unknown Object (File)
Sun, Apr 12, 1:08 AM
Unknown Object (File)
Mon, Mar 30, 3:18 PM
Subscribers

Details

Summary

During recent testing as part of an IETF NFSv4 testing event,
I got an assert panic in sleepq_add(), due to a call with a
different mutex for the same wchan.

This turned out to be harmless, since the msleep() is on
a fake wchan that never gets a wakeup() and is simply a
time delay sleep.

To avoid the assert panic, this patch replaces the msleep()
with a tsleep().

Test Plan

With this patch, the test runs no longer assert panic'd.

Diff Detail

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