Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754911Ab0FVTcM (ORCPT ); Tue, 22 Jun 2010 15:32:12 -0400 Received: from bar.sig21.net ([80.81.252.164]:40746 "EHLO bar.sig21.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750986Ab0FVTcL (ORCPT ); Tue, 22 Jun 2010 15:32:11 -0400 Date: Tue, 22 Jun 2010 21:31:43 +0200 From: Johannes Stezenbach To: Matthew Garrett Cc: "Luis R. Rodriguez" , Jussi Kivilinna , Maxim Levitsky , David Quan , Bob Copeland , "Luis R. Rodriguez" , ath5k-devel@venema.h4ckr.net, linux-wireless@vger.kernel.org, linux-kernel , Jonathan May , Tim Gardner Subject: Re: [ath5k-devel] [PATCH v2] ath5k: disable ASPM Message-ID: <20100622193143.GA17803@sig21.net> References: <20100622163138.GD20668@srcf.ucam.org> <20100622165213.GA21842@srcf.ucam.org> <20100622172545.GA22680@srcf.ucam.org> <20100622175058.GA23499@srcf.ucam.org> <20100622184426.GA24546@srcf.ucam.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20100622184426.GA24546@srcf.ucam.org> User-Agent: Mutt/1.5.20 (2009-06-14) X-Spam-21-Score: -3.6 (---) X-Spam-21-Report: No, score=-3.6 required=5.0 tests=ALL_TRUSTED=-1.8,AWL=0.775,BAYES_00=-2.599 autolearn=no Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2082 Lines: 44 On Tue, Jun 22, 2010 at 07:44:26PM +0100, Matthew Garrett wrote: > On Tue, Jun 22, 2010 at 11:28:20AM -0700, Luis R. Rodriguez wrote: > > > > Heh, this whole patch and thread was started because Jussi tested > > ath5k with pcie_aspm=force (on a pre PCIE 1.1 device (?)) . I have > > been trying to explain all along why this is a terrible idea to the > > point we should probably just remove that code from the kernel. Hence > > my side rants and explanations to justify my reasonings. > > Well, there's two things here. If you use force then you might get > inappropriate ASPM. But if your BIOS enables ASPM on an old device, then > booting *without* CONFIG_PCIE_ASPM will leave it turned on, and booting > *with* CONFIG_PCIE_ASPM will turn it off. The Kconfig description is > confusing - reality is that CONFIG_PCIE_ASPM enables logic that allows > the kernel to modify the BIOS default, and disabling it makes the > assumption that your BIOS did something sensible. Does CONFIG_PCIEASPM provide a way for the user to modifiy the settings at runtime? I have a Samsung N130 netbook which has a BIOS setting called "CPU Power Saving Mode". When enabled it activates ASPM L1 and L0s for the ethernet chip (Realtek RTL8102e, 100Mbit) and the PCIE bridge (with the BIOS setting off it's just L1). The result is that the ethernet througput is reduced to 25Mbit/s. (The BIOS setting does not activa L0s for the Atheros AR9285 WLAN.) 99,9% of the time I want to enjoy the power savings, but occationally I have to transfer some bulk data and would like to switch the setting for a few minutes. Or, well, ideally I'd like to have power savings _and_ performance at the same time without any manual intervention. I'm not sure if this is a quirk of the N130 or if ASPM L0s always causes performance degradation? Thanks Johannes -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/