Page MenuHomeFreeBSD
Differential D53554

sound: Retire feedertab_entry
ClosedPublic
Actions

Authored by christos on Nov 3 2025, 3:30 PM.
Tags
None
Referenced Files
F146512826: D53554.id.diff
Tue, Mar 3, 6:56 AM
F146512206: D53554.id165738.diff
Tue, Mar 3, 6:48 AM
F146512202: D53554.id165803.diff
Tue, Mar 3, 6:48 AM
F146512195: D53554.id166217.diff
Tue, Mar 3, 6:48 AM
Unknown Object (File)
Fri, Feb 20, 12:16 AM
Unknown Object (File)
Mon, Feb 16, 6:16 PM
Unknown Object (File)
Mon, Feb 16, 11:37 AM
Unknown Object (File)
Tue, Feb 3, 12:19 PM
View All 35 Files
Subscribers
imp

Details

Reviewers
markj
emaste
Commits
rG2acd16ce68be: sound: Retire feedertab_entry
rGc24ca02c4c85: sound: Retire feedertab_entry
Summary

Have an SLIST of feeder_class directly. This way we simplify the code,
and also avoid the additional malloc()/free() for each entry.

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 68405
Build 65288: arc lint + arc unit

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 B68378: Diff 165738.Nov 3 2025, 3:30 PM
christos added a parent revision: D53553: sound: Simplify feeder_getclass().Nov 3 2025, 3:30 PM
christos added a child revision: D53555: sound: Retire feeder_class->data.
markj added inline comments.Nov 3 2025, 5:37 PM
sys/dev/sound/pcm/feeder.c
44

How is this list locked?

71

This loop can be replaced by SLIST_INIT(&feedertab);

christos updated this revision to Diff 165803.Nov 4 2025, 12:44 PM
christos marked an inline comment as done.

Address Mark's comment.

Harbormaster completed remote builds in B68405: Diff 165803.Nov 4 2025, 12:44 PM
christos added inline comments.Nov 4 2025, 12:45 PM
sys/dev/sound/pcm/feeder.c
44

It isn't. This list is created once after sound.ko has been loaded, and destroyed on unload. There is no adding/removing in the meantime.

markj accepted this revision.Nov 4 2025, 12:51 PM
This revision is now accepted and ready to land.Nov 4 2025, 12:51 PM
Closed by commit rGc24ca02c4c85: sound: Retire feedertab_entry (authored by christos). · Explain WhyNov 11 2025, 12:08 PM
This revision was automatically updated to reflect the committed changes.
christos added a commit: rGc24ca02c4c85: sound: Retire feedertab_entry.
christos added a commit: rG2acd16ce68be: sound: Retire feedertab_entry.Nov 18 2025, 12:52 AM

Revision Contents
Changeset List

PathSize
sys/
dev/
sound/
pcm/
1 line
37 lines

Diff 165803

View Options

sys/dev/sound/pcm/feeder.h

Loading...
View Options

sys/dev/sound/pcm/feeder.c

Loading...