From nobody Sun Feb 8 05:59:58 2026 Received: from mail-pj1-f48.google.com (mail-pj1-f48.google.com [209.85.216.48]) (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 753D539F30F for ; Thu, 22 Jan 2026 20:16:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.48 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769112970; cv=none; b=iardfS3zf8GFykSyaRO4vr7pcUaX/dDBunIsBi6Lt9YvvIiXdEUfP2R8fqAtP6tub6oJ4sJBbP13V5GBipLWp6QbHa8zmrztCHYPdHBSvaUZyHDDO24tFWUA2PskQp2zVmjsn47dPxmc4Ob8RqzXit/gr+v5oQsGJ75vkq70Xz4= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769112970; c=relaxed/simple; bh=Tehk9JrlgFpNSfXPDNiROc3FDw4ecgDHXj6NwGxDsyw=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=Sc1gdJyhhheydyZWh8SZgkqH77ecw5StE4SbkG+sdxuRge7hsHiRSpYirXqeEZAZqzv2t5T8H3TCTSspCrlEkQ4kpmpnnKs8thtlWu0CRh4Bco9iAeBE/+FyxxHJwEAcSjjjOwx13WijVyGCDjsfHQ0ltaDNQF0hzimCqH0DbZs= 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=PkuwXz4H; arc=none smtp.client-ip=209.85.216.48 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="PkuwXz4H" Received: by mail-pj1-f48.google.com with SMTP id 98e67ed59e1d1-352e2c59264so963119a91.0 for ; Thu, 22 Jan 2026 12:16:02 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1769112960; x=1769717760; 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=Sm1p55Lhtr7bC52ssSuFAxbhrBupFEQ8dwcLX8l+9PY=; b=PkuwXz4HCt4jVia5m5Rg/A+HmLEpYCtTFZQe1irdbNC/a1xZ0++tfXcyiVByjliNO8 JAHMj5Nzd7tTlIASo4PQgIaHoUBAXi0eDnWpn8OTiF9+/6PESGhBWJkgU6Wh+ogY8+nW aYuPOIZz08rC7CK73fnK2FJi7qyQ+RqVbJXSUik+1kJhQp9s/t/fP5Tjs7zCsdZGCOg+ 5XWBOnzazrTSRZbYZrbDz+qAFA7XuJGKUDvmeLUhLVFtnb5jPb7J7K44QtHKeak1EmZb crHGkEZR7lpU6SSPCrRasC8D+JY2n+W97rpUuI9GieY924s9FYvf31yA94g50xKy3jJ0 9gmw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1769112960; x=1769717760; 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=Sm1p55Lhtr7bC52ssSuFAxbhrBupFEQ8dwcLX8l+9PY=; b=diQXPZd/WqOMcvugYHm/W2K73I1u2JlGMdtnxHYgPKC2U2Ex4SVC2nsKww+DqCBOzm JWj/sA5712ujt1AyfKxj/oxFPpOH0+qygL1Gi0vlmQTHv9nhfvaak5OU4LE+CuoPDqws bYut2hnRpS/KkBZmmWlhhLQTMop4w/4AO+nV0UJwifhN0+bUNQ153bjzmLlrFBGWYsSb PY395DGRQoAl1FEZW8AnyFtyxFDaWB/U3Rf5GA8AL6wPYfoEq01FMfOn6GJQgMaeoMZ1 M6XnEiD0gHTsI0nGjpcHP5MET5RTmiDqEEqk/v5KzsMvygq2Y3LO+2TFnKtyE2jfMg2P eisQ== X-Forwarded-Encrypted: i=1; AJvYcCUPBh/WY6uucp49m+Jklz9pC926vTqPqMRPdkuKlPa/XP6rUEq3BmClpbJXWM02MNz20S5nG6j49zw2f/Q=@vger.kernel.org X-Gm-Message-State: AOJu0YxljTvVx97UxdpAXIyQhdQaB/FwA6GVSah9J3BGwNHPBDF+oWwt 9X2ZuHYzuVgukyOZgsQp1oA5Xj8VUvl/d4GLDPUZmwV2yQFCHuqyTWgYDlp54w== X-Gm-Gg: AZuq6aJBE1WkCgAPfq21rpn44jENTfLsHQxWKq0C9PF2VTlBFhdNKowu1UfXme3tQO7 IOB8GCsnEGjQZSQd7gzaZnBuWzQ23iInajtIHoBsLPvEXMf8rD8IJqS2hWrI+andL025Bo/9sns cCeafIoW/1xPra2XpgZ9ssO4Sa9mfcokJU4YVFOQWmT6wODUm8y7DR6oXKdjCG33oNU/tJG4g2Z Mi645hz00WJDcU8fPgvrXZSeTI5uu5TyT1QBB90Y7+VT2X/qarjtnwHP+w+yEdS/uaJ/8qn2ZK1 cxY8X44Y5ZV+8ANM01XwsUbT/0d2KtmQKw7OsguoDJX24HYSEseiAdSvVPty7W8TsN6QvHMUeNF fhqXyUGoCcXT1tk7qQ48ucCRozP0HcYaVmTQu2AdiMiF1JaOhITsWTR8AyBxN4MMHl826ZBKBoG gQOPMMrGrKTeEAg3KUrmrDBqKmBa/CFOp1nZaKfsM0BVfcwAdGwMLBMw== X-Received: by 2002:a17:90b:1f88:b0:349:3fe8:e7df with SMTP id 98e67ed59e1d1-35368953071mr522308a91.22.1769112959695; Thu, 22 Jan 2026 12:15:59 -0800 (PST) Received: from d.home.mmyangfl.tk ([2001:19f0:8001:1644:5400:5ff:fe3e:12b1]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-3536b00fdb1sm104462a91.1.2026.01.22.12.15.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 22 Jan 2026 12:15:59 -0800 (PST) From: David Yang To: netdev@vger.kernel.org Cc: David Yang , Florian Fainelli , Broadcom internal kernel review list , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , linux-kernel@vger.kernel.org Subject: [RFC net-next] net: systemport: Use u64_stats_t with u64_stats_sync properly Date: Fri, 23 Jan 2026 04:15:41 +0800 Message-ID: <20260122201545.2792428-1-mmyangfl@gmail.com> X-Mailer: git-send-email 2.51.0 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" On 64bit arches, struct u64_stats_sync is empty and provides no help against load/store tearing. Convert to u64_stats_t to ensure atomic operations. Signed-off-by: David Yang --- RFC Comment: I couldn't find the lock associated with u64_stats_sync. Should this be considered an issue? drivers/net/ethernet/broadcom/bcmsysport.c | 56 ++++++++++++---------- drivers/net/ethernet/broadcom/bcmsysport.h | 12 ++--- 2 files changed, 37 insertions(+), 31 deletions(-) diff --git a/drivers/net/ethernet/broadcom/bcmsysport.c b/drivers/net/ether= net/broadcom/bcmsysport.c index bc4e1f3b3752..7017b2d2a1b5 100644 --- a/drivers/net/ethernet/broadcom/bcmsysport.c +++ b/drivers/net/ethernet/broadcom/bcmsysport.c @@ -435,8 +435,8 @@ static void bcm_sysport_update_tx_stats(struct bcm_sysp= ort_priv *priv, ring =3D &priv->tx_rings[q]; do { start =3D u64_stats_fetch_begin(&priv->syncp); - bytes =3D ring->bytes; - packets =3D ring->packets; + bytes =3D u64_stats_read(&ring->bytes); + packets =3D u64_stats_read(&ring->packets); } while (u64_stats_fetch_retry(&priv->syncp, start)); =20 *tx_bytes +=3D bytes; @@ -458,8 +458,10 @@ static void bcm_sysport_get_stats(struct net_device *d= ev, if (netif_running(dev)) { bcm_sysport_update_mib_counters(priv); bcm_sysport_update_tx_stats(priv, &tx_bytes, &tx_packets); - stats64->tx_bytes =3D tx_bytes; - stats64->tx_packets =3D tx_packets; + u64_stats_update_begin(&priv->syncp); + u64_stats_set(&stats64->tx_bytes, tx_bytes); + u64_stats_set(&stats64->tx_packets, tx_packets); + u64_stats_update_end(&priv->syncp); } =20 for (i =3D 0, j =3D 0; i < BCM_SYSPORT_STATS_LEN; i++) { @@ -482,28 +484,32 @@ static void bcm_sysport_get_stats(struct net_device *= dev, s->type =3D=3D BCM_SYSPORT_STAT_NETDEV64) { do { start =3D u64_stats_fetch_begin(syncp); - data[i] =3D *(u64 *)p; + data[i] =3D u64_stats_read((u64_stats_t *)p); } while (u64_stats_fetch_retry(syncp, start)); } else data[i] =3D *(u32 *)p; j++; } =20 - /* For SYSTEMPORT Lite since we have holes in our statistics, j would - * be equal to BCM_SYSPORT_STATS_LEN at the end of the loop, but it - * needs to point to how many total statistics we have minus the - * number of per TX queue statistics - */ - j =3D bcm_sysport_get_sset_count(dev, ETH_SS_STATS) - - dev->num_tx_queues * NUM_SYSPORT_TXQ_STAT; + do { + /* For SYSTEMPORT Lite since we have holes in our statistics, j + * would be equal to BCM_SYSPORT_STATS_LEN at the end of the + * loop, but it needs to point to how many total statistics we + * have minus the number of per TX queue statistics + */ + j =3D bcm_sysport_get_sset_count(dev, ETH_SS_STATS) - + dev->num_tx_queues * NUM_SYSPORT_TXQ_STAT; =20 - for (i =3D 0; i < dev->num_tx_queues; i++) { - ring =3D &priv->tx_rings[i]; - data[j] =3D ring->packets; - j++; - data[j] =3D ring->bytes; - j++; - } + start =3D u64_stats_fetch_begin(syncp); + + for (i =3D 0; i < dev->num_tx_queues; i++) { + ring =3D &priv->tx_rings[i]; + data[j] =3D u64_stats_read(&ring->packets); + j++; + data[j] =3D u64_stats_read(&ring->bytes); + j++; + } + } while (u64_stats_fetch_retry(syncp, start)); } =20 static void bcm_sysport_get_wol(struct net_device *dev, @@ -829,8 +835,8 @@ static unsigned int bcm_sysport_desc_rx(struct bcm_sysp= ort_priv *priv, ndev->stats.rx_packets++; ndev->stats.rx_bytes +=3D len; u64_stats_update_begin(&priv->syncp); - stats64->rx_packets++; - stats64->rx_bytes +=3D len; + u64_stats_inc(&stats64->rx_packets); + u64_stats_add(&stats64->rx_bytes, len); u64_stats_update_end(&priv->syncp); =20 napi_gro_receive(&priv->napi, skb); @@ -914,8 +920,8 @@ static unsigned int __bcm_sysport_tx_reclaim(struct bcm= _sysport_priv *priv, } =20 u64_stats_update_begin(&priv->syncp); - ring->packets +=3D pkts_compl; - ring->bytes +=3D bytes_compl; + u64_stats_add(&ring->packets, pkts_compl); + u64_stats_add(&ring->bytes, bytes_compl); u64_stats_update_end(&priv->syncp); =20 ring->c_index =3D c_index; @@ -1857,8 +1863,8 @@ static void bcm_sysport_get_stats64(struct net_device= *dev, =20 do { start =3D u64_stats_fetch_begin(&priv->syncp); - stats->rx_packets =3D stats64->rx_packets; - stats->rx_bytes =3D stats64->rx_bytes; + stats->rx_packets =3D u64_stats_read(&stats64->rx_packets); + stats->rx_bytes =3D u64_stats_read(&stats64->rx_bytes); } while (u64_stats_fetch_retry(&priv->syncp, start)); } =20 diff --git a/drivers/net/ethernet/broadcom/bcmsysport.h b/drivers/net/ether= net/broadcom/bcmsysport.h index a34296f989f1..91b1c8293a23 100644 --- a/drivers/net/ethernet/broadcom/bcmsysport.h +++ b/drivers/net/ethernet/broadcom/bcmsysport.h @@ -652,10 +652,10 @@ struct bcm_sysport_stats { =20 struct bcm_sysport_stats64 { /* 64bit stats on 32bit/64bit Machine */ - u64 rx_packets; - u64 rx_bytes; - u64 tx_packets; - u64 tx_bytes; + u64_stats_t rx_packets; + u64_stats_t rx_bytes; + u64_stats_t tx_packets; + u64_stats_t tx_bytes; }; =20 /* Software house keeping helper structure */ @@ -698,8 +698,8 @@ struct bcm_sysport_tx_ring { unsigned int clean_index; /* Current clean index */ struct bcm_sysport_cb *cbs; /* Transmit control blocks */ struct bcm_sysport_priv *priv; /* private context backpointer */ - unsigned long packets; /* packets statistics */ - unsigned long bytes; /* bytes statistics */ + u64_stats_t packets; /* packets statistics */ + u64_stats_t bytes; /* bytes statistics */ unsigned int switch_queue; /* switch port queue number */ unsigned int switch_port; /* switch port queue number */ bool inspect; /* inspect switch port and queue */ --=20 2.51.0