Commit 9382a506 authored by David S. Miller's avatar David S. Miller

Merge branch 'sch_api-style'

Alexander Aring says:

====================
net: sched: sch_api: fix coding style issues for extack

this patch prepares to handle extack for qdiscs and fixes checkpatch
issues.

There are a bunch of warnings issued by checkpatch which bothered me.
This first patchset is to get rid of those warnings to make way for
the next patchsets.

I plan to followup with qdiscs, classifiers and actions after this.
====================
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents 045203e9 54160ef6
...@@ -161,7 +161,8 @@ struct Qdisc_class_ops { ...@@ -161,7 +161,8 @@ struct Qdisc_class_ops {
void (*walk)(struct Qdisc *, struct qdisc_walker * arg); void (*walk)(struct Qdisc *, struct qdisc_walker * arg);
/* Filter manipulation */ /* Filter manipulation */
struct tcf_block * (*tcf_block)(struct Qdisc *, unsigned long); struct tcf_block * (*tcf_block)(struct Qdisc *sch,
unsigned long arg);
unsigned long (*bind_tcf)(struct Qdisc *, unsigned long, unsigned long (*bind_tcf)(struct Qdisc *, unsigned long,
u32 classid); u32 classid);
void (*unbind_tcf)(struct Qdisc *, unsigned long); void (*unbind_tcf)(struct Qdisc *, unsigned long);
...@@ -185,11 +186,12 @@ struct Qdisc_ops { ...@@ -185,11 +186,12 @@ struct Qdisc_ops {
struct sk_buff * (*dequeue)(struct Qdisc *); struct sk_buff * (*dequeue)(struct Qdisc *);
struct sk_buff * (*peek)(struct Qdisc *); struct sk_buff * (*peek)(struct Qdisc *);
int (*init)(struct Qdisc *, struct nlattr *arg); int (*init)(struct Qdisc *sch, struct nlattr *arg);
void (*reset)(struct Qdisc *); void (*reset)(struct Qdisc *);
void (*destroy)(struct Qdisc *); void (*destroy)(struct Qdisc *);
int (*change)(struct Qdisc *, struct nlattr *arg); int (*change)(struct Qdisc *sch,
void (*attach)(struct Qdisc *); struct nlattr *arg);
void (*attach)(struct Qdisc *sch);
int (*dump)(struct Qdisc *, struct sk_buff *); int (*dump)(struct Qdisc *, struct sk_buff *);
int (*dump_stats)(struct Qdisc *, struct gnet_dump *); int (*dump_stats)(struct Qdisc *, struct gnet_dump *);
......
...@@ -1020,7 +1020,7 @@ static struct Qdisc *qdisc_create(struct net_device *dev, ...@@ -1020,7 +1020,7 @@ static struct Qdisc *qdisc_create(struct net_device *dev,
#endif #endif
err = -ENOENT; err = -ENOENT;
if (ops == NULL) if (!ops)
goto err_out; goto err_out;
sch = qdisc_alloc(dev_queue, ops); sch = qdisc_alloc(dev_queue, ops);
...@@ -1060,54 +1060,60 @@ static struct Qdisc *qdisc_create(struct net_device *dev, ...@@ -1060,54 +1060,60 @@ static struct Qdisc *qdisc_create(struct net_device *dev,
netdev_info(dev, "Caught tx_queue_len zero misconfig\n"); netdev_info(dev, "Caught tx_queue_len zero misconfig\n");
} }
if (!ops->init || (err = ops->init(sch, tca[TCA_OPTIONS])) == 0) { if (ops->init) {
if (qdisc_is_percpu_stats(sch)) { err = ops->init(sch, tca[TCA_OPTIONS]);
sch->cpu_bstats = if (err != 0)
netdev_alloc_pcpu_stats(struct gnet_stats_basic_cpu); goto err_out5;
if (!sch->cpu_bstats) }
goto err_out4;
sch->cpu_qstats = alloc_percpu(struct gnet_stats_queue);
if (!sch->cpu_qstats)
goto err_out4;
}
if (tca[TCA_STAB]) { if (qdisc_is_percpu_stats(sch)) {
stab = qdisc_get_stab(tca[TCA_STAB]); sch->cpu_bstats =
if (IS_ERR(stab)) { netdev_alloc_pcpu_stats(struct gnet_stats_basic_cpu);
err = PTR_ERR(stab); if (!sch->cpu_bstats)
goto err_out4; goto err_out4;
}
rcu_assign_pointer(sch->stab, stab);
}
if (tca[TCA_RATE]) {
seqcount_t *running;
err = -EOPNOTSUPP; sch->cpu_qstats = alloc_percpu(struct gnet_stats_queue);
if (sch->flags & TCQ_F_MQROOT) if (!sch->cpu_qstats)
goto err_out4; goto err_out4;
}
if ((sch->parent != TC_H_ROOT) && if (tca[TCA_STAB]) {
!(sch->flags & TCQ_F_INGRESS) && stab = qdisc_get_stab(tca[TCA_STAB]);
(!p || !(p->flags & TCQ_F_MQROOT))) if (IS_ERR(stab)) {
running = qdisc_root_sleeping_running(sch); err = PTR_ERR(stab);
else goto err_out4;
running = &sch->running;
err = gen_new_estimator(&sch->bstats,
sch->cpu_bstats,
&sch->rate_est,
NULL,
running,
tca[TCA_RATE]);
if (err)
goto err_out4;
} }
rcu_assign_pointer(sch->stab, stab);
}
if (tca[TCA_RATE]) {
seqcount_t *running;
qdisc_hash_add(sch, false); err = -EOPNOTSUPP;
if (sch->flags & TCQ_F_MQROOT)
goto err_out4;
return sch; if (sch->parent != TC_H_ROOT &&
!(sch->flags & TCQ_F_INGRESS) &&
(!p || !(p->flags & TCQ_F_MQROOT)))
running = qdisc_root_sleeping_running(sch);
else
running = &sch->running;
err = gen_new_estimator(&sch->bstats,
sch->cpu_bstats,
&sch->rate_est,
NULL,
running,
tca[TCA_RATE]);
if (err)
goto err_out4;
} }
qdisc_hash_add(sch, false);
return sch;
err_out5:
/* ops->init() failed, we call ->destroy() like qdisc_create_dflt() */ /* ops->init() failed, we call ->destroy() like qdisc_create_dflt() */
if (ops->destroy) if (ops->destroy)
ops->destroy(sch); ops->destroy(sch);
...@@ -1139,7 +1145,7 @@ static int qdisc_change(struct Qdisc *sch, struct nlattr **tca) ...@@ -1139,7 +1145,7 @@ static int qdisc_change(struct Qdisc *sch, struct nlattr **tca)
int err = 0; int err = 0;
if (tca[TCA_OPTIONS]) { if (tca[TCA_OPTIONS]) {
if (sch->ops->change == NULL) if (!sch->ops->change)
return -EINVAL; return -EINVAL;
err = sch->ops->change(sch, tca[TCA_OPTIONS]); err = sch->ops->change(sch, tca[TCA_OPTIONS]);
if (err) if (err)
...@@ -1344,7 +1350,8 @@ static int tc_modify_qdisc(struct sk_buff *skb, struct nlmsghdr *n, ...@@ -1344,7 +1350,8 @@ static int tc_modify_qdisc(struct sk_buff *skb, struct nlmsghdr *n,
goto create_n_graft; goto create_n_graft;
if (n->nlmsg_flags & NLM_F_EXCL) if (n->nlmsg_flags & NLM_F_EXCL)
return -EEXIST; return -EEXIST;
if (tca[TCA_KIND] && nla_strcmp(tca[TCA_KIND], q->ops->id)) if (tca[TCA_KIND] &&
nla_strcmp(tca[TCA_KIND], q->ops->id))
return -EINVAL; return -EINVAL;
if (q == p || if (q == p ||
(p && check_loop(q, p, 0))) (p && check_loop(q, p, 0)))
...@@ -1389,7 +1396,7 @@ static int tc_modify_qdisc(struct sk_buff *skb, struct nlmsghdr *n, ...@@ -1389,7 +1396,7 @@ static int tc_modify_qdisc(struct sk_buff *skb, struct nlmsghdr *n,
} }
/* Change qdisc parameters */ /* Change qdisc parameters */
if (q == NULL) if (!q)
return -ENOENT; return -ENOENT;
if (n->nlmsg_flags & NLM_F_EXCL) if (n->nlmsg_flags & NLM_F_EXCL)
return -EEXIST; return -EEXIST;
......
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