mirror of
git://projects.qi-hardware.com/openwrt-xburst.git
synced 2024-11-28 04:16:13 +02:00
reduce the total cpu usage of ipt_do_table under high network load from 3.1% down to 1.6% (tested with oprofile)
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@15822 3c298f89-4303-0410-b956-a3cf2f4a3e73
This commit is contained in:
parent
c2db6d509f
commit
e036e1ee00
@ -75,3 +75,46 @@
|
|||||||
}
|
}
|
||||||
|
|
||||||
#if defined(CONFIG_NETFILTER_XT_TARGET_TRACE) || \
|
#if defined(CONFIG_NETFILTER_XT_TARGET_TRACE) || \
|
||||||
|
@@ -332,8 +358,27 @@ ipt_do_table(struct sk_buff *skb,
|
||||||
|
struct xt_match_param mtpar;
|
||||||
|
struct xt_target_param tgpar;
|
||||||
|
|
||||||
|
- /* Initialization */
|
||||||
|
ip = ip_hdr(skb);
|
||||||
|
+
|
||||||
|
+ read_lock_bh(&table->lock);
|
||||||
|
+ IP_NF_ASSERT(table->valid_hooks & (1 << hook));
|
||||||
|
+ private = table->private;
|
||||||
|
+ table_base = (void *)private->entries[smp_processor_id()];
|
||||||
|
+ e = get_entry(table_base, private->hook_entry[hook]);
|
||||||
|
+ if (e->target_offset <= sizeof(struct ipt_entry) &&
|
||||||
|
+ (e->ip.flags & IPT_F_NO_DEF_MATCH)) {
|
||||||
|
+ struct ipt_entry_target *t = ipt_get_target(e);
|
||||||
|
+ if (!t->u.kernel.target->target) {
|
||||||
|
+ int v = ((struct ipt_standard_target *)t)->verdict;
|
||||||
|
+ if ((v < 0) && (v != IPT_RETURN)) {
|
||||||
|
+ ADD_COUNTER(e->counters, ntohs(ip->tot_len), 1);
|
||||||
|
+ read_unlock_bh(&table->lock);
|
||||||
|
+ return (unsigned)(-v) - 1;
|
||||||
|
+ }
|
||||||
|
+ }
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ /* Initialization */
|
||||||
|
datalen = skb->len - ip->ihl * 4;
|
||||||
|
indev = in ? in->name : nulldevname;
|
||||||
|
outdev = out ? out->name : nulldevname;
|
||||||
|
@@ -351,12 +396,6 @@ ipt_do_table(struct sk_buff *skb,
|
||||||
|
mtpar.family = tgpar.family = NFPROTO_IPV4;
|
||||||
|
tgpar.hooknum = hook;
|
||||||
|
|
||||||
|
- read_lock_bh(&table->lock);
|
||||||
|
- IP_NF_ASSERT(table->valid_hooks & (1 << hook));
|
||||||
|
- private = table->private;
|
||||||
|
- table_base = (void *)private->entries[smp_processor_id()];
|
||||||
|
- e = get_entry(table_base, private->hook_entry[hook]);
|
||||||
|
-
|
||||||
|
/* For return from builtin chain */
|
||||||
|
back = get_entry(table_base, private->underflow[hook]);
|
||||||
|
|
||||||
|
|
||||||
|
@ -75,3 +75,45 @@
|
|||||||
}
|
}
|
||||||
|
|
||||||
#if defined(CONFIG_NETFILTER_XT_TARGET_TRACE) || \
|
#if defined(CONFIG_NETFILTER_XT_TARGET_TRACE) || \
|
||||||
|
@@ -328,8 +354,27 @@ ipt_do_table(struct sk_buff *skb,
|
||||||
|
struct xt_match_param mtpar;
|
||||||
|
struct xt_target_param tgpar;
|
||||||
|
|
||||||
|
- /* Initialization */
|
||||||
|
ip = ip_hdr(skb);
|
||||||
|
+
|
||||||
|
+ read_lock_bh(&table->lock);
|
||||||
|
+ IP_NF_ASSERT(table->valid_hooks & (1 << hook));
|
||||||
|
+ private = table->private;
|
||||||
|
+ table_base = (void *)private->entries[smp_processor_id()];
|
||||||
|
+ e = get_entry(table_base, private->hook_entry[hook]);
|
||||||
|
+ if (e->target_offset <= sizeof(struct ipt_entry) &&
|
||||||
|
+ (e->ip.flags & IPT_F_NO_DEF_MATCH)) {
|
||||||
|
+ struct ipt_entry_target *t = ipt_get_target(e);
|
||||||
|
+ if (!t->u.kernel.target->target) {
|
||||||
|
+ int v = ((struct ipt_standard_target *)t)->verdict;
|
||||||
|
+ if ((v < 0) && (v != IPT_RETURN)) {
|
||||||
|
+ ADD_COUNTER(e->counters, ntohs(ip->tot_len), 1);
|
||||||
|
+ read_unlock_bh(&table->lock);
|
||||||
|
+ return (unsigned)(-v) - 1;
|
||||||
|
+ }
|
||||||
|
+ }
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ /* Initialization */
|
||||||
|
datalen = skb->len - ip->ihl * 4;
|
||||||
|
indev = in ? in->name : nulldevname;
|
||||||
|
outdev = out ? out->name : nulldevname;
|
||||||
|
@@ -347,12 +392,6 @@ ipt_do_table(struct sk_buff *skb,
|
||||||
|
mtpar.family = tgpar.family = NFPROTO_IPV4;
|
||||||
|
tgpar.hooknum = hook;
|
||||||
|
|
||||||
|
- read_lock_bh(&table->lock);
|
||||||
|
- IP_NF_ASSERT(table->valid_hooks & (1 << hook));
|
||||||
|
- private = table->private;
|
||||||
|
- table_base = (void *)private->entries[smp_processor_id()];
|
||||||
|
- e = get_entry(table_base, private->hook_entry[hook]);
|
||||||
|
-
|
||||||
|
/* For return from builtin chain */
|
||||||
|
back = get_entry(table_base, private->underflow[hook]);
|
||||||
|
|
||||||
|
@ -75,3 +75,47 @@
|
|||||||
}
|
}
|
||||||
|
|
||||||
#if defined(CONFIG_NETFILTER_XT_TARGET_TRACE) || \
|
#if defined(CONFIG_NETFILTER_XT_TARGET_TRACE) || \
|
||||||
|
@@ -318,8 +344,28 @@ ipt_do_table(struct sk_buff *skb,
|
||||||
|
struct xt_match_param mtpar;
|
||||||
|
struct xt_target_param tgpar;
|
||||||
|
|
||||||
|
- /* Initialization */
|
||||||
|
ip = ip_hdr(skb);
|
||||||
|
+
|
||||||
|
+ IP_NF_ASSERT(table->valid_hooks & (1 << hook));
|
||||||
|
+ xt_info_rdlock_bh();
|
||||||
|
+ private = table->private;
|
||||||
|
+ table_base = private->entries[smp_processor_id()];
|
||||||
|
+ e = get_entry(table_base, private->hook_entry[hook]);
|
||||||
|
+
|
||||||
|
+ if (e->target_offset <= sizeof(struct ipt_entry) &&
|
||||||
|
+ (e->ip.flags & IPT_F_NO_DEF_MATCH)) {
|
||||||
|
+ struct ipt_entry_target *t = ipt_get_target(e);
|
||||||
|
+ if (!t->u.kernel.target->target) {
|
||||||
|
+ int v = ((struct ipt_standard_target *)t)->verdict;
|
||||||
|
+ if ((v < 0) && (v != IPT_RETURN)) {
|
||||||
|
+ ADD_COUNTER(e->counters, ntohs(ip->tot_len), 1);
|
||||||
|
+ read_unlock_bh(&table->lock);
|
||||||
|
+ return (unsigned)(-v) - 1;
|
||||||
|
+ }
|
||||||
|
+ }
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ /* Initialization */
|
||||||
|
datalen = skb->len - ip->ihl * 4;
|
||||||
|
indev = in ? in->name : nulldevname;
|
||||||
|
outdev = out ? out->name : nulldevname;
|
||||||
|
@@ -337,13 +383,6 @@ ipt_do_table(struct sk_buff *skb,
|
||||||
|
mtpar.family = tgpar.family = NFPROTO_IPV4;
|
||||||
|
tgpar.hooknum = hook;
|
||||||
|
|
||||||
|
- IP_NF_ASSERT(table->valid_hooks & (1 << hook));
|
||||||
|
- xt_info_rdlock_bh();
|
||||||
|
- private = table->private;
|
||||||
|
- table_base = private->entries[smp_processor_id()];
|
||||||
|
-
|
||||||
|
- e = get_entry(table_base, private->hook_entry[hook]);
|
||||||
|
-
|
||||||
|
/* For return from builtin chain */
|
||||||
|
back = get_entry(table_base, private->underflow[hook]);
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user