netdev - Re: [PATCH 1/2] r8169: reinstate ALDPS for power saving

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <B39421A6-A863-414E-956F-0F185F09EB18@canonical.com>
Date:   Thu, 22 Mar 2018 18:19:56 +0800
From:   Kai-Heng Feng <kai.heng.feng@...onical.com>
To:     Hayes Wang <hayeswang@...ltek.com>
Cc:     nic_swsd <nic_swsd@...ltek.com>,
        Linux Netdev List <netdev@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        romieu@...zoreil.com, jrg.otte@...il.com,
        David Miller <davem@...emloft.net>
Subject: Re: [PATCH 1/2] r8169: reinstate ALDPS for power saving

Kai Heng Feng <kai.heng.feng@...onical.com> wrote:

> Hopefully Hayes (or Realtek) can shed more lights on the issue. Apparently
> ALDPS and ASPM for r8169 is enabled in different commercial products, just
> not in Linux mainline.

Hayes and Realtek folks,

How do we make this patch going forward?
Do you find the root cause that make this patch got reverted?

I guess ALDPS is no longer needed after commit a92a08499b1f ("r8169:  
improve runtime pm in general and suspend unused ports"), now the device  
gets runtime suspended when link is down.

OTOH, ASPM is still quite useful though. When it's enabled, it can save 1W  
power usage, which is quite substantial for a laptop.

So, I'd like to hear your feedback and make ASPM for r8169 eventually gets  
upstreamed.

Kai-Heng