On Fri, 2013-03-29 at 14:08 +0100, Martin Mokrejs wrote: > > Huang Ying wrote: > > On Thu, 2013-03-28 at 19:38 +0100, Martin Mokrejs wrote: > >> Hi Ying, > >> would you please tell me how this report relate to this patch? > >> > >> [PATCH] PCI / ACPI: Always resume devices on ACPI wakeup notifications > >> > >> Could you tell me why this PME was being flipped back and forth now? > >> Actually, does that make finally some sense to you, pci/acpi devs? > >> > >> > >> Does is help to say that on the SandyBridge chip I have the following root ports > >> hooked to the following end devices?: > >> > >> 1.c1 -> rtl8169 05:00.0 > >> 1.c3 -> iwlwifi 09:00.0 > >> 1.c4 -> xhci_hcd 0b:00.0 > >> 1.c7 -> 00:11: express card slot > >> > >> > >> Why didn't I see interleraved lines with 1.c7 *and* 00:11? See the interleaving > >> happening with the network card on 3.7.10 kernel (not broken kernel): > >> > >> [138268.870070] r8169 0000:05:00.0 eth0: link down > >> [138270.809811] r8169 0000:05:00.0 eth0: link up > >> [138365.599744] r8169 0000:05:00.0 eth0: link down > >> [138370.594343] r8169 0000:05:00.0: PME# enabled > >> [138370.623852] pcieport 0000:00:1c.1: PME# enabled > >> [169885.247386] pcieport 0000:00:1c.1: PME# disabled > >> [169885.267374] r8169 0000:05:00.0: PME# disabled > >> [169885.330160] r8169 0000:05:00.0 eth0: link down > >> [169886.992531] r8169 0000:05:00.0 eth0: link up > >> [169904.405769] r8169 0000:05:00.0 eth0: link down > >> [169909.401237] r8169 0000:05:00.0: PME# enabled > >> [169909.430782] pcieport 0000:00:1c.1: PME# enabled > >> [170090.538980] pcieport 0000:00:1c.1: PME# disabled > >> [170090.559088] r8169 0000:05:00.0: PME# disabled > >> [170090.640494] r8169 0000:05:00.0: PME# enabled > >> [170090.678425] pcieport 0000:00:1c.1: PME# enabled > >> [170090.829959] pcieport 0000:00:1c.1: PME# disabled > >> [170090.848479] r8169 0000:05:00.0: PME# disabled > >> [170090.892011] r8169 0000:05:00.0 eth0: link down > >> [170090.892134] r8169 0000:05:00.0 eth0: link down > >> [170090.930998] r8169 0000:05:00.0 eth0: link down > >> [170092.554553] r8169 0000:05:00.0 eth0: link up > > > > I don't know exactly. Can you give me the follow output? > > > > grep . /sys/bus/pci/devices/*/power/control > > grep . /sys/bus/pci/devices/*/power/runtime_status > > > Hi Ying, > here is the output from the not-broken 3.7.10 kernel: > > vostro ~ # uname -a > Linux vostro 3.7.10-default #2 SMP Tue Mar 5 09:52:12 MET 2013 x86_64 Intel(R) Core(TM) i7-2640M CPU @ 2.80GHz GenuineIntel GNU/Linux > vostro ~ # grep . /sys/bus/pci/devices/*/power/control > /sys/bus/pci/devices/0000:00:00.0/power/control:auto > /sys/bus/pci/devices/0000:00:02.0/power/control:auto > /sys/bus/pci/devices/0000:00:16.0/power/control:auto > /sys/bus/pci/devices/0000:00:1a.0/power/control:auto > /sys/bus/pci/devices/0000:00:1b.0/power/control:auto > /sys/bus/pci/devices/0000:00:1c.0/power/control:auto > /sys/bus/pci/devices/0000:00:1c.1/power/control:auto > /sys/bus/pci/devices/0000:00:1c.3/power/control:auto > /sys/bus/pci/devices/0000:00:1c.4/power/control:auto > /sys/bus/pci/devices/0000:00:1c.7/power/control:auto > /sys/bus/pci/devices/0000:00:1d.0/power/control:auto > /sys/bus/pci/devices/0000:00:1f.0/power/control:auto > /sys/bus/pci/devices/0000:00:1f.2/power/control:auto > /sys/bus/pci/devices/0000:00:1f.3/power/control:auto > /sys/bus/pci/devices/0000:05:00.0/power/control:auto > /sys/bus/pci/devices/0000:09:00.0/power/control:auto > /sys/bus/pci/devices/0000:0b:00.0/power/control:auto > /sys/bus/pci/devices/0000:11:00.0/power/control:auto I guess you use laptop-mode-tool to turn on runtime PM for all PCI devices. Is possible for you to turn off runtime PM for PCI devices in laptop-mode-tool if you use it or by hand as follow and test it again to check whether it is related to runtime PM support? runtime PM for PCI devices can be turned off via the following command line as root: for ctrl in /sys/bus/pci/devices/*/power/control; do echo on > $ctrl done Best Regards, Huang Ying > vostro ~ # grep . /sys/bus/pci/devices/*/power/runtime_status > /sys/bus/pci/devices/0000:00:00.0/power/runtime_status:suspended > /sys/bus/pci/devices/0000:00:02.0/power/runtime_status:active > /sys/bus/pci/devices/0000:00:16.0/power/runtime_status:suspended > /sys/bus/pci/devices/0000:00:1a.0/power/runtime_status:active > /sys/bus/pci/devices/0000:00:1b.0/power/runtime_status:active > /sys/bus/pci/devices/0000:00:1c.0/power/runtime_status:suspended > /sys/bus/pci/devices/0000:00:1c.1/power/runtime_status:active > /sys/bus/pci/devices/0000:00:1c.3/power/runtime_status:active > /sys/bus/pci/devices/0000:00:1c.4/power/runtime_status:active > /sys/bus/pci/devices/0000:00:1c.7/power/runtime_status:active > /sys/bus/pci/devices/0000:00:1d.0/power/runtime_status:active > /sys/bus/pci/devices/0000:00:1f.0/power/runtime_status:active > /sys/bus/pci/devices/0000:00:1f.2/power/runtime_status:active > /sys/bus/pci/devices/0000:00:1f.3/power/runtime_status:suspended > /sys/bus/pci/devices/0000:05:00.0/power/runtime_status:active > /sys/bus/pci/devices/0000:09:00.0/power/runtime_status:active > /sys/bus/pci/devices/0000:0b:00.0/power/runtime_status:active > /sys/bus/pci/devices/0000:11:00.0/power/runtime_status:active > vostro ~ # > > > > Now will go and test your patch for the 3.9-rc1 PME# storm when the coldplugged eSATA > is unplugged and rmmod sata_sil24 triggers the messages. > > Martin -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
