Page MenuHomeFreeBSD
Differential D53554

sound: Retire feedertab_entry
ClosedPublic
Actions

Authored by christos on Nov 3 2025, 3:30 PM.
Tags
None
Referenced Files
F163145783: D53554.id165803.diff
Mon, Jul 20, 12:32 PM
F163135603: D53554.diff
Mon, Jul 20, 10:05 AM
F163082718: D53554.diff
Sun, Jul 19, 10:24 PM
Unknown Object (File)
Sat, Jul 18, 4:28 AM
Unknown Object (File)
Wed, Jul 15, 6:53 AM
Unknown Object (File)
Wed, Jul 15, 6:53 AM
Unknown Object (File)
Sun, Jun 21, 2:55 PM
Unknown Object (File)
Jun 19 2026, 8:35 PM
View All 75 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 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 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 166217

View Options

sys/dev/sound/pcm/feeder.h

Loading...
View Options

sys/dev/sound/pcm/feeder.c

Loading...