Date: Sat, 22 Nov 2025 00:50:19 +0100 From: "Christos Margiolis" <christos@freebsd.org> To: "Shawn Webb" <shawn.webb@hardenedbsd.org> Cc: <src-committers@freebsd.org>, <dev-commits-src-all@freebsd.org>, <dev-commits-src-main@freebsd.org> Subject: Re: git: 9d18115ca0ab - main - sound: Retire snd_mtx* wrappers Message-ID: <DEERXPN275JN.27945BFY6JELE@freebsd.org> In-Reply-To: <zc4flp64tzpveyezhojzbbgppvlvfiqyhygljy3t3xdtufxsyv@46knkawsq5kx> References: <69209007.369f8.65e30118@gitrepo.freebsd.org> <tfjtmgfx6pvyypqh6ugh574u6k7vrcb43uny4nhqmx6k24qjex@es6i2jzhsra2> <zc4flp64tzpveyezhojzbbgppvlvfiqyhygljy3t3xdtufxsyv@46knkawsq5kx>
index | next in thread | previous in thread | raw e-mail
On Fri Nov 21, 2025 at 9:46 PM CET, Shawn Webb wrote: > On Fri, Nov 21, 2025 at 08:45:27PM +0000, Shawn Webb wrote: >> On Fri, Nov 21, 2025 at 04:15:03PM +0000, Christos Margiolis wrote: >> > The branch main has been updated by christos: >> > >> > URL: https://cgit.FreeBSD.org/src/commit/?id=9d18115ca0ab0ef3f34173d4e2bdabec916d0b60 >> > >> > commit 9d18115ca0ab0ef3f34173d4e2bdabec916d0b60 >> > Author: Christos Margiolis <christos@FreeBSD.org> >> > AuthorDate: 2025-11-21 16:14:33 +0000 >> > Commit: Christos Margiolis <christos@FreeBSD.org> >> > CommitDate: 2025-11-21 16:14:47 +0000 >> > >> > sound: Retire snd_mtx* wrappers >> > >> > Do not create mutexes with snd_mtxcreate(). It doesn't provide any >> > value, plus it first allocates the mutex with malloc(9). Allocate >> > mutexes in the stack and use mtx_* functions directly instead of the >> > snd_mtx* wrappers. >> > >> > Sponsored by: The FreeBSD Foundation >> > MFC after: 1 week >> > Reviewed by: kib, markj >> > Differential Revision: https://reviews.freebsd.org/D53855 >> >> Hey Christos, >> >> I'm getting the following error on buildkernel: >> >> ==== BEGIN LOG ==== >> /usr/src/sys/dev/sound/pci/hdspe-pcm.c:726:3: error: call to undeclared function 'snd_mtxunlock'; ISO C99 and later do not support implicit function declarations [-Werror,-Wimplicit-function-declaration] >> 726 | snd_mtxunlock(sc->lock); >> | ^ >> 1 error generated. >> --- hdspe-pcm.o --- >> ==== END LOG ==== > > Never mind. I didn't realize HardenedBSD had a change to that file. > Sorry for the noise! So all good? Out of curiousity, what change is there on HardenedBSD? Christoshelp
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?DEERXPN275JN.27945BFY6JELE>