Received-SPF: pass (google.com: domain of linux-kernel+bounces-98768-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: Subbaraya Sundeep Bhatta <sbhatta@marvell.com>
To: Breno Leitao <leitao@debian.org>, "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>, Jakub Kicinski <kuba@kernel.org>,
        Paolo
 Abeni <pabeni@redhat.com>
CC: "keescook@chromium.org" <keescook@chromium.org>,
        Ido Schimmel
	<idosch@nvidia.com>,
        Nikolay Aleksandrov <razor@blackwall.org>,
        Amit Cohen
	<amcohen@nvidia.com>, Petr Machata <petrm@nvidia.com>,
        Jiri Benc
	<jbenc@redhat.com>, Beniamino Galvani <b.galvani@gmail.com>,
        Gavin Li
	<gavinl@nvidia.com>,
        "open list:NETWORKING DRIVERS" <netdev@vger.kernel.org>,
        open list <linux-kernel@vger.kernel.org>
Subject: RE: [EXTERNAL] [PATCH net-next 1/2] vxlan: Do not alloc tstats
 manually
Thread-Topic: [EXTERNAL] [PATCH net-next 1/2] vxlan: Do not alloc tstats
 manually
Thread-Index: AQHac6bNog3RCIA29U+/uKYpWGz3vLEyaF9Q
Date: Mon, 11 Mar 2024 11:34:15 +0000
Message-ID: 
 <CO1PR18MB46665FCB882E8E293EEDE521A1242@CO1PR18MB4666.namprd18.prod.outlook.com>
References: <20240311112437.3813987-1-leitao@debian.org>
In-Reply-To: <20240311112437.3813987-1-leitao@debian.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: 
 =?us-ascii?Q?Q9P5Su15wtiUm2aQMWSm2RR+YsIi2/MUFvn1UX/qFEsDMCwhJkXVmDZCkCwj?=
 =?us-ascii?Q?gy/NOiIRBTZdUfxR/yHlfR2mONxILTIGE544dY9c7X6kIYyfqf45AvhcTHDe?=
 =?us-ascii?Q?AM+yYn2FU1sZDPJstz3MBPQUbuboYAA3iR4k+L3HVmf+6kMXQ+ICY6mPeipy?=
 =?us-ascii?Q?15JtXRdFfmO/an84J+EujHVv3+dpgjqGcXc4fa+jrBCtwys9YXQr4Zc38wHU?=
 =?us-ascii?Q?1Zw/fYNdw2PEqPxL4Ao26bLstnlgV5Sz9+3FkxWg5SdWRwtXQXDDUoWo2cP4?=
 =?us-ascii?Q?/rF+gUlLSMqX0xzFp7NeAuiWzHHK40ZESHFhy1se531sUsO+XqWnUs5LsM9I?=
 =?us-ascii?Q?Zrov9o6MjwxWJTv812zgGA+BKU5y4EaV45gAsxj0399ZQ9EHdYKTXd+z+z76?=
 =?us-ascii?Q?QQXFAGT/OMZKUQDCc/hFcFjbR7gVguoEAqGw+bXCjwx60EV4fPBIujXUgcJe?=
 =?us-ascii?Q?pu8RP5Po3Vo0vXncxAYcYlpvWargGtUW6ODtKwvdjNpm1kDtyrRXiGRXsjUN?=
 =?us-ascii?Q?oE/QnpyyYE8xNi4iWbDaPx2DtGmjREKVhO4zy1xIJ4f1CWqsnE57CAMba77Z?=
 =?us-ascii?Q?CKJa+3Tr94S51W+ZkNvbzsaQsnsLA+wujZ+NyiGXxrIriqRHlCy2BGbix/6s?=
 =?us-ascii?Q?Bc9gRYFZuvwjYhm7tIMWj2+T7PPEb52hUhVMWB6/emSD0kwX/oxxmFUug0K5?=
 =?us-ascii?Q?yEeru0n3o0NUp2Rymc34eiJU6fKiYElend3nlziUcdI8FjJodS28QCg1rHJv?=
 =?us-ascii?Q?1baVK9LpQqT8fgcGe8cHfRkA+OiPKHSomf6cs4NHhnTTEi1ke16zGgcq6qX4?=
 =?us-ascii?Q?ojZZB8e00X5ixSMxStr0EJIRIT9wAIv96iRQUflv32MYQg++yau075FUE6P1?=
 =?us-ascii?Q?qKXxu3AMoXpzL9xE6YR9nFvsHaQbEp6MCRoUGYiPmsAuX1mLGx7K/41fxUCI?=
 =?us-ascii?Q?highTVoAnkg5dvy7sa1A28DLSB/HapUeeAdvsJpy02X+L8G4KnNqILFMP1iF?=
 =?us-ascii?Q?VkyC1f+mbimXofRQ5aWWkGfy00tWCppY3mQ4SJnndA0ew8zVA5B13usD4WH3?=
 =?us-ascii?Q?01zyFf7X0A3hAeHxb6IIstyXIAL2TM7iB29oJNzyrKaltfJI+x+iRlasb3J5?=
 =?us-ascii?Q?pRzXuCeQlnEAyS8T2e1SkoHTJwYQbhy20gdL5mFiAZ5QEkWB93lx0qmJn0CD?=
 =?us-ascii?Q?sBC0WqhXSLUFQhWTiqWRnDCwQXrT2oyAdFqlXeqlzMYiX30qODSQHaRYLX/9?=
 =?us-ascii?Q?OXqPCz3HFxqGaqSAGvbGhnuMBXUGxQt/L9T10mLq9UVudiLmuOjxiOYqkJGs?=
 =?us-ascii?Q?uR5ySAw0HZHpi4gI9oWR/B0Xzj84PL8R8ixQ9HNE7lU8aTWmD6WJODfmtu9x?=
 =?us-ascii?Q?H9EcLVh6u3E+8hxL+daLC4VUdu1KiKLY979EM1cVsiFeguAl3lE+PQ059XVj?=
 =?us-ascii?Q?fJps8HfgaYzd5ND5tuh/C6EtLRxVFWOyXbj0RPK4rIEQMneRZi9fNJet3fVd?=
 =?us-ascii?Q?ZlU+1/o57okKAwql6m/kS06w8oYL+KzE+L3yxyjb543dDvsN2hLO8LnvX3vI?=
 =?us-ascii?Q?TNGZfPi/+tMeY5rqlA4=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: marvell.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CO1PR18MB4666.namprd18.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2bf9b7df-5121-4e8f-85c3-08dc41bf2e8b
X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Mar 2024 11:34:15.4907
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 70e1fb47-1155-421d-87fc-2e58f638b6e0
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: KnbCSkkaymIFIhKG+W+TTajQL4wwyEWHjEq5KsRId5NsVUaU3CABkvS73qytQoLANgdNehW07hzvVuMAcNxOKQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN0PR18MB5944
X-Proofpoint-ORIG-GUID: GlHP1GkT6xWLWYBIUrFEuCksDHKNDysT
X-Proofpoint-GUID: GlHP1GkT6xWLWYBIUrFEuCksDHKNDysT
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2024-03-11_08,2024-03-06_01,2023-05-22_02


>-----Original Message-----
>From: Breno Leitao <leitao@debian.org>
>Sent: Monday, March 11, 2024 4:55 PM
>To: David S. Miller <davem@davemloft.net>; Eric Dumazet
><edumazet@google.com>; Jakub Kicinski <kuba@kernel.org>; Paolo Abeni
><pabeni@redhat.com>
>Cc: keescook@chromium.org; Ido Schimmel <idosch@nvidia.com>; Nikolay
>Aleksandrov <razor@blackwall.org>; Amit Cohen <amcohen@nvidia.com>; Petr
>Machata <petrm@nvidia.com>; Jiri Benc <jbenc@redhat.com>; Beniamino
>Galvani <b.galvani@gmail.com>; Gavin Li <gavinl@nvidia.com>; open
>list:NETWORKING DRIVERS <netdev@vger.kernel.org>; open list <linux-
>kernel@vger.kernel.org>
>Subject: [EXTERNAL] [PATCH net-next 1/2] vxlan: Do not alloc tstats manual=
ly
>
>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@debian.org>

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

>---