On Sun, Dec 11, 2016 at 02:18:00PM +0200, Saeed Mahameed wrote: > On Wed, Dec 7, 2016 at 4:41 AM, Andrew Lunn <and...@lunn.ch> wrote: > > On Wed, Dec 07, 2016 at 12:33:08AM +0200, Saeed Mahameed wrote: > >> Hi Dave, > >> > >> This series adds the support for setting device registers from user > >> space ethtool. > > > > Is this not the start of allowing binary only drivers in user space? > > > > It is not, we want to do same as set_eeprom already do, > Just set some HW registers, for analysis/debug/tweak/configure HW > dependent register for the NIC netdev sake.
Mellanox has a good reputation of open drivers. However, this API sounds like it would be the first step towards user space drivers. This is an API which can peek and poke registers, so it probably could be mis-used to put part of a driver in user space. Hence we should avoid this sort of API to start with. > where can i find some details about "HW performance counters for CPUs" ? https://perf.wiki.kernel.org/index.php/Tutorial Andrew