Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

CVS commit: src/sys/rump/librump/rumpkern



Module Name:    src
Committed By:   riastradh
Date:           Sun Apr  6 01:58:23 UTC 2025

Modified Files:
        src/sys/rump/librump/rumpkern: intr.c

Log Message:
rump: Set up the softint CPU bouncer only once, not once per CPU.

This is used for softint_schedule_cpu to schedule softints on other
CPUs.  There is one global lock, and one global condition variable,
and really there only needs to be one thread to handle the requests.
Without this change, we reinitialize the lock and condition variable,
leaking them (and possibly leave some threads confused if they are
currently taking the lock or waiting on the condition variable as
they get reinitialized).

Leak found by lsan while investigating:

PR misc/59252: tests/rump/rumpkern/t_sp:sigsafe: out of memory


To generate a diff of this commit:
cvs rdiff -u -r1.56 -r1.57 src/sys/rump/librump/rumpkern/intr.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.




Home | Main Index | Thread Index | Old Index