On Thu, Apr 15, 2021 at 4:17 PM Du Cheng <duche...@gmail.com> wrote: > diff --git a/net/sched/sch_taprio.c b/net/sched/sch_taprio.c > index 8287894541e3..abd6b176383c 100644 > --- a/net/sched/sch_taprio.c > +++ b/net/sched/sch_taprio.c > @@ -901,6 +901,10 @@ static int parse_taprio_schedule(struct taprio_sched *q, > struct nlattr **tb, > > list_for_each_entry(entry, &new->entries, list) > cycle = ktime_add_ns(cycle, entry->interval); > + > + if (!cycle) > + return -EINVAL;
Just a nit: please add an extack to explain why we return EINVAL here. Thanks.