From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oa1-f46.google.com (mail-oa1-f46.google.com [209.85.160.46]) (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 EC01443F8CF for ; Fri, 14 Aug 2026 11:44:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.160.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786707894; cv=none; b=ptGqBpEg3EFDTOW6+OEUYsVzApk7ZjskNurG5olOri1Co5NDXM2BYf/HXLWEpk/KalTT/zFNPjiGzQ2HjeJkRw7LUG3wdEmKlH3Gij4+U4WYnxYJFz0YBn34zBo+s1+QtBKPTGbdNoocvF+GnEsSqd3lQwypF74CdMWjmmwJilM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786707894; c=relaxed/simple; bh=9Y7Psrpz7ZVGA1MqX5Ovz7nLg4icry+Wn6dT4jpWro0=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:To:Cc; b=ovzNLPzKWp+L5xRpXQiIENJo/15cMpBK+BSz5cRNbbWA2oriz2F0YT4bHEgIfv3Zp+5LNwCvELL4kMk/siCLgNVc1Q0bNYLpm+37gv7txEQ9gvAqMswU7Vgs4+YpUvLUuyg8D/Jc8I055WpduzWDgHSE16W5HXMrDHC2A2EVx+4= 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=Yo1mUrna; arc=none smtp.client-ip=209.85.160.46 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="Yo1mUrna" Received: by mail-oa1-f46.google.com with SMTP id 586e51a60fabf-446f87b6de1so965122fac.3 for ; Fri, 14 Aug 2026 04:44:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786707892; x=1787312692; darn=vger.kernel.org; h=cc:to:message-id:content-transfer-encoding:content-type :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to:content-type; bh=kz9ix/CS/l9p6EaI99ss8zgME0LR5yKN+YEGKtDT9Kg=; b=Yo1mUrnaQ9tqL1jsKArQR4f2xsi3HkUzjvysoerFTciCtvcutHiRxO/tla42yYmxJu jfcFNjdOJTzTExnbRBICRdQ87XFu6S+6N24H4tNqm3Dw9X95wPsMkg/ku/QPRCd/7nbK 6c5wh48WEIDYNDUceST2xYysItlzVtJo2aWYTVJMD6JyRISzHryA8q9Tjw97o1sSd5Wt nMoYV5rSIVnnCUYUlL2QVdPFNYNNGsGxGQ+vmpfnsSgIvYFdx9LgjVvymmLUpQzj2hcI ntsjRMce03oz4d3iAgM9d5I5RENaxMBhF5uwGw8ypn5Hiwd4QXYy+3RhOENtK/w6gQG3 ifaw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786707892; x=1787312692; h=cc:to:message-id:content-transfer-encoding:content-type :mime-version:subject:date:from:x-gm-gg:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to:content-type; bh=kz9ix/CS/l9p6EaI99ss8zgME0LR5yKN+YEGKtDT9Kg=; b=XlnSZKkd8fQgCIeGVeRp614r8ggZvthLaTz19WFbWQeOCd7DDvqXDC09nTo+5urJl6 TuQwNWCw5kiLAitSMnDSfBF47h1TNHsXGVXhTvs9Nqghjkn7vlBgRtrUE6gOfzJFhpGE KN6uNe1LLGo4d+YMDWHNY7opgLtFT9/7Ljt6GZjj+alVI4l1Pz4DgxwFXUu0MaTWUEaX WcK4ER027L9rvp34W/vD4C8pCtHRMi3xfEVSkaajroJF4AV/U+4nXXOF1v7p2spwmlmP WMFRPO3fs+K4NXf6EPsDsFv+2qmYYDb3NETYIb7JYUvug4HXXxJgGd+1EGX/mr/RJnxO BjfQ== X-Forwarded-Encrypted: i=1; AHgh+RqHb0zYvl6Z2GkMkyJw3oZrn+X/PQkTg6DyKT9/TEvluT3YuxrVX3HCxAlIhWTZ9YMN/MsN0M3XaT+y1X0=@vger.kernel.org X-Gm-Message-State: AOJu0Yx+hUW8PHbLZo+rBus4SqiCRmuO75M/Em6RBlj6ZZEG716Xi7pr YIGtTL1BuvJerfZGICtnj54Mmp8fqziPSsTohy+VIXO1cqxTebJlUfhRz07SAw== X-Gm-Gg: AR+sD11aRVxbxnYiTJpio0a8HeSair72GbXR6URpWG91bZAh3t70msviwChnbdtiZC8 VTVF9hdy7k37XgbYLu6iuE9UWpseo97cJt5XcBZk0bsZGOQ8lDTJdL+j/C2w6h6GcVUEWqsbL5C ZO9lv4libsKz/YwKHJmcEWge0SfAM0HodvuMx1QJTcyYDaVp5ipMdp1pH95gvlyS9LTkhsiGCA1 QWkR0tN6FqUzzFFu25Ra+GjkuMg3PNjbhRSqYF9O/GUnt+36LcA55mtQia7HWhO7m/BAGZjLSGf MVHd1/NoM+JC9wP0maAExGxxHNHuXjenWCeOY1X3oi159DyPBA7EdVmPu0I/mn8gTowp/vZciOc SQzGjpnPSZ2I0hO+JHE/f1whyntsTmtPySEseuZyMBbDQQb7+8D6zsN39LIhJVY6HjRiLuRTjHI D26AcHpzMwX9CdjRMpR38ARpTgMVd3kJnq4bAAYTNfUqwaqtvYsei2B7Vlvg== X-Received: by 2002:a05:6820:c3c3:10b0:6b0:ded9:6d1f with SMTP id 006d021491bc7-6b0ded96d71mr1103751eaf.11.1786707891760; Fri, 14 Aug 2026 04:44:51 -0700 (PDT) Received: from localhost ([2a03:2880:10ff:43::]) by smtp.gmail.com with ESMTPSA id 586e51a60fabf-45e8ff322c0sm2148455fac.17.2026.08.14.04.44.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 14 Aug 2026 04:44:50 -0700 (PDT) From: Daniel Zahka Date: Fri, 14 Aug 2026 04:44:06 -0700 Subject: [PATCH net-next] net: psp: use psp_dev_is_registered() in psp_assoc_free() Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20260814-psp-dev-is-reg-v1-1-5029e1f1eb01@gmail.com> X-B4-Tracking: v=1; b=H4sIAIX/fmoC/x3MOwqAMBBF0a3I1A6YqCG4FbHw89RpYsiICOLeD ZanuPchRRIodcVDCZeoHCHDlAXN+xg2sCzZZCvrKm8ajhp5wcWinLCxNdPsatf4tvWUo5iwyv0 Pewo4OeA+aXjfDzKhjCRqAAAA To: Jakub Kicinski , Willem de Bruijn , "David S. Miller" , Eric Dumazet , Paolo Abeni , Simon Horman Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org X-Mailer: b4 0.13.0 No functional changes. In code paths that use a psp_dev reference that wasn't obtained from the psp_devs xarray, e.g. not via psp_device_get_and_lock(), there is no guarantee that the psp_dev has not been unregistered. The check here is correct, but it doesn't match other code paths that use psp_dev_is_registered(). Commit b89769f936a8 ("net: psp: check for device unregister when creating assoc") is an example of a fix that adds a check for this after locking a psp_dev. if (psp_dev_is_registered(psd)) vs if (psd->ops) makes it clear what we are really checking for. Signed-off-by: Daniel Zahka --- net/psp/psp_sock.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/psp/psp_sock.c b/net/psp/psp_sock.c index 07dc4cf741f3..1a2a6b7516b0 100644 --- a/net/psp/psp_sock.c +++ b/net/psp/psp_sock.c @@ -96,7 +96,7 @@ static void psp_assoc_free(struct work_struct *work) struct psp_dev *psd = pas->psd; mutex_lock(&psd->lock); - if (psd->ops) + if (psp_dev_is_registered(psd)) psp_dev_tx_key_del(psd, pas); mutex_unlock(&psd->lock); psp_dev_put(psd); --- base-commit: 4f93b12cf7b25fbf8e73d222722805b049f0a6d3 change-id: 20260814-psp-dev-is-reg-21bc63648558 Best regards, -- Daniel Zahka