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

	F133413202: D48966.diff
	Sat, Oct 25, 3:11 PM

	F133405572: D48966.id150911.diff
	Sat, Oct 25, 1:47 PM

	Unknown Object (File)
	Sun, Oct 12, 3:37 PM

	Unknown Object (File)
	Fri, Oct 10, 1:26 AM

	Unknown Object (File)
	Tue, Sep 30, 8:52 AM

	Unknown Object (File)
	Tue, Sep 30, 12:50 AM

	Unknown Object (File)
	Sep 25 2025, 12:03 AM

	Unknown Object (File)
	Sep 17 2025, 5:11 PM

View All 72 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 Not Applicable

Unit

Tests Not Applicable

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
1311	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
1181–1183	The FALLTHROUGH comment has to appear at the end of the case.
1185	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
1185	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/

50 lines

Diff 151470

sys/dev/sound/pcm/channel.c

Loading...