From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932496AbcKMCCd (ORCPT ); Sat, 12 Nov 2016 21:02:33 -0500 Received: from shards.monkeyblade.net ([184.105.139.130]:58154 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932167AbcKMCCc (ORCPT ); Sat, 12 Nov 2016 21:02:32 -0500 Date: Sat, 12 Nov 2016 21:02:24 -0500 (EST) Message-Id: <20161112.210224.1889131711533608896.davem@davemloft.net> To: kastbernd@gmx.de Cc: nic_swsd@realtek.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] r8169: Module parameter for opt-in of ASPM From: David Miller In-Reply-To: <20161103233306.GA9368@ASUS-Bernd.fritz.box> References: <20161103233306.GA9368@ASUS-Bernd.fritz.box> X-Mailer: Mew version 6.7 on Emacs 25.1 / Mule 6.0 (HANACHIRUSATO) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.12 (shards.monkeyblade.net [149.20.54.216]); Sat, 12 Nov 2016 17:02:58 -0800 (PST) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Kast Bernd Date: Fri, 4 Nov 2016 00:33:06 +0100 > This patch adds a module parameter in order to activate ASPM. By that > the CPU can enter deep sleep modes (PC6) and power consumption can be > reduced (for example from 13W to 8W on my notebook with a Haswell CPU). > Basically, it reapplies d64ec841517a25f6d468bde9f67e5b4cffdc67c7, which > was reverted due to delayed link status detection and increased boot > times on some systems. These bugs are avoided by two actions: > 1) ASPM is turned off by default to avoid any problems with the > default configuration. > 2) Flags for ASPM and clock request are set after ephy_init, > which wasn't respected on the previous patch. Thus ASPM with > this patch could work even with previously failing systems. This feels like grasping at straws. If you feel you've corrected a flaw in the previous ASPM support, then let's reinstate it unconditionally without some module parameter. We'll never find out if you actually did fix ASPM support sufficiently if it's off by default. Only experts like you will ever enable the option. It's usage will be minimal, and therefore the benefits will not be sufficient to justify these changes in the first place.