Differential D54131

sound: Improve snd_midi->{in,out}q allocation
ClosedPublic
Actions

Authored by christos on Dec 8 2025, 5:13 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Sep 14, 2:35 PM

	Unknown Object (File)
	Mon, Sep 14, 2:29 PM

	Unknown Object (File)
	Mon, Sep 14, 2:24 PM

	Unknown Object (File)
	Fri, Sep 11, 4:44 AM

	Unknown Object (File)
	Thu, Sep 10, 2:28 PM

	Unknown Object (File)
	Tue, Sep 8, 1:45 PM

	Unknown Object (File)
	Sun, Sep 6, 2:11 PM

	Unknown Object (File)
	Sun, Sep 6, 3:40 AM

View All 89 Files

Subscribers

Details

Reviewers

markj
emaste

Commits

rG0ed90f4379c4: sound: Improve snd_midi->{in,out}q allocation
rGb9db6c212873: sound: Improve snd_midi->{in,out}q allocation

Summary

Currently we lock and allocate the buffers with M_NOWAIT, without
checking if the return value of malloc(). This is not necessary as
subsequent check below will eventually check that. However, for
correctness, allocate the buffers with M_WAITOK (there is no reason not
to) and lock afterwards.

Sponsored by: The FreeBSD Foundation
MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

christos created this revision.Dec 8 2025, 5:13 PM

Herald added a subscriber: imp. · View Herald TranscriptDec 8 2025, 5:13 PM

christos requested review of this revision.Dec 8 2025, 5:13 PM

Harbormaster completed remote builds in B69129: Diff 167722.Dec 8 2025, 5:13 PM

christos mentioned this in D54129: sound: Retire snd_midi->qlock.Dec 11 2025, 11:37 AM

markj accepted this revision.Dec 14 2025, 4:16 PM

markj added inline comments.

sys/dev/sound/midi/midi.c
128	Why did you add M_ZERO?

This revision is now accepted and ready to land.Dec 14 2025, 4:16 PM

Closed by commit rGb9db6c212873: sound: Improve snd_midi->{in,out}q allocation (authored by christos). · Explain WhyJan 2 2026, 4:59 PM

This revision was automatically updated to reflect the committed changes.

christos added a commit: rGb9db6c212873: sound: Improve snd_midi->{in,out}q allocation.

christos added a commit: rG0ed90f4379c4: sound: Improve snd_midi->{in,out}q allocation.Jan 9 2026, 11:28 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

sound/

midi/

24 lines

Diff 168888

sys/dev/sound/midi/midi.c

Loading...