[Bug 109513] [CI][BAT] igt at pm_rpm@module-reload - dmesg-warn - Use count on domain AUDIO is already zero

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=109513

--- Comment #19 from Martin Peres <martin.peres at free.fr> ---
(In reply to Chris Wilson from comment #18)
> commit 4bdc42094d9c4af75f90ee49b9009ea8dfe41444 (HEAD ->
> drm-intel-next-queued, 
> drm-intel/for-linux-next, drm-intel/drm-intel-next-queued)
> Author: Takashi Iwai <tiwai at suse.de>
> 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 <chris at chris-wilson.co.uk>
>     Cc: Imre Deak <imre.deak at intel.com>
>     Signed-off-by: Takashi Iwai <tiwai at suse.de>
>     Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
>     Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
>     Link:
> https://patchwork.freedesktop.org/patch/msgid/s5h8swiunph.wl-tiwai@suse.de

It definitely fixed the issue! Thanks!

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-gfx-bugs/attachments/20190417/77a3c3ae/attachment-0001.html>