Paths

Table of Contentst

Differential D46834

sound: Remove useless newspd check in sysctl_dev_pcm_vchanrate()
ClosedPublic
Actions

Authored by christos on Sep 28 2024, 3:16 PM.

Tags

None

Referenced Files

	F102924231: D46834.diff
	Mon, Nov 18, 7:34 PM

	Unknown Object (File)
	Sun, Nov 17, 8:21 AM

	Unknown Object (File)
	Tue, Nov 5, 2:19 PM

	Unknown Object (File)
	Mon, Oct 28, 11:03 AM

	Unknown Object (File)
	Fri, Oct 25, 2:36 AM

	Unknown Object (File)
	Thu, Oct 24, 8:24 PM

	Unknown Object (File)
	Oct 18 2024, 11:56 AM

	Unknown Object (File)
	Oct 18 2024, 3:02 AM

View All 12 Files

Subscribers

Details

Reviewers

markj
emaste
dev_submerge.ch

Commits

rG3b5b485d7436: sound: Remove useless newspd check in sysctl_dev_pcm_vchanrate()
rGf3092614bcae: sound: Remove useless newspd check in sysctl_dev_pcm_vchanrate()

Summary

feeder_rate_min functions as the lower boundary.

Sponsored by: The FreeBSD Foundation
MFC after: 2 days

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 59644
Build 56531: arc lint + arc unit

Event Timeline

christos created this revision.Sep 28 2024, 3:16 PM

Herald added a subscriber: imp. · View Herald TranscriptSep 28 2024, 3:16 PM

christos requested review of this revision.Sep 28 2024, 3:16 PM

Harbormaster completed remote builds in B59644: Diff 143867.Sep 28 2024, 3:16 PM

christos added inline comments.Sep 28 2024, 3:18 PM

sys/dev/sound/pcm/vchan.c
511	We could just as well use RANGE() to fit the speed into the acceptable range, instead of returning an error.

dev_submerge.ch accepted this revision.Sep 29 2024, 3:08 PM

dev_submerge.ch added inline comments.

sys/dev/sound/pcm/vchan.c
511	I'd prefer the EINVAL return here. The values close to `feeder_rate_min` and `feeder_rate_max` are unrealistic and never used in practice, going out of bounds here is almost certainly a user error.

This revision is now accepted and ready to land.Sep 29 2024, 3:08 PM

markj accepted this revision.Sep 30 2024, 9:05 AM

emaste accepted this revision.Sep 30 2024, 1:10 PM

Closed by commit rGf3092614bcae: sound: Remove useless newspd check in sysctl_dev_pcm_vchanrate() (authored by christos). · Explain WhyOct 18 2024, 8:45 AM

This revision was automatically updated to reflect the committed changes.

christos added a commit: rGf3092614bcae: sound: Remove useless newspd check in sysctl_dev_pcm_vchanrate().

christos added a commit: rG3b5b485d7436: sound: Remove useless newspd check in sysctl_dev_pcm_vchanrate().Sun, Oct 20, 11:22 AM

Revision Contents
Changeset List

Path

Size

sys/

dev/

sound/

pcm/

3 lines

Diff 143867

sys/dev/sound/pcm/vchan.c

Loading...