Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

From: Takashi Iwai
Date: Tue Jul 04 2017 - 11:28:21 EST

On Tue, 04 Jul 2017 17:14:39 +0200,
Daniel Vetter wrote:
> On Thu, Jun 29, 2017 at 12:25 PM, Daniel Vetter <daniel@xxxxxxxx> wrote:
> >> Now I took a closer look, and this appears rather like a brown paper
> >> bag bug, not about the deferred probe or module dependency.
> >> The fix patch is below. Could you check whether it works?
> >
> > Yay, this works!
> >
> > Tested-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
> Next one: i915 module reloading is broken because something is holding
> onto a module reference and doesn't drop it. Didn't check which sets
> of patches introduced this, but iirc this worked last week. Disabling
> hda-intel gets in Kconfig gets rid of the problem, so I assume
> something in the sound driver is leaking that reference ...
> It's also causing lots and lots of red in our CI :( If we can't fix
> this we need to disable snd-hda-intel there too.

I spotted out a typo in my previous patch that leads to the module
reference unbalance. The fix is already in sound.git tree today:

The bug was introduced after 4.12.


Takashi Iwai <tiwai@xxxxxxx>