Huang Ying wrote: > 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 > Hmm, the startup script claimed that while on AC they are turned off. Here is part of my /etc/laptop-mode/laptop-mode.conf # # Enable laptop mode when on battery power. # ENABLE_LAPTOP_MODE_ON_BATTERY=1 # # Enable laptop mode when on AC power. # ENABLE_LAPTOP_MODE_ON_AC=0 # # Enable laptop mode when the laptop's lid is closed, even when we're on AC # power? (ACPI-ONLY) # ENABLE_LAPTOP_MODE_WHEN_LID_CLOSED=0 # # Enable all simple zero-configuration auto modules # This option enables all simple modules (listed below) without requiring # the user to enable each module individually # # List of modules which can be automatically enabled with this setting are: # # ac97-powersave # cpufreq # dpms-standby # eee-superhe # ethernet # exec-commands # hal-polling # hdparm # intel-hda-powersave # intel-sata-powermgmt # nmi-watchdog # runtime-pm # sched-mc-power-savings # sched-smt-power-savings # terminal-blanking # usb-autosuspend # wireless-ipw-power # wireless-iwl-power # wireless-power # # Set this to 1 to enable all simple zero-configuration auto modules listed above. # # NOTE: You can explicitly enable/disable any of the above modules by changing their # values in the individual settings file # ENABLE_AUTO_MODULES=1 OK, will uninstall app-laptop/laptop-mode-tools-1.62-r1 now. I am sorry for the confusion. > 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
