Message ID | 20190320072626.31589-1-skalluru@marvell.com |
---|---|
Headers | show |
Series | qed* enhancements. | expand |
From: Sudarsana Reddy Kalluru <skalluru@marvell.com> Date: Wed, 20 Mar 2019 00:26:24 -0700 > The patch series adds couple of enhancements for qed/qede drivers. > Please consider applying it to 'net-next' tree. > > Sudarsana Reddy Kalluru (2): > qede: Populate mbi version in ethtool driver query data. > qed: Define new MF bit for no_vlan config Series appied. I don't know how big your MBI version strings are, but if you are printing now both it and the MFW versions into that one string you might hit the ETHTOOL_FWVERS_LEN limit often. Just a thought...
> -----Original Message----- > From: David Miller <davem@davemloft.net> > Sent: Wednesday, March 20, 2019 11:44 PM > To: Sudarsana Reddy Kalluru <skalluru@marvell.com> > Cc: netdev@vger.kernel.org; Ariel Elior <aelior@marvell.com>; Michal > Kalderon <mkalderon@marvell.com> > Subject: Re: [PATCH net-next 0/2] qed* enhancements. > > From: Sudarsana Reddy Kalluru <skalluru@marvell.com> > Date: Wed, 20 Mar 2019 00:26:24 -0700 > > > The patch series adds couple of enhancements for qed/qede drivers. > > Please consider applying it to 'net-next' tree. > > > > Sudarsana Reddy Kalluru (2): > > qede: Populate mbi version in ethtool driver query data. > > qed: Define new MF bit for no_vlan config > > Series appied. > > I don't know how big your MBI version strings are, but if you are printing now > both it and the MFW versions into that one string you might hit the > ETHTOOL_FWVERS_LEN limit often. > > Just a thought... Dave, Thanks for your comments and accepting the changes. Size of mbi version string is 7 bytes and the mfw is 10 bytes. Hence the total string size including the spaces would be under ETHTOOL_FWVERS_LEN. Thanks, Sudarsana