From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qt1-f169.google.com (mail-qt1-f169.google.com [209.85.160.169]) (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 0949F31355D for ; Mon, 10 Aug 2026 13:42:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.160.169 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786369360; cv=none; b=s9WS1oqDs3FA6IaSBtxnJ/tmLsqKaJgMT9tF5s9lXIdrZqsjneElB8YFAN+yABAVsPOO4uUffDu4zkAJTNmX03uS6JZvbianfq0yFq0smSeOALIDB6NzhxgK85RD9tTt1x22bseY359pp0K5UdvJ8FZiwUCnt4DdL9hyKhRb9yk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786369360; c=relaxed/simple; bh=w4RTm1SdmnPbUq7p6a2Lo87lUDniWqHAOY5MD7rV81E=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=mj3s3Rxn1nn60FBl6gXnm3X0RSAzv6pIfRCT9dUUj+y1FS0SCI4GCTFmQ0vlIwtvVSTEmQv6cxAYsXqMQwktDZWYGu9S9i5cfkbfdPpXIgqPX/MhZm8OqkMkIOoflXQe87t3U7QKWb1jDz/jA66v4wyz4zNs15uqoPrNoTw2YXg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ziepe.ca; spf=pass smtp.mailfrom=ziepe.ca; dkim=pass (2048-bit key) header.d=ziepe.ca header.i=@ziepe.ca header.b=cn2kZ6bE; arc=none smtp.client-ip=209.85.160.169 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ziepe.ca Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ziepe.ca Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=ziepe.ca header.i=@ziepe.ca header.b="cn2kZ6bE" Received: by mail-qt1-f169.google.com with SMTP id d75a77b69052e-5276d598b96so8551931cf.3 for ; Mon, 10 Aug 2026 06:42:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ziepe.ca; s=google; t=1786369357; x=1786974157; darn=vger.kernel.org; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to:content-type; bh=BCZXwY4oNGw2YYfFqKfcu4fpZvmors3q1AOve/GpS3E=; b=cn2kZ6bE/fnC6Tb10sEle1ggcS1nGM8G/wtZSWHg14KuZzJzo2hZttI6FxpGehCzrO 1ZRrXur/s2Dm6auTot7E+3ZfSb07C1ku592gpr0PpsHtz0FNxa2Cv44uTELPTl5EbuGt kFjxHffECFJw9QJnGIXkw5TMT7FpX25z5fe0aMzigx0+uqKFWwYXZPDWpVQeDoVuGUQB CTxZzmmrf+beLqfEHXI7lwcqR+wjqcupaGyh87saDognX6+MQYvOMixRjD4vet1SkHdL QkOjfEtXF2ZvEQUD5HqoCPRkVqbIMsiySjse/QJCHEAZSAhrtjvsUdJOSL/a9OsCsgOT PFEQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786369357; x=1786974157; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=BCZXwY4oNGw2YYfFqKfcu4fpZvmors3q1AOve/GpS3E=; b=e4bLDmJSMChkoQGPEHtW6SoMN33S4E8zSenu1BXGCrDyg1JDUtUUdh7YBRrFcxS47u LJm4laLpxhAGkcGnB3cs/uo2vlHdL4EwavmrGhJXXIqshJS6r1jj4weKuHm5jXIs4Add OvM+R9/9sOr544OQAWGl1WInviHdCEnzDvwW3AnGNZ3aYb3NpQcpyTAU/0t6/4G9EsBt E4/YcS0/REXBEjzXg+HzihZOgZmye40gviWrE5yoPfjRFVcsM539/KMV1attbG84Iohb AuPYrPvIHhOm5R9U2NPZUsdblpszrH/Hc/Z/saALXDbRo+giaPicKuQj+CaCob1Ba5tb 5d9Q== X-Forwarded-Encrypted: i=1; AHgh+Rpa/t3qswOxgXwkv7QmLhpVlc+NuexhfvAY4GBceHtGN1ye3epq63h3VALjd3smrImzuf0sIYeva9g/rkM=@vger.kernel.org X-Gm-Message-State: AOJu0Yw1qJLSVmEqkGdMKeEFtk2cWtceYE/LzU6r+4mHfJDCiVyHVtjI kxIGd/bBUflX6luR9dwa+HfhVfSTnVfcGYpWR4911QEyBd7CaPB22Z2DvJMfBaEY18o= X-Gm-Gg: AR+sD1181dfS1IaVr5REuY8kNM2Jq0mVn/vFJtMyCG2KqGCuk5uqKSHjIblpnIwEdTZ n9a8RO4117GV3ucyQ6/i1z3+rR7r0c8h1YcSobiEWpP45/sSXaZWjl5vLGUue6nNgRDTQImUsXr Ojc+OQR1krEZSBwmWYyH4H1+mODcxJ1EPjWVb/P5D9daKdR5Fhu7ie5ZdRhM6SKJNq0JPf9Ht2q 3YIfRFmueM14qCUTK8njbPx7Cb44UPjNdTSgeKu1pYvYhyUWobhQe4iVDzcq788VMW8wyXvmEeS 8Q8jRVk41x7e9p8EH9fGG4yo/4cCtq/GvmALR/JbaPl17BJBSqm3eex3tQJOVapK16FpW0DvY7m hdnUP+EMfR5aeBqs8xPRV80Z8tWn9sIZIfkzrqmh4Mg+Iu8Ju7oNfG/H3+zgp2L7Z6owQ6zSPHf KcHYk7yxPUIEKxLj0JB93ewzhsGe7gNVtHGCJVPA== X-Received: by 2002:ac8:7f4d:0:b0:51c:1c2c:a8bc with SMTP id d75a77b69052e-52d4bca21d5mr17658741cf.37.1786369356751; Mon, 10 Aug 2026 06:42:36 -0700 (PDT) Received: from ziepe.ca ([142.166.156.215]) by smtp.gmail.com with ESMTPSA id 6a1803df08f44-908a9301d83sm72812166d6.26.2026.08.10.06.42.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 10 Aug 2026 06:42:36 -0700 (PDT) Received: from jgg by wakko with local (Exim 4.97) (envelope-from ) id 1wtQGx-000000022xs-25Hm; Mon, 10 Aug 2026 10:42:35 -0300 Date: Mon, 10 Aug 2026 10:42:35 -0300 From: Jason Gunthorpe To: Krystian Kaniewski Cc: Leon Romanovsky , linux-rdma@vger.kernel.org, Andrew Lunn , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, syzkaller-bugs@googlegroups.com, syzbot+5fe14f2ff4ccbace9a26@syzkaller.appspotmail.com Subject: Re: [PATCH net v3] RDMA/core: Reject unregistering netdevs in ib_get_eth_speed Message-ID: <20260810134235.GO200537@ziepe.ca> References: <20260810133124.44513-1-krystianmkaniewski@gmail.com> 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=us-ascii Content-Disposition: inline In-Reply-To: <20260810133124.44513-1-krystianmkaniewski@gmail.com> On Mon, Aug 10, 2026 at 03:31:20PM +0200, Krystian Kaniewski wrote: > @@ -2049,7 +2050,15 @@ int ib_get_eth_speed(struct ib_device *dev, u32 port_num, u16 *speed, u8 *width) > return -ENODEV; > > rtnl_lock(); > + if (READ_ONCE(netdev->reg_state) != NETREG_REGISTERED) { > + dev_put(netdev); > + rtnl_unlock(); > + return -ENODEV; > + } > + > rc = __ethtool_get_link_ksettings(netdev, &lksettings); > + if (rc) > + strscpy(name, netdev->name, sizeof(name)); > rtnl_unlock(); > > dev_put(netdev); > @@ -2060,7 +2069,7 @@ int ib_get_eth_speed(struct ib_device *dev, u32 port_num, u16 *speed, u8 *width) > netdev_speed = SPEED_1000; > if (rc) > pr_warn("%s speed is unknown, defaulting to %u\n", > - netdev->name, netdev_speed); > + name, netdev_speed); > } I would probably just put this block inside the rtnl lock and not copy the netdev->name Jason