Commit c1b52739 authored by Benjamin LaHaise's avatar Benjamin LaHaise Committed by David S. Miller

pkt_sched: namespace aware act_mirred

Eric Dumazet pointed out that act_mirred needs to find the current net_ns,
and struct net pointer is not provided in the call chain.  His original
patch made use of current->nsproxy->net_ns to find the network namespace,
but this fails to work correctly for userspace code that makes use of
netlink sockets in different network namespaces.  Instead, pass the
"struct net *" down along the call chain to where it is needed.

This version removes the ifb changes as Eric has submitted that patch
separately, but is otherwise identical to the previous version.
Signed-off-by: default avatarBenjamin LaHaise <bcrl@kvack.org>
Tested-by: default avatarEric Dumazet <eric.dumazet@gmail.com>
Acked-by: default avatarJamal Hadi Salim <jhs@mojatatu.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 60592833
...@@ -91,7 +91,9 @@ struct tc_action_ops { ...@@ -91,7 +91,9 @@ struct tc_action_ops {
int (*dump)(struct sk_buff *, struct tc_action *, int, int); int (*dump)(struct sk_buff *, struct tc_action *, int, int);
int (*cleanup)(struct tc_action *, int bind); int (*cleanup)(struct tc_action *, int bind);
int (*lookup)(struct tc_action *, u32); int (*lookup)(struct tc_action *, u32);
int (*init)(struct nlattr *, struct nlattr *, struct tc_action *, int , int); int (*init)(struct net *net, struct nlattr *nla,
struct nlattr *est, struct tc_action *act, int ovr,
int bind);
int (*walk)(struct sk_buff *, struct netlink_callback *, int, struct tc_action *); int (*walk)(struct sk_buff *, struct netlink_callback *, int, struct tc_action *);
}; };
...@@ -116,8 +118,12 @@ extern int tcf_register_action(struct tc_action_ops *a); ...@@ -116,8 +118,12 @@ extern int tcf_register_action(struct tc_action_ops *a);
extern int tcf_unregister_action(struct tc_action_ops *a); extern int tcf_unregister_action(struct tc_action_ops *a);
extern void tcf_action_destroy(struct tc_action *a, int bind); extern void tcf_action_destroy(struct tc_action *a, int bind);
extern int tcf_action_exec(struct sk_buff *skb, const struct tc_action *a, struct tcf_result *res); extern int tcf_action_exec(struct sk_buff *skb, const struct tc_action *a, struct tcf_result *res);
extern struct tc_action *tcf_action_init(struct nlattr *nla, struct nlattr *est, char *n, int ovr, int bind); extern struct tc_action *tcf_action_init(struct net *net, struct nlattr *nla,
extern struct tc_action *tcf_action_init_1(struct nlattr *nla, struct nlattr *est, char *n, int ovr, int bind); struct nlattr *est, char *n, int ovr,
int bind);
extern struct tc_action *tcf_action_init_1(struct net *net, struct nlattr *nla,
struct nlattr *est, char *n, int ovr,
int bind);
extern int tcf_action_dump(struct sk_buff *skb, struct tc_action *a, int, int); extern int tcf_action_dump(struct sk_buff *skb, struct tc_action *a, int, int);
extern int tcf_action_dump_old(struct sk_buff *skb, struct tc_action *a, int, int); extern int tcf_action_dump_old(struct sk_buff *skb, struct tc_action *a, int, int);
extern int tcf_action_dump_1(struct sk_buff *skb, struct tc_action *a, int, int); extern int tcf_action_dump_1(struct sk_buff *skb, struct tc_action *a, int, int);
......
...@@ -126,8 +126,9 @@ tcf_exts_exec(struct sk_buff *skb, struct tcf_exts *exts, ...@@ -126,8 +126,9 @@ tcf_exts_exec(struct sk_buff *skb, struct tcf_exts *exts,
return 0; return 0;
} }
extern int tcf_exts_validate(struct tcf_proto *tp, struct nlattr **tb, extern int tcf_exts_validate(struct net *net, struct tcf_proto *tp,
struct nlattr *rate_tlv, struct tcf_exts *exts, struct nlattr **tb, struct nlattr *rate_tlv,
struct tcf_exts *exts,
const struct tcf_ext_map *map); const struct tcf_ext_map *map);
extern void tcf_exts_destroy(struct tcf_proto *tp, struct tcf_exts *exts); extern void tcf_exts_destroy(struct tcf_proto *tp, struct tcf_exts *exts);
extern void tcf_exts_change(struct tcf_proto *tp, struct tcf_exts *dst, extern void tcf_exts_change(struct tcf_proto *tp, struct tcf_exts *dst,
......
...@@ -195,7 +195,7 @@ struct tcf_proto_ops { ...@@ -195,7 +195,7 @@ struct tcf_proto_ops {
unsigned long (*get)(struct tcf_proto*, u32 handle); unsigned long (*get)(struct tcf_proto*, u32 handle);
void (*put)(struct tcf_proto*, unsigned long); void (*put)(struct tcf_proto*, unsigned long);
int (*change)(struct sk_buff *, int (*change)(struct net *net, struct sk_buff *,
struct tcf_proto*, unsigned long, struct tcf_proto*, unsigned long,
u32 handle, struct nlattr **, u32 handle, struct nlattr **,
unsigned long *); unsigned long *);
......
...@@ -485,8 +485,9 @@ tcf_action_dump(struct sk_buff *skb, struct tc_action *act, int bind, int ref) ...@@ -485,8 +485,9 @@ tcf_action_dump(struct sk_buff *skb, struct tc_action *act, int bind, int ref)
return err; return err;
} }
struct tc_action *tcf_action_init_1(struct nlattr *nla, struct nlattr *est, struct tc_action *tcf_action_init_1(struct net *net, struct nlattr *nla,
char *name, int ovr, int bind) struct nlattr *est, char *name, int ovr,
int bind)
{ {
struct tc_action *a; struct tc_action *a;
struct tc_action_ops *a_o; struct tc_action_ops *a_o;
...@@ -542,9 +543,9 @@ struct tc_action *tcf_action_init_1(struct nlattr *nla, struct nlattr *est, ...@@ -542,9 +543,9 @@ struct tc_action *tcf_action_init_1(struct nlattr *nla, struct nlattr *est,
/* backward compatibility for policer */ /* backward compatibility for policer */
if (name == NULL) if (name == NULL)
err = a_o->init(tb[TCA_ACT_OPTIONS], est, a, ovr, bind); err = a_o->init(net, tb[TCA_ACT_OPTIONS], est, a, ovr, bind);
else else
err = a_o->init(nla, est, a, ovr, bind); err = a_o->init(net, nla, est, a, ovr, bind);
if (err < 0) if (err < 0)
goto err_free; goto err_free;
...@@ -566,8 +567,9 @@ struct tc_action *tcf_action_init_1(struct nlattr *nla, struct nlattr *est, ...@@ -566,8 +567,9 @@ struct tc_action *tcf_action_init_1(struct nlattr *nla, struct nlattr *est,
return ERR_PTR(err); return ERR_PTR(err);
} }
struct tc_action *tcf_action_init(struct nlattr *nla, struct nlattr *est, struct tc_action *tcf_action_init(struct net *net, struct nlattr *nla,
char *name, int ovr, int bind) struct nlattr *est, char *name, int ovr,
int bind)
{ {
struct nlattr *tb[TCA_ACT_MAX_PRIO + 1]; struct nlattr *tb[TCA_ACT_MAX_PRIO + 1];
struct tc_action *head = NULL, *act, *act_prev = NULL; struct tc_action *head = NULL, *act, *act_prev = NULL;
...@@ -579,7 +581,7 @@ struct tc_action *tcf_action_init(struct nlattr *nla, struct nlattr *est, ...@@ -579,7 +581,7 @@ struct tc_action *tcf_action_init(struct nlattr *nla, struct nlattr *est,
return ERR_PTR(err); return ERR_PTR(err);
for (i = 1; i <= TCA_ACT_MAX_PRIO && tb[i]; i++) { for (i = 1; i <= TCA_ACT_MAX_PRIO && tb[i]; i++) {
act = tcf_action_init_1(tb[i], est, name, ovr, bind); act = tcf_action_init_1(net, tb[i], est, name, ovr, bind);
if (IS_ERR(act)) if (IS_ERR(act))
goto err; goto err;
act->order = i; act->order = i;
...@@ -960,7 +962,7 @@ tcf_action_add(struct net *net, struct nlattr *nla, struct nlmsghdr *n, ...@@ -960,7 +962,7 @@ tcf_action_add(struct net *net, struct nlattr *nla, struct nlmsghdr *n,
struct tc_action *a; struct tc_action *a;
u32 seq = n->nlmsg_seq; u32 seq = n->nlmsg_seq;
act = tcf_action_init(nla, NULL, NULL, ovr, 0); act = tcf_action_init(net, nla, NULL, NULL, ovr, 0);
if (act == NULL) if (act == NULL)
goto done; goto done;
if (IS_ERR(act)) { if (IS_ERR(act)) {
......
...@@ -51,7 +51,7 @@ static const struct nla_policy csum_policy[TCA_CSUM_MAX + 1] = { ...@@ -51,7 +51,7 @@ static const struct nla_policy csum_policy[TCA_CSUM_MAX + 1] = {
[TCA_CSUM_PARMS] = { .len = sizeof(struct tc_csum), }, [TCA_CSUM_PARMS] = { .len = sizeof(struct tc_csum), },
}; };
static int tcf_csum_init(struct nlattr *nla, struct nlattr *est, static int tcf_csum_init(struct net *n, struct nlattr *nla, struct nlattr *est,
struct tc_action *a, int ovr, int bind) struct tc_action *a, int ovr, int bind)
{ {
struct nlattr *tb[TCA_CSUM_MAX + 1]; struct nlattr *tb[TCA_CSUM_MAX + 1];
......
...@@ -58,8 +58,9 @@ static const struct nla_policy gact_policy[TCA_GACT_MAX + 1] = { ...@@ -58,8 +58,9 @@ static const struct nla_policy gact_policy[TCA_GACT_MAX + 1] = {
[TCA_GACT_PROB] = { .len = sizeof(struct tc_gact_p) }, [TCA_GACT_PROB] = { .len = sizeof(struct tc_gact_p) },
}; };
static int tcf_gact_init(struct nlattr *nla, struct nlattr *est, static int tcf_gact_init(struct net *net, struct nlattr *nla,
struct tc_action *a, int ovr, int bind) struct nlattr *est, struct tc_action *a,
int ovr, int bind)
{ {
struct nlattr *tb[TCA_GACT_MAX + 1]; struct nlattr *tb[TCA_GACT_MAX + 1];
struct tc_gact *parm; struct tc_gact *parm;
......
...@@ -102,7 +102,7 @@ static const struct nla_policy ipt_policy[TCA_IPT_MAX + 1] = { ...@@ -102,7 +102,7 @@ static const struct nla_policy ipt_policy[TCA_IPT_MAX + 1] = {
[TCA_IPT_TARG] = { .len = sizeof(struct xt_entry_target) }, [TCA_IPT_TARG] = { .len = sizeof(struct xt_entry_target) },
}; };
static int tcf_ipt_init(struct nlattr *nla, struct nlattr *est, static int tcf_ipt_init(struct net *net, struct nlattr *nla, struct nlattr *est,
struct tc_action *a, int ovr, int bind) struct tc_action *a, int ovr, int bind)
{ {
struct nlattr *tb[TCA_IPT_MAX + 1]; struct nlattr *tb[TCA_IPT_MAX + 1];
......
...@@ -62,8 +62,9 @@ static const struct nla_policy mirred_policy[TCA_MIRRED_MAX + 1] = { ...@@ -62,8 +62,9 @@ static const struct nla_policy mirred_policy[TCA_MIRRED_MAX + 1] = {
[TCA_MIRRED_PARMS] = { .len = sizeof(struct tc_mirred) }, [TCA_MIRRED_PARMS] = { .len = sizeof(struct tc_mirred) },
}; };
static int tcf_mirred_init(struct nlattr *nla, struct nlattr *est, static int tcf_mirred_init(struct net *net, struct nlattr *nla,
struct tc_action *a, int ovr, int bind) struct nlattr *est, struct tc_action *a, int ovr,
int bind)
{ {
struct nlattr *tb[TCA_MIRRED_MAX + 1]; struct nlattr *tb[TCA_MIRRED_MAX + 1];
struct tc_mirred *parm; struct tc_mirred *parm;
...@@ -88,7 +89,7 @@ static int tcf_mirred_init(struct nlattr *nla, struct nlattr *est, ...@@ -88,7 +89,7 @@ static int tcf_mirred_init(struct nlattr *nla, struct nlattr *est,
return -EINVAL; return -EINVAL;
} }
if (parm->ifindex) { if (parm->ifindex) {
dev = __dev_get_by_index(&init_net, parm->ifindex); dev = __dev_get_by_index(net, parm->ifindex);
if (dev == NULL) if (dev == NULL)
return -ENODEV; return -ENODEV;
switch (dev->type) { switch (dev->type) {
......
...@@ -44,7 +44,7 @@ static const struct nla_policy nat_policy[TCA_NAT_MAX + 1] = { ...@@ -44,7 +44,7 @@ static const struct nla_policy nat_policy[TCA_NAT_MAX + 1] = {
[TCA_NAT_PARMS] = { .len = sizeof(struct tc_nat) }, [TCA_NAT_PARMS] = { .len = sizeof(struct tc_nat) },
}; };
static int tcf_nat_init(struct nlattr *nla, struct nlattr *est, static int tcf_nat_init(struct net *net, struct nlattr *nla, struct nlattr *est,
struct tc_action *a, int ovr, int bind) struct tc_action *a, int ovr, int bind)
{ {
struct nlattr *tb[TCA_NAT_MAX + 1]; struct nlattr *tb[TCA_NAT_MAX + 1];
......
...@@ -38,8 +38,9 @@ static const struct nla_policy pedit_policy[TCA_PEDIT_MAX + 1] = { ...@@ -38,8 +38,9 @@ static const struct nla_policy pedit_policy[TCA_PEDIT_MAX + 1] = {
[TCA_PEDIT_PARMS] = { .len = sizeof(struct tc_pedit) }, [TCA_PEDIT_PARMS] = { .len = sizeof(struct tc_pedit) },
}; };
static int tcf_pedit_init(struct nlattr *nla, struct nlattr *est, static int tcf_pedit_init(struct net *net, struct nlattr *nla,
struct tc_action *a, int ovr, int bind) struct nlattr *est, struct tc_action *a,
int ovr, int bind)
{ {
struct nlattr *tb[TCA_PEDIT_MAX + 1]; struct nlattr *tb[TCA_PEDIT_MAX + 1];
struct tc_pedit *parm; struct tc_pedit *parm;
......
...@@ -130,8 +130,9 @@ static const struct nla_policy police_policy[TCA_POLICE_MAX + 1] = { ...@@ -130,8 +130,9 @@ static const struct nla_policy police_policy[TCA_POLICE_MAX + 1] = {
[TCA_POLICE_RESULT] = { .type = NLA_U32 }, [TCA_POLICE_RESULT] = { .type = NLA_U32 },
}; };
static int tcf_act_police_locate(struct nlattr *nla, struct nlattr *est, static int tcf_act_police_locate(struct net *net, struct nlattr *nla,
struct tc_action *a, int ovr, int bind) struct nlattr *est, struct tc_action *a,
int ovr, int bind)
{ {
unsigned int h; unsigned int h;
int ret = 0, err; int ret = 0, err;
......
...@@ -95,8 +95,9 @@ static const struct nla_policy simple_policy[TCA_DEF_MAX + 1] = { ...@@ -95,8 +95,9 @@ static const struct nla_policy simple_policy[TCA_DEF_MAX + 1] = {
[TCA_DEF_DATA] = { .type = NLA_STRING, .len = SIMP_MAX_DATA }, [TCA_DEF_DATA] = { .type = NLA_STRING, .len = SIMP_MAX_DATA },
}; };
static int tcf_simp_init(struct nlattr *nla, struct nlattr *est, static int tcf_simp_init(struct net *net, struct nlattr *nla,
struct tc_action *a, int ovr, int bind) struct nlattr *est, struct tc_action *a,
int ovr, int bind)
{ {
struct nlattr *tb[TCA_DEF_MAX + 1]; struct nlattr *tb[TCA_DEF_MAX + 1];
struct tc_defact *parm; struct tc_defact *parm;
......
...@@ -67,8 +67,9 @@ static const struct nla_policy skbedit_policy[TCA_SKBEDIT_MAX + 1] = { ...@@ -67,8 +67,9 @@ static const struct nla_policy skbedit_policy[TCA_SKBEDIT_MAX + 1] = {
[TCA_SKBEDIT_MARK] = { .len = sizeof(u32) }, [TCA_SKBEDIT_MARK] = { .len = sizeof(u32) },
}; };
static int tcf_skbedit_init(struct nlattr *nla, struct nlattr *est, static int tcf_skbedit_init(struct net *net, struct nlattr *nla,
struct tc_action *a, int ovr, int bind) struct nlattr *est, struct tc_action *a,
int ovr, int bind)
{ {
struct nlattr *tb[TCA_SKBEDIT_MAX + 1]; struct nlattr *tb[TCA_SKBEDIT_MAX + 1];
struct tc_skbedit *parm; struct tc_skbedit *parm;
......
...@@ -321,7 +321,7 @@ static int tc_ctl_tfilter(struct sk_buff *skb, struct nlmsghdr *n, void *arg) ...@@ -321,7 +321,7 @@ static int tc_ctl_tfilter(struct sk_buff *skb, struct nlmsghdr *n, void *arg)
} }
} }
err = tp->ops->change(skb, tp, cl, t->tcm_handle, tca, &fh); err = tp->ops->change(net, skb, tp, cl, t->tcm_handle, tca, &fh);
if (err == 0) { if (err == 0) {
if (tp_created) { if (tp_created) {
spin_lock_bh(root_lock); spin_lock_bh(root_lock);
...@@ -508,7 +508,7 @@ void tcf_exts_destroy(struct tcf_proto *tp, struct tcf_exts *exts) ...@@ -508,7 +508,7 @@ void tcf_exts_destroy(struct tcf_proto *tp, struct tcf_exts *exts)
} }
EXPORT_SYMBOL(tcf_exts_destroy); EXPORT_SYMBOL(tcf_exts_destroy);
int tcf_exts_validate(struct tcf_proto *tp, struct nlattr **tb, int tcf_exts_validate(struct net *net, struct tcf_proto *tp, struct nlattr **tb,
struct nlattr *rate_tlv, struct tcf_exts *exts, struct nlattr *rate_tlv, struct tcf_exts *exts,
const struct tcf_ext_map *map) const struct tcf_ext_map *map)
{ {
...@@ -519,7 +519,7 @@ int tcf_exts_validate(struct tcf_proto *tp, struct nlattr **tb, ...@@ -519,7 +519,7 @@ int tcf_exts_validate(struct tcf_proto *tp, struct nlattr **tb,
struct tc_action *act; struct tc_action *act;
if (map->police && tb[map->police]) { if (map->police && tb[map->police]) {
act = tcf_action_init_1(tb[map->police], rate_tlv, act = tcf_action_init_1(net, tb[map->police], rate_tlv,
"police", TCA_ACT_NOREPLACE, "police", TCA_ACT_NOREPLACE,
TCA_ACT_BIND); TCA_ACT_BIND);
if (IS_ERR(act)) if (IS_ERR(act))
...@@ -528,8 +528,9 @@ int tcf_exts_validate(struct tcf_proto *tp, struct nlattr **tb, ...@@ -528,8 +528,9 @@ int tcf_exts_validate(struct tcf_proto *tp, struct nlattr **tb,
act->type = TCA_OLD_COMPAT; act->type = TCA_OLD_COMPAT;
exts->action = act; exts->action = act;
} else if (map->action && tb[map->action]) { } else if (map->action && tb[map->action]) {
act = tcf_action_init(tb[map->action], rate_tlv, NULL, act = tcf_action_init(net, tb[map->action], rate_tlv,
TCA_ACT_NOREPLACE, TCA_ACT_BIND); NULL, TCA_ACT_NOREPLACE,
TCA_ACT_BIND);
if (IS_ERR(act)) if (IS_ERR(act))
return PTR_ERR(act); return PTR_ERR(act);
......
...@@ -132,15 +132,16 @@ static const struct nla_policy basic_policy[TCA_BASIC_MAX + 1] = { ...@@ -132,15 +132,16 @@ static const struct nla_policy basic_policy[TCA_BASIC_MAX + 1] = {
[TCA_BASIC_EMATCHES] = { .type = NLA_NESTED }, [TCA_BASIC_EMATCHES] = { .type = NLA_NESTED },
}; };
static int basic_set_parms(struct tcf_proto *tp, struct basic_filter *f, static int basic_set_parms(struct net *net, struct tcf_proto *tp,
unsigned long base, struct nlattr **tb, struct basic_filter *f, unsigned long base,
struct nlattr **tb,
struct nlattr *est) struct nlattr *est)
{ {
int err = -EINVAL; int err = -EINVAL;
struct tcf_exts e; struct tcf_exts e;
struct tcf_ematch_tree t; struct tcf_ematch_tree t;
err = tcf_exts_validate(tp, tb, est, &e, &basic_ext_map); err = tcf_exts_validate(net, tp, tb, est, &e, &basic_ext_map);
if (err < 0) if (err < 0)
return err; return err;
...@@ -162,7 +163,7 @@ static int basic_set_parms(struct tcf_proto *tp, struct basic_filter *f, ...@@ -162,7 +163,7 @@ static int basic_set_parms(struct tcf_proto *tp, struct basic_filter *f,
return err; return err;
} }
static int basic_change(struct sk_buff *in_skb, static int basic_change(struct net *net, struct sk_buff *in_skb,
struct tcf_proto *tp, unsigned long base, u32 handle, struct tcf_proto *tp, unsigned long base, u32 handle,
struct nlattr **tca, unsigned long *arg) struct nlattr **tca, unsigned long *arg)
{ {
...@@ -182,7 +183,7 @@ static int basic_change(struct sk_buff *in_skb, ...@@ -182,7 +183,7 @@ static int basic_change(struct sk_buff *in_skb,
if (f != NULL) { if (f != NULL) {
if (handle && f->handle != handle) if (handle && f->handle != handle)
return -EINVAL; return -EINVAL;
return basic_set_parms(tp, f, base, tb, tca[TCA_RATE]); return basic_set_parms(net, tp, f, base, tb, tca[TCA_RATE]);
} }
err = -ENOBUFS; err = -ENOBUFS;
...@@ -208,7 +209,7 @@ static int basic_change(struct sk_buff *in_skb, ...@@ -208,7 +209,7 @@ static int basic_change(struct sk_buff *in_skb,
f->handle = head->hgenerator; f->handle = head->hgenerator;
} }
err = basic_set_parms(tp, f, base, tb, tca[TCA_RATE]); err = basic_set_parms(net, tp, f, base, tb, tca[TCA_RATE]);
if (err < 0) if (err < 0)
goto errout; goto errout;
......
...@@ -178,7 +178,7 @@ static const struct nla_policy cgroup_policy[TCA_CGROUP_MAX + 1] = { ...@@ -178,7 +178,7 @@ static const struct nla_policy cgroup_policy[TCA_CGROUP_MAX + 1] = {
[TCA_CGROUP_EMATCHES] = { .type = NLA_NESTED }, [TCA_CGROUP_EMATCHES] = { .type = NLA_NESTED },
}; };
static int cls_cgroup_change(struct sk_buff *in_skb, static int cls_cgroup_change(struct net *net, struct sk_buff *in_skb,
struct tcf_proto *tp, unsigned long base, struct tcf_proto *tp, unsigned long base,
u32 handle, struct nlattr **tca, u32 handle, struct nlattr **tca,
unsigned long *arg) unsigned long *arg)
...@@ -215,7 +215,8 @@ static int cls_cgroup_change(struct sk_buff *in_skb, ...@@ -215,7 +215,8 @@ static int cls_cgroup_change(struct sk_buff *in_skb,
if (err < 0) if (err < 0)
return err; return err;
err = tcf_exts_validate(tp, tb, tca[TCA_RATE], &e, &cgroup_ext_map); err = tcf_exts_validate(net, tp, tb, tca[TCA_RATE], &e,
&cgroup_ext_map);
if (err < 0) if (err < 0)
return err; return err;
......
...@@ -351,7 +351,7 @@ static const struct nla_policy flow_policy[TCA_FLOW_MAX + 1] = { ...@@ -351,7 +351,7 @@ static const struct nla_policy flow_policy[TCA_FLOW_MAX + 1] = {
[TCA_FLOW_PERTURB] = { .type = NLA_U32 }, [TCA_FLOW_PERTURB] = { .type = NLA_U32 },
}; };
static int flow_change(struct sk_buff *in_skb, static int flow_change(struct net *net, struct sk_buff *in_skb,
struct tcf_proto *tp, unsigned long base, struct tcf_proto *tp, unsigned long base,
u32 handle, struct nlattr **tca, u32 handle, struct nlattr **tca,
unsigned long *arg) unsigned long *arg)
...@@ -397,7 +397,7 @@ static int flow_change(struct sk_buff *in_skb, ...@@ -397,7 +397,7 @@ static int flow_change(struct sk_buff *in_skb,
return -EOPNOTSUPP; return -EOPNOTSUPP;
} }
err = tcf_exts_validate(tp, tb, tca[TCA_RATE], &e, &flow_ext_map); err = tcf_exts_validate(net, tp, tb, tca[TCA_RATE], &e, &flow_ext_map);
if (err < 0) if (err < 0)
return err; return err;
......
...@@ -192,7 +192,7 @@ static const struct nla_policy fw_policy[TCA_FW_MAX + 1] = { ...@@ -192,7 +192,7 @@ static const struct nla_policy fw_policy[TCA_FW_MAX + 1] = {
}; };
static int static int
fw_change_attrs(struct tcf_proto *tp, struct fw_filter *f, fw_change_attrs(struct net *net, struct tcf_proto *tp, struct fw_filter *f,
struct nlattr **tb, struct nlattr **tca, unsigned long base) struct nlattr **tb, struct nlattr **tca, unsigned long base)
{ {
struct fw_head *head = (struct fw_head *)tp->root; struct fw_head *head = (struct fw_head *)tp->root;
...@@ -200,7 +200,7 @@ fw_change_attrs(struct tcf_proto *tp, struct fw_filter *f, ...@@ -200,7 +200,7 @@ fw_change_attrs(struct tcf_proto *tp, struct fw_filter *f,
u32 mask; u32 mask;
int err; int err;
err = tcf_exts_validate(tp, tb, tca[TCA_RATE], &e, &fw_ext_map); err = tcf_exts_validate(net, tp, tb, tca[TCA_RATE], &e, &fw_ext_map);
if (err < 0) if (err < 0)
return err; return err;
...@@ -233,7 +233,7 @@ fw_change_attrs(struct tcf_proto *tp, struct fw_filter *f, ...@@ -233,7 +233,7 @@ fw_change_attrs(struct tcf_proto *tp, struct fw_filter *f,
return err; return err;
} }
static int fw_change(struct sk_buff *in_skb, static int fw_change(struct net *net, struct sk_buff *in_skb,
struct tcf_proto *tp, unsigned long base, struct tcf_proto *tp, unsigned long base,
u32 handle, u32 handle,
struct nlattr **tca, struct nlattr **tca,
...@@ -255,7 +255,7 @@ static int fw_change(struct sk_buff *in_skb, ...@@ -255,7 +255,7 @@ static int fw_change(struct sk_buff *in_skb,
if (f != NULL) { if (f != NULL) {
if (f->id != handle && handle) if (f->id != handle && handle)
return -EINVAL; return -EINVAL;
return fw_change_attrs(tp, f, tb, tca, base); return fw_change_attrs(net, tp, f, tb, tca, base);
} }
if (!handle) if (!handle)
...@@ -282,7 +282,7 @@ static int fw_change(struct sk_buff *in_skb, ...@@ -282,7 +282,7 @@ static int fw_change(struct sk_buff *in_skb,
f->id = handle; f->id = handle;
err = fw_change_attrs(tp, f, tb, tca, base); err = fw_change_attrs(net, tp, f, tb, tca, base);
if (err < 0) if (err < 0)
goto errout; goto errout;
......
...@@ -335,8 +335,9 @@ static const struct nla_policy route4_policy[TCA_ROUTE4_MAX + 1] = { ...@@ -335,8 +335,9 @@ static const struct nla_policy route4_policy[TCA_ROUTE4_MAX + 1] = {
[TCA_ROUTE4_IIF] = { .type = NLA_U32 }, [TCA_ROUTE4_IIF] = { .type = NLA_U32 },
}; };
static int route4_set_parms(struct tcf_proto *tp, unsigned long base, static int route4_set_parms(struct net *net, struct tcf_proto *tp,
struct route4_filter *f, u32 handle, struct route4_head *head, unsigned long base, struct route4_filter *f,
u32 handle, struct route4_head *head,
struct nlattr **tb, struct nlattr *est, int new) struct nlattr **tb, struct nlattr *est, int new)
{ {
int err; int err;
...@@ -346,7 +347,7 @@ static int route4_set_parms(struct tcf_proto *tp, unsigned long base, ...@@ -346,7 +347,7 @@ static int route4_set_parms(struct tcf_proto *tp, unsigned long base,
struct route4_bucket *b; struct route4_bucket *b;
struct tcf_exts e; struct tcf_exts e;
err = tcf_exts_validate(tp, tb, est, &e, &route_ext_map); err = tcf_exts_validate(net, tp, tb, est, &e, &route_ext_map);
if (err < 0) if (err < 0)
return err; return err;
...@@ -427,7 +428,7 @@ static int route4_set_parms(struct tcf_proto *tp, unsigned long base, ...@@ -427,7 +428,7 @@ static int route4_set_parms(struct tcf_proto *tp, unsigned long base,
return err; return err;
} }
static int route4_change(struct sk_buff *in_skb, static int route4_change(struct net *net, struct sk_buff *in_skb,
struct tcf_proto *tp, unsigned long base, struct tcf_proto *tp, unsigned long base,
u32 handle, u32 handle,
struct nlattr **tca, struct nlattr **tca,
...@@ -457,7 +458,7 @@ static int route4_change(struct sk_buff *in_skb, ...@@ -457,7 +458,7 @@ static int route4_change(struct sk_buff *in_skb,
if (f->bkt) if (f->bkt)
old_handle = f->handle; old_handle = f->handle;
err = route4_set_parms(tp, base, f, handle, head, tb, err = route4_set_parms(net, tp, base, f, handle, head, tb,
tca[TCA_RATE], 0); tca[TCA_RATE], 0);
if (err < 0) if (err < 0)
return err; return err;
...@@ -480,7 +481,7 @@ static int route4_change(struct sk_buff *in_skb, ...@@ -480,7 +481,7 @@ static int route4_change(struct sk_buff *in_skb,
if (f == NULL) if (f == NULL)
goto errout; goto errout;
err = route4_set_parms(tp, base, f, handle, head, tb, err = route4_set_parms(net, tp, base, f, handle, head, tb,
tca[TCA_RATE], 1); tca[TCA_RATE], 1);
if (err < 0) if (err < 0)
goto errout; goto errout;
......
...@@ -416,7 +416,7 @@ static const struct nla_policy rsvp_policy[TCA_RSVP_MAX + 1] = { ...@@ -416,7 +416,7 @@ static const struct nla_policy rsvp_policy[TCA_RSVP_MAX + 1] = {
[TCA_RSVP_PINFO] = { .len = sizeof(struct tc_rsvp_pinfo) }, [TCA_RSVP_PINFO] = { .len = sizeof(struct tc_rsvp_pinfo) },
}; };
static int rsvp_change(struct sk_buff *in_skb, static int rsvp_change(struct net *net, struct sk_buff *in_skb,
struct tcf_proto *tp, unsigned long base, struct tcf_proto *tp, unsigned long base,
u32 handle, u32 handle,
struct nlattr **tca, struct nlattr **tca,
...@@ -440,7 +440,7 @@ static int rsvp_change(struct sk_buff *in_skb, ...@@ -440,7 +440,7 @@ static int rsvp_change(struct sk_buff *in_skb,
if (err < 0) if (err < 0)
return err; return err;
err = tcf_exts_validate(tp, tb, tca[TCA_RATE], &e, &rsvp_ext_map); err = tcf_exts_validate(net, tp, tb, tca[TCA_RATE], &e, &rsvp_ext_map);
if (err < 0) if (err < 0)
return err; return err;
......
...@@ -197,9 +197,10 @@ static const struct nla_policy tcindex_policy[TCA_TCINDEX_MAX + 1] = { ...@@ -197,9 +197,10 @@ static const struct nla_policy tcindex_policy[TCA_TCINDEX_MAX + 1] = {
}; };
static int static int
tcindex_set_parms(struct tcf_proto *tp, unsigned long base, u32 handle, tcindex_set_parms(struct net *net, struct tcf_proto *tp, unsigned long base,
struct tcindex_data *p, struct tcindex_filter_result *r, u32 handle, struct tcindex_data *p,
struct nlattr **tb, struct nlattr *est) struct tcindex_filter_result *r, struct nlattr **tb,
struct nlattr *est)
{ {
int err, balloc = 0; int err, balloc = 0;
struct tcindex_filter_result new_filter_result, *old_r = r; struct tcindex_filter_result new_filter_result, *old_r = r;
...@@ -208,7 +209,7 @@ tcindex_set_parms(struct tcf_proto *tp, unsigned long base, u32 handle, ...@@ -208,7 +209,7 @@ tcindex_set_parms(struct tcf_proto *tp, unsigned long base, u32 handle,
struct tcindex_filter *f = NULL; /* make gcc behave */ struct tcindex_filter *f = NULL; /* make gcc behave */
struct tcf_exts e; struct tcf_exts e;
err = tcf_exts_validate(tp, tb, est, &e, &tcindex_ext_map); err = tcf_exts_validate(net, tp, tb, est, &e, &tcindex_ext_map);
if (err < 0) if (err < 0)
return err; return err;
...@@ -332,7 +333,7 @@ tcindex_set_parms(struct tcf_proto *tp, unsigned long base, u32 handle, ...@@ -332,7 +333,7 @@ tcindex_set_parms(struct tcf_proto *tp, unsigned long base, u32 handle,
} }
static int static int
tcindex_change(struct sk_buff *in_skb, tcindex_change(struct net *net, struct sk_buff *in_skb,
struct tcf_proto *tp, unsigned long base, u32 handle, struct tcf_proto *tp, unsigned long base, u32 handle,
struct nlattr **tca, unsigned long *arg) struct nlattr **tca, unsigned long *arg)
{ {
...@@ -353,7 +354,8 @@ tcindex_change(struct sk_buff *in_skb, ...@@ -353,7 +354,8 @@ tcindex_change(struct sk_buff *in_skb,
if (err < 0) if (err < 0)
return err; return err;
return tcindex_set_parms(tp, base, handle, p, r, tb, tca[TCA_RATE]); return tcindex_set_parms(net, tp, base, handle, p, r, tb,
tca[TCA_RATE]);
} }
......
...@@ -488,15 +488,15 @@ static const struct nla_policy u32_policy[TCA_U32_MAX + 1] = { ...@@ -488,15 +488,15 @@ static const struct nla_policy u32_policy[TCA_U32_MAX + 1] = {
[TCA_U32_MARK] = { .len = sizeof(struct tc_u32_mark) }, [TCA_U32_MARK] = { .len = sizeof(struct tc_u32_mark) },
}; };
static int u32_set_parms(struct tcf_proto *tp, unsigned long base, static int u32_set_parms(struct net *net, struct tcf_proto *tp,
struct tc_u_hnode *ht, unsigned long base, struct tc_u_hnode *ht,
struct tc_u_knode *n, struct nlattr **tb, struct tc_u_knode *n, struct nlattr **tb,
struct nlattr *est) struct nlattr *est)
{ {
int err; int err;
struct tcf_exts e; struct tcf_exts e;
err = tcf_exts_validate(tp, tb, est, &e, &u32_ext_map); err = tcf_exts_validate(net, tp, tb, est, &e, &u32_ext_map);
if (err < 0) if (err < 0)
return err; return err;
...@@ -544,7 +544,7 @@ static int u32_set_parms(struct tcf_proto *tp, unsigned long base, ...@@ -544,7 +544,7 @@ static int u32_set_parms(struct tcf_proto *tp, unsigned long base,
return err; return err;
} }
static int u32_change(struct sk_buff *in_skb, static int u32_change(struct net *net, struct sk_buff *in_skb,
struct tcf_proto *tp, unsigned long base, u32 handle, struct tcf_proto *tp, unsigned long base, u32 handle,
struct nlattr **tca, struct nlattr **tca,
unsigned long *arg) unsigned long *arg)
...@@ -570,7 +570,8 @@ static int u32_change(struct sk_buff *in_skb, ...@@ -570,7 +570,8 @@ static int u32_change(struct sk_buff *in_skb,
if (TC_U32_KEY(n->handle) == 0) if (TC_U32_KEY(n->handle) == 0)
return -EINVAL; return -EINVAL;
return u32_set_parms(tp, base, n->ht_up, n, tb, tca[TCA_RATE]); return u32_set_parms(net, tp, base, n->ht_up, n, tb,
tca[TCA_RATE]);
} }
if (tb[TCA_U32_DIVISOR]) { if (tb[TCA_U32_DIVISOR]) {
...@@ -656,7 +657,7 @@ static int u32_change(struct sk_buff *in_skb, ...@@ -656,7 +657,7 @@ static int u32_change(struct sk_buff *in_skb,
} }
#endif #endif
err = u32_set_parms(tp, base, ht, n, tb, tca[TCA_RATE]); err = u32_set_parms(net, tp, base, ht, n, tb, tca[TCA_RATE]);
if (err == 0) { if (err == 0) {
struct tc_u_knode **ins; struct tc_u_knode **ins;
for (ins = &ht->ht[TC_U32_HASH(handle)]; *ins; ins = &(*ins)->next) for (ins = &ht->ht[TC_U32_HASH(handle)]; *ins; ins = &(*ins)->next)
......
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