lkml.org 
[lkml]   [2018]   [Aug]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [RFC PATCH net-next v2 10/17] ethtool: implement GET_SETTINGS message
    On Tue, Aug 21, 2018 at 04:10:22PM +0200, Andrew Lunn wrote:
    > On Tue, Aug 21, 2018 at 11:32:46AM +0200, Michal Kubecek wrote:
    > > I have prepared a patch converting 8390/etherh driver to use
    > > {g,s}et_link_ksettings and I'm going to submit it when net-next opens.
    > > Do you think we can then drop {g,s}et_settings callbacks completely
    > > (i.e. also from ioctl() code and ethtool_ops)? Do we care about
    > > unconverted out of tree drivers?
    >
    > We cannot break ethtool, the ABI it uses. But there is already code to
    > use get_link_ksettings() and only fall back to get_settings if it does
    > not exist. So we can clean up all the fallback code, remove the
    > ethtool_ops, etc.

    Yes, that's what I have in mind: keep the ethtool interface as is (both
    ETHTOOL_{G,S}SET and ETHTOOL_{G,S}LINKSETTINGS) but drop get_settings
    and set_settings from ethtool_ops and the code that falls back to them.
    This way both old and new versions of ethtool would still work and the
    only problem would be with out of tree drivers not converted to
    {s,g}et_link_ksettings (which won't build so that it won't be silent
    breakage).

    Michal Kubecek

    \
     
     \ /
      Last update: 2018-08-21 16:52    [W:2.381 / U:0.324 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site