Paths

Table of Contentst

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

	Unknown Object (File)
	Fri, Apr 18, 3:13 AM

	Unknown Object (File)
	Wed, Mar 26, 5:27 AM

	Unknown Object (File)
	Mar 19 2025, 6:54 PM

	Unknown Object (File)
	Mar 19 2025, 6:50 PM

	Unknown Object (File)
	Mar 19 2025, 8:17 AM

	Unknown Object (File)
	Mar 6 2025, 11:59 PM

	Unknown Object (File)
	Mar 1 2025, 7:20 AM

	Unknown Object (File)
	Feb 26 2025, 10:40 AM

View All 20 Files

Subscribers

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

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

Path

Size

sys/

dev/

sound/

pcm/

53 lines

Diff 150911

sys/dev/sound/pcm/channel.c

Loading...