From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A800FC4360F for ; Tue, 2 Apr 2019 19:55:42 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 5DA252084B for ; Tue, 2 Apr 2019 19:55:42 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="s75pF5Yf" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726732AbfDBTzk (ORCPT ); Tue, 2 Apr 2019 15:55:40 -0400 Received: from mail-wr1-f67.google.com ([209.85.221.67]:38370 "EHLO mail-wr1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726083AbfDBTzk (ORCPT ); Tue, 2 Apr 2019 15:55:40 -0400 Received: by mail-wr1-f67.google.com with SMTP id k11so18230011wro.5 for ; Tue, 02 Apr 2019 12:55:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:subject:to:cc:message-id:date:user-agent:mime-version :content-language:content-transfer-encoding; bh=SE6f8UgVyO8vPZi4c0MxIDXSqMOo4rcjBMLDJZxw1Dw=; b=s75pF5YfWNlTMqL7M3jZdMZGdpJiZvKsU+w645TE+GBxhodisOtxQzVHQxgU2nuSbG PAhY+xnGfxwPeyNVRy2m3BuzUhF8M/hFrjks5coSwqutnr665HSRtZ1SRWMJQG6+OGJ/ Tctnal+qP/l3z/1Hr6ZHa6uj9Tv1vpTCELTuy7znbG8fwnJQlzglDPKfPoez8b7YGesA n+hYfXuEKXWk7flTf3U2GZoNyfykVlNgP3Cv7bWAa4dQj1BUSmELulsAbVyPYTj7pmPp 4K0DDah/eDTC1jK9qceE3U0UKNWK4mNDJwfymhRWZdF59NiODPXIGWshqkjl63t5xPRE kH9w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:subject:to:cc:message-id:date:user-agent :mime-version:content-language:content-transfer-encoding; bh=SE6f8UgVyO8vPZi4c0MxIDXSqMOo4rcjBMLDJZxw1Dw=; b=HtrG0wha1/kh+LAqlI/9YUfwMsL+662fXag6W7zmXvJ4SepHKQidykhE2ZfItmO31V uTu5Pm77rrghVse1jd+CF5jQ6Ly3R6NLapDrt51jNZcWPE55Ny62mOqnCE5S6hTAuTUX Zmh+6uyLngTeGG4BpUNr98JFvq19qtSeJ1EknOuElUCj/gYrpjg5A7PBQ5RZGsvefE1n +stA6HoU+SX9ARYFdUXfec8CH+3+DhqRXAjyvBrchsZGDbVTcE8nIVQEPqKIRTlPsO0S 5f61pDd2ZW1wFQoxDqbSbH2WBlAxsSojMAHkkUUv5ugFY4j4NSPbJ0LCNqvK1XLbYDLm HYbQ== X-Gm-Message-State: APjAAAUD66p3YbAMt211Gc2CGeUwief0aMAlLfOm1+nP0alsN/WGGs/W cgUAhUy1pIB+4XLCxfwF+r4hX+/1 X-Google-Smtp-Source: APXvYqwKoAQyABCEZeIaqbLKc25S7HkUh1zculaqxdALizJXDpVQ9OSWCBZUq8bgFLEms+Epg0mYYA== X-Received: by 2002:adf:e790:: with SMTP id n16mr21516295wrm.292.1554234937441; Tue, 02 Apr 2019 12:55:37 -0700 (PDT) Received: from ?IPv6:2003:ea:8be1:dd00:e07d:a1a:1bc0:370? (p200300EA8BE1DD00E07D0A1A1BC00370.dip0.t-ipconnect.de. [2003:ea:8be1:dd00:e07d:a1a:1bc0:370]) by smtp.googlemail.com with ESMTPSA id m13sm14665414wmg.42.2019.04.02.12.55.36 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 02 Apr 2019 12:55:36 -0700 (PDT) From: Heiner Kallweit Subject: [PATCH net] r8169: switch off ASPM by default and add sysfs attribute to control ASPM To: David Miller , Realtek linux nic maintainers Cc: "netdev@vger.kernel.org" Message-ID: <275eb60f-8cfe-9ff6-97bc-39d9ef280d36@gmail.com> Date: Tue, 2 Apr 2019 21:55:25 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org There are numerous reports about different problems caused by ASPM incompatibilities between certain network chip versions and board chipsets. On the other hand on (especially mobile) systems where ASPM works properly it can significantly contribute to power-saving and increased battery runtime. One problem so far to make ASPM configurable was to find an acceptable way of configuration (e.g. module parameters are discouraged for that purpose). As a new attempt let's switch off ASPM per default and make it configurable by a sysfs attribute. The attribute is documented in new file Documentation/networking/device_drivers/realtek/r8169.txt. Fixes: a99790bf5c7f ("r8169: Reinstate ASPM Support") Signed-off-by: Heiner Kallweit --- .../device_drivers/realtek/r8169.txt | 19 +++++ drivers/net/ethernet/realtek/r8169.c | 75 +++++++++++++++++-- 2 files changed, 86 insertions(+), 8 deletions(-) create mode 100644 Documentation/networking/device_drivers/realtek/r8169.txt diff --git a/Documentation/networking/device_drivers/realtek/r8169.txt b/Documentation/networking/device_drivers/realtek/r8169.txt new file mode 100644 index 000000000..669995d0c --- /dev/null +++ b/Documentation/networking/device_drivers/realtek/r8169.txt @@ -0,0 +1,19 @@ +Written by Heiner Kallweit + +Version 04/02/2019 + +Driver-specific sysfs attributes +================================ + +rtl_aspm (bool) +--------------- + +Certain combinations of network chip versions and board chipsets result in +increased packet latency, PCIe errors, or significantly reduced network +performance. Therefore ASPM is off by default. On the other hand ASPM can +significantly contribute to power-saving and thus increased battery +runtime on notebooks. Therefore this sysfs attribute allows to switch on +ASPM on systems where ASPM works properly. The attribute accepts any form +of bool value, e.g. 1/y/on. See also kerneldoc of kstrtobool(). +Note that the attribute is accessible only if interface is up. +Else network chip and PCIe link may be powered-down and not reachable. diff --git a/drivers/net/ethernet/realtek/r8169.c b/drivers/net/ethernet/realtek/r8169.c index 19efa88f3..e2de94dc3 100644 --- a/drivers/net/ethernet/realtek/r8169.c +++ b/drivers/net/ethernet/realtek/r8169.c @@ -678,6 +678,8 @@ struct rtl8169_private { struct work_struct work; } wk; + unsigned aspm_supported:1; + unsigned aspm_enabled:1; unsigned irq_enabled:1; unsigned supports_gmii:1; dma_addr_t counters_phys_addr; @@ -5091,7 +5093,7 @@ static void rtl_hw_start_8168e_2(struct rtl8169_private *tp) RTL_W32(tp, MISC, RTL_R32(tp, MISC) | PWM_EN); RTL_W8(tp, Config5, RTL_R8(tp, Config5) & ~Spi_en); - rtl_hw_aspm_clkreq_enable(tp, true); + tp->aspm_supported = 1; } static void rtl_hw_start_8168f(struct rtl8169_private *tp) @@ -5205,7 +5207,7 @@ static void rtl_hw_start_8168g_1(struct rtl8169_private *tp) /* disable aspm and clock request before access ephy */ rtl_hw_aspm_clkreq_enable(tp, false); rtl_ephy_init(tp, e_info_8168g_1, ARRAY_SIZE(e_info_8168g_1)); - rtl_hw_aspm_clkreq_enable(tp, true); + tp->aspm_supported = 1; } static void rtl_hw_start_8168g_2(struct rtl8169_private *tp) @@ -5240,7 +5242,7 @@ static void rtl_hw_start_8411_2(struct rtl8169_private *tp) /* disable aspm and clock request before access ephy */ rtl_hw_aspm_clkreq_enable(tp, false); rtl_ephy_init(tp, e_info_8411_2, ARRAY_SIZE(e_info_8411_2)); - rtl_hw_aspm_clkreq_enable(tp, true); + tp->aspm_supported = 1; } static void rtl_hw_start_8168h_1(struct rtl8169_private *tp) @@ -5337,7 +5339,7 @@ static void rtl_hw_start_8168h_1(struct rtl8169_private *tp) r8168_mac_ocp_write(tp, 0xc094, 0x0000); r8168_mac_ocp_write(tp, 0xc09e, 0x0000); - rtl_hw_aspm_clkreq_enable(tp, true); + tp->aspm_supported = 1; } static void rtl_hw_start_8168ep(struct rtl8169_private *tp) @@ -5394,7 +5396,7 @@ static void rtl_hw_start_8168ep_1(struct rtl8169_private *tp) rtl_hw_start_8168ep(tp); - rtl_hw_aspm_clkreq_enable(tp, true); + tp->aspm_supported = 1; } static void rtl_hw_start_8168ep_2(struct rtl8169_private *tp) @@ -5414,7 +5416,7 @@ static void rtl_hw_start_8168ep_2(struct rtl8169_private *tp) RTL_W8(tp, DLLPR, RTL_R8(tp, DLLPR) & ~PFM_EN); RTL_W8(tp, MISC_1, RTL_R8(tp, MISC_1) & ~PFM_D3COLD_EN); - rtl_hw_aspm_clkreq_enable(tp, true); + tp->aspm_supported = 1; } static void rtl_hw_start_8168ep_3(struct rtl8169_private *tp) @@ -5449,7 +5451,7 @@ static void rtl_hw_start_8168ep_3(struct rtl8169_private *tp) data |= 0x0080; r8168_mac_ocp_write(tp, 0xe860, data); - rtl_hw_aspm_clkreq_enable(tp, true); + tp->aspm_supported = 1; } static void rtl_hw_start_8168(struct rtl8169_private *tp) @@ -5696,7 +5698,7 @@ static void rtl_hw_start_8106(struct rtl8169_private *tp) RTL_W8(tp, DLLPR, RTL_R8(tp, DLLPR) & ~PFM_EN); rtl_pcie_state_l2l3_disable(tp); - rtl_hw_aspm_clkreq_enable(tp, true); + tp->aspm_supported = 1; } static void rtl_hw_start_8101(struct rtl8169_private *tp) @@ -7097,6 +7099,49 @@ static const struct rtl_cfg_info { } }; +static ssize_t rtl_aspm_show(struct device *dev, struct device_attribute *attr, + char *buf) +{ + struct net_device *ndev = dev_get_drvdata(dev); + struct rtl8169_private *tp = netdev_priv(ndev); + + return snprintf(buf, PAGE_SIZE, "%d\n", tp->aspm_enabled); +} + +static ssize_t rtl_aspm_store(struct device *dev, struct device_attribute *attr, + const char *buf, size_t size) +{ + struct net_device *ndev = dev_get_drvdata(dev); + struct rtl8169_private *tp = netdev_priv(ndev); + bool aspm; + + if (!tp->aspm_supported) + return -EOPNOTSUPP; + + if (strtobool(buf, &aspm) < 0) + return -EINVAL; + + pm_runtime_get_noresume(dev); + + if (!pm_runtime_active(dev)) { + size = -ENETDOWN; + goto out; + } + + rtl_lock_work(tp); + rtl_unlock_config_regs(tp); + rtl_hw_aspm_clkreq_enable(tp, aspm); + tp->aspm_enabled = aspm ? 1 : 0; + rtl_lock_config_regs(tp); + rtl_unlock_work(tp); +out: + pm_runtime_put_noidle(dev); + + return size; +} + +static DEVICE_ATTR_RW(rtl_aspm); + static int rtl_alloc_irq(struct rtl8169_private *tp) { unsigned int flags; @@ -7325,6 +7370,11 @@ static int rtl_get_ether_clk(struct rtl8169_private *tp) return rc; } +static void rtl_remove_sysfs_aspm_file(void *data) +{ + device_remove_file(data, &dev_attr_rtl_aspm); +} + static int rtl_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) { const struct rtl_cfg_info *cfg = rtl_cfg_infos + ent->driver_data; @@ -7498,6 +7548,15 @@ static int rtl_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) pci_set_drvdata(pdev, dev); + rc = device_create_file(&pdev->dev, &dev_attr_rtl_aspm); + if (rc) + return rc; + + rc = devm_add_action_or_reset(&pdev->dev, rtl_remove_sysfs_aspm_file, + &pdev->dev); + if (rc) + return rc; + rc = r8169_mdio_register(tp); if (rc) return rc; -- 2.21.0