Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-12 Thread Borislav Petkov
On Wed, Dec 12, 2012 at 03:20:25PM +, Lin, Mengdong wrote: > I remember I didn't observed repetitive attempts to suspend if the > azx_runtime_suspend() returns -EAGAIN. > > Because the HD-A driver does not implement the runtime idle > callback... It does now :-)

RE: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-12 Thread Lin, Mengdong
> -Original Message- > From: Borislav Petkov [mailto:b...@alien8.de] > Sent: Wednesday, December 12, 2012 6:52 PM > > It looks like azx_runtime_suspend() is new in 3.7 and it returns > > -EAGAIN to indicate that it actually can't be suspended (if my > > understanding the code is

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-12 Thread Takashi Iwai
PM check to runtime_idle callback > > > > The runtime_idle callback is the right place to check the suspend > > capability, but currently we do it wrongly in the runtime_suspend > > callback. This leads to a kernel error message like: > >pci_pm_runtime_suspend()

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-12 Thread Borislav Petkov
nd > capability, but currently we do it wrongly in the runtime_suspend > callback. This leads to a kernel error message like: > pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] > returns -11 > and the runtime PM core would even repeat the attempts. >

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-12 Thread Takashi Iwai
elow? thanks, Takashi --- From: Takashi Iwai Subject: [PATCH] ALSA: hda - Move runtime PM check to runtime_idle callback The runtime_idle callback is the right place to check the suspend capability, but currently we do it wrongly in the runtime_suspend callback. This leads to a kernel error messag

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-12 Thread Borislav Petkov
On Wed, Dec 12, 2012 at 12:44:33AM +0100, Rafael J. Wysocki wrote: > On Tuesday, December 11, 2012 06:55:08 PM Borislav Petkov wrote: > > On Tue, Dec 11, 2012 at 06:48:23PM +0100, Rafael J. Wysocki wrote: > > > Boris, please send the output of "lspci -vvv' from that box. > > > > Attached. > > So

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-12 Thread Borislav Petkov
On Wed, Dec 12, 2012 at 12:44:33AM +0100, Rafael J. Wysocki wrote: On Tuesday, December 11, 2012 06:55:08 PM Borislav Petkov wrote: On Tue, Dec 11, 2012 at 06:48:23PM +0100, Rafael J. Wysocki wrote: Boris, please send the output of lspci -vvv' from that box. Attached. So the audio is

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-12 Thread Takashi Iwai
- Move runtime PM check to runtime_idle callback The runtime_idle callback is the right place to check the suspend capability, but currently we do it wrongly in the runtime_suspend callback. This leads to a kernel error message like: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-12 Thread Borislav Petkov
capability, but currently we do it wrongly in the runtime_suspend callback. This leads to a kernel error message like: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11 and the runtime PM core would even repeat the attempts. Reported-by: Borislav Petkov b

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-12 Thread Takashi Iwai
The runtime_idle callback is the right place to check the suspend capability, but currently we do it wrongly in the runtime_suspend callback. This leads to a kernel error message like: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11 and the runtime PM

RE: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-12 Thread Lin, Mengdong
-Original Message- From: Borislav Petkov [mailto:b...@alien8.de] Sent: Wednesday, December 12, 2012 6:52 PM It looks like azx_runtime_suspend() is new in 3.7 and it returns -EAGAIN to indicate that it actually can't be suspended (if my understanding the code is correct).

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-12 Thread Borislav Petkov
On Wed, Dec 12, 2012 at 03:20:25PM +, Lin, Mengdong wrote: I remember I didn't observed repetitive attempts to suspend if the azx_runtime_suspend() returns -EAGAIN. Because the HD-A driver does not implement the runtime idle callback... It does now :-)

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-11 Thread Rafael J. Wysocki
On Tuesday, December 11, 2012 06:55:08 PM Borislav Petkov wrote: > On Tue, Dec 11, 2012 at 06:48:23PM +0100, Rafael J. Wysocki wrote: > > Boris, please send the output of "lspci -vvv' from that box. > > Attached. So the audio is a Root Complex Integrated Endpoind and there shouldn't be any

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-11 Thread Borislav Petkov
On Tue, Dec 11, 2012 at 06:48:23PM +0100, Rafael J. Wysocki wrote: > Boris, please send the output of "lspci -vvv' from that box. Attached. -- Regards/Gruss, Boris. Sent from a fat crate under my desk. Formatting is fine. -- 00:00.0 Host bridge: Advanced Micro Devices [AMD] Family 14h

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-11 Thread Rafael J. Wysocki
tual/input/input14 > [ 29.550494] pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 > [snd_hda_intel] returns -11 <-- HERE > [ 29.610630] powernow-k8: this CPU is not supported anymore, using > acpi-cpufreq instead. > [ 29.676307] acpi-cpufreq: overriding BIOS provided _P

pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-11 Thread Borislav Petkov
Hi guys, I've got the message tagged "HERE" in dmesg running 3.7 (of course, this is a new message): ... [ 27.41] input: ACPI Virtual Keyboard Device as /devices/virtual/input/input14 [ 29.550494] pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] r

pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-11 Thread Borislav Petkov
Hi guys, I've got the message tagged HERE in dmesg running 3.7 (of course, this is a new message): ... [ 27.41] input: ACPI Virtual Keyboard Device as /devices/virtual/input/input14 [ 29.550494] pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-11 Thread Rafael J. Wysocki
] pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11 -- HERE [ 29.610630] powernow-k8: this CPU is not supported anymore, using acpi-cpufreq instead. [ 29.676307] acpi-cpufreq: overriding BIOS provided _PSD data [ 32.575161] Program wdm tried to access /dev/mem between

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-11 Thread Borislav Petkov
On Tue, Dec 11, 2012 at 06:48:23PM +0100, Rafael J. Wysocki wrote: Boris, please send the output of lspci -vvv' from that box. Attached. -- Regards/Gruss, Boris. Sent from a fat crate under my desk. Formatting is fine. -- 00:00.0 Host bridge: Advanced Micro Devices [AMD] Family 14h

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-11 Thread Rafael J. Wysocki
On Tuesday, December 11, 2012 06:55:08 PM Borislav Petkov wrote: On Tue, Dec 11, 2012 at 06:48:23PM +0100, Rafael J. Wysocki wrote: Boris, please send the output of lspci -vvv' from that box. Attached. So the audio is a Root Complex Integrated Endpoind and there shouldn't be any problems