sound: Retire MAXFEEDERS and feedercnt
ClosedPublic
Actions

Authored by christos on Nov 3 2025, 3:30 PM.

Details

Reviewers

markj
emaste

Commits

rGebe59a72abf7: sound: Retire MAXFEEDERS and feedercnt
rG84282785cc49: sound: Retire MAXFEEDERS and feedercnt

Summary

Apart from the fact that it's unrealistic to reach MAXFEEDERS (256),
since sound(4) comes with maximum 7 feeders (including feeder_root),
there is also no reason to cap it.

Sponsored by: The FreeBSD Foundation
MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

christos created this revision.Nov 3 2025, 3:30 PM

Herald added a subscriber: imp. · View Herald TranscriptNov 3 2025, 3:30 PM

christos requested review of this revision.Nov 3 2025, 3:30 PM

Harbormaster completed remote builds in B68372: Diff 165732.Nov 3 2025, 3:30 PM

christos added a parent revision: D53547: sound: Retire unused {pcm_feederdesc,feedertab_entry}->idx.Nov 3 2025, 3:30 PM

christos added a child revision: D53550: sound: Remove unnecessary pcm/vchan.h include from pcm/feeder.c.

markj accepted this revision.Nov 3 2025, 4:08 PM

This revision is now accepted and ready to land.Nov 3 2025, 4:08 PM

Closed by commit rG84282785cc49: sound: Retire MAXFEEDERS and feedercnt (authored by christos). · Explain WhyNov 11 2025, 12:08 PM

This revision was automatically updated to reflect the committed changes.

christos added a commit: rG84282785cc49: sound: Retire MAXFEEDERS and feedercnt.

christos added a commit: rGebe59a72abf7: sound: Retire MAXFEEDERS and feedercnt.Nov 18 2025, 12:52 AM

Revision Contents
Changeset List

Path

Size

sys/

dev/

sound/

pcm/

feeder.c

12 lines

Diff 166212

View Options

sys/dev/sound/pcm/feeder.c

sound: Retire MAXFEEDERS and feedercntClosedPublicActions