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)
	Mon, Jun 30, 1:34 AM

	Unknown Object (File)
	Sun, Jun 29, 7:20 AM

	Unknown Object (File)
	Thu, Jun 26, 5:38 AM

	Unknown Object (File)
	Sun, Jun 22, 8:50 PM

	Unknown Object (File)
	Sun, Jun 22, 12:15 PM

	Unknown Object (File)
	Sun, Jun 22, 9:06 AM

	Unknown Object (File)
	Tue, Jun 17, 11:19 PM

	Unknown Object (File)
	Mon, Jun 16, 1:25 AM

View All 43 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...