Re: [Alsa-user] hda-intel: MSI + controller power savings issue

[Takashi I. <ti...@su...>]

At Sun, 06 Jan 2013 04:29:57 +0100,
Mau wrote:
> 
> Hello,
> 
> I'm running current Debian testing (Wheezy), KDE with pulseaudio, with
> the latest Liquorix kernel (3.7.0-1.dmz.6-liquorix-amd64) on a Clevo
> W150HRM laptop (Realtek ALC269VB codec).
> 
> As soon as I unplug ac, the hda_intel power saving features get turned
> on by the /usr/lib/pm-utils/power.d/intel-audio-powersave script
> supplied by the pm-tools package, which echoes "1" to
> /sys/module/snd_hda_intel/parameters/power_save; after that, and until I
> disable hda_intel power savings, all the sounds I get are prolonged and
> interrupted by pauses of silence. I'm quite sure that this issue wasn't
> present before 3.7 kernel.
> 
> Either disabling MSIs or disabling controller power savings seem to fix
> the issue (I'm currently running with enable_msi=0).
> 
> Any idea?

Which HD-audio controller do you have?  Give lspci -v output.
If it's a Cougar Point or Panther Point, it's likely the regression in
3.7 kernel due to the runtime D3 support.  The patch below should fix
that problem.


Takashi

---
From: Takashi Iwai <ti...@su...>
Subject: [PATCH] ALSA: hda - Disable runtime D3 for Intel CPT & co

We've got a few bug reports that the runtime D3 results in the dead
HD-audio controller.  It seems that the problem is in a deeper level
than the sound driver itself, so as a temporal solution, disable the
feature for these controllers again.

Reported-by: Vincent Blut <vin...@fr...>
Reported-by: Maurizio Avogadro <ma...@gm...>
Cc: <st...@vg...> [v3.7]
Signed-off-by: Takashi Iwai <ti...@su...>
---
 sound/pci/hda/hda_intel.c | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
index cca8727..0b6aeba 100644
--- a/sound/pci/hda/hda_intel.c
+++ b/sound/pci/hda/hda_intel.c
@@ -573,9 +573,12 @@ enum {
 #define AZX_DCAPS_PM_RUNTIME	(1 << 26)	/* runtime PM support */
 
 /* quirks for Intel PCH */
-#define AZX_DCAPS_INTEL_PCH \
+#define AZX_DCAPS_INTEL_PCH_NOPM \
 	(AZX_DCAPS_SCH_SNOOP | AZX_DCAPS_BUFSIZE | \
-	 AZX_DCAPS_COUNT_LPIB_DELAY | AZX_DCAPS_PM_RUNTIME)
+	 AZX_DCAPS_COUNT_LPIB_DELAY)
+
+#define AZX_DCAPS_INTEL_PCH \
+	(AZX_DCAPS_INTEL_PCH_NOPM | AZX_DCAPS_PM_RUNTIME)
 
 /* quirks for ATI SB / AMD Hudson */
 #define AZX_DCAPS_PRESET_ATI_SB \
@@ -3586,13 +3589,13 @@ static void azx_remove(struct pci_dev *pci)
 static DEFINE_PCI_DEVICE_TABLE(azx_ids) = {
 	/* CPT */
 	{ PCI_DEVICE(0x8086, 0x1c20),
-	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH },
+	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH_NOPM },
 	/* PBG */
 	{ PCI_DEVICE(0x8086, 0x1d20),
-	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH },
+	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH_NOPM },
 	/* Panther Point */
 	{ PCI_DEVICE(0x8086, 0x1e20),
-	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH },
+	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH_NOPM },
 	/* Lynx Point */
 	{ PCI_DEVICE(0x8086, 0x8c20),
 	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH },
-- 
1.8.0.1