Message ID | 50A1A816.3070900@gmail.com |
---|---|
State | Not Applicable, archived |
Delegated to: | David Miller |
Headers | show |
On Tuesday, November 13, 2012 09:53:26 Shan Wei wrote: > From: Shan Wei <davidshan@tencent.com> > > this_cpu_add is an atomic operation. > and be more faster than per_cpu_ptr operation. > > Signed-off-by: Shan Wei <davidshan@tencent.com> > Reviewed-by: Christoph Lameter <cl@linux.com> > --- > v4: no changes vs v3. > --- > net/batman-adv/main.h | 4 +--- > 1 files changed, 1 insertions(+), 3 deletions(-) Applied in our tree (revision f8b19e1). We will pass along this patch through our standard workflow. No need to resend this patch in the future. Thanks, Marek -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Marek Lindner said, at 2012/11/13 17:55: >> net/batman-adv/main.h | 4 +--- >> 1 files changed, 1 insertions(+), 3 deletions(-) > > Applied in our tree (revision f8b19e1). We will pass along this patch through > our standard workflow. No need to resend this patch in the future. OK thanks~ > > Thanks, > Marek > -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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)