[alsa-devel] [regression] ALSA: hda - Handle open while transitioning to D3.

[Dylan Reid]

On Sat, Jul 14, 2012 at 6:36 AM, Oleksij Rempel (fishor)
<bug-track at fisher-privat.net> wrote:
> dmesg is in attachment
Thanks Oleksij,

I didn't get much from that though.  What is your powersave timeout
set to?  If you have time could you try adding the following and send
me the output, I have two guesses about where it is spending it's time
(no guesses about why yet though).  I've got a couple of systems with
a 269, I'll try to get some tests run on those tomorrow.

diff --git a/sound/pci/hda/hda_codec.c b/sound/pci/hda/hda_codec.c
index 20580bf..fd8ce9e 100644
--- a/sound/pci/hda/hda_codec.c
+++ b/sound/pci/hda/hda_codec.c
@@ -4383,9 +4383,11 @@ static void hda_power_work(struct work_struct *work)
        }
        spin_unlock(&codec->power_lock);

+       snd_printd("power_work start %p\n", codec);;
        hda_call_codec_suspend(codec);
        if (bus->ops.pm_notify)
                bus->ops.pm_notify(bus);
+       snd_printd("power_work done %p\n", codec);;
 }

 static void hda_keep_power_on(struct hda_codec *codec)
@@ -4426,7 +4428,9 @@ static void __snd_hda_power_up(struct hda_codec
*codec, bool wait_power_down)
        }
        spin_unlock(&codec->power_lock);

+       snd_printd("power_up cancel sync start %p\n", codec);
        cancel_delayed_work_sync(&codec->power_work);
+       snd_printd("power_up cancel sync done %p\n", codec);

        spin_lock(&codec->power_lock);
        /* If the power down delayed work was cancelled above before starting,

>
>
> On 14.07.2012 11:21, Dylan Reid wrote:
>>
>> On Jul 13, 2012 11:01 PM, "Oleksij Rempel (fishor)"
>> <bug-track at fisher-privat.net <mailto:bug-track at fisher-privat.net>> wrote:
>>  >
>>  > Hallo Dylan, Takashi,
>>  >
>>  > I have regression after patch b4a91cf05c3 "ALSA: hda - Handle open
>> while transitioning to D3."
>>  >
>>  > This regression i can reproduce only if my laptop works on batteries.
>> After this patch it needs almost 1 minute more to start. The login
>> screen usually plays start sound, but after this patch it wait until
>> sound card is ready.
>>  >
>>  > My hardware: asus zenbook ux31e.
>>  >
>>  > Codec: Realtek ALC269VB
>>  >
>>  > 00:1b.0 0403: 8086:1c20 (rev 05)
>>  >         Subsystem: 1043:1427
>>  >         Control: I/O- Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop-
>> ParErr- Stepping- SERR- FastB2B- DisINTx+
>>  >         Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=fast
>>  >TAbort- <TAbort- <MAbort- >SERR- <PERR- INTx-
>>  >         Latency: 0, Cache Line Size: 64 bytes
>>  >         Interrupt: pin A routed to IRQ 50
>>  >         Region 0: Memory at dfe00000 (64-bit, non-prefetchable)
>> [size=16K]
>>  >         Capabilities: <access denied>
>>  >         Kernel driver in use: snd_hda_intel
>>  >         Kernel modules: snd-hda-intel
>>  >
>>
>> Could you enable config_snd_debug and send the output during startup?
>> I'm actually using the same codec on one of our systems.
>>
>> Thanks for testing.
>>
>> Dylan
>>
>
>