Page MenuHomeFreeBSD
Differential D45862

sound: Fix min/max rate for SNDCTL_AUDIOINFO and SNDCTL_ENGINEINFO
ClosedPublic
Actions

Authored by christos on Jul 3 2024, 7:10 PM.
Tags
None
Referenced Files
F163270639: D45862.id140535.diff
Tue, Jul 21, 3:09 PM
Unknown Object (File)
Mon, Jul 20, 12:19 PM
Unknown Object (File)
Mon, Jul 20, 6:25 AM
Unknown Object (File)
Sun, Jul 19, 8:30 PM
Unknown Object (File)
Sun, Jul 19, 5:47 PM
Unknown Object (File)
Sat, Jul 18, 7:15 PM
Unknown Object (File)
Sat, Jul 18, 5:31 AM
Unknown Object (File)
Sun, Jul 5, 4:01 AM
View All Files
Subscribers
imp

Details

Reviewers
markj
emaste
dev_submerge.ch
Commits
rG95d3abff7375: sound: Fix min/max rate for SNDCTL_AUDIOINFO and SNDCTL_ENGINEINFO
rG86585210fd56: sound: Fix min/max rate for SNDCTL_AUDIOINFO and SNDCTL_ENGINEINFO
Summary

Since we allow feeding of any rate within the [feeder_rate_min,
feeder_rate_max] range, report this as the min/max rate as well. Only
exceptions are when we the device is opened in exclusive or bitperfect
mode.

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 58487
Build 55375: arc lint + arc unit

Event Timeline

christos created this revision.Jul 3 2024, 7:10 PM
Herald added a subscriber: imp. · View Herald TranscriptJul 3 2024, 7:10 PM
christos requested review of this revision.Jul 3 2024, 7:10 PM
Harbormaster completed remote builds in B58485: Diff 140532.Jul 3 2024, 7:10 PM
christos mentioned this in D45664: sound: Fix min/max sample rate assignment for VCHANs.Jul 3 2024, 7:10 PM
christos added inline comments.
sys/dev/sound/pcm/dsp.c
2124

I did this for AUDIOINFO since we can feed arbitrary rates to physical channels as well.

christos added inline comments.Jul 3 2024, 7:21 PM
sys/dev/sound/pcm/dsp.c
2124

Didn't get the EXCL and BITPERFECT cases into account. Uploading fixed diff in a sec.

christos updated this revision to Diff 140535.Jul 3 2024, 7:40 PM

Handle EXCLUSIVE and BITPERFECT cases.

Harbormaster completed remote builds in B58487: Diff 140535.Jul 3 2024, 7:40 PM
christos edited the summary of this revision. (Show Details)Jul 3 2024, 7:41 PM
christos added a comment.Jul 3 2024, 7:47 PM

sndstat_get_caps() probably also needs fixing, but I will do this in a separate patch.

dev_submerge.ch accepted this revision.Jul 4 2024, 3:23 PM

Looks good to me, in my tests with ossinfo this returns the expected sample rate values.

sys/dev/sound/pcm/dsp.c
2210–2212

I think this comment should be adapted on occasion.

This revision is now accepted and ready to land.Jul 4 2024, 3:23 PM
christos added inline comments.Jul 4 2024, 3:28 PM
sys/dev/sound/pcm/dsp.c
2210–2212

I think we could even get rid of it. It also exists in AUDIOINFO.

christos added a comment.Jul 4 2024, 4:00 PM
In D45862#1045693, @christos wrote:

sndstat_get_caps() probably also needs fixing, but I will do this in a separate patch.

D45872

Closed by commit rG86585210fd56: sound: Fix min/max rate for SNDCTL_AUDIOINFO and SNDCTL_ENGINEINFO (authored by christos). · Explain WhyJul 6 2024, 6:24 PM
This revision was automatically updated to reflect the committed changes.
christos added a commit: rG86585210fd56: sound: Fix min/max rate for SNDCTL_AUDIOINFO and SNDCTL_ENGINEINFO.
christos added a commit: rG95d3abff7375: sound: Fix min/max rate for SNDCTL_AUDIOINFO and SNDCTL_ENGINEINFO.Jul 10 2024, 4:49 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
sound/
pcm/
20 lines

Diff 140535

View Options

sys/dev/sound/pcm/dsp.c

Loading...