On Tue, Nov 12, 2013 at 12:37 PM, Emmanuel Grumbach <egrumbach@xxxxxxxxx> wrote: > On 11/12/2013 09:14 PM, Bjorn Helgaas wrote: >> On Tue, Nov 12, 2013 at 11:25 AM, Grumbach, Emmanuel >> <emmanuel.grumbach@xxxxxxxxx> wrote: >> >>> Right - I remember the discussion we had on that. >>> On this device (7260 that has an issue with ASPM), we don't call pci_disable_link_state, because we know it is supposed to work... >> >> If ASPM is supposed to work as far as the hardware is concerned, I >> guess you're saying this must be an iwlwifi driver issue. Right? > > ASPM is supposed to work as far as the hardware is concerned. > We might very well have an issue in iwlwifi - and I am checking this > internally with our System guys. > It can be a PCI core problem too, and it could also be a platform / BIOS > / Lenovo issue. > Of course, I have no clue which of these is the culprit here. > Our System folks seemed to say that this new device uses L1 substates > which can be enabled in Haswell platform which the user owns. > Now - L1 substates is a new feature and might introduce issues > (apparently) - and this is why they (System folks) wanted the try > without L1 substates. But disabling L1 substates doesn't seem trivial > with the production BIOS of Lenovo. So I am pretty stuck here. For debugging purposes, we could configure L1 substates with setpci, as we did for ASPM. The Linux kernel knows nothing about L1 substates, so the PCI core isn't doing anything with them. It's possible the driver itself could muck with L1 substate configuration, but that would be discouraged, and I don't see anything in iwlwifi that is doing that. The lspci output in https://bugzilla.kernel.org/attachment.cgi?id=114061 shows an L1 PM Substates extended capability (capability ID 0x1e) for the Root Port leading to the 7260 device, but not for the 7260 device itself: 00:1c.1 PCI bridge: Intel Corporation Lynx Point-LP PCI Express Root Port 3 (rev e4) (prog-if 00 [Normal decode]) Capabilities: [200 v1] #1e Per sec 5.5.4 of the ECN for L1 PM Substates (15 Aug 2012), I think L1 substates must be configured on both ends of the link, and if the 7260 device doesn't have that capability, I don't see how it could be enabled. The lspci version wzyboy has doesn't decode the L1 PM Substates capability, but there is a newer version at git://git.kernel.org/pub/scm/utils/pciutils/pciutils.git that should decode it. Also, "lspci -vvxxx" didn't hexdump this capability, which should be at offset 0x200. Using "lspci -xxxx" (four "x"s) should dump it, and we can decode it manually. wzyboy, can you run these commands before the bug occurs and before using the "setpci" workaround: lspci -vvxxxx -s00:1c.1 lspci -vvxxxx -s03:00.0 Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
