From: Shan Wei davidshan@tencent.com
As Christoph Lameter said:
In addition, following usage of per_cpu_ptr can be replaced by this_cpu_read.
cpu=get_cpu() .... *per_cpu_ptr(p,cpu) .... .... put_cpu()
Right.
Signed-off-by: Shan Wei davidshan@tencent.com --- net/batman-adv/main.h | 4 +--- 1 files changed, 1 insertions(+), 3 deletions(-)
diff --git a/net/batman-adv/main.h b/net/batman-adv/main.h index 897ba6a..3aef5b2 100644 --- a/net/batman-adv/main.h +++ b/net/batman-adv/main.h @@ -263,9 +263,7 @@ static inline bool batadv_has_timed_out(unsigned long timestamp, static inline void batadv_add_counter(struct batadv_priv *bat_priv, size_t idx, size_t count) { - int cpu = get_cpu(); - per_cpu_ptr(bat_priv->bat_counters, cpu)[idx] += count; - put_cpu(); + this_cpu_add(bat_priv->bat_counters[idx], count); }
#define batadv_inc_counter(b, i) batadv_add_counter(b, i, 1)