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

	F171076568: D54131.id167722.diff
	Tue, Sep 8, 1:45 PM

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

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

	Unknown Object (File)
	Sat, Sep 5, 11:19 PM

	Unknown Object (File)
	Sat, Sep 5, 1:51 PM

	Unknown Object (File)
	Fri, Sep 4, 11:37 PM

	Unknown Object (File)
	Fri, Sep 4, 8:39 PM

	Unknown Object (File)
	Fri, Sep 4, 6:52 AM

View All 84 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 Skipped

Unit

Tests Skipped

Build Status

Buildable 69129
Build 66012: arc lint + arc unit

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
129	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/

30 lines

Diff 167722

sys/dev/sound/midi/midi.c

Loading...