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
| ||
|
Date: Tue, 28 Mar 2017 14:10:29 +0200 From: Andrew Lunn <andrew@...n.ch> To: Thomas Scariah <thomasscariah@...il.com> Cc: Florian Fainelli <f.fainelli@...il.com>, nsekhar@...com, grygorii.strashko@...com, davem@...emloft.net, drivshin@...worx.com, mugunthanvnm@...com, ivan.khoronzhuk@...aro.org, Thomas Scariah <thomas.scariah@...man.com>, netdev@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH 1/1] ethtool : added get_phy_stats,get_strings,get_sset_count On Tue, Mar 28, 2017 at 11:56:40AM +0530, Thomas Scariah wrote: > Hello Florian/Andrew, > > > On Mon, Mar 27, 2017 at 10:14 PM, Florian Fainelli <f.fainelli@...il.com> > wrote: > > > Hello, > > > > On 03/27/2017 03:12 AM, Thomas Scariah wrote: > > > From: "Scariah, Thomas" <thomas.scariah@...man.com> > > > > > > Added functions to support ethtool to print the phy statistics and error > > > information along with other ethtool statistics. This will help ethtool > > > information to know the error is from physical layer or MAC layer. > > > This is an enahancement for ethtool to accommodate phy statistics > > > > It sounds like your patch should actually be 3 different patches: > > > > - add helper function to the core PHY library > > - add statistics support to the Micrel PHY driver > > - hook the proper ndo operations in cpsw to allow querying the PHY > > driver's statistics > > > > Yes agreed. This can be 3 different patches . > Anyway the latest kernel already has the same change. > I have developed this patch against Kernel 4.4.14 and the similar > functionality was not available. Hi Thomas O.K, using v4.4 explains a lot. However, we only accept patches against: git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git I still suggest you only do the PHY driver parts, and don't touch the MAC driver. Andrew
Powered by blists - more mailing lists