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
F98916538: D45862.diff
Sat, Oct 5, 7:13 AM
Unknown Object (File)
Tue, Sep 24, 5:28 AM
Unknown Object (File)
Mon, Sep 23, 10:11 AM
Unknown Object (File)
Sun, Sep 22, 10:56 PM
Unknown Object (File)
Sun, Sep 22, 7:32 PM
Unknown Object (File)
Sun, Sep 22, 9:02 AM
Unknown Object (File)
Sat, Sep 21, 6:07 PM
Unknown Object (File)
Wed, Sep 18, 5:45 PM
View All 32 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 58485
Build 55373: 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
2123

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
2123

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
2203–2205

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
2203–2205

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/
11 lines

Diff 140532

View Options

sys/dev/sound/pcm/dsp.c

Loading...