netdev - Re: [PATCHv2 0/3] usbnet: speed reporting for devices without MDIO

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CANEJEGuXmBwYdb3Vr7QhwEUFNXyLgQ0PkqP+H1Qya+Kceh7G7w@mail.gmail.com>
Date:   Tue, 9 Feb 2021 23:47:34 +0000
From:   Grant Grundler <grundler@...omium.org>
To:     Grant Grundler <grundler@...omium.org>
Cc:     Andrew Lunn <andrew@...n.ch>, Oliver Neukum <oneukum@...e.com>,
        Hayes Wang <hayeswang@...ltek.com>,
        David Miller <davem@...emloft.net>,
        netdev <netdev@...r.kernel.org>, linux-usb@...r.kernel.org
Subject: Re: [PATCHv2 0/3] usbnet: speed reporting for devices without MDIO

Oliver,
it's been a few weeks, do you have time to post an updated patchset or
is there someone else you would trust to help land this series?

cheers,
grant

On Fri, Jan 22, 2021 at 2:10 AM Grant Grundler <grundler@...omium.org> wrote:
>
> On Fri, Jan 22, 2021 at 1:29 AM Andrew Lunn <andrew@...n.ch> wrote:
> >
> > On Thu, Jan 21, 2021 at 01:57:28PM +0100, Oliver Neukum wrote:
> > > This series introduces support for USB network devices that report
> > > speed as a part of their protocol, not emulating an MII to be accessed
> > > over MDIO.
> > >
> > > v2: adjusted to recent changes
> >
> > Hi Oliver
> >
> > Please give more details what actually changed.  Does this mean you
> > just rebased it on net-next? Or have you made real changes?
>
> My apologies to Oliver - the changes he's referring to are the ones I submitted:
>    https://www.spinics.net/lists/netdev/msg715248.html
>
> which is related to this series:
>    https://www.spinics.net/lists/netdev/msg714493.html
>
> I wasn't aware of and didn't look for the series Oliver had previously
> posted. *sigh*  I have been talking to Realtek about getting the issue
> of RTL8156 spewing notifications every 32ms fixed (thinking a FW
> change could fix it) for nearly three months.  It is unfortunate
> timing that Roland Dreier decided to do something about it in December
> - which I didn't expect to happen given this problem was reported
> nearly two years ago.
>
> > The discussion with v1 suggested that this framework should also be
> > used by anything which gets notified in CDC style. So i was expecting
> > to see cdc_ether.c also use this.
>
> Agreed. That's a two lines change to cdc_ether.c. I can submit this if
> Oliver doesn't want to spin the series.
>
> I've reviewed all three patches and besides one nit (which could be
> ignored or fixed later), I'm offering my
>    Reviewed-by: Grant Grundler <grundler@...omium.org>
>
> in the off chance that helps get this accepted into net-next (and/or
> 5.11 RC release).
>
> cheers,
> grant
>
> >
> >             Andrew