taprio: boolean values to a bool variable
authorJiapeng Zhong <abaci-bugfix@linux.alibaba.com>
Mon, 18 Jan 2021 08:31:02 +0000 (16:31 +0800)
committerJakub Kicinski <kuba@kernel.org>
Wed, 20 Jan 2021 01:43:48 +0000 (17:43 -0800)
Fix the following coccicheck warnings:

./net/sched/sch_taprio.c:393:3-16: WARNING: Assignment of 0/1 to bool
variable.

./net/sched/sch_taprio.c:375:2-15: WARNING: Assignment of 0/1 to bool
variable.

./net/sched/sch_taprio.c:244:4-19: WARNING: Assignment of 0/1 to bool
variable.

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Signed-off-by: Jiapeng Zhong <abaci-bugfix@linux.alibaba.com>
Link: https://lore.kernel.org/r/1610958662-71166-1-git-send-email-abaci-bugfix@linux.alibaba.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/sched/sch_taprio.c

index 6f77527..8287894 100644 (file)
@@ -241,7 +241,7 @@ static struct sched_entry *find_entry_to_transmit(struct sk_buff *skb,
                                /* Here, we are just trying to find out the
                                 * first available interval in the next cycle.
                                 */
-                               entry_available = 1;
+                               entry_available = true;
                                entry_found = entry;
                                *interval_start = ktime_add_ns(curr_intv_start, cycle);
                                *interval_end = ktime_add_ns(curr_intv_end, cycle);
@@ -372,7 +372,7 @@ static long get_packet_txtime(struct sk_buff *skb, struct Qdisc *sch)
        packet_transmit_time = length_to_duration(q, len);
 
        do {
-               sched_changed = 0;
+               sched_changed = false;
 
                entry = find_entry_to_transmit(skb, sch, sched, admin,
                                               minimum_time,
@@ -390,7 +390,7 @@ static long get_packet_txtime(struct sk_buff *skb, struct Qdisc *sch)
                if (admin && admin != sched &&
                    ktime_after(txtime, admin->base_time)) {
                        sched = admin;
-                       sched_changed = 1;
+                       sched_changed = true;
                        continue;
                }