linux-kernel - Re: [PATCH net-next 0/9] r8169:update hardware ephy parameter

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] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 09 Dec 2014 18:35:49 -0500 (EST)
From:	David Miller <davem@...emloft.net>
To:	hau@...ltek.com
Cc:	netdev@...r.kernel.org, nic_swsd@...ltek.com,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH net-next 0/9] r8169:update hardware ephy parameter

From: Chunhao Lin <hau@...ltek.com>
Date: Wed, 10 Dec 2014 00:45:54 +0800

> Update hardware ephy parameter to improve pcie compatibility.

This really doesn't tell me anything, I really dislike patch
series like this one.

All of the programming is magic values to magic offsets.

You aren't even trying to describe in the commit log message exactly
what kind of settings are being changed, and exactly how those changes
achieve the stated goal.

Furthermore, the commit description makes no sense at all to me.

How can programming the ethernet MAC PHY have any influence on PCI-E
bus compatability?  Or are you programming the PCI bus interface's
PHY?

In what way are you adjusting which settings and in what way do those
adjustments help improve PCI-E bus behavior?

You absolutely must describe exactly what the new programming
is actually doing, precisely, and in detail.  I want to know
if some kind of timings are being adjusted, and in what way.
Are some fifo limits being changes?  If so, in what way, and
why does that help.

You have to describe what you are doing. Short and non-informative
commit log messages alongside random changes to undocumented magic
constant registers is simply unacceptable.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/