Commit 1f747c26 authored by WANG Cong's avatar WANG Cong Committed by David S. Miller

net_sched: convert tc_action_ops to use struct list_head

We don't need to maintain our own singly linked list code.

Cc: Jamal Hadi Salim <jhs@mojatatu.com>
Cc: David S. Miller <davem@davemloft.net>
Signed-off-by: default avatarCong Wang <xiyou.wangcong@gmail.com>
Signed-off-by: default avatarJamal Hadi Salim <jhs@mojatatu.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 89819dc0
...@@ -85,7 +85,7 @@ struct tc_action { ...@@ -85,7 +85,7 @@ struct tc_action {
#define TCA_CAP_NONE 0 #define TCA_CAP_NONE 0
struct tc_action_ops { struct tc_action_ops {
struct tc_action_ops *next; struct list_head head;
struct tcf_hashinfo *hinfo; struct tcf_hashinfo *hinfo;
char kind[IFNAMSIZ]; char kind[IFNAMSIZ];
__u32 type; /* TBD to match kind */ __u32 type; /* TBD to match kind */
......
...@@ -255,12 +255,12 @@ void tcf_hash_insert(struct tcf_common *p, struct tcf_hashinfo *hinfo) ...@@ -255,12 +255,12 @@ void tcf_hash_insert(struct tcf_common *p, struct tcf_hashinfo *hinfo)
} }
EXPORT_SYMBOL(tcf_hash_insert); EXPORT_SYMBOL(tcf_hash_insert);
static struct tc_action_ops *act_base = NULL; static LIST_HEAD(act_base);
static DEFINE_RWLOCK(act_mod_lock); static DEFINE_RWLOCK(act_mod_lock);
int tcf_register_action(struct tc_action_ops *act) int tcf_register_action(struct tc_action_ops *act)
{ {
struct tc_action_ops *a, **ap; struct tc_action_ops *a;
/* Must supply act, dump, cleanup and init */ /* Must supply act, dump, cleanup and init */
if (!act->act || !act->dump || !act->cleanup || !act->init) if (!act->act || !act->dump || !act->cleanup || !act->init)
...@@ -273,14 +273,13 @@ int tcf_register_action(struct tc_action_ops *act) ...@@ -273,14 +273,13 @@ int tcf_register_action(struct tc_action_ops *act)
act->walk = tcf_generic_walker; act->walk = tcf_generic_walker;
write_lock(&act_mod_lock); write_lock(&act_mod_lock);
for (ap = &act_base; (a = *ap) != NULL; ap = &a->next) { list_for_each_entry(a, &act_base, head) {
if (act->type == a->type || (strcmp(act->kind, a->kind) == 0)) { if (act->type == a->type || (strcmp(act->kind, a->kind) == 0)) {
write_unlock(&act_mod_lock); write_unlock(&act_mod_lock);
return -EEXIST; return -EEXIST;
} }
} }
act->next = NULL; list_add_tail(&act->head, &act_base);
*ap = act;
write_unlock(&act_mod_lock); write_unlock(&act_mod_lock);
return 0; return 0;
} }
...@@ -288,16 +287,15 @@ EXPORT_SYMBOL(tcf_register_action); ...@@ -288,16 +287,15 @@ EXPORT_SYMBOL(tcf_register_action);
int tcf_unregister_action(struct tc_action_ops *act) int tcf_unregister_action(struct tc_action_ops *act)
{ {
struct tc_action_ops *a, **ap; struct tc_action_ops *a;
int err = -ENOENT; int err = -ENOENT;
write_lock(&act_mod_lock); write_lock(&act_mod_lock);
for (ap = &act_base; (a = *ap) != NULL; ap = &a->next) list_for_each_entry(a, &act_base, head)
if (a == act) if (a == act)
break; break;
if (a) { if (a) {
*ap = a->next; list_del(&act->head);
a->next = NULL;
err = 0; err = 0;
} }
write_unlock(&act_mod_lock); write_unlock(&act_mod_lock);
...@@ -312,7 +310,7 @@ static struct tc_action_ops *tc_lookup_action_n(char *kind) ...@@ -312,7 +310,7 @@ static struct tc_action_ops *tc_lookup_action_n(char *kind)
if (kind) { if (kind) {
read_lock(&act_mod_lock); read_lock(&act_mod_lock);
for (a = act_base; a; a = a->next) { list_for_each_entry(a, &act_base, head) {
if (strcmp(kind, a->kind) == 0) { if (strcmp(kind, a->kind) == 0) {
if (!try_module_get(a->owner)) { if (!try_module_get(a->owner)) {
read_unlock(&act_mod_lock); read_unlock(&act_mod_lock);
...@@ -333,7 +331,7 @@ static struct tc_action_ops *tc_lookup_action(struct nlattr *kind) ...@@ -333,7 +331,7 @@ static struct tc_action_ops *tc_lookup_action(struct nlattr *kind)
if (kind) { if (kind) {
read_lock(&act_mod_lock); read_lock(&act_mod_lock);
for (a = act_base; a; a = a->next) { list_for_each_entry(a, &act_base, head) {
if (nla_strcmp(kind, a->kind) == 0) { if (nla_strcmp(kind, a->kind) == 0) {
if (!try_module_get(a->owner)) { if (!try_module_get(a->owner)) {
read_unlock(&act_mod_lock); read_unlock(&act_mod_lock);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment