HomeFreeBSD

clnt_vc.c: Replace msleep() with pause() to avoid assert panic

Description

clnt_vc.c: Replace msleep() with pause() to avoid assert panic

An msleep() in clnt_vc.c used a global "fake_wchan" wchan argument
along with the mutex in a CLIENT structure. As such, it was
possible to use different mutexes for the same wchan and
cause a panic assert. Since this is in a rarely executed code
path, the assert panic was only recently observed.

Since "fake_wchan" never gets a wakeup, this msleep() can
be replaced with a pause() to avoid the panic assert,
which is what this patch does.

(cherry picked from commit 82512c17ea39fcc272483024cb55d567dfd55366)

Details

Provenance
rmacklemAuthored on Oct 14 2022, 10:46 PM
Parents
rG83ac15a799e3: Bump __FreeBSD_version to 1301508
Branches
Unknown
Tags
Unknown