netdev - Re: [PATCH v2 net] net: stmmac: Modify configuration method of EEE timers

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]

Date:   Sat, 03 Oct 2020 16:42:40 -0700 (PDT)
From:   David Miller <davem@...emloft.net>
To:     weifeng.voon@...el.com
Cc:     mcoquelin.stm32@...il.com, netdev@...r.kernel.org,
        linux-kernel@...r.kernel.org, joabreu@...opsys.com,
        peppe.cavallaro@...com, andrew@...n.ch, alexandre.torgue@...com,
        boon.leong.ong@...el.com, chen.yong.seow@...el.com,
        mgross@...ux.intel.com, vee.khee.wong@...el.com
Subject: Re: [PATCH v2 net] net: stmmac: Modify configuration method of EEE
 timers

From: Voon Weifeng <weifeng.voon@...el.com>
Date: Thu,  1 Oct 2020 23:56:09 +0800

> From: "Vineetha G. Jaya Kumaran" <vineetha.g.jaya.kumaran@...el.com>
> 
> Ethtool manual stated that the tx-timer is the "the amount of time the
> device should stay in idle mode prior to asserting its Tx LPI". The
> previous implementation for "ethtool --set-eee tx-timer" sets the LPI TW
> timer duration which is not correct. Hence, this patch fixes the
> "ethtool --set-eee tx-timer" to configure the EEE LPI timer.
> 
> The LPI TW Timer will be using the defined default value instead of
> "ethtool --set-eee tx-timer" which follows the EEE LS timer implementation.
> 
> Fixes: d765955d2ae0 ("stmmac: add the Energy Efficient Ethernet support")
> Signed-off-by: Vineetha G. Jaya Kumaran <vineetha.g.jaya.kumaran@...el.com>
> Signed-off-by: Voon Weifeng <weifeng.voon@...el.com>
> 
> Changelog V2
> *Not removing/modifying the eee_timer.
> *EEE LPI timer can be configured through ethtool and also the eee_timer
> module param.
> *EEE TW Timer will be configured with default value only, not able to be
> configured through ethtool or module param. This follows the implementation
> of the EEE LS Timer.

Please put the Changelog above the various signoffs and other tags, as those
should be at the end of the commit log message.

Just out of curiousity, where did you see put the changelog after the
tags, and thus caused you to use this layout?  If you decided that on
your own, this is pretty much always a bad idea.  Look to other
patches which have been accepted as a guide for how to format your commit
log message.

Applied, thank you.