]> asedeno.scripts.mit.edu Git - linux.git/blobdiff - net/netfilter/nft_counter.c
netfilter: nft_counter: rework atomic dump and reset
[linux.git] / net / netfilter / nft_counter.c
index 77db8358ab14fa600601ad73ef91c26c843db605..7f84222133414d4a65ed96508c83dea711295c9f 100644 (file)
 #include <net/netfilter/nf_tables.h>
 
 struct nft_counter {
-       u64             bytes;
-       u64             packets;
-};
-
-struct nft_counter_percpu {
-       struct nft_counter      counter;
-       struct u64_stats_sync   syncp;
+       s64             bytes;
+       s64             packets;
 };
 
 struct nft_counter_percpu_priv {
-       struct nft_counter_percpu __percpu *counter;
+       struct nft_counter __percpu *counter;
 };
 
-static void nft_counter_eval(const struct nft_expr *expr,
-                            struct nft_regs *regs,
-                            const struct nft_pktinfo *pkt)
+static DEFINE_PER_CPU(seqcount_t, nft_counter_seq);
+
+static inline void nft_counter_do_eval(struct nft_counter_percpu_priv *priv,
+                                      struct nft_regs *regs,
+                                      const struct nft_pktinfo *pkt)
 {
-       struct nft_counter_percpu_priv *priv = nft_expr_priv(expr);
-       struct nft_counter_percpu *this_cpu;
+       struct nft_counter *this_cpu;
+       seqcount_t *myseq;
 
        local_bh_disable();
        this_cpu = this_cpu_ptr(priv->counter);
-       u64_stats_update_begin(&this_cpu->syncp);
-       this_cpu->counter.bytes += pkt->skb->len;
-       this_cpu->counter.packets++;
-       u64_stats_update_end(&this_cpu->syncp);
+       myseq = this_cpu_ptr(&nft_counter_seq);
+
+       write_seqcount_begin(myseq);
+
+       this_cpu->bytes += pkt->skb->len;
+       this_cpu->packets++;
+
+       write_seqcount_end(myseq);
        local_bh_enable();
 }
 
-static void nft_counter_fetch(const struct nft_counter_percpu __percpu *counter,
+static inline void nft_counter_obj_eval(struct nft_object *obj,
+                                       struct nft_regs *regs,
+                                       const struct nft_pktinfo *pkt)
+{
+       struct nft_counter_percpu_priv *priv = nft_obj_data(obj);
+
+       nft_counter_do_eval(priv, regs, pkt);
+}
+
+static int nft_counter_do_init(const struct nlattr * const tb[],
+                              struct nft_counter_percpu_priv *priv)
+{
+       struct nft_counter __percpu *cpu_stats;
+       struct nft_counter *this_cpu;
+
+       cpu_stats = alloc_percpu(struct nft_counter);
+       if (cpu_stats == NULL)
+               return -ENOMEM;
+
+       preempt_disable();
+       this_cpu = this_cpu_ptr(cpu_stats);
+       if (tb[NFTA_COUNTER_PACKETS]) {
+               this_cpu->packets =
+                       be64_to_cpu(nla_get_be64(tb[NFTA_COUNTER_PACKETS]));
+       }
+       if (tb[NFTA_COUNTER_BYTES]) {
+               this_cpu->bytes =
+                       be64_to_cpu(nla_get_be64(tb[NFTA_COUNTER_BYTES]));
+       }
+       preempt_enable();
+       priv->counter = cpu_stats;
+       return 0;
+}
+
+static int nft_counter_obj_init(const struct nlattr * const tb[],
+                               struct nft_object *obj)
+{
+       struct nft_counter_percpu_priv *priv = nft_obj_data(obj);
+
+       return nft_counter_do_init(tb, priv);
+}
+
+static void nft_counter_do_destroy(struct nft_counter_percpu_priv *priv)
+{
+       free_percpu(priv->counter);
+}
+
+static void nft_counter_obj_destroy(struct nft_object *obj)
+{
+       struct nft_counter_percpu_priv *priv = nft_obj_data(obj);
+
+       nft_counter_do_destroy(priv);
+}
+
+static void nft_counter_reset(struct nft_counter_percpu_priv __percpu *priv,
                              struct nft_counter *total)
 {
-       const struct nft_counter_percpu *cpu_stats;
+       struct nft_counter *this_cpu;
+
+       local_bh_disable();
+       this_cpu = this_cpu_ptr(priv->counter);
+       this_cpu->packets -= total->packets;
+       this_cpu->bytes -= total->bytes;
+       local_bh_enable();
+}
+
+static void nft_counter_fetch(struct nft_counter_percpu_priv *priv,
+                             struct nft_counter *total)
+{
+       struct nft_counter *this_cpu;
+       const seqcount_t *myseq;
        u64 bytes, packets;
        unsigned int seq;
        int cpu;
 
        memset(total, 0, sizeof(*total));
        for_each_possible_cpu(cpu) {
-               cpu_stats = per_cpu_ptr(counter, cpu);
+               myseq = per_cpu_ptr(&nft_counter_seq, cpu);
+               this_cpu = per_cpu_ptr(priv->counter, cpu);
                do {
-                       seq     = u64_stats_fetch_begin_irq(&cpu_stats->syncp);
-                       bytes   = cpu_stats->counter.bytes;
-                       packets = cpu_stats->counter.packets;
-               } while (u64_stats_fetch_retry_irq(&cpu_stats->syncp, seq));
+                       seq     = read_seqcount_begin(myseq);
+                       bytes   = this_cpu->bytes;
+                       packets = this_cpu->packets;
+               } while (read_seqcount_retry(myseq, seq));
 
-               total->packets += packets;
-               total->bytes += bytes;
+               total->bytes    += bytes;
+               total->packets  += packets;
        }
 }
 
-static int nft_counter_dump(struct sk_buff *skb, const struct nft_expr *expr)
+static int nft_counter_do_dump(struct sk_buff *skb,
+                              struct nft_counter_percpu_priv *priv,
+                              bool reset)
 {
-       struct nft_counter_percpu_priv *priv = nft_expr_priv(expr);
        struct nft_counter total;
 
-       nft_counter_fetch(priv->counter, &total);
+       nft_counter_fetch(priv, &total);
 
        if (nla_put_be64(skb, NFTA_COUNTER_BYTES, cpu_to_be64(total.bytes),
                         NFTA_COUNTER_PAD) ||
            nla_put_be64(skb, NFTA_COUNTER_PACKETS, cpu_to_be64(total.packets),
                         NFTA_COUNTER_PAD))
                goto nla_put_failure;
+
+       if (reset)
+               nft_counter_reset(priv, &total);
+
        return 0;
 
 nla_put_failure:
        return -1;
 }
 
+static int nft_counter_obj_dump(struct sk_buff *skb,
+                               struct nft_object *obj, bool reset)
+{
+       struct nft_counter_percpu_priv *priv = nft_obj_data(obj);
+
+       return nft_counter_do_dump(skb, priv, reset);
+}
+
 static const struct nla_policy nft_counter_policy[NFTA_COUNTER_MAX + 1] = {
        [NFTA_COUNTER_PACKETS]  = { .type = NLA_U64 },
        [NFTA_COUNTER_BYTES]    = { .type = NLA_U64 },
 };
 
+static struct nft_object_type nft_counter_obj __read_mostly = {
+       .type           = NFT_OBJECT_COUNTER,
+       .size           = sizeof(struct nft_counter_percpu_priv),
+       .maxattr        = NFTA_COUNTER_MAX,
+       .policy         = nft_counter_policy,
+       .eval           = nft_counter_obj_eval,
+       .init           = nft_counter_obj_init,
+       .destroy        = nft_counter_obj_destroy,
+       .dump           = nft_counter_obj_dump,
+       .owner          = THIS_MODULE,
+};
+
+static void nft_counter_eval(const struct nft_expr *expr,
+                            struct nft_regs *regs,
+                            const struct nft_pktinfo *pkt)
+{
+       struct nft_counter_percpu_priv *priv = nft_expr_priv(expr);
+
+       nft_counter_do_eval(priv, regs, pkt);
+}
+
+static int nft_counter_dump(struct sk_buff *skb, const struct nft_expr *expr)
+{
+       struct nft_counter_percpu_priv *priv = nft_expr_priv(expr);
+
+       return nft_counter_do_dump(skb, priv, false);
+}
+
 static int nft_counter_init(const struct nft_ctx *ctx,
                            const struct nft_expr *expr,
                            const struct nlattr * const tb[])
 {
        struct nft_counter_percpu_priv *priv = nft_expr_priv(expr);
-       struct nft_counter_percpu __percpu *cpu_stats;
-       struct nft_counter_percpu *this_cpu;
-
-       cpu_stats = netdev_alloc_pcpu_stats(struct nft_counter_percpu);
-       if (cpu_stats == NULL)
-               return -ENOMEM;
 
-       preempt_disable();
-       this_cpu = this_cpu_ptr(cpu_stats);
-       if (tb[NFTA_COUNTER_PACKETS]) {
-               this_cpu->counter.packets =
-                       be64_to_cpu(nla_get_be64(tb[NFTA_COUNTER_PACKETS]));
-       }
-       if (tb[NFTA_COUNTER_BYTES]) {
-               this_cpu->counter.bytes =
-                       be64_to_cpu(nla_get_be64(tb[NFTA_COUNTER_BYTES]));
-       }
-       preempt_enable();
-       priv->counter = cpu_stats;
-       return 0;
+       return nft_counter_do_init(tb, priv);
 }
 
 static void nft_counter_destroy(const struct nft_ctx *ctx,
@@ -124,28 +216,27 @@ static void nft_counter_destroy(const struct nft_ctx *ctx,
 {
        struct nft_counter_percpu_priv *priv = nft_expr_priv(expr);
 
-       free_percpu(priv->counter);
+       nft_counter_do_destroy(priv);
 }
 
 static int nft_counter_clone(struct nft_expr *dst, const struct nft_expr *src)
 {
        struct nft_counter_percpu_priv *priv = nft_expr_priv(src);
        struct nft_counter_percpu_priv *priv_clone = nft_expr_priv(dst);
-       struct nft_counter_percpu __percpu *cpu_stats;
-       struct nft_counter_percpu *this_cpu;
+       struct nft_counter __percpu *cpu_stats;
+       struct nft_counter *this_cpu;
        struct nft_counter total;
 
-       nft_counter_fetch(priv->counter, &total);
+       nft_counter_fetch(priv, &total);
 
-       cpu_stats = __netdev_alloc_pcpu_stats(struct nft_counter_percpu,
-                                             GFP_ATOMIC);
+       cpu_stats = alloc_percpu_gfp(struct nft_counter, GFP_ATOMIC);
        if (cpu_stats == NULL)
                return -ENOMEM;
 
        preempt_disable();
        this_cpu = this_cpu_ptr(cpu_stats);
-       this_cpu->counter.packets = total.packets;
-       this_cpu->counter.bytes = total.bytes;
+       this_cpu->packets = total.packets;
+       this_cpu->bytes = total.bytes;
        preempt_enable();
 
        priv_clone->counter = cpu_stats;
@@ -174,12 +265,29 @@ static struct nft_expr_type nft_counter_type __read_mostly = {
 
 static int __init nft_counter_module_init(void)
 {
-       return nft_register_expr(&nft_counter_type);
+       int cpu, err;
+
+       for_each_possible_cpu(cpu)
+               seqcount_init(per_cpu_ptr(&nft_counter_seq, cpu));
+
+       err = nft_register_obj(&nft_counter_obj);
+       if (err < 0)
+               return err;
+
+       err = nft_register_expr(&nft_counter_type);
+       if (err < 0)
+               goto err1;
+
+       return 0;
+err1:
+       nft_unregister_obj(&nft_counter_obj);
+       return err;
 }
 
 static void __exit nft_counter_module_exit(void)
 {
        nft_unregister_expr(&nft_counter_type);
+       nft_unregister_obj(&nft_counter_obj);
 }
 
 module_init(nft_counter_module_init);
@@ -188,3 +296,4 @@ module_exit(nft_counter_module_exit);
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Patrick McHardy <kaber@trash.net>");
 MODULE_ALIAS_NFT_EXPR("counter");
+MODULE_ALIAS_NFT_OBJ(NFT_OBJECT_COUNTER);