From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f49.google.com (mail-wm1-f49.google.com [209.85.128.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E9B682652AF for ; Sun, 26 Oct 2025 12:51:01 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761483063; cv=none; b=a25xP2BLAzW4Omp8a/+AmY3wO7E7aMAHtOzE3uIsbfOFUAWP325NOYCoCbP/JuLwa5O+FgJRv+1z86HCHy6mCRpjGHT7I5h65X57sEFfkkyE/cqlBU5aeLGYgOdTMuuo56iSaLnX35/llujItrr7tjW6cX4n4b+Q8d5odzIf0AY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761483063; c=relaxed/simple; bh=sC0JIKkXBimdhEy3Volg1/Z15jb+ADR/MxZiHjq4n8U=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=R2QJ3+3LwO7pRvxJAkEgT3sLp4g4OtJjGTkhWHk4ZKY3Uv/JQ2NgWkbuwBT6F43oblAKYEdE5XZeUzPcKINrdRTzc79fkBlodqBPdTrMDTYb+0yjougu+JxlGMxpTmCYJxOK/xMjahbp7ngokXr4qyXvWCi3x04uqARvuKdKWKE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=YTbbJnJR; arc=none smtp.client-ip=209.85.128.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="YTbbJnJR" Received: by mail-wm1-f49.google.com with SMTP id 5b1f17b1804b1-47103b6058fso26483285e9.1 for ; Sun, 26 Oct 2025 05:51:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1761483060; x=1762087860; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=kXj5Wf+44uD1y6UQsG1pQ//IkxTEmThHrLRBULvH6Rk=; b=YTbbJnJRBGsFBY58Qgo3Hod4DkpyoJwuK5hjSYg74yh2EGkM1MAJAYaCamzkzuHl/K 7mJuGB3A4LJ1I63xFBHSQWF+Jn39mBtqt21Sc8/uBwoGlQ5did1Csac0CkZiW8NJPFdD OxVLQXUxJZpAerIc2qdcmqSAeKVOF6WBxwH5kpwVaI4nD1puSMskC/7rltTa5oiSAJv+ 0ixGkC24eFkT4WkWhYpdgEaHh6F3hEu1OT+9Jq4ZNK6C5AHBl6RXpxFBryJczQWDZ7SD KpCWQTAm7vZu2BaVZMJmsBInO2tDQAMYRQmDwGxJiUJL63MGWOnNqRsc492h1Lq49+m/ syxA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1761483060; x=1762087860; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=kXj5Wf+44uD1y6UQsG1pQ//IkxTEmThHrLRBULvH6Rk=; b=WKTTKqVRtKDCeREGpw2+Nz0TX5XCHQFYyA2Pm+3H1eBGa6S96/i/cnTBljPh5Y1jHU 6Lxp4leATsFepFkX/foZcGucKM9kRAA71BfLC0Czb0kGZCFIIa77RLEvHf1pw0kdVQq5 LUiGrjGGd/pxPhZDMQvM+JY9YUIdsGscuYqMew4kj6uOH12ZT3YDVCKS0B2RFXoR9Vc0 0PUW8BnRb93u5rpq9wtHR8eCLYOaUN4vc3fNlpRLNPLgxUTD8UWqBWSBT60TUS68QqZw n5FiF1H0jD71aVhifa4WC0lw3eI2NsDH9f14Ir/b3qmIBB/yZZDSaQ4o7UFGMRDJNpbV 9kzg== X-Forwarded-Encrypted: i=1; AJvYcCUpK0by1uJimaRBrWrI2mNgUyrwZYQjHPAfKiPF3P8+yn2a2ZgcjzaAK+XSNUH9dasreRtrf6FvhBzfzyc=@vger.kernel.org X-Gm-Message-State: AOJu0YzvgWuCaPgb3bIv9wV2eOLyzmo7WxVyjXoiqD6PweoInNdmF88x gaB5c85+IpkaB4+3dVuSTf036bGs12EIeS9A+OtYOpDaOsi0BbuP03t/ X-Gm-Gg: ASbGncvy5jVt4XsWne0DQegeGZb3p+2nGv/WWur3w5V7n3cdpbFlMVVPQ6R9OQAG7e0 +T5fM0LEW3VsIDuiaG4rok4SCxWyBsRV4VN6avKu3L8KRrwnOttSUQR08bs7VhqQaBToxHeLs6e P/KzNAaCGGuy7iCvrFgTc0JgBv9Q5jmpS4E7Wwg4Y+XKR9zPAVYOpdPaC4/JakvJsPDwsqj1U43 8t525XhNhlWZdI6MP6Nmw3KuWIYO2hTr6SQuj7kJEcnBN5oiuDVIYqmaMycYj8ByeLwCctJXXcx EQ1l69yKZJAy7nVwgcUIFFBndfCRXrtF2mLBkhVxDGYotbph6Hki7NIJM5SXTADYXzKhiPTodub G3dJARt5K7gcHcOTWa6NphZzQsUkvvNFLtQQ1L63I1WES2qOBIWNp2mG0RKQPonUqNHO3HQ2Rl3 WMGE4EeluNAQgofvbS/HRhef/y88UDu1OGLQ== X-Google-Smtp-Source: AGHT+IG91g4W6yyRy1O/9J13waij055moyyErvrzJtpVIyLJEy5+LYF+7HxHgefiVJ+Er3vsDKPDFg== X-Received: by 2002:a05:600c:a112:b0:46e:24a4:c247 with SMTP id 5b1f17b1804b1-475d2411199mr55666785e9.5.1761483060052; Sun, 26 Oct 2025 05:51:00 -0700 (PDT) Received: from [10.221.206.54] ([165.85.126.46]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-429952d5768sm8715765f8f.24.2025.10.26.05.50.58 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 26 Oct 2025 05:50:59 -0700 (PDT) Message-ID: <83aad5ca-21e5-41a2-89c9-e3c8e9006e6a@gmail.com> Date: Sun, 26 Oct 2025 14:50:58 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH net-next 2/7] net/mlx5e: Use TIR API in mlx5e_modify_tirs_lb() To: Simon Horman , Tariq Toukan Cc: Eric Dumazet , Jakub Kicinski , Paolo Abeni , Andrew Lunn , "David S. Miller" , Saeed Mahameed , Leon Romanovsky , Mark Bloch , netdev@vger.kernel.org, linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org, Gal Pressman , Carolina Jubran , Dragos Tatulea References: <1761201820-923638-1-git-send-email-tariqt@nvidia.com> <1761201820-923638-3-git-send-email-tariqt@nvidia.com> Content-Language: en-US From: Tariq Toukan In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 23/10/2025 16:31, Simon Horman wrote: > On Thu, Oct 23, 2025 at 09:43:35AM +0300, Tariq Toukan wrote: > > ... > >> diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_common.c b/drivers/net/ethernet/mellanox/mlx5/core/en_common.c >> index 376a018b2db1..fad6b761f622 100644 >> --- a/drivers/net/ethernet/mellanox/mlx5/core/en_common.c >> +++ b/drivers/net/ethernet/mellanox/mlx5/core/en_common.c >> @@ -250,43 +250,30 @@ void mlx5e_destroy_mdev_resources(struct mlx5_core_dev *mdev) >> int mlx5e_modify_tirs_lb(struct mlx5_core_dev *mdev, bool enable_uc_lb, >> bool enable_mc_lb) > > ... > >> list_for_each_entry(tir, &mdev->mlx5e_res.hw_objs.td.tirs_list, list) { >> - tirn = tir->tirn; >> - err = mlx5_core_modify_tir(mdev, tirn, in); >> + err = mlx5e_tir_modify(tir, builder); >> if (err) >> break; >> } >> mutex_unlock(&mdev->mlx5e_res.hw_objs.td.list_lock); >> >> - kvfree(in); >> + mlx5e_tir_builder_free(builder); >> if (err) >> mlx5_core_err(mdev, >> "modify tir(0x%x) enable_lb uc(%d) mc(%d) failed, %d\n", >> - tirn, >> + mlx5e_tir_get_tirn(tir), > > Sorry, for not noticing this before sending my previous email. > > Coccinelle complains about the line above like this: > > .../en_common.c:276:28-31: ERROR: invalid reference to the index variable of the iterator on line 265 > > I think this is a false positive because the problem only occurs if > the list iteration runs to completion. But err guards against > tir being used in that case. > Exactly. > But, perhaps, to be on the safe side, it would be good practice > to stash tir somewhere? > I tried to keep the error print out of the critical lock section. It's not time-sensitive so optimization is not really needed. I can simply move the print inside where it belongs. >> enable_uc_lb, enable_mc_lb, err); >> >> return err; >> -- >> 2.31.1 >> >> >