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  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Tue, 12 Jan 2021 01:47:27 +0100
From:   Andrew Lunn <andrew@...n.ch>
To:     Jakub Kicinski <kuba@...nel.org>
Cc:     stefanc@...vell.com, netdev@...r.kernel.org,
        thomas.petazzoni@...tlin.com, davem@...emloft.net,
        nadavh@...vell.com, ymarkman@...vell.com,
        linux-kernel@...r.kernel.org, linux@...linux.org.uk,
        mw@...ihalf.com, rmk+kernel@...linux.org.uk, atenart@...nel.org
Subject: Re: [PATCH net ] net: mvpp2: Remove Pause and Asym_Pause support

On Mon, Jan 11, 2021 at 04:36:53PM -0800, Jakub Kicinski wrote:
> On Sun, 10 Jan 2021 21:23:02 +0200 stefanc@...vell.com wrote:
> > From: Stefan Chulski <stefanc@...vell.com>
> > 
> > Packet Processor hardware not connected to MAC flow control unit and
> > cannot support TX flow control.
> > This patch disable flow control support.
> > 
> > Fixes: 3f518509dedc ("ethernet: Add new driver for Marvell Armada 375 network unit")
> > Signed-off-by: Stefan Chulski <stefanc@...vell.com>
> 
> I'm probably missing something, but why not 4bb043262878 ("net: mvpp2:
> phylink support")?

Hi Jakub

Before that change, it used phylib. The same is true with phylib, you
need to tell phylib it should not advertise pause. How you do it is
different, but the basic issue is the same. Anybody doing a backport
past 4bb043262878is going to need a different fix, but the basic issue
is there all the way back to when the driver was added.

      Andrew

Powered by blists - more mailing lists