LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

[PATCH 02/24] ipvs: remove _bh from percpu stats reading

To: netdev@xxxxxxxxxxxxxxx, netfilter-devel@xxxxxxxxxxxxxxx, netfilter@xxxxxxxxxxxxxxx, lvs-devel@xxxxxxxxxxxxxxx
Subject: [PATCH 02/24] ipvs: remove _bh from percpu stats reading
Cc: Eric Dumazet <eric.dumazet@xxxxxxxxx>, Julian Anastasov <ja@xxxxxx>, Hans Schillstrom <hans@xxxxxxxxxxxxxxx>, Simon Horman <horms@xxxxxxxxxxxx>
From: Simon Horman <horms@xxxxxxxxxxxx>
Date: Mon, 14 Mar 2011 18:06:14 +0900
From: Julian Anastasov <ja@xxxxxx>

        ip_vs_read_cpu_stats is called only from timer, so
no need for _bh locks.

Signed-off-by: Julian Anastasov <ja@xxxxxx>
Signed-off-by: Hans Schillstrom <hans@xxxxxxxxxxxxxxx>
Signed-off-by: Simon Horman <horms@xxxxxxxxxxxx>
---
 net/netfilter/ipvs/ip_vs_est.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/net/netfilter/ipvs/ip_vs_est.c b/net/netfilter/ipvs/ip_vs_est.c
index f560a05..88bd716 100644
--- a/net/netfilter/ipvs/ip_vs_est.c
+++ b/net/netfilter/ipvs/ip_vs_est.c
@@ -69,10 +69,10 @@ static void ip_vs_read_cpu_stats(struct ip_vs_stats_user 
*sum,
                        sum->inpkts += s->ustats.inpkts;
                        sum->outpkts += s->ustats.outpkts;
                        do {
-                               start = u64_stats_fetch_begin_bh(&s->syncp);
+                               start = u64_stats_fetch_begin(&s->syncp);
                                inbytes = s->ustats.inbytes;
                                outbytes = s->ustats.outbytes;
-                       } while (u64_stats_fetch_retry_bh(&s->syncp, start));
+                       } while (u64_stats_fetch_retry(&s->syncp, start));
                        sum->inbytes += inbytes;
                        sum->outbytes += outbytes;
                } else {
@@ -80,10 +80,10 @@ static void ip_vs_read_cpu_stats(struct ip_vs_stats_user 
*sum,
                        sum->inpkts = s->ustats.inpkts;
                        sum->outpkts = s->ustats.outpkts;
                        do {
-                               start = u64_stats_fetch_begin_bh(&s->syncp);
+                               start = u64_stats_fetch_begin(&s->syncp);
                                sum->inbytes = s->ustats.inbytes;
                                sum->outbytes = s->ustats.outbytes;
-                       } while (u64_stats_fetch_retry_bh(&s->syncp, start));
+                       } while (u64_stats_fetch_retry(&s->syncp, start));
                }
        }
 }
-- 
1.7.2.3

--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

<Prev in Thread] Current Thread [Next in Thread>