* [PATCH net-next] tg3: fix return value in tg3_get_stats64
@ 2014-08-13 7:34 Govindarajulu Varadarajan
2014-08-13 20:52 ` Prashant Sreedharan
2014-08-13 23:17 ` David Miller
0 siblings, 2 replies; 3+ messages in thread
From: Govindarajulu Varadarajan @ 2014-08-13 7:34 UTC (permalink / raw)
To: davem, prashant, mchan, netdev; +Cc: Govindarajulu Varadarajan
When tp->hw_stats is 0, tg3_get_stats64 should display previously
recorded stats. So it returns &tp->net_stats_prev. But the caller,
dev_get_stats, ignores the return value.
Fix this by assigning tp->net_stats_prev to stats and returning stats.
Signed-off-by: Govindarajulu Varadarajan <_govind@gmx.com>
---
drivers/net/ethernet/broadcom/tg3.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/broadcom/tg3.c b/drivers/net/ethernet/broadcom/tg3.c
index a3dd5dc..4296b3d26 100644
--- a/drivers/net/ethernet/broadcom/tg3.c
+++ b/drivers/net/ethernet/broadcom/tg3.c
@@ -14093,8 +14093,9 @@ static struct rtnl_link_stats64 *tg3_get_stats64(struct net_device *dev,
spin_lock_bh(&tp->lock);
if (!tp->hw_stats) {
+ *stats = tp->net_stats_prev;
spin_unlock_bh(&tp->lock);
- return &tp->net_stats_prev;
+ return stats;
}
tg3_get_nstats(tp, stats);
--
2.0.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] tg3: fix return value in tg3_get_stats64
2014-08-13 7:34 [PATCH net-next] tg3: fix return value in tg3_get_stats64 Govindarajulu Varadarajan
@ 2014-08-13 20:52 ` Prashant Sreedharan
2014-08-13 23:17 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: Prashant Sreedharan @ 2014-08-13 20:52 UTC (permalink / raw)
To: Govindarajulu Varadarajan; +Cc: davem, mchan, netdev
On Wed, 2014-08-13 at 13:04 +0530, Govindarajulu Varadarajan wrote:
> When tp->hw_stats is 0, tg3_get_stats64 should display previously
> recorded stats. So it returns &tp->net_stats_prev. But the caller,
> dev_get_stats, ignores the return value.
>
> Fix this by assigning tp->net_stats_prev to stats and returning stats.
>
> Signed-off-by: Govindarajulu Varadarajan <_govind@gmx.com>
Acked-by: Prashant Sreedharan <prashant@broadcom.com>
> ---
> drivers/net/ethernet/broadcom/tg3.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/broadcom/tg3.c b/drivers/net/ethernet/broadcom/tg3.c
> index a3dd5dc..4296b3d26 100644
> --- a/drivers/net/ethernet/broadcom/tg3.c
> +++ b/drivers/net/ethernet/broadcom/tg3.c
> @@ -14093,8 +14093,9 @@ static struct rtnl_link_stats64 *tg3_get_stats64(struct net_device *dev,
>
> spin_lock_bh(&tp->lock);
> if (!tp->hw_stats) {
> + *stats = tp->net_stats_prev;
> spin_unlock_bh(&tp->lock);
> - return &tp->net_stats_prev;
> + return stats;
> }
>
> tg3_get_nstats(tp, stats);
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] tg3: fix return value in tg3_get_stats64
2014-08-13 7:34 [PATCH net-next] tg3: fix return value in tg3_get_stats64 Govindarajulu Varadarajan
2014-08-13 20:52 ` Prashant Sreedharan
@ 2014-08-13 23:17 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2014-08-13 23:17 UTC (permalink / raw)
To: _govind; +Cc: prashant, mchan, netdev
From: Govindarajulu Varadarajan <_govind@gmx.com>
Date: Wed, 13 Aug 2014 13:04:56 +0530
> When tp->hw_stats is 0, tg3_get_stats64 should display previously
> recorded stats. So it returns &tp->net_stats_prev. But the caller,
> dev_get_stats, ignores the return value.
>
> Fix this by assigning tp->net_stats_prev to stats and returning stats.
>
> Signed-off-by: Govindarajulu Varadarajan <_govind@gmx.com>
Applied, but please target 'net' for bug fixes in the future.
Thanks.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-08-13 23:17 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-13 7:34 [PATCH net-next] tg3: fix return value in tg3_get_stats64 Govindarajulu Varadarajan
2014-08-13 20:52 ` Prashant Sreedharan
2014-08-13 23:17 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).