From nobody Sun Feb 8 14:53:06 2026 Received: from mail-pl1-f172.google.com (mail-pl1-f172.google.com [209.85.214.172]) (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 7734D42A82 for ; Sun, 18 Jan 2026 01:39:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768700390; cv=none; b=IQJ+oB9zZVI3n1Wp6bTD2ouz4n7nuuVVh11ZOk13hd7j5D/4cDSGaFzd4sXDuKT/lTNXb4Ecu/fXjNQUwK3FkI3ywXRJZMY/yNoKqIkay6nqc0PxyuBg8U3kHIjuamWVFuHNmx7jsynoT56TwLpxVC57MlX8Y90bzVmuSTxvMFU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768700390; c=relaxed/simple; bh=mHd5/FDiQ86yScP8/xKWBTcg+ow54G9KDtNwccSg1Uk=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=QbnQwESPmzqKFEpc2plwPQQjM2LZaqbRAIfxGnG6/8QVxIn3TNLQPpPuUwuXaLmLiop6qeMxWE6wAk9jdzzcAvcnnt9RF/30F9AGWSSyXn3P4i/vC6zKL6gpv4c6uZi0MCsZdd+Qcjx1JqJSR5jfyKTHDS1rXAXK5NNnlKwJwQk= 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=F1F6wLHt; arc=none smtp.client-ip=209.85.214.172 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="F1F6wLHt" Received: by mail-pl1-f172.google.com with SMTP id d9443c01a7336-29efd139227so20419775ad.1 for ; Sat, 17 Jan 2026 17:39:49 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1768700389; x=1769305189; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=qykhmziqocBs83eOMsCXS+riegj3dQCRKqwsythsgqI=; b=F1F6wLHtBVff3S8i3w5YUDm6RO/A5gmWxKXksgHJnpJUzu9Qti7ETGMZAzKIpz9tXp HKmvttyUi5Z+1PdhY7eZbmVYvUbJqqxFymBdz97XlfsWgiKGOMHTh/i12wBMygnuTfnP tKKF2qjgNKnbQUnOAOzI2AcBfhxcaH5I9V8n3U5DIbmlVyJot7j9RuaVdt6UO0fYLfZG oXZ3jmHByuGlSxNRLWDxTZPoeZ4rIzl98jPuZqyD2hmrLVib1p6RryjLhEiz4Qo0/Ztb 4Zs6D8qhQ3gyHXZqJ2r4xvBX79mXMJ/LyYOE8uldNeICpumRIDCcENTXehY7Oro3Z8tv ZOvQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1768700389; x=1769305189; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=qykhmziqocBs83eOMsCXS+riegj3dQCRKqwsythsgqI=; b=QqnXdPlZCKIEVjilKRx+TW2G/K0Gx8PycIYOdjG0tKfrnCkFUCHo6HescGdYtOJDcd nsNQ0QbCl7E6nbNfcwsg/KMpYQePq+yNRrsjVCddqeNNAGhDOUrZ03+z/EANOgG8rsU6 KSJt5Kov6+PHd1MmwA3ecF3noXhPvGfC1NzwmAqa7Ej2dOgCPn8R2uue+YJEh1hwdDH1 uXbGbBgUvSlq3hgzB2f1jJznW9r2gGOAoULG135PraUWrz8AvvWYji6blqwHbP7Jf7Tz HFtll1yD8e8V/kUFc1rp9tef0He6Ro5DJ9IvyXI6vvrgZRuUljMfoKufTUKFtEPWWrV0 4AFg== X-Forwarded-Encrypted: i=1; AJvYcCVzlglGKSz5TdsXKmI4ztA580KgAqMiwTEilDRvW+K+AThrLgY84orHH8T7gUqWluan7Iijc63onN4zDJU=@vger.kernel.org X-Gm-Message-State: AOJu0YxuMqllvOxAXB9rC9wIh3Vkwwt6VV9WeQd0ecwmbHIvBTGBnA3u ccnx8RnFlpcyKqan+rhkXOaWMST8Q2EnEwBPe1hNjK/50TXsSKjXGX3Q X-Gm-Gg: AY/fxX6AcpiTIvA3Fn9eGlNXCicugaFdAlXRSOIBVSJ/tRB7YTkTt07wkqa+Uz0eWfS YPIVLwctqifAMGKsP9ErjU2a5lOV3GG+JlCDiK1B+nKxIPrWBa2ndsliR8oy/h7TLzuds3D+bn5 R2Ano1bEgWvVc+++y0dpIsuGoQfNSqmA+nhzAHxmxRrrEN5z6K5tz/T15iF8uCKdt0G+gxpPZAt MZ4M3PiFhLe+53cgeCdZQ44cdxxgaTh3g2pNpfbsec1m2NR68S1kc3H7V4cjgXmPWVzOudLGurQ Fw8RjVBBAaiKKngUXoUuu/6TZpkdfN2LcBCutycXNW/nTFlOtcD0YzJrXXvVvgM878zz5wmtT8g z/RmLct8Zj8hYwV9nShl0s9+SWRqWcMv4mycDIVD5u49BlmxdWlX0rXI5cuaigrfHgFkJsa7VN5 EH6bl4Jx0sYuPEliqgDcj5SY6O8OmJwVt5haYyMBIRMAlbKpJK4g== X-Received: by 2002:a17:902:ccc7:b0:2a0:d5b0:dd82 with SMTP id d9443c01a7336-2a718a743e4mr64000315ad.61.1768700388730; Sat, 17 Jan 2026 17:39:48 -0800 (PST) Received: from d.home.mmyangfl.tk ([45.32.227.231]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2a7193dbbf5sm56030615ad.47.2026.01.17.17.39.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 17 Jan 2026 17:39:48 -0800 (PST) From: David Yang To: netdev@vger.kernel.org Cc: David Yang , Andrew Lunn , Vladimir Oltean , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , linux-kernel@vger.kernel.org, Dan Carpenter , David Laight Subject: [PATCH net-next v6 1/3] net: dsa: yt921x: Fix MIB overflow wraparound routine Date: Sun, 18 Jan 2026 09:30:14 +0800 Message-ID: <20260118013019.1078847-2-mmyangfl@gmail.com> X-Mailer: git-send-email 2.51.0 In-Reply-To: <20260118013019.1078847-1-mmyangfl@gmail.com> References: <20260118013019.1078847-1-mmyangfl@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Reported by the following Smatch static checker warning: drivers/net/dsa/yt921x.c:702 yt921x_read_mib() warn: was expecting a 64 bit value instead of '(~0)' Fixes: 186623f4aa72 ("net: dsa: yt921x: Add support for Motorcomm YT921x") Reported-by: Dan Carpenter Closes: https://lore.kernel.org/netdev/aPsjYKQMzpY0nSXm@stanley.mountain/ Suggested-by: David Laight Signed-off-by: David Yang --- drivers/net/dsa/yt921x.c | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/drivers/net/dsa/yt921x.c b/drivers/net/dsa/yt921x.c index 0b3df732c0d1..5e4e8093ba16 100644 --- a/drivers/net/dsa/yt921x.c +++ b/drivers/net/dsa/yt921x.c @@ -682,21 +682,22 @@ static int yt921x_read_mib(struct yt921x_priv *priv, = int port) const struct yt921x_mib_desc *desc =3D &yt921x_mib_descs[i]; u32 reg =3D YT921X_MIBn_DATA0(port) + desc->offset; u64 *valp =3D &((u64 *)mib)[i]; - u64 val =3D *valp; u32 val0; - u32 val1; + u64 val; =20 res =3D yt921x_reg_read(priv, reg, &val0); if (res) break; =20 if (desc->size <=3D 1) { - if (val < (u32)val) - /* overflow */ - val +=3D (u64)U32_MAX + 1; - val &=3D ~U32_MAX; - val |=3D val0; + u64 old_val =3D *valp; + + val =3D (old_val & ~(u64)U32_MAX) | val0; + if (val < old_val) + val +=3D 1ull << 32; } else { + u32 val1; + res =3D yt921x_reg_read(priv, reg + 4, &val1); if (res) break; --=20 2.51.0 From nobody Sun Feb 8 14:53:06 2026 Received: from mail-pl1-f174.google.com (mail-pl1-f174.google.com [209.85.214.174]) (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 E0E4B21B9FD for ; Sun, 18 Jan 2026 01:39:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.174 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768700394; cv=none; b=dzPVRbmrwlmNKJtq+W5YHW1zYTtfu0rzbnk3IszksQTi0cJR5jjlNB1B1SzKsENYNyOlpmwdnkCwdHrLUar+5xKAt7C3Q8qqRQLCEQgu1jJoA6n+6L8KfNMPldPH8/lMe70pdOcWjX38FAfl+W6ejjIUF/j3ou9bBzNw9NrjgWY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768700394; c=relaxed/simple; bh=Itni9SNfJdb7Dhz/FCbUKIbVg6MfARfvhe9M09DKCfI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=o/gNzsj06N00VMXeFZXnGM6WQZ1+bC4/179jPOSCGJ4Xzf7LP8wpsEXkr6eSGAWgibLqA2G/BQkpApgzeKpaBgFWH4GBRj6ERJ0SwGsUEk4mtMV84RcccXUIkuaqkjqr6AJpI8nDzMhh+4pPMIEQRBezyJy7kNQeJzuCtlpqTIc= 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=hFNaxSaA; arc=none smtp.client-ip=209.85.214.174 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="hFNaxSaA" Received: by mail-pl1-f174.google.com with SMTP id d9443c01a7336-2a09757004cso28985835ad.3 for ; Sat, 17 Jan 2026 17:39:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1768700392; x=1769305192; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=tYwkrpDpNTFizksR2j091lBfcUhZu5HCF7/lQUtqnk4=; b=hFNaxSaAJG3wt8VPn6sBEjEtckIrbao5POZXfza9fItW8lc0rFg3eZl62nKE7M9py2 iTgpBvSy4DPB6kZ3rLZIcRFSKPCVRI2yJOGFNSJOdd026dRIDf53DkGkSIhKz2lM+1MC Ti/5E6eZMoxy21Y9klNGRTLUQvk52mOcInk3dGYHdNaH+VXTw9qh7OMq+DBUdVZbM8Ah QiabPYwPg6lPZSeKZTZ6mO9cQ/dQVRIUyAQ5YX66umjdkFaxRZP6G8cmMx+Zq/d9FwRb Uae/FX0AP7oTGt/F3EuCgt1CGn5N6aK9T3JtWxVK6KPX45Ofum+HT6Lh2gVBfC9OXc2V L1Tg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1768700392; x=1769305192; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=tYwkrpDpNTFizksR2j091lBfcUhZu5HCF7/lQUtqnk4=; b=NWEEswUcBiZaTIEvXLAzgGXkZZP5RFdwtnYYjGBPGSkKmK5RpeTWpf+9BDX8qUfgbS 7+uTzWT+587zgjB+4bzeyf0Qkpv93QzZ3J9WZbMqj14q4qoNvOKKlM3pvqQl9l9lgdGa bSfZ4fqaUrD6wuHc/4jt/hYPhTvQxVM1mmPzmJVLvuuix+MG34ebdFQo48ehg324yeV0 EEQEPqeLcetMjohqp9IMT91YOsIIDG3TRdBssQ7RkOifWK06xBSKB09MgmFNWQtCg1UC gd04jJgaDKWpqCH8CETsLR4J9PYeb90UXLvFaLSIMaAJwUJIQzRuB7iiDGkk+R2aMWmH ep9g== X-Forwarded-Encrypted: i=1; AJvYcCUugsi9ymtPQ85YsMaPt+x+TnSSSGumc2pUOkbhQMcuXPyvUhRdCcZxHTz0v1Uk+nlGjvsaEHc3hAGRoBo=@vger.kernel.org X-Gm-Message-State: AOJu0Yw3SbZoRmm0HmNay6Mo38es4kqI4tONVryUcydxIdWTyPlby30Q eUdmtcXwOGvrlPNzCuNElco2wFhbhiCUT1iCYSfDf7bt7z5IBeCgDmN5 X-Gm-Gg: AY/fxX66ey+UyARHUOdylhF1CswUQs4xhPsLpyu+Tp4T2Xihwzgp+HSuJhSBgkZ3Rj4 qZ4LpM6ivUgSIB5E4L6dJi+7pT+fK289fpGyWavphCnkDHpxiO1os+I+TOPq1BLGOb/YnuTgqMd ec0NZQxwGaP73u+0YT8HAeWRdgWlRIGDqSedUbTIH9rmF0VBHzhWYnBGlQiAerXWYp3ZcfMBTzX vRjrGPbEbxHN7f/mIwoQ+qiIAIerhfyftK9A7u5YfzNBZ+HXv4sQ0mRImWOX89bmP/u47gJxGmQ VUGvk+J1jbLD+QMVoHwa+RNzafjChhLg5MAKVHj2IPDNU2+5V7rnsdiU2qZDyAJKtB+O8LeiXTX 6beMKlRRjrIWfRxcHNzCXExmzI6ry6PTG9VmQslqb8b3Lb7UmQtgYRMT/eIrhUCLeSZBgAksHS5 2ZzXJu0ePzQf/i6Bgs3YXGj5LgpSdTcdHDLfaEx9d01t5fMH0iJQ== X-Received: by 2002:a17:902:db0a:b0:2a3:bf5f:9269 with SMTP id d9443c01a7336-2a7174f0127mr83993075ad.3.1768700392217; Sat, 17 Jan 2026 17:39:52 -0800 (PST) Received: from d.home.mmyangfl.tk ([45.32.227.231]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2a7193dbbf5sm56030615ad.47.2026.01.17.17.39.49 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 17 Jan 2026 17:39:51 -0800 (PST) From: David Yang To: netdev@vger.kernel.org Cc: David Yang , Andrew Lunn , Vladimir Oltean , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , linux-kernel@vger.kernel.org Subject: [PATCH net-next v6 2/3] net: dsa: yt921x: Return early for failed MIB read Date: Sun, 18 Jan 2026 09:30:15 +0800 Message-ID: <20260118013019.1078847-3-mmyangfl@gmail.com> X-Mailer: git-send-email 2.51.0 In-Reply-To: <20260118013019.1078847-1-mmyangfl@gmail.com> References: <20260118013019.1078847-1-mmyangfl@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" This patch does not change anything effectively, but serves as a prerequisite for another patch. Signed-off-by: David Yang --- drivers/net/dsa/yt921x.c | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/drivers/net/dsa/yt921x.c b/drivers/net/dsa/yt921x.c index 5e4e8093ba16..fe08385445d2 100644 --- a/drivers/net/dsa/yt921x.c +++ b/drivers/net/dsa/yt921x.c @@ -707,6 +707,12 @@ static int yt921x_read_mib(struct yt921x_priv *priv, i= nt port) WRITE_ONCE(*valp, val); } =20 + if (res) { + dev_err(dev, "Failed to %s port %d: %i\n", "read stats for", + port, res); + return res; + } + pp->rx_frames =3D mib->rx_64byte + mib->rx_65_127byte + mib->rx_128_255byte + mib->rx_256_511byte + mib->rx_512_1023byte + mib->rx_1024_1518byte + @@ -716,10 +722,7 @@ static int yt921x_read_mib(struct yt921x_priv *priv, i= nt port) mib->tx_512_1023byte + mib->tx_1024_1518byte + mib->tx_jumbo; =20 - if (res) - dev_err(dev, "Failed to %s port %d: %i\n", "read stats for", - port, res); - return res; + return 0; } =20 static void yt921x_poll_mib(struct work_struct *work) --=20 2.51.0 From nobody Sun Feb 8 14:53:06 2026 Received: from mail-pl1-f172.google.com (mail-pl1-f172.google.com [209.85.214.172]) (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 8390E2222D2 for ; Sun, 18 Jan 2026 01:39:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768700398; cv=none; b=hu2ZmTmFWjdYDydKhSaZ46x2y4iMz/2DIBDDGyamUSPdgOHWExzvmB/w++yzjyrir6kxwlJBp5awtXAYdhXHHcuPVM6OFFwJKc/WQUkpuVUPNopX4o5yFPAuhfNQ3OSV1CosC+7nNEfq1YIhaF7aIZPZB22Ma4C+9r0LHJ2SUMs= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768700398; c=relaxed/simple; bh=1dMRjURizLmyAKEXN7wjz53SViwwPTPux9yr0Zr60gY=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=pLA+tp79eM/gFaMSzNc9xov6+3xJwULI4aDwiAVrerayR+s4gUCHIcGfjxHmuBMxNsuQdrc6cv20N36j4eRpLfbvD/3F3RgN1bs5SGwbC5AaHe36FByysxYv1YydR1RVjDRvfrQw24qvkzKwFnPD/iDX2qq/PuJ1skRiwIWpDIc= 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=GANmK4Jq; arc=none smtp.client-ip=209.85.214.172 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="GANmK4Jq" Received: by mail-pl1-f172.google.com with SMTP id d9443c01a7336-29f30233d8aso22004395ad.0 for ; Sat, 17 Jan 2026 17:39:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1768700396; x=1769305196; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=umLx+BGjT4uYx/x7sL81vw16fF3imM/rUa8KKEKKGMc=; b=GANmK4JqJwvG1sHDEQiQJeA3wSaYbQy/Zkr8MorppD54fRpYnG+P4Vq+jONsGeG/ZU maZQeZ5ZfEoArN8dRSj2lDW7YohH282YwWvm4ckaZzfCvxVnqHuwNXiyxHZSotJOkpvb /itZtjnXKLauEMfqonTNpYmj4Yfrema4GcAjHfIC0T/ixY+RTliawZL1CT8iEEubxbeo +xVCiKOYffJhB1aPhi3fFA7BoWVSAwVBU3PHP9S3h7ygDgZA+QieYzdfo8qfa+DRuODJ EcEG4I0IVTLb4iA3z/qyuHEedP602oyf0kZJTYNzZGxQ27xeEOydCPkymel8kFPG0ucq eP7w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1768700396; x=1769305196; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=umLx+BGjT4uYx/x7sL81vw16fF3imM/rUa8KKEKKGMc=; b=XjdVVMSr889sm5bwVKyC9GDNNnEN44hBori8L4OEcKZ0AHeqkdTM3jYr0ipTKF4naV yyVCMYsWMA5xNDWT2S7PasI0rImMlQ1Ug1P046Rz2TqlqWHriLIPvH6IC0zAj6AGkg1b Bjshpnbsn+pAqQHtaapPo890ZyK86iqGPPNaxFxzE+rEB5UtssDrK3uMAoz1KswEYf8X AZ+3GUdTG5EahEbl7p0jiIvhdEssn6IYyLUl4e+5hSymk1BbEb27YHH/SDvDkYjiEJ43 YYEvrYhG2YmmZRJ3mgpIgbpR8uy/B1wxPR5wrtIKUCJebLFMH72hrDFIDunohIX/Ba79 hIug== X-Forwarded-Encrypted: i=1; AJvYcCW2C6lf8kh8J9NssKG1YqxWBIyDQEFPfAFhBmUJS263utXrjQE/Dhz2A2NcwHrA7qVTUzpGmnE84ZqKtAY=@vger.kernel.org X-Gm-Message-State: AOJu0Yw2Gwwn9oPc97E19Ql78Vo4SSpeHulSM3pZ67P6UVVPunRhfCgP /kPlTRbOB/Q/cgLgRMczxQMiEzHPbnI4hIF9IAYIhCAbgcOA0Fy2hiH6qoNfKw== X-Gm-Gg: AY/fxX4eTIwTizHuLSeqmqwPL2OR30B6iyt3b0mzfS6Z4VMgj2yYly9Ww1/0BMTX/qG 58zEEFNSd1RKptpPUJlfe51yy+9nFHjS9wrFnPt0h/sfgTli2M2OB/U4zUQG79DDWS4GAuT+VJj VMvV7hKxWNndS/iL1fRCcO1v6+HheYLAUik7bKs9merXFT//wOSGUpubLULmVfVdaylBOSS6yMN heYxl0f+TV2JaiKCtqBEkBY0yS0WdQxgt2PqEeZ/xtqVcu2lfBCx9sJacUJZhxRzC1xACqxdpwh yCR6pEu8cwZ/E2yoKX3yrB+VFuTm8wIuy+OlYzJCz8E6J2EO5CRv6H5kXizIhPS0eClBVDQQ5SE YLy7K/dDbTzNbVHeUeDGsrPI1jp8ivzf/u3+fhUZxVksj8QA0eQADYee0/S0stWD9QhPo/v+X73 tfGIW3lnrHHVjXAVhVMvGv/jWpt2S6w57oUAwbdVkdeHjaI6gvXg== X-Received: by 2002:a17:902:f652:b0:269:8d1b:40c3 with SMTP id d9443c01a7336-2a71888abfdmr77013255ad.12.1768700395992; Sat, 17 Jan 2026 17:39:55 -0800 (PST) Received: from d.home.mmyangfl.tk ([45.32.227.231]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2a7193dbbf5sm56030615ad.47.2026.01.17.17.39.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 17 Jan 2026 17:39:55 -0800 (PST) From: David Yang To: netdev@vger.kernel.org Cc: David Yang , Andrew Lunn , Vladimir Oltean , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , linux-kernel@vger.kernel.org Subject: [PATCH net-next v6 3/3] net: dsa: yt921x: Use u64_stats_t for MIB stats Date: Sun, 18 Jan 2026 09:30:16 +0800 Message-ID: <20260118013019.1078847-4-mmyangfl@gmail.com> X-Mailer: git-send-email 2.51.0 In-Reply-To: <20260118013019.1078847-1-mmyangfl@gmail.com> References: <20260118013019.1078847-1-mmyangfl@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" 64-bit variables might not be atomic on 32-bit architectures, thus cannot be unconditionally made lock-free. Use u64_stats_t so it would still be lock-free on 64-bit architectures. Signed-off-by: David Yang --- drivers/net/dsa/yt921x.c | 236 +++++++++++++++++++++++---------------- drivers/net/dsa/yt921x.h | 114 ++++++++++--------- 2 files changed, 205 insertions(+), 145 deletions(-) diff --git a/drivers/net/dsa/yt921x.c b/drivers/net/dsa/yt921x.c index fe08385445d2..f850117e7a68 100644 --- a/drivers/net/dsa/yt921x.c +++ b/drivers/net/dsa/yt921x.c @@ -18,6 +18,7 @@ #include #include #include +#include =20 #include =20 @@ -666,22 +667,20 @@ yt921x_mbus_ext_init(struct yt921x_priv *priv, struct= device_node *mnp) static int yt921x_read_mib(struct yt921x_priv *priv, int port) { struct yt921x_port *pp =3D &priv->ports[port]; + struct yt921x_mib *mib_new =3D &pp->mib_new; struct device *dev =3D to_device(priv); struct yt921x_mib *mib =3D &pp->mib; + u64 rx_frames; + u64 tx_frames; int res =3D 0; =20 - /* Reading of yt921x_port::mib is not protected by a lock and it's vain - * to keep its consistency, since we have to read registers one by one - * and there is no way to make a snapshot of MIB stats. - * - * Writing (by this function only) is and should be protected by - * reg_lock. + /* u64_stats_read/set is redundant for mib_new, but I don't want to + * declare a plain u64 yt921x_mib variant. */ =20 for (size_t i =3D 0; i < ARRAY_SIZE(yt921x_mib_descs); i++) { const struct yt921x_mib_desc *desc =3D &yt921x_mib_descs[i]; u32 reg =3D YT921X_MIBn_DATA0(port) + desc->offset; - u64 *valp =3D &((u64 *)mib)[i]; u32 val0; u64 val; =20 @@ -690,7 +689,7 @@ static int yt921x_read_mib(struct yt921x_priv *priv, in= t port) break; =20 if (desc->size <=3D 1) { - u64 old_val =3D *valp; + u64 old_val =3D u64_stats_read(&mib->stats[i]); =20 val =3D (old_val & ~(u64)U32_MAX) | val0; if (val < old_val) @@ -704,7 +703,7 @@ static int yt921x_read_mib(struct yt921x_priv *priv, in= t port) val =3D ((u64)val1 << 32) | val0; } =20 - WRITE_ONCE(*valp, val); + u64_stats_set(&mib_new->stats[i], val); } =20 if (res) { @@ -713,14 +712,29 @@ static int yt921x_read_mib(struct yt921x_priv *priv, = int port) return res; } =20 - pp->rx_frames =3D mib->rx_64byte + mib->rx_65_127byte + - mib->rx_128_255byte + mib->rx_256_511byte + - mib->rx_512_1023byte + mib->rx_1024_1518byte + - mib->rx_jumbo; - pp->tx_frames =3D mib->tx_64byte + mib->tx_65_127byte + - mib->tx_128_255byte + mib->tx_256_511byte + - mib->tx_512_1023byte + mib->tx_1024_1518byte + - mib->tx_jumbo; + rx_frames =3D u64_stats_read(&mib_new->rx_64byte) + + u64_stats_read(&mib_new->rx_65_127byte) + + u64_stats_read(&mib_new->rx_128_255byte) + + u64_stats_read(&mib_new->rx_256_511byte) + + u64_stats_read(&mib_new->rx_512_1023byte) + + u64_stats_read(&mib_new->rx_1024_1518byte) + + u64_stats_read(&mib_new->rx_jumbo); + tx_frames =3D u64_stats_read(&mib_new->tx_64byte) + + u64_stats_read(&mib_new->tx_65_127byte) + + u64_stats_read(&mib_new->tx_128_255byte) + + u64_stats_read(&mib_new->tx_256_511byte) + + u64_stats_read(&mib_new->tx_512_1023byte) + + u64_stats_read(&mib_new->tx_1024_1518byte) + + u64_stats_read(&mib_new->tx_jumbo); + + u64_stats_update_begin(&pp->syncp); + for (size_t i =3D 0; i < ARRAY_SIZE(yt921x_mib_descs); i++) { + u64_stats_set(&mib->stats[i], + u64_stats_read(&mib_new->stats[i])); + } + u64_stats_set(&pp->rx_frames, rx_frames); + u64_stats_set(&pp->tx_frames, tx_frames); + u64_stats_update_end(&pp->syncp); =20 return 0; } @@ -765,22 +779,27 @@ yt921x_dsa_get_ethtool_stats(struct dsa_switch *ds, i= nt port, uint64_t *data) struct yt921x_priv *priv =3D to_yt921x_priv(ds); struct yt921x_port *pp =3D &priv->ports[port]; struct yt921x_mib *mib =3D &pp->mib; + unsigned int start; size_t j; =20 mutex_lock(&priv->reg_lock); yt921x_read_mib(priv, port); mutex_unlock(&priv->reg_lock); =20 - j =3D 0; - for (size_t i =3D 0; i < ARRAY_SIZE(yt921x_mib_descs); i++) { - const struct yt921x_mib_desc *desc =3D &yt921x_mib_descs[i]; + do { + start =3D u64_stats_fetch_begin(&pp->syncp); =20 - if (!desc->name) - continue; + j =3D 0; + for (size_t i =3D 0; i < ARRAY_SIZE(yt921x_mib_descs); i++) { + const struct yt921x_mib_desc *desc =3D &yt921x_mib_descs[i]; =20 - data[j] =3D ((u64 *)mib)[i]; - j++; - } + if (!desc->name) + continue; + + data[j] =3D u64_stats_read(&((u64_stats_t *)mib)[i]); + j++; + } + } while (u64_stats_fetch_retry(&pp->syncp, start)); } =20 static int yt921x_dsa_get_sset_count(struct dsa_switch *ds, int port, int = sset) @@ -807,33 +826,38 @@ yt921x_dsa_get_eth_mac_stats(struct dsa_switch *ds, i= nt port, struct yt921x_priv *priv =3D to_yt921x_priv(ds); struct yt921x_port *pp =3D &priv->ports[port]; struct yt921x_mib *mib =3D &pp->mib; + unsigned int start; =20 mutex_lock(&priv->reg_lock); yt921x_read_mib(priv, port); mutex_unlock(&priv->reg_lock); =20 - mac_stats->FramesTransmittedOK =3D pp->tx_frames; - mac_stats->SingleCollisionFrames =3D mib->tx_single_collisions; - mac_stats->MultipleCollisionFrames =3D mib->tx_multiple_collisions; - mac_stats->FramesReceivedOK =3D pp->rx_frames; - mac_stats->FrameCheckSequenceErrors =3D mib->rx_crc_errors; - mac_stats->AlignmentErrors =3D mib->rx_alignment_errors; - mac_stats->OctetsTransmittedOK =3D mib->tx_good_bytes; - mac_stats->FramesWithDeferredXmissions =3D mib->tx_deferred; - mac_stats->LateCollisions =3D mib->tx_late_collisions; - mac_stats->FramesAbortedDueToXSColls =3D mib->tx_aborted_errors; - /* mac_stats->FramesLostDueToIntMACXmitError */ - /* mac_stats->CarrierSenseErrors */ - mac_stats->OctetsReceivedOK =3D mib->rx_good_bytes; - /* mac_stats->FramesLostDueToIntMACRcvError */ - mac_stats->MulticastFramesXmittedOK =3D mib->tx_multicast; - mac_stats->BroadcastFramesXmittedOK =3D mib->tx_broadcast; - /* mac_stats->FramesWithExcessiveDeferral */ - mac_stats->MulticastFramesReceivedOK =3D mib->rx_multicast; - mac_stats->BroadcastFramesReceivedOK =3D mib->rx_broadcast; - /* mac_stats->InRangeLengthErrors */ - /* mac_stats->OutOfRangeLengthField */ - mac_stats->FrameTooLongErrors =3D mib->rx_oversize_errors; + do { + start =3D u64_stats_fetch_begin(&pp->syncp); + + mac_stats->FramesTransmittedOK =3D u64_stats_read(&pp->tx_frames); + mac_stats->SingleCollisionFrames =3D u64_stats_read(&mib->tx_single_coll= isions); + mac_stats->MultipleCollisionFrames =3D u64_stats_read(&mib->tx_multiple_= collisions); + mac_stats->FramesReceivedOK =3D u64_stats_read(&pp->rx_frames); + mac_stats->FrameCheckSequenceErrors =3D u64_stats_read(&mib->rx_crc_erro= rs); + mac_stats->AlignmentErrors =3D u64_stats_read(&mib->rx_alignment_errors); + mac_stats->OctetsTransmittedOK =3D u64_stats_read(&mib->tx_good_bytes); + mac_stats->FramesWithDeferredXmissions =3D u64_stats_read(&mib->tx_defer= red); + mac_stats->LateCollisions =3D u64_stats_read(&mib->tx_late_collisions); + mac_stats->FramesAbortedDueToXSColls =3D u64_stats_read(&mib->tx_aborted= _errors); + /* mac_stats->FramesLostDueToIntMACXmitError */ + /* mac_stats->CarrierSenseErrors */ + mac_stats->OctetsReceivedOK =3D u64_stats_read(&mib->rx_good_bytes); + /* mac_stats->FramesLostDueToIntMACRcvError */ + mac_stats->MulticastFramesXmittedOK =3D u64_stats_read(&mib->tx_multicas= t); + mac_stats->BroadcastFramesXmittedOK =3D u64_stats_read(&mib->tx_broadcas= t); + /* mac_stats->FramesWithExcessiveDeferral */ + mac_stats->MulticastFramesReceivedOK =3D u64_stats_read(&mib->rx_multica= st); + mac_stats->BroadcastFramesReceivedOK =3D u64_stats_read(&mib->rx_broadca= st); + /* mac_stats->InRangeLengthErrors */ + /* mac_stats->OutOfRangeLengthField */ + mac_stats->FrameTooLongErrors =3D u64_stats_read(&mib->rx_oversize_error= s); + } while (u64_stats_fetch_retry(&pp->syncp, start)); } =20 static void @@ -843,14 +867,19 @@ yt921x_dsa_get_eth_ctrl_stats(struct dsa_switch *ds, = int port, struct yt921x_priv *priv =3D to_yt921x_priv(ds); struct yt921x_port *pp =3D &priv->ports[port]; struct yt921x_mib *mib =3D &pp->mib; + unsigned int start; =20 mutex_lock(&priv->reg_lock); yt921x_read_mib(priv, port); mutex_unlock(&priv->reg_lock); =20 - ctrl_stats->MACControlFramesTransmitted =3D mib->tx_pause; - ctrl_stats->MACControlFramesReceived =3D mib->rx_pause; - /* ctrl_stats->UnsupportedOpcodesReceived */ + do { + start =3D u64_stats_fetch_begin(&pp->syncp); + + ctrl_stats->MACControlFramesTransmitted =3D u64_stats_read(&mib->tx_paus= e); + ctrl_stats->MACControlFramesReceived =3D u64_stats_read(&mib->rx_pause); + /* ctrl_stats->UnsupportedOpcodesReceived */ + } while (u64_stats_fetch_retry(&pp->syncp, start)); } =20 static const struct ethtool_rmon_hist_range yt921x_rmon_ranges[] =3D { @@ -872,6 +901,7 @@ yt921x_dsa_get_rmon_stats(struct dsa_switch *ds, int po= rt, struct yt921x_priv *priv =3D to_yt921x_priv(ds); struct yt921x_port *pp =3D &priv->ports[port]; struct yt921x_mib *mib =3D &pp->mib; + unsigned int start; =20 mutex_lock(&priv->reg_lock); yt921x_read_mib(priv, port); @@ -879,26 +909,30 @@ yt921x_dsa_get_rmon_stats(struct dsa_switch *ds, int = port, =20 *ranges =3D yt921x_rmon_ranges; =20 - rmon_stats->undersize_pkts =3D mib->rx_undersize_errors; - rmon_stats->oversize_pkts =3D mib->rx_oversize_errors; - rmon_stats->fragments =3D mib->rx_alignment_errors; - /* rmon_stats->jabbers */ - - rmon_stats->hist[0] =3D mib->rx_64byte; - rmon_stats->hist[1] =3D mib->rx_65_127byte; - rmon_stats->hist[2] =3D mib->rx_128_255byte; - rmon_stats->hist[3] =3D mib->rx_256_511byte; - rmon_stats->hist[4] =3D mib->rx_512_1023byte; - rmon_stats->hist[5] =3D mib->rx_1024_1518byte; - rmon_stats->hist[6] =3D mib->rx_jumbo; - - rmon_stats->hist_tx[0] =3D mib->tx_64byte; - rmon_stats->hist_tx[1] =3D mib->tx_65_127byte; - rmon_stats->hist_tx[2] =3D mib->tx_128_255byte; - rmon_stats->hist_tx[3] =3D mib->tx_256_511byte; - rmon_stats->hist_tx[4] =3D mib->tx_512_1023byte; - rmon_stats->hist_tx[5] =3D mib->tx_1024_1518byte; - rmon_stats->hist_tx[6] =3D mib->tx_jumbo; + do { + start =3D u64_stats_fetch_begin(&pp->syncp); + + rmon_stats->undersize_pkts =3D u64_stats_read(&mib->rx_undersize_errors); + rmon_stats->oversize_pkts =3D u64_stats_read(&mib->rx_oversize_errors); + rmon_stats->fragments =3D u64_stats_read(&mib->rx_alignment_errors); + /* rmon_stats->jabbers */ + + rmon_stats->hist[0] =3D u64_stats_read(&mib->rx_64byte); + rmon_stats->hist[1] =3D u64_stats_read(&mib->rx_65_127byte); + rmon_stats->hist[2] =3D u64_stats_read(&mib->rx_128_255byte); + rmon_stats->hist[3] =3D u64_stats_read(&mib->rx_256_511byte); + rmon_stats->hist[4] =3D u64_stats_read(&mib->rx_512_1023byte); + rmon_stats->hist[5] =3D u64_stats_read(&mib->rx_1024_1518byte); + rmon_stats->hist[6] =3D u64_stats_read(&mib->rx_jumbo); + + rmon_stats->hist_tx[0] =3D u64_stats_read(&mib->tx_64byte); + rmon_stats->hist_tx[1] =3D u64_stats_read(&mib->tx_65_127byte); + rmon_stats->hist_tx[2] =3D u64_stats_read(&mib->tx_128_255byte); + rmon_stats->hist_tx[3] =3D u64_stats_read(&mib->tx_256_511byte); + rmon_stats->hist_tx[4] =3D u64_stats_read(&mib->tx_512_1023byte); + rmon_stats->hist_tx[5] =3D u64_stats_read(&mib->tx_1024_1518byte); + rmon_stats->hist_tx[6] =3D u64_stats_read(&mib->tx_jumbo); + } while (u64_stats_fetch_retry(&pp->syncp, start)); } =20 static void @@ -908,33 +942,41 @@ yt921x_dsa_get_stats64(struct dsa_switch *ds, int por= t, struct yt921x_priv *priv =3D to_yt921x_priv(ds); struct yt921x_port *pp =3D &priv->ports[port]; struct yt921x_mib *mib =3D &pp->mib; + unsigned int start; + + do { + start =3D u64_stats_fetch_begin(&pp->syncp); + + stats->rx_length_errors =3D u64_stats_read(&mib->rx_undersize_errors) + + u64_stats_read(&mib->rx_fragment_errors); + stats->rx_over_errors =3D u64_stats_read(&mib->rx_oversize_errors); + stats->rx_crc_errors =3D u64_stats_read(&mib->rx_crc_errors); + stats->rx_frame_errors =3D u64_stats_read(&mib->rx_alignment_errors); + /* stats->rx_fifo_errors */ + /* stats->rx_missed_errors */ + + stats->tx_aborted_errors =3D u64_stats_read(&mib->tx_aborted_errors); + /* stats->tx_carrier_errors */ + stats->tx_fifo_errors =3D u64_stats_read(&mib->tx_undersize_errors); + /* stats->tx_heartbeat_errors */ + stats->tx_window_errors =3D u64_stats_read(&mib->tx_late_collisions); + + stats->rx_packets =3D u64_stats_read(&pp->rx_frames); + stats->tx_packets =3D u64_stats_read(&pp->tx_frames); + stats->rx_bytes =3D u64_stats_read(&mib->rx_good_bytes) - + ETH_FCS_LEN * stats->rx_packets; + stats->tx_bytes =3D u64_stats_read(&mib->tx_good_bytes) - + ETH_FCS_LEN * stats->tx_packets; + stats->rx_dropped =3D u64_stats_read(&mib->rx_dropped); + /* stats->tx_dropped */ + stats->multicast =3D u64_stats_read(&mib->rx_multicast); + stats->collisions =3D u64_stats_read(&mib->tx_collisions); + } while (u64_stats_fetch_retry(&pp->syncp, start)); =20 - stats->rx_length_errors =3D mib->rx_undersize_errors + - mib->rx_fragment_errors; - stats->rx_over_errors =3D mib->rx_oversize_errors; - stats->rx_crc_errors =3D mib->rx_crc_errors; - stats->rx_frame_errors =3D mib->rx_alignment_errors; - /* stats->rx_fifo_errors */ - /* stats->rx_missed_errors */ - - stats->tx_aborted_errors =3D mib->tx_aborted_errors; - /* stats->tx_carrier_errors */ - stats->tx_fifo_errors =3D mib->tx_undersize_errors; - /* stats->tx_heartbeat_errors */ - stats->tx_window_errors =3D mib->tx_late_collisions; - - stats->rx_packets =3D pp->rx_frames; - stats->tx_packets =3D pp->tx_frames; - stats->rx_bytes =3D mib->rx_good_bytes - ETH_FCS_LEN * stats->rx_packets; - stats->tx_bytes =3D mib->tx_good_bytes - ETH_FCS_LEN * stats->tx_packets; stats->rx_errors =3D stats->rx_length_errors + stats->rx_over_errors + stats->rx_crc_errors + stats->rx_frame_errors; stats->tx_errors =3D stats->tx_aborted_errors + stats->tx_fifo_errors + stats->tx_window_errors; - stats->rx_dropped =3D mib->rx_dropped; - /* stats->tx_dropped */ - stats->multicast =3D mib->rx_multicast; - stats->collisions =3D mib->tx_collisions; } =20 static void @@ -944,13 +986,18 @@ yt921x_dsa_get_pause_stats(struct dsa_switch *ds, int= port, struct yt921x_priv *priv =3D to_yt921x_priv(ds); struct yt921x_port *pp =3D &priv->ports[port]; struct yt921x_mib *mib =3D &pp->mib; + unsigned int start; =20 mutex_lock(&priv->reg_lock); yt921x_read_mib(priv, port); mutex_unlock(&priv->reg_lock); =20 - pause_stats->tx_pause_frames =3D mib->tx_pause; - pause_stats->rx_pause_frames =3D mib->rx_pause; + do { + start =3D u64_stats_fetch_begin(&pp->syncp); + + pause_stats->tx_pause_frames =3D u64_stats_read(&mib->tx_pause); + pause_stats->rx_pause_frames =3D u64_stats_read(&mib->rx_pause); + } while (u64_stats_fetch_retry(&pp->syncp, start)); } =20 static int @@ -2971,6 +3018,7 @@ static int yt921x_mdio_probe(struct mdio_device *mdio= dev) =20 pp->index =3D i; INIT_DELAYED_WORK(&pp->mib_read, yt921x_poll_mib); + u64_stats_init(&pp->syncp); } =20 ds =3D &priv->ds; diff --git a/drivers/net/dsa/yt921x.h b/drivers/net/dsa/yt921x.h index 61bb0ab3b09a..29d82f2b7733 100644 --- a/drivers/net/dsa/yt921x.h +++ b/drivers/net/dsa/yt921x.h @@ -6,6 +6,8 @@ #ifndef __YT921X_H #define __YT921X_H =20 +#include + #include =20 #define YT921X_SMI_SWITCHID_M GENMASK(3, 2) @@ -475,55 +477,61 @@ enum yt921x_fdb_entry_status { #define yt921x_port_is_external(port) (8 <=3D (port) && (port) < 9) =20 struct yt921x_mib { - u64 rx_broadcast; - u64 rx_pause; - u64 rx_multicast; - u64 rx_crc_errors; - - u64 rx_alignment_errors; - u64 rx_undersize_errors; - u64 rx_fragment_errors; - u64 rx_64byte; - - u64 rx_65_127byte; - u64 rx_128_255byte; - u64 rx_256_511byte; - u64 rx_512_1023byte; - - u64 rx_1024_1518byte; - u64 rx_jumbo; - u64 rx_good_bytes; - - u64 rx_bad_bytes; - u64 rx_oversize_errors; - - u64 rx_dropped; - u64 tx_broadcast; - u64 tx_pause; - u64 tx_multicast; - - u64 tx_undersize_errors; - u64 tx_64byte; - u64 tx_65_127byte; - u64 tx_128_255byte; - - u64 tx_256_511byte; - u64 tx_512_1023byte; - u64 tx_1024_1518byte; - u64 tx_jumbo; - - u64 tx_good_bytes; - u64 tx_collisions; - - u64 tx_aborted_errors; - u64 tx_multiple_collisions; - u64 tx_single_collisions; - u64 tx_good; - - u64 tx_deferred; - u64 tx_late_collisions; - u64 rx_oam; - u64 tx_oam; + union { + struct { + u64_stats_t rx_broadcast; + u64_stats_t rx_pause; + u64_stats_t rx_multicast; + u64_stats_t rx_crc_errors; + + u64_stats_t rx_alignment_errors; + u64_stats_t rx_undersize_errors; + u64_stats_t rx_fragment_errors; + u64_stats_t rx_64byte; + + u64_stats_t rx_65_127byte; + u64_stats_t rx_128_255byte; + u64_stats_t rx_256_511byte; + u64_stats_t rx_512_1023byte; + + u64_stats_t rx_1024_1518byte; + u64_stats_t rx_jumbo; + u64_stats_t rx_good_bytes; + + u64_stats_t rx_bad_bytes; + u64_stats_t rx_oversize_errors; + + u64_stats_t rx_dropped; + u64_stats_t tx_broadcast; + u64_stats_t tx_pause; + u64_stats_t tx_multicast; + + u64_stats_t tx_undersize_errors; + u64_stats_t tx_64byte; + u64_stats_t tx_65_127byte; + u64_stats_t tx_128_255byte; + + u64_stats_t tx_256_511byte; + u64_stats_t tx_512_1023byte; + u64_stats_t tx_1024_1518byte; + u64_stats_t tx_jumbo; + + u64_stats_t tx_good_bytes; + u64_stats_t tx_collisions; + + u64_stats_t tx_aborted_errors; + u64_stats_t tx_multiple_collisions; + u64_stats_t tx_single_collisions; + u64_stats_t tx_good; + + u64_stats_t tx_deferred; + u64_stats_t tx_late_collisions; + u64_stats_t rx_oam; + u64_stats_t tx_oam; + }; + + u64_stats_t stats[39]; + }; }; =20 struct yt921x_port { @@ -533,9 +541,13 @@ struct yt921x_port { bool isolated; =20 struct delayed_work mib_read; + struct u64_stats_sync syncp; struct yt921x_mib mib; - u64 rx_frames; - u64 tx_frames; + u64_stats_t rx_frames; + u64_stats_t tx_frames; + + /* only used by read routine to avoid huge allocations on the stack */ + struct yt921x_mib mib_new; }; =20 struct yt921x_reg_ops { --=20 2.51.0