> -----Original Message----- > From: Jakub Kicinski <[email protected]> > Sent: Thursday, May 30, 2019 1:10 AM > To: Jubran, Samih <[email protected]> > Cc: [email protected]; [email protected]; Kiyanovski, Arthur > <[email protected]>; Woodhouse, David <[email protected]>; > Machulsky, Zorik <[email protected]>; Matushevsky, Alexander > <[email protected]>; Bshara, Saeed <[email protected]>; Wilson, Matt > <[email protected]>; Liguori, Anthony <[email protected]>; Bshara, > Nafea <[email protected]>; Tzalik, Guy <[email protected]>; Belgazal, > Netanel <[email protected]>; Saidi, Ali <[email protected]>; > Herrenschmidt, Benjamin <[email protected]> > Subject: Re: [PATCH V1 net-next 02/11] net: ena: ethtool: add extra properties > retrieval via get_priv_flags > > On Wed, 29 May 2019 12:49:55 +0300, [email protected] wrote: > > From: Arthur Kiyanovski <[email protected]> > > > > This commit adds a mechanism for exposing different driver properties > > via ethtool's priv_flags. > > > > In this commit we: > > > > Add commands, structs and defines necessary for handling extra > > properties > > > > Add functions for: > > Allocation/destruction of a buffer for extra properties strings. > > Retreival of extra properties strings and flags from the network device. > > > > Handle the allocation of a buffer for extra properties strings. > > > > * Initialize buffer with extra properties strings from the > > network device at driver startup. > > > > Use ethtool's get_priv_flags to expose extra properties of the ENA > > device > > > > Signed-off-by: Arthur Kiyanovski <[email protected]> > > Signed-off-by: Sameeh Jubran <[email protected]> > > This commit DMAs in the string set blindly from the FW and exposes it to user > space, without any interpretation by the driver, correct? > Making the driver a mere proxy for the FW. I think it should be clearly > mentioned in the commit message, to make sure we know what what we are > accepting here. I'm always a little uncomfortable with such changes :) (I'm > not > actually sure there is a precedent for this).
Ack. We will update the commit message in the next version of the patch with more information.
