sound: Retire MAXFEEDERS and feedercnt
AcceptedPublic
Actions

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

Tags

None

Referenced Files

	F134778174: D53549.id.diff
	Tue, Nov 4, 2:01 PM

	F134778156: D53549.id165732.diff
	Tue, Nov 4, 2:00 PM

	F134756093: D53549.diff
	Tue, Nov 4, 10:20 AM

	F134712026: D53549.id.diff
	Tue, Nov 4, 3:18 AM

	F134685332: D53549.id.diff
	Tue, Nov 4, 12:16 AM

	F134684956: D53549.diff
	Tue, Nov 4, 12:15 AM

	F134684649: D53549.id.diff
	Tue, Nov 4, 12:13 AM

	F134684191: D53549.id165732.diff
	Tue, Nov 4, 12:12 AM

View All 14 Files

Subscribers

imp

Details

Reviewers

markj
emaste

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 Skipped

Unit

Tests Skipped

Build Status

Buildable 68372
Build 65255: arc lint + arc unit

Event Timeline

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

Herald added a subscriber: imp. · View Herald TranscriptMon, Nov 3, 3:30 PM

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

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

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

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

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

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

Revision Contents
Changeset List

Path

Size

sys/

dev/

sound/

pcm/

feeder.c

12 lines

Diff 165732

View Options

sys/dev/sound/pcm/feeder.c

sound: Retire MAXFEEDERS and feedercntAcceptedPublicActions