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

	Unknown Object (File)
	Fri, Dec 13, 12:32 PM

	Unknown Object (File)
	Thu, Nov 28, 11:49 AM

	Unknown Object (File)
	Nov 19 2024, 5:17 PM

	Unknown Object (File)
	Nov 18 2024, 9:36 PM

	Unknown Object (File)
	Nov 18 2024, 9:35 PM

	Unknown Object (File)
	Nov 18 2024, 7:34 PM

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

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

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

Unit

Tests Not Applicable

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().Oct 20 2024, 11:22 AM

Revision Contents
Changeset List

Path

Size

sys/

dev/

sound/

pcm/

3 lines

Diff 145090

sys/dev/sound/pcm/vchan.c

Loading...