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
F102078739: D36977.id111777.diff
Thu, Nov 7, 8:50 AM
Unknown Object (File)
Oct 1 2024, 6:56 AM
Unknown Object (File)
Sep 27 2024, 11:13 AM
Unknown Object (File)
Sep 25 2024, 3:09 AM
Unknown Object (File)
Sep 24 2024, 3:51 AM
Unknown Object (File)
Sep 23 2024, 7:16 PM
Unknown Object (File)
Sep 23 2024, 11:19 AM
Unknown Object (File)
Sep 23 2024, 10:42 AM
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