All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick McHardy <kaber@trash.net>
To: Michal Soltys <soltys@ziu.info>
Cc: davem@davemloft.net, netdev@vger.kernel.org
Subject: Re: [PATCH/RFC 00/11] HFSC patches
Date: Sat, 05 Nov 2011 04:48:00 +0100	[thread overview]
Message-ID: <4EB4B1F0.20404@trash.net> (raw)
In-Reply-To: <1320460377-8682-1-git-send-email-soltys@ziu.info>

On 05.11.2011 03:32, Michal Soltys wrote:
> Those are most of the patches I've been sitting on for a while. For the most
> part they are either small corrections or simplifications (marked with s) - but
> there are also some new things (marked with !), the most interesting being #8
> probably.
> 
> All changes are richly commented in respective commit messages. I've been using
> them for a while - so unless I missed some subtlety, all should be fine.

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.

> 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.

  parent reply	other threads:[~2011-11-05  3:56 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 ` Patrick McHardy [this message]
2011-11-05 17:43   ` [PATCH/RFC 00/11] HFSC patches Michal Soltys

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=4EB4B1F0.20404@trash.net \
    --to=kaber@trash.net \
    --cc=davem@davemloft.net \
    --cc=netdev@vger.kernel.org \
    --cc=soltys@ziu.info \
    /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.