From: Michal Soltys <soltys@ziu.info>
To: Patrick McHardy <kaber@trash.net>
Cc: davem@davemloft.net, netdev@vger.kernel.org
Subject: Re: [PATCH/RFC 00/11] HFSC patches
Date: Sat, 05 Nov 2011 18:43:11 +0100 [thread overview]
Message-ID: <4EB575AF.1010903@ziu.info> (raw)
In-Reply-To: <4EB4B1F0.20404@trash.net>
On 11-11-05 04:48, Patrick McHardy wrote:
>
> Thanks Michal. It has been quite a while since I've last looked
> at this and this is complicated stuff, please give me a few days
> to review your patches.
Of course, and thanks for doing it. If I have any corrections, I'll add
them as v2 versions in this thread (under respective patches).
>
>> Apart from these, there's still one subtle thing to do w.r.t. cl_cvtmin (during
>> init_vf(), as this value is lagged relatively to the situation at the time of
>> enqueue).
>>
>> On a side note, I was thinking about something like hfsc-strict or so - where
>> [uplink] interface could be upperlimited on hfsc qdisc level, but all the class
>> upperlimit would be otherwise gone. Not sure if anyone would be even interested
>> in something like that at all.
>
> So classes would just use link-sharing curves? That's
> already possible, so I probably don't get your idea.
>
I mean, that upperlimit's main use is for matching [upstream] router's
capability (as far as I've always seen this). Other scenarios where
upperlimit is used somewhere lower, can be transformed to just proper
linksharing ratios and realtime leaves w/o linksharking part (if
applicable) - so thus the idea of no upperlimit at class level at all
(and related code), but ability to define one at qdisc level (added
during tc qdisc add hfsc ...) and executed during hfsc_dequeue().
Note - this is just a purist's idea, and I realize unacceptable in
context of existing hfsc scheduler for many reasons (compatibility with
exisiting configurations for once). But the idea about
hfsc-{light,strict,pure,etc.} has been crawling in my head for a while.
Apart from that - in the sch_hfsc.c code there're few things you once
commented out - related to myf adjustments that "overshoot" and made
classes stay way too much under their respective linksharing curves. Do
you have the configuration examples saved somewhere, under which it
happened ?
prev parent reply other threads:[~2011-11-05 17:43 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-05 2:32 [PATCH/RFC 00/11] HFSC patches Michal Soltys
2011-11-05 2:32 ` [PATCH 01/11] sch_hfsc.c: update_d() fixup Michal Soltys
2011-11-05 2:32 ` [PATCH 02/11] sch_hfsc.c: go passive after cl_vt update in update_vf() Michal Soltys
2011-11-05 2:32 ` [PATCH 03/11] sch_hfsc.c: rtsc_min() convex/convex case fixup Michal Soltys
2011-11-05 2:32 ` [PATCH 04/11] sch_hfsc.c: remove initial check in vttree_get_minvt() (optional) Michal Soltys
2011-11-05 2:32 ` [PATCH 05/11] sch_hfsc.c: don't subtract from cl_vtoff and cl_cvtoff Michal Soltys
2011-11-05 2:32 ` [PATCH 06/11] sch_hfsc.c: seg_y2x(), rtsc_y2x(), hfsc_change_class() minor changes Michal Soltys
2011-11-05 2:32 ` [PATCH 07/11] sch_hfsc.c: make cl_vt keep all periods in itself Michal Soltys
2011-11-05 2:32 ` [PATCH 08/11] sch_hfsc.c: update speed table, add explanations, increase accuracy Michal Soltys
2011-11-06 1:09 ` [PATCH 12/11] sch_hfsc.c: converter fixups Michal Soltys
2011-11-05 2:32 ` [PATCH 09/11] sch_hfsc.c: split update_vf() Michal Soltys
2011-11-05 2:32 ` [PATCH 10/11] sch_hfsc.c: make sure classes are able to use 1st segment on fresh backlog period Michal Soltys
2011-11-05 2:32 ` [PATCH 11/11] sch_hfsc.c: remove cl_vtadj, shift virtual curve instead Michal Soltys
2011-11-05 3:48 ` [PATCH/RFC 00/11] HFSC patches Patrick McHardy
2011-11-05 17:43 ` Michal Soltys [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4EB575AF.1010903@ziu.info \
--to=soltys@ziu.info \
--cc=davem@davemloft.net \
--cc=kaber@trash.net \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.