netdev - RE: [EXTERNAL] [PATCH net-next 1/2] vxlan: Do not alloc tstats manually

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

Message-ID: 
 <CO1PR18MB46665FCB882E8E293EEDE521A1242@CO1PR18MB4666.namprd18.prod.outlook.com>
Date: Mon, 11 Mar 2024 11:34:15 +0000
From: Subbaraya Sundeep Bhatta <sbhatta@...vell.com>
To: Breno Leitao <leitao@...ian.org>, "David S. Miller" <davem@...emloft.net>,
        Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>,
        Paolo
 Abeni <pabeni@...hat.com>
CC: "keescook@...omium.org" <keescook@...omium.org>,
        Ido Schimmel
	<idosch@...dia.com>,
        Nikolay Aleksandrov <razor@...ckwall.org>,
        Amit Cohen
	<amcohen@...dia.com>, Petr Machata <petrm@...dia.com>,
        Jiri Benc
	<jbenc@...hat.com>, Beniamino Galvani <b.galvani@...il.com>,
        Gavin Li
	<gavinl@...dia.com>,
        "open list:NETWORKING DRIVERS" <netdev@...r.kernel.org>,
        open list <linux-kernel@...r.kernel.org>
Subject: RE: [EXTERNAL] [PATCH net-next 1/2] vxlan: Do not alloc tstats
 manually



>-----Original Message-----
>From: Breno Leitao <leitao@...ian.org>
>Sent: Monday, March 11, 2024 4:55 PM
>To: David S. Miller <davem@...emloft.net>; Eric Dumazet
><edumazet@...gle.com>; Jakub Kicinski <kuba@...nel.org>; Paolo Abeni
><pabeni@...hat.com>
>Cc: keescook@...omium.org; Ido Schimmel <idosch@...dia.com>; Nikolay
>Aleksandrov <razor@...ckwall.org>; Amit Cohen <amcohen@...dia.com>; Petr
>Machata <petrm@...dia.com>; Jiri Benc <jbenc@...hat.com>; Beniamino
>Galvani <b.galvani@...il.com>; Gavin Li <gavinl@...dia.com>; open
>list:NETWORKING DRIVERS <netdev@...r.kernel.org>; open list <linux-
>kernel@...r.kernel.org>
>Subject: [EXTERNAL] [PATCH net-next 1/2] vxlan: Do not alloc tstats manually
>
>With commit 34d21de99cea9 ("net: Move {l,t,d}stats allocation to core and
>convert veth & vrf"), stats allocation could be done on net core
>instead of in this driver.
>
>With this new approach, the driver doesn't have to bother with error
>handling (allocation failure checking, making sure free happens in the
>right spot, etc). This is core responsibility now.
>
>Remove the allocation in the vxlan driver and leverage the network
>core allocation instead.
>
>Signed-off-by: Breno Leitao <leitao@...ian.org>

Reviewed-by: Subbaraya Sundeep <sbhatta@...vell.com>

>---