diff options
| author | Changli Gao <xiaosuo@gmail.com> | 2010-07-23 14:04:03 +0200 |
|---|---|---|
| committer | Patrick McHardy <kaber@trash.net> | 2010-07-23 14:04:03 +0200 |
| commit | b0c81aa566cccbe26bd99237873fb8428827d82f (patch) | |
| tree | bd0c7f6528f1f7283dcc3163929b627e6bb1c756 /net | |
| parent | f667009ecc3304248727236ff88f9070f918355f (diff) | |
| download | linux-b0c81aa566cccbe26bd99237873fb8428827d82f.tar.gz linux-b0c81aa566cccbe26bd99237873fb8428827d82f.tar.bz2 linux-b0c81aa566cccbe26bd99237873fb8428827d82f.zip | |
netfilter: xt_quota: use per-rule spin lock
Use per-rule spin lock to improve the scalability.
Signed-off-by: Changli Gao <xiaosuo@gmail.com>
Signed-off-by: Patrick McHardy <kaber@trash.net>
Diffstat (limited to 'net')
| -rw-r--r-- | net/netfilter/xt_quota.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/net/netfilter/xt_quota.c b/net/netfilter/xt_quota.c index b4f7dfea5980..304b1fda1a0d 100644 --- a/net/netfilter/xt_quota.c +++ b/net/netfilter/xt_quota.c @@ -11,7 +11,8 @@ #include <linux/netfilter/xt_quota.h> struct xt_quota_priv { - uint64_t quota; + spinlock_t lock; + uint64_t quota; }; MODULE_LICENSE("GPL"); @@ -20,8 +21,6 @@ MODULE_DESCRIPTION("Xtables: countdown quota match"); MODULE_ALIAS("ipt_quota"); MODULE_ALIAS("ip6t_quota"); -static DEFINE_SPINLOCK(quota_lock); - static bool quota_mt(const struct sk_buff *skb, struct xt_action_param *par) { @@ -29,7 +28,7 @@ quota_mt(const struct sk_buff *skb, struct xt_action_param *par) struct xt_quota_priv *priv = q->master; bool ret = q->flags & XT_QUOTA_INVERT; - spin_lock_bh("a_lock); + spin_lock_bh(&priv->lock); if (priv->quota >= skb->len) { priv->quota -= skb->len; ret = !ret; @@ -39,7 +38,7 @@ quota_mt(const struct sk_buff *skb, struct xt_action_param *par) } /* Copy quota back to matchinfo so that iptables can display it */ q->quota = priv->quota; - spin_unlock_bh("a_lock); + spin_unlock_bh(&priv->lock); return ret; } @@ -55,6 +54,7 @@ static int quota_mt_check(const struct xt_mtchk_param *par) if (q->master == NULL) return -ENOMEM; + spin_lock_init(&q->master->lock); q->master->quota = q->quota; return 0; } |