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

["Taylor R Campbell" <riastradh%netbsd.org@localhost>]

The following reply was made to PR misc/59252; it has been noted by GNATS.

From: "Taylor R Campbell" <riastradh%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/59252 CVS commit: src/sys/rump/librump/rumpkern
Date: Sun, 6 Apr 2025 01:58:23 +0000

 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.