Page MenuHomeFreeBSD

sound: Standardize mixer_init() call order
Needs ReviewPublic

Authored by christos on Fri, Aug 21, 12:41 PM.
Tags
None
Referenced Files
F168953829: D59069.diff
Sun, Aug 30, 10:45 PM
F168935229: D59069.id184635.diff
Sun, Aug 30, 8:21 PM
Unknown Object (File)
Sun, Aug 30, 3:37 AM
Unknown Object (File)
Fri, Aug 28, 11:00 PM
Unknown Object (File)
Fri, Aug 28, 7:23 PM
Unknown Object (File)
Fri, Aug 28, 7:46 AM
Unknown Object (File)
Thu, Aug 27, 5:06 PM
Unknown Object (File)
Thu, Aug 27, 2:37 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