Page MenuHomeFreeBSD

sound: Standardize mixer_init() call order
Needs ReviewPublic

Authored by christos on Fri, Aug 21, 12:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 11, 3:25 AM
Unknown Object (File)
Thu, Sep 10, 8:57 AM
Unknown Object (File)
Wed, Sep 9, 12:31 PM
Unknown Object (File)
Tue, Sep 8, 12:57 PM
Unknown Object (File)
Mon, Sep 7, 1:28 PM
Unknown Object (File)
Sun, Sep 6, 2:30 AM
Unknown Object (File)
Sun, Sep 6, 1:45 AM
Unknown Object (File)
Sat, Sep 5, 2:50 PM
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 75934
Build 72817: arc lint + arc unit