sound: Bail out early if a format is not supported
Needs ReviewPublic
Actions

Authored by christos on Dec 11 2024, 4:47 PM.

Details

Reviewers

markj
emaste
dev_submerge.ch

Summary

If a format is unsupported, the feeder_build_* functions will fail
anyway, so bail out early to avoid unnecessary computation.

This is also needed by a series of upcoming patches to the feeder
framework.

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 61398
Build 58282: arc lint + arc unit

Event Timeline

christos created this revision.Dec 11 2024, 4:47 PM

Herald added a subscriber: imp. · View Herald TranscriptDec 11 2024, 4:47 PM

christos requested review of this revision.Dec 11 2024, 4:47 PM

Harbormaster completed remote builds in B61087: Diff 147819.Dec 11 2024, 4:47 PM

christos added inline comments.Dec 11 2024, 5:03 PM

sys/dev/sound/pcm/feeder_chain.c
712	Maybe `ENODEV` would be better.

christos mentioned this in D47932: sound: Refactor the format conversion framework.Dec 11 2024, 7:45 PM

dev_submerge.ch added inline comments.Dec 13 2024, 2:17 PM

sys/dev/sound/pcm/feeder_chain.c
578–579	This seems redundant with `AFMT_CONVERTIBLE` from `sound.h`?
706–712	Below we handle any format with bitperfect / passthrough. If these cases never hit `feeder_chain()` then the code below is superfluous, but I suspect it's not. Which means we cannot bail out early here?

christos added inline comments.Dec 15 2024, 11:01 PM

sys/dev/sound/pcm/feeder_chain.c
578–579	I am aware of it, but I thought the "convertible" part in the name would make the meaning of the operation non-obvious. I can however, use `AFMT_CONVERTIBLE` and add a comment, which should be fine.
706–712	That's a good point. I think the check could be moved to the `else` (non-bitperfect/non-passthrough) case instead.

Use AFMT_CONVERTIBLE.
Move check inside non-bitperfect/passthrough case.

Harbormaster completed remote builds in B61266: Diff 148253.Dec 19 2024, 5:51 PM

Bump.

Rebase.

Harbormaster completed remote builds in B61398: Diff 148537.Dec 29 2024, 8:29 PM

Bump.

Revision Contents
Changeset List

Path

Size

sys/

dev/

sound/

pcm/

feeder_chain.c

11 lines

Diff 148537

View Options

sys/dev/sound/pcm/feeder_chain.c

sound: Bail out early if a format is not supportedNeeds ReviewPublicActions