<html> <head> <base href="https://bugs.freedesktop.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_RESOLVED bz_closed" title="RESOLVED FIXED - [CI][BAT] igt@pm_rpm@module-reload - dmesg-warn - Use count on domain AUDIO is already zero" href="https://bugs.freedesktop.org/show_bug.cgi?id=109513#c19">Comment # 19</a> on <a class="bz_bug_link bz_status_RESOLVED bz_closed" title="RESOLVED FIXED - [CI][BAT] igt@pm_rpm@module-reload - dmesg-warn - Use count on domain AUDIO is already zero" href="https://bugs.freedesktop.org/show_bug.cgi?id=109513">bug 109513</a> from <a class="email" href="mailto:martin.peres@free.fr" title="Martin Peres <martin.peres@free.fr>"> Martin Peres</a> <pre>(In reply to Chris Wilson from <a href="show_bug.cgi?id=109513#c18">comment #18</a>) > commit 4bdc42094d9c4af75f90ee49b9009ea8dfe41444 (HEAD -> > drm-intel-next-queued, > drm-intel/for-linux-next, drm-intel/drm-intel-next-queued) > Author: Takashi Iwai <<a href="mailto:tiwai@suse.de">tiwai@suse.de</a>> > Date: Wed Apr 10 13:03:22 2019 +0200 > > ALSA: hda: Fix racy display power access > > snd_hdac_display_power() doesn't handle the concurrent calls carefully > enough, and it may lead to the doubly get_power or put_power calls, > when a runtime PM and an async work get called in racy way. > > This patch addresses it by reusing the bus->lock mutex that has been > used for protecting the link state change in ext bus code, so that it > can protect against racy display state changes. The initialization of > bus->lock was moved from snd_hdac_ext_bus_init() to > snd_hdac_bus_init() as well accordingly. > > Testcase: igt/i915_pm_rpm/module-reload #glk-dsi > Reported-by: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>> > Cc: Imre Deak <<a href="mailto:imre.deak@intel.com">imre.deak@intel.com</a>> > Signed-off-by: Takashi Iwai <<a href="mailto:tiwai@suse.de">tiwai@suse.de</a>> > Reviewed-by: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>> > Signed-off-by: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>> > Link: > <a href="https://patchwork.freedesktop.org/patch/msgid/s5h8swiunph.wl-tiwai@suse.de">https://patchwork.freedesktop.org/patch/msgid/s5h8swiunph.wl-tiwai@suse.de</a> It definitely fixed the issue! Thanks!</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> <li>You are the QA Contact for the bug.</li> <li>You are on the CC list for the bug.</li> </ul> </body> </html>