[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <1386658768-29584-1-git-send-email-yangyingliang@huawei.com>
Date: Tue, 10 Dec 2013 14:59:26 +0800
From: Yang Yingliang <yangyingliang@...wei.com>
To: <davem@...emloft.net>, <netdev@...r.kernel.org>
CC: <eric.dumazet@...il.com>, <brouer@...hat.com>, <jpirko@...hat.com>,
<jbrouer@...hat.com>
Subject: [PATCH net v7 0/2] net: sched: fix two issues
fix calculation of max_size in tbf
fix quantum calculation introduced by 64bit rates.
v7
patch 1/2:
1.Set members of "struct tbf_sched_data *q" after calculating
max_size successfully, or it will change the old settings while
calculating failed.
2.If time_in_ns is bigger than ~0U(about 4 sec), it's useless,
use ~0U instead.
v6
patch 1/2:
1.Use a simpler and better way to calculate length in psched_ns_t2l().
2.Replace pr_err() with pr_warn_ratelimited().
patch 2/2: Replace div64_u64() with do_div() suggested by Eric.
v5
patch 1/2:
1.Remove rtab and ptab use suggested by Eric.
2.Don't reduce max_size to 65536, as Eric suggested that
if a burst set to 200KB, we do not want tbf use 64KB
or even less. So add a helper psched_ns_t2l to calculate
max_size directly.
v4
patch 1/2:
1.Update commit message suggested by Jesper.
2.Use a macro to replace 65535 constant.
3.Add condition that when peakrate is lower than rate, return -EINVAL.
4.Don't use cell_log anymore.
v3
patch 1/2: use psched_l2t_ns to calculate max_size
and cleanup exit/done section suggested by Jesper.
v2
patch 1/2: redescribe the regression.
patch 2/2: add Eric's ack.
Yang Yingliang (2):
net: sched: tbf: fix the calculation of max_size
net: sched: htb: fix the calculation of quantum
net/sched/sch_htb.c | 20 +++++----
net/sched/sch_tbf.c | 115 ++++++++++++++++++++++++++++++++--------------------
2 files changed, 82 insertions(+), 53 deletions(-)
--
1.8.0
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists