Return-path: Received: from mail-la0-f52.google.com ([209.85.215.52]:52535 "EHLO mail-la0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757021Ab3C3Sie (ORCPT ); Sat, 30 Mar 2013 14:38:34 -0400 MIME-Version: 1.0 In-Reply-To: References: <20130304134949.GB3021@redhat.com> <20130304154832.GD3021@redhat.com> Date: Sat, 30 Mar 2013 21:38:32 +0300 Message-ID: (sfid-20130330_193839_551413_699AC9D0) Subject: Re: is L1 really disabled in iwlwifi From: Emmanuel Grumbach To: Bjorn Helgaas Cc: Stanislaw Gruszka , "linux-pci@vger.kernel.org" , linux-wireless , John Linville , Roman Yepishev Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-wireless-owner@vger.kernel.org List-ID: >> >> Anyway - I think I will just remove this pci_disable_link_state call >> and hopefully, it will stay long enough in linux-next so that people >> will report issues before it get into linux. >> I don't like leaving that code if it doesn't do anything. > > I think we broke at least some cases of pci_disable_link_state() a > while back. I'd like to fix it rather than just removing calls to it. > > Can someone collect a complete dmesg log and "lspci -vv" output? > I will. But I am on vacation right now and my linux box is not accessible right now... In any case, what you'll see is that ASPM is enabled (L1 enabled with or without driver). We are also seeing some really weird stuff (which I can't reproduce of course) like HW becoming not accessible, I guess it would be worth trying to *really* disable L1. I can tweak that in the code of the driver and tell the NIC that L1 is disabled. /* Disable L1-Active */ iwl_set_bits_prph(trans, APMG_PCIDEV_STT_REG, APMG_PCIDEV_STT_VAL_L1_ACT_DIS); Do you have an idea about *when* did pci_disable_link_state() break?