Page MenuHomeFreeBSD
Differential D46821

sound: Move root feeder initialization to separate function
AcceptedPublic
Actions

Authored by christos on Fri, Sep 27, 5:55 PM.
Tags
None
Referenced Files
F97722384: D46821.diff
Mon, Sep 30, 11:38 PM
Unknown Object (File)
Sun, Sep 29, 7:12 PM
Unknown Object (File)
Sun, Sep 29, 4:24 PM
Subscribers
imp
zlei

Details

Reviewers
markj
emaste
dev_submerge.ch
zlei
Summary

While here, fix some style errors.

Sponsored by: The FreeBSD Foundation
MFC after: 2 days

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 59632
Build 56519: arc lint + arc unit

Event Timeline

christos created this revision.Fri, Sep 27, 5:55 PM
Herald added a subscriber: imp. · View Herald TranscriptFri, Sep 27, 5:55 PM
christos requested review of this revision.Fri, Sep 27, 5:55 PM
Harbormaster completed remote builds in B59632: Diff 143821.Fri, Sep 27, 5:55 PM
christos mentioned this in D46749: sound: Move global variable initialization to sound_modevent().Fri, Sep 27, 5:55 PM
zlei accepted this revision.Sun, Sep 29, 3:15 AM
zlei added a subscriber: zlei.
zlei added inline comments.
sys/dev/sound/pcm/feeder.c
64

The SYSINIT is the slow path. I've not tested yet but I bet M_WAITOK will be OK. So that we can guarantee that the root feeder is always available.

This revision is now accepted and ready to land.Sun, Sep 29, 3:15 AM
christos added inline comments.Sun, Sep 29, 10:52 AM
sys/dev/sound/pcm/feeder.c
64

I think we could change to M_WAITOK in feeder_register() as well.

christos marked an inline comment as done.Sun, Sep 29, 1:34 PM
christos added inline comments.
sys/dev/sound/pcm/feeder.c
64

@zlei D46845

dev_submerge.ch accepted this revision.Sun, Sep 29, 2:40 PM
zlei added inline comments.Sun, Sep 29, 3:16 PM
sys/dev/sound/pcm/feeder.c
63

A nit here, you can assert, aka MPASS(feedercnt == 0), to ensure in future the change of order of SYSINIT will not break. The current change is absolutely OK, but just in case.

christos marked an inline comment as done.Sun, Sep 29, 3:33 PM
christos added inline comments.
sys/dev/sound/pcm/feeder.c
63

Good idea. Will commit the patch with the MPASS as well, but will the review here as-is here since it's accepted.

markj accepted this revision.Mon, Sep 30, 8:16 AM

When making a non-functional change like this, it's nice to state why in the commit message.

Revision Contents
Changeset List

PathSize
sys/
dev/
sound/
pcm/
68 lines

Diff 143821

View Options

sys/dev/sound/pcm/feeder.c

Loading...