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, Aug 11, 4:57 PM
Unknown Object (File)
Mon, Aug 10, 10:45 AM
Unknown Object (File)
Mon, Aug 10, 10:45 AM
Unknown Object (File)
Mon, Aug 10, 10:42 AM
Unknown Object (File)
Sun, Aug 9, 3:54 PM
Unknown Object (File)
Sat, Aug 8, 4:15 PM
Unknown Object (File)
Fri, Aug 7, 7:48 PM
Unknown Object (File)
Fri, Aug 7, 5:55 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