Page MenuHomeFreeBSD
Differential D53558

sound: Remove unnecessary initializations in feeder_create()
ClosedPublic
Actions

Authored by christos on Nov 3 2025, 3:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 28, 7:14 AM
Unknown Object (File)
Fri, Dec 26, 9:18 AM
Unknown Object (File)
Wed, Dec 17, 12:10 AM
Unknown Object (File)
Dec 9 2025, 3:52 PM
Unknown Object (File)
Dec 5 2025, 1:41 PM
Unknown Object (File)
Dec 2 2025, 5:15 AM
Unknown Object (File)
Dec 1 2025, 1:26 PM
Unknown Object (File)
Nov 30 2025, 1:17 AM
View All 24 Files
Subscribers
imp

Details

Reviewers
markj
emaste
Commits
rGad348a08586f: sound: Remove unnecessary initializations in feeder_create()
rG699fd108c50c: sound: Remove unnecessary initializations in feeder_create()
Summary

We do not need to zero out fields since the struct is allocated with
M_ZERO. Also we no longer need to have a special case for the root
feeder.

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 68382
Build 65265: arc lint + arc unit

Event Timeline

christos created this revision.Nov 3 2025, 3:31 PM
Herald added a subscriber: imp. · View Herald TranscriptNov 3 2025, 3:31 PM
christos requested review of this revision.Nov 3 2025, 3:31 PM
Harbormaster completed remote builds in B68382: Diff 165742.Nov 3 2025, 3:31 PM
christos added a parent revision: D53557: sound: Retire pcm_feederdesc->type.Nov 3 2025, 3:31 PM
christos added a child revision: D53559: sound: Retire unused feeder_printchain().
markj added inline comments.Nov 3 2025, 6:35 PM
sys/dev/sound/pcm/feeder.c
104

It doesn't seem nice to assume that FEEDER_ROOT == 0?

christos added inline comments.Nov 4 2025, 12:45 PM
sys/dev/sound/pcm/feeder.c
104

What do you mean?

markj accepted this revision.Nov 4 2025, 12:54 PM
markj added inline comments.
sys/dev/sound/pcm/feeder.c
104

I misread the change, never mind.

This revision is now accepted and ready to land.Nov 4 2025, 12:54 PM
Closed by commit rG699fd108c50c: sound: Remove unnecessary initializations in feeder_create() (authored by christos). · Explain WhyNov 11 2025, 12:08 PM
This revision was automatically updated to reflect the committed changes.
christos added a commit: rG699fd108c50c: sound: Remove unnecessary initializations in feeder_create().
christos added a commit: rGad348a08586f: sound: Remove unnecessary initializations in feeder_create().Nov 18 2025, 12:57 AM

Revision Contents
Changeset List

PathSize
sys/
dev/
sound/
pcm/
11 lines

Diff 165742

View Options

sys/dev/sound/pcm/feeder.c

Loading...