Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1701353 > unrolled thread

[PATCH net 1/7] b44: Initialize 64-bit stats seqcount

Started byFlorian Fainelli <f.fainelli@gmail.com>
First post2017-08-01 21:20 +0200
Last post2017-08-01 22:50 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH net 1/7] b44: Initialize 64-bit stats seqcount Florian Fainelli <f.fainelli@gmail.com> - 2017-08-01 21:20 +0200
    Re: [PATCH net 1/7] b44: Initialize 64-bit stats seqcount Michael Chan <michael.chan@broadcom.com> - 2017-08-01 22:50 +0200

#1701353 — [PATCH net 1/7] b44: Initialize 64-bit stats seqcount

FromFlorian Fainelli <f.fainelli@gmail.com>
Date2017-08-01 21:20 +0200
Subject[PATCH net 1/7] b44: Initialize 64-bit stats seqcount
Message-ID<u9KRX-82o-7@gated-at.bofh.it>
On 32-bit hosts and with CONFIG_DEBUG_LOCK_ALLOC we should be seeing a
lockdep splat indicating this seqcount is not correctly initialized, fix
that.

Fixes: eeda8585522b ("b44: add 64 bit stats")
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
---
 drivers/net/ethernet/broadcom/b44.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/net/ethernet/broadcom/b44.c b/drivers/net/ethernet/broadcom/b44.c
index f411936b744c..a1125d10c825 100644
--- a/drivers/net/ethernet/broadcom/b44.c
+++ b/drivers/net/ethernet/broadcom/b44.c
@@ -2368,6 +2368,7 @@ static int b44_init_one(struct ssb_device *sdev,
 	bp->msg_enable = netif_msg_init(b44_debug, B44_DEF_MSG_ENABLE);
 
 	spin_lock_init(&bp->lock);
+	u64_stats_init(&bp->hw_stats.syncp);
 
 	bp->rx_pending = B44_DEF_RX_RING_PENDING;
 	bp->tx_pending = B44_DEF_TX_RING_PENDING;
-- 
2.9.3

[toc] | [next] | [standalone]


#1701463

FromMichael Chan <michael.chan@broadcom.com>
Date2017-08-01 22:50 +0200
Message-ID<u9Mh5-jv-31@gated-at.bofh.it>
In reply to#1701353
On Tue, Aug 1, 2017 at 12:11 PM, Florian Fainelli <f.fainelli@gmail.com> wrote:
> On 32-bit hosts and with CONFIG_DEBUG_LOCK_ALLOC we should be seeing a
> lockdep splat indicating this seqcount is not correctly initialized, fix
> that.
>
> Fixes: eeda8585522b ("b44: add 64 bit stats")
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>

Acked-by: Michael Chan <michael.chan@broadcom.com>

Thanks.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web