Page MenuHomeFreeBSD
Differential D50337

sound: Call PCM_RELEASE() if pcm_addchan() fails
ClosedPublic
Actions

Authored by christos on May 13 2025, 8:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jun 9, 4:44 PM
Unknown Object (File)
Sat, Jun 7, 5:39 PM
Unknown Object (File)
Mon, Jun 2, 11:43 AM
Unknown Object (File)
Sun, May 25, 10:24 AM
Unknown Object (File)
Sun, May 25, 9:08 AM
Unknown Object (File)
Sat, May 24, 8:55 AM
Unknown Object (File)
Sat, May 24, 5:52 AM
Unknown Object (File)
Thu, May 22, 1:53 AM
View All 9 Files
Subscribers
imp

Details

Reviewers
markj
emaste
jhibbits
Commits
rGaff92a9f3c6c: sound: Call PCM_RELEASE() if pcm_addchan() fails
rG3de66f369c1e: sound: Call PCM_RELEASE() if pcm_addchan() fails
rGdbfe79bbe5d5: sound: Call PCM_RELEASE() if pcm_addchan() fails
Summary

Fixes: fd906e47b18f ("sound: Simplify locking during device creation")
Sponsored by: The FreeBSD Foundation
MFC after: 1 day

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

christos created this revision.May 13 2025, 8:24 PM
Herald added a subscriber: imp. · View Herald TranscriptMay 13 2025, 8:24 PM
christos requested review of this revision.May 13 2025, 8:24 PM
Harbormaster completed remote builds in B64149: Diff 155413.May 13 2025, 8:24 PM
christos mentioned this in D48482: sound: Simplify locking during device creation.May 13 2025, 8:25 PM

@markj If this fixes the issue mentioned in D48482, should we MFC it to releng/14.3 as well?

markj accepted this revision.May 13 2025, 9:54 PM
In D50337#1148520, @christos wrote:

@markj If this fixes the issue mentioned in D48482, should we MFC it to releng/14.3 as well?

Yes, definitely.

This revision is now accepted and ready to land.May 13 2025, 9:54 PM
christos added a comment.May 13 2025, 10:03 PM
In D50337#1148551, @markj wrote:
In D50337#1148520, @christos wrote:

@markj If this fixes the issue mentioned in D48482, should we MFC it to releng/14.3 as well?

Yes, definitely.

Alright. Can I do it directly or do I need to get formal approval from re@?

Closed by commit rGdbfe79bbe5d5: sound: Call PCM_RELEASE() if pcm_addchan() fails (authored by christos). · Explain WhyMay 13 2025, 10:03 PM
This revision was automatically updated to reflect the committed changes.
christos added a commit: rGdbfe79bbe5d5: sound: Call PCM_RELEASE() if pcm_addchan() fails.
markj added a comment.May 13 2025, 10:05 PM
In D50337#1148554, @christos wrote:
In D50337#1148551, @markj wrote:
In D50337#1148520, @christos wrote:

@markj If this fixes the issue mentioned in D48482, should we MFC it to releng/14.3 as well?

Yes, definitely.

Alright. Can I do it directly or do I need to get formal approval from re@?

You need to MFC to stable/14 first, then request approval from re@: https://wiki.freebsd.org/Releng/ChangeRequestGuidelines

christos added a commit: rG3de66f369c1e: sound: Call PCM_RELEASE() if pcm_addchan() fails.May 14 2025, 11:45 AM
christos added a commit: rGaff92a9f3c6c: sound: Call PCM_RELEASE() if pcm_addchan() fails.May 14 2025, 2:24 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
sound/
pcm/
6 lines

Diff 155417

View Options

sys/dev/sound/pcm/sound.c

Loading...