netdev - Re: [PATCH net-next 0/2] net: bridge: drop hosts limit sysfs and add a comment

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>] [day] [month] [year] [list]

Date:   Fri, 29 Jan 2021 21:37:42 -0800
From:   Jakub Kicinski <kuba@...nel.org>
To:     Nikolay Aleksandrov <nikolay@...dia.com>
Cc:     Nikolay Aleksandrov <razor@...ckwall.org>,
        <netdev@...r.kernel.org>, <roopa@...dia.com>,
        <bridge@...ts.linux-foundation.org>, <davem@...emloft.net>
Subject: Re: [PATCH net-next 0/2] net: bridge: drop hosts limit sysfs and
 add a comment

On Fri, 29 Jan 2021 13:55:24 +0200 Nikolay Aleksandrov wrote:
> On 29/01/2021 13:51, Nikolay Aleksandrov wrote:
> > From: Nikolay Aleksandrov <nikolay@...dia.com>
> > 
> > Hi,
> > As recently discussed[1] we should stop extending the bridge sysfs
> > support for new options and move to using netlink only, so patch 01
> > drops the recently added hosts limit sysfs support which is still in
> > net-next only and patch 02 adds comments in br_sysfs_br/if.c to warn
> > against adding new sysfs options.
> > 
> > Thanks,
> >  Nik
> > 
> > Nikolay Aleksandrov (2):
> >   net: bridge: mcast: drop hosts limit sysfs support
> >   net: bridge: add warning comments to avoid extending sysfs
> > 
> >  net/bridge/br_sysfs_br.c |  4 ++++
> >  net/bridge/br_sysfs_if.c | 30 ++++--------------------------
> >  2 files changed, 8 insertions(+), 26 deletions(-)
> >   
> 
> Oops :) the [1] addendum should be:
> 
> [1] https://lore.kernel.org/netdev/20210128105201.7c6bed82@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com/T/#mda7265b2e57b52bdab863f286efa85291cf83822
> 
> Since this is in the cover letter I don't think v2 is needed.
> Please let me know if you'd like me to resend.

Seems that vger ate the cover letter completely, but not the patches,
so I was putting the merge commit message together by hand, anyway :)

Applied, thanks!