From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f176.google.com (mail-pl1-f176.google.com [209.85.214.176]) (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 412072BD0B for ; Sat, 2 May 2026 05:07:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.176 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777698469; cv=none; b=URCNt1HpwCQs9OhC6JVYLEqERtljRJV9vbAdS0vzzOmrnFjwNsp29ujSatqNzD37hWCc9fnbOB0TjTMzcojBUkMdlPkzKC4IR0RrxRoYiDfikihM05NLo1AXM4pMSNKQP2y2MA+sx00KL3HjT7/tVcxFhsw1ZUX5MdLefUViatc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777698469; c=relaxed/simple; bh=M7eXefBZbvtTLvDUAWBFVtM+mK72e8gioWagVIzC0bo=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=ne0dm8b0Ofp4CiEqpAXkPyfD+sl53ODesKtTf93WRpAFu/DXMJFMS4BH+d2gIiufyC+AScFxzUxPH3ILX80Zz9lrJPtrFAxvD14z+dLF+vJo40EdVGEHYHpPEmed8W09w6twgbG7sOVji2MnqBjVzS1x2rlQFeUFm98mCE3gFxI= 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=H2g6+uwd; arc=none smtp.client-ip=209.85.214.176 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="H2g6+uwd" Received: by mail-pl1-f176.google.com with SMTP id d9443c01a7336-2b4583f0a1aso15314725ad.3 for ; Fri, 01 May 2026 22:07:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1777698468; x=1778303268; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=A9PB/D/JhFEvy+R3A7+7Tb+xDlORxmPJZApX2c7GDnA=; b=H2g6+uwd/ompmj3TnorczF0Jjk7X5sLjKaPhBIwkMvYy99jCJhTaU4XlLYdnWwrvwc gT23Ym2dLMHTVsmKS9dxxDADWLzLVyYhyaGhbwMLl2KEHfn9gOxItyF16HnMr3XY/KW1 kl1l3zYWwiEYgeRQMq1l/19RqCdiPyyhfbc4/LJ3ssEz6Kvr2avxXNMmULIMuNo5fZnI 62oxsZdsMayc9bTwVpuKvDF2rdJEhzL3qPNu27W9DkBNt6OMGfu6daqHBAqzqLOpnD1r UjlD9Hn9KuPpi3V1lIrQTpFCI+Rled4k3/vDXb9iOAWIQWMZCvN43C2Gooc0avZFYuEO VwPA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777698468; x=1778303268; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=A9PB/D/JhFEvy+R3A7+7Tb+xDlORxmPJZApX2c7GDnA=; b=QSM5pglR36J1lIXC3DFJTTxYByPBKhjHL4ZqX4Cl2QQWlgJGwTgbL1jnyr+gWhryyc U35fF6WLwK33S7ULX7Hzcg7DO6OGS7JSjFENVddTY1rAXvHzNDIxruf3nf72UyqoBhC1 5QvQCFrtRtjY/dmVNsGvqs1N2P1jd2gMF/OV+Cp1q1F0Mh9MEfWMEF7IznoFoLV2BEl0 mu3nN3hciAeb7ERLYT6p46S/0npGzruHlI1ei1UYcVsxY0A8CY2u2ZL5S4NNLp61T4ZK UQ/Inc3NVkRmm9w6zQXFTrl0G9/q6iq+Y42mOT8ABouNY3fioY/lhdkb+D4GRFldzazE xxdw== X-Forwarded-Encrypted: i=1; AFNElJ/BI+9BMcPNu5O/fiA/dW1VX1FcL8wRHp4dkOk4YbT3FGBtd47dbMjc4Gu7ZEocp930h6JFnmkspVcscIQ=@vger.kernel.org X-Gm-Message-State: AOJu0YzMWXHM2yRhB+XvOeOySfsC0qKKbNDBJJ/tSI1E9rJOcrX6BH29 qfK2JZKFuhiEiRj0/nWAL6jZWOIu0d+nh/SsaXmGHfHUOrWjyhrkaJ8= X-Gm-Gg: AeBDieuvv+fF3UCeiofp4IrIvcDPflO/dwUSuieepnyTxEijdYYq6HuN0Ls6iNylCG9 ITFd01J5pjJrAclp/EcQhbw2zYrIIozgUxKAxMCP3y3kqeQ9NYCbnbbeL9cf8ufOKiRFIhxcyg8 AEy4BDCs3RhLbRMeNWL/pS4fHuLYgSzKm8A5iDc6zQBnVIlnDFw4OaaSvF7qO7f+AddwVb7TDyD wokzAUl1WXk3InnVC2DDO6w4l/jkMTqmnYwKPkgF36+8UcRrLXnR7ZlxMz7Yr3olL4WvcWSmsZL On26T3GAG0NNCX5c0T63RLRZ3KgtULep5g2lnY+XWEWWcobw5OSLwEqvOKMdW3TzRL2OcHFxYG7 OnFnPUyfZBZF+JltK1dOJV16F/SN4HsUrr8BxeazevA+qI+aKUx+LEYm9nkyNDQ/LiEuhJNKitq f0/IHl2eKvKLMMqzp+vDoV6GsSBooU9HBZ39i1d2d7+fCiFQ2zo1+G/3ygRaCppWoPINBxBpIl2 9rj7XLYUg== X-Received: by 2002:a17:902:ce05:b0:2a9:e8b:5326 with SMTP id d9443c01a7336-2b9f2808c9dmr18991085ad.23.1777698467520; Fri, 01 May 2026 22:07:47 -0700 (PDT) Received: from localhost.localdomain ([1.226.165.54]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2b9caa7e44bsm38263715ad.11.2026.05.01.22.07.44 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Fri, 01 May 2026 22:07:46 -0700 (PDT) From: Myeonghun Pak To: Horatiu Vultur , UNGLinuxDriver@microchip.com, Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Ijae Kim Cc: Myeonghun Pak , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v2] net: lan966x: avoid unregistering netdev on register failure Date: Sat, 2 May 2026 14:07:27 +0900 Message-ID: <20260502050741.76945-1-mhun512@gmail.com> X-Mailer: git-send-email 2.47.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit lan966x_probe_port() stores the newly allocated net_device in the port before calling register_netdev(). If register_netdev() fails, the probe error path calls lan966x_cleanup_ports(), which sees port->dev and calls unregister_netdev() for a device that was never registered. Destroy the phylink instance created for this port and clear port->dev before returning the registration error, matching the existing guard used by the common cleanup path. Also require port->dev to be non-NULL before matching it against the FDMA netdev, so the error path cannot treat an uninitialized FDMA netdev and a failed port as a valid match. Fixes: d28d6d2e37d1 ("net: lan966x: add port module support") Co-developed-by: Ijae Kim Signed-off-by: Ijae Kim Signed-off-by: Myeonghun Pak --- Changes in v2: - Require port->dev to be non-NULL before calling the FDMA netdev cleanup path, avoiding a NULL == NULL match when register_netdev() fails before FDMA NAPI has been initialized. drivers/net/ethernet/microchip/lan966x/lan966x_main.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/net/ethernet/microchip/lan966x/lan966x_main.c b/drivers/net/ethernet/microchip/lan966x/lan966x_main.c index 47752d3fde..22c496f588 100644 --- a/drivers/net/ethernet/microchip/lan966x/lan966x_main.c +++ b/drivers/net/ethernet/microchip/lan966x/lan966x_main.c @@ -756,7 +756,7 @@ static void lan966x_cleanup_ports(struct lan966x *lan966x) unregister_netdev(port->dev); lan966x_xdp_port_deinit(port); - if (lan966x->fdma && lan966x->fdma_ndev == port->dev) + if (lan966x->fdma && port->dev && lan966x->fdma_ndev == port->dev) lan966x_fdma_netdev_deinit(lan966x, port->dev); if (port->phylink) { @@ -873,6 +873,9 @@ static int lan966x_probe_port(struct lan966x *lan966x, u32 p, err = register_netdev(dev); if (err) { dev_err(lan966x->dev, "register_netdev failed\n"); + phylink_destroy(phylink); + port->phylink = NULL; + port->dev = NULL; return err; }