Skip to content

Commit

Permalink
net: sched: pie: fix commenting
Browse files Browse the repository at this point in the history
Fix punctuation and logical mistakes in the comments. The
logical mistake was that "dequeue_rate" is no longer the default
way to calculate queuing delay and is not needed. The default
way to calculate queue delay was changed in commit cec2975
("net: sched: pie: enable timestamp based delay calculation").

Signed-off-by: Mohit P. Tahiliani <tahiliani@nitk.edu.in>
Signed-off-by: Leslie Monis <lesliemonis@gmail.com>
Signed-off-by: Gautam Ramakrishnan <gautamramk@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Mohit P. Tahiliani authored and David S. Miller committed Jan 23, 2020
1 parent b42a3d7 commit 55f780c
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions net/sched/sch_pie.c
Original file line number Diff line number Diff line change
Expand Up @@ -248,10 +248,10 @@ static void pie_process_dequeue(struct Qdisc *sch, struct sk_buff *skb)
q->vars.dq_count = 0;
}

/* Calculate the average drain rate from this value. If queue length
* has receded to a small value viz., <= QUEUE_THRESHOLD bytes,reset
/* Calculate the average drain rate from this value. If queue length
* has receded to a small value viz., <= QUEUE_THRESHOLD bytes, reset
* the dq_count to -1 as we don't have enough packets to calculate the
* drain rate anymore The following if block is entered only when we
* drain rate anymore. The following if block is entered only when we
* have a substantial queue built up (QUEUE_THRESHOLD bytes or more)
* and we calculate the drain rate for the threshold here. dq_count is
* in bytes, time difference in psched_time, hence rate is in
Expand Down Expand Up @@ -329,8 +329,8 @@ static void calculate_probability(struct Qdisc *sch)
qdelay_old = q->vars.qdelay_old;
}

/* If qdelay is zero and qlen is not, it means qlen is very small, less
* than dequeue_rate, so we do not update probabilty in this round
/* If qdelay is zero and qlen is not, it means qlen is very small,
* so we do not update probabilty in this round.
*/
if (qdelay == 0 && qlen != 0)
update_prob = false;
Expand Down

0 comments on commit 55f780c

Please sign in to comment.