Page MenuHomeFreeBSD

sound: Standardize mixer_init() call order
AbandonedPublic

Authored by christos on Fri, Aug 21, 12:43 PM.
Tags
None
Referenced Files
F171003174: D59072.diff
Tue, Sep 8, 3:20 AM
Unknown Object (File)
Sun, Sep 6, 10:36 AM
Unknown Object (File)
Sat, Sep 5, 11:30 AM
Unknown Object (File)
Sat, Sep 5, 7:01 AM
Unknown Object (File)
Sat, Sep 5, 3:52 AM
Unknown Object (File)
Fri, Sep 4, 11:17 PM
Unknown Object (File)
Fri, Sep 4, 8:31 PM
Unknown Object (File)
Fri, Sep 4, 9:01 AM
Subscribers

Details

Summary

Device drivers call mixer_init() in a non-standard order - some before
pcm_init(), and some after pcm_register(). However, both approaches are
potentially risky, and logically weak, since pcm_register() is supposed
to be the function that finalizes sound(4) attach.

Standardize the call ordering by moving all mixer_init() calls after
pcm_init(). This is also necessary for a follow-up patch, which expects
pcm_init() to have run first and initialized the PCM lock.

Sponsored by: The FreeBSD Foundation
MFC after: 1 month

Diff Detail

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