Page MenuHomeFreeBSD
Differential D48966

sound: Call chn_kill() in chn_init() failure
ClosedPublic
Actions

Authored by christos on Feb 12 2025, 4:04 PM.
Tags
None
Referenced Files
F153497921: D48966.id150911.diff
Tue, Apr 21, 11:41 AM
Unknown Object (File)
Tue, Apr 21, 4:15 AM
Unknown Object (File)
Sun, Apr 19, 8:38 PM
Unknown Object (File)
Sun, Apr 19, 8:38 PM
Unknown Object (File)
Sun, Apr 19, 5:50 PM
Unknown Object (File)
Fri, Apr 17, 1:18 AM
Unknown Object (File)
Fri, Apr 17, 12:54 AM
Unknown Object (File)
Wed, Apr 15, 9:55 PM
View All Files
Subscribers
imp

Details

Reviewers
markj
emaste
dev_submerge.ch
Commits
rG858504a1d89f: sound: Call chn_kill() in chn_init() failure
rGbc7e65e95015: sound: Call chn_kill() in chn_init() failure
Summary

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 62397
Build 59281: arc lint + arc unit

Event Timeline

christos created this revision.Feb 12 2025, 4:04 PM
Herald added a subscriber: imp. · View Herald TranscriptFeb 12 2025, 4:04 PM
christos requested review of this revision.Feb 12 2025, 4:04 PM
Harbormaster completed remote builds in B62397: Diff 150911.Feb 12 2025, 4:05 PM
christos added a parent revision: D48962: sound: Update comment and channel insertion in vchan_create().Feb 12 2025, 4:06 PM
christos added inline comments.Feb 12 2025, 4:08 PM
sys/dev/sound/pcm/channel.c
1326

I removed this and moved these increments inside the switch at the top of chn_init(). This is to avoid inconsistencies since chn_kill() decrements these counters.

markj added inline comments.Feb 13 2025, 4:49 PM
sys/dev/sound/pcm/channel.c
1188

The FALLTHROUGH comment has to appear at the end of the case.

1191

This comment doesn't seem very useful?

christos marked 2 inline comments as done.Feb 13 2025, 5:01 PM
christos added inline comments.
sys/dev/sound/pcm/channel.c
1191

This was a leftover I forgot to correct. Thank you.

christos marked an inline comment as done.Feb 13 2025, 5:02 PM
christos updated this revision to Diff 150982.Feb 13 2025, 5:02 PM

Address Mark's comments.

Harbormaster completed remote builds in B62426: Diff 150982.Feb 13 2025, 5:02 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 25 2025, 11:49 AM
Closed by commit rGbc7e65e95015: sound: Call chn_kill() in chn_init() failure (authored by christos). · Explain Why
This revision was automatically updated to reflect the committed changes.
christos added a commit: rGbc7e65e95015: sound: Call chn_kill() in chn_init() failure.
christos added a commit: rG858504a1d89f: sound: Call chn_kill() in chn_init() failure.Mar 4 2025, 3:47 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
sound/
pcm/
53 lines

Diff 150911

View Options

sys/dev/sound/pcm/channel.c

Loading...