net: sched: convert tasklets to use new tasklet_setup() API
authorAllen Pais <apais@linux.microsoft.com>
Tue, 3 Nov 2020 09:18:21 +0000 (14:48 +0530)
committerJakub Kicinski <kuba@kernel.org>
Sat, 7 Nov 2020 18:41:15 +0000 (10:41 -0800)
In preparation for unconditionally passing the
struct tasklet_struct pointer to all tasklet
callbacks, switch to using the new tasklet_setup()
and from_tasklet() to pass the tasklet pointer explicitly.

Signed-off-by: Romain Perier <romain.perier@gmail.com>
Signed-off-by: Allen Pais <apais@linux.microsoft.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
include/net/pkt_sched.h
net/sched/sch_atm.c

index 4ed32e6b020145afb015c3c07d2ec3a613f1311d..15b1b30f454e4837cd1fc07bb3ff6b4f178b1d39 100644 (file)
@@ -24,6 +24,11 @@ static inline void *qdisc_priv(struct Qdisc *q)
        return &q->privdata;
 }
 
+static inline struct Qdisc *qdisc_from_priv(void *priv)
+{
+       return container_of(priv, struct Qdisc, privdata);
+}
+
 /* 
    Timer resolution MUST BE < 10% of min_schedulable_packet_size/bandwidth
    
index 1c281cc81f57789b787c77622cc2c026414f9c9d..007bd2d9f1ff970bfdfce45331b892936987c956 100644 (file)
@@ -466,10 +466,10 @@ drop: __maybe_unused
  * non-ATM interfaces.
  */
 
-static void sch_atm_dequeue(unsigned long data)
+static void sch_atm_dequeue(struct tasklet_struct *t)
 {
-       struct Qdisc *sch = (struct Qdisc *)data;
-       struct atm_qdisc_data *p = qdisc_priv(sch);
+       struct atm_qdisc_data *p = from_tasklet(p, t, task);
+       struct Qdisc *sch = qdisc_from_priv(p);
        struct atm_flow_data *flow;
        struct sk_buff *skb;
 
@@ -563,7 +563,7 @@ static int atm_tc_init(struct Qdisc *sch, struct nlattr *opt,
        if (err)
                return err;
 
-       tasklet_init(&p->task, sch_atm_dequeue, (unsigned long)sch);
+       tasklet_setup(&p->task, sch_atm_dequeue);
        return 0;
 }