mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Uwaysi Bin Kareem <uwaysi.bin.kareem@paradoxuncreated.com>
To: Mike Galbraith <efault@gmx.de>
Subject: Re: The 10ms averager in fair.c
Date: Mon, 01 Oct 2012 15:24:28 +0200	[thread overview]
Message-ID: <op.wlh4u2xj6426ze@millennium.getinternet.no> (raw)
In-Reply-To: <1349064397.6957.26.camel@marge.simpson.net>

On Mon, 01 Oct 2012 06:06:37 +0200, Mike Galbraith <efault@gmx.de> wrote:

> On Sun, 2012-09-30 at 13:44 +0200, Uwaysi Bin Kareem wrote:
>> Hiya. I just had an initial look at fair.c
>>
>> There seems to be a 10ms averager in there?
>>
>> You are aware that that means you work on delayed values?
>>
>> Isn`t that counterintuitive to the principle of sharing?
>
> Not if you want to be able to use lots of groups, and still do something
> other than in-kernel arithmetic.
>
> -Mike
>

"Use lots of groups"? I don`t even see the point with that. Currently  
doesn`t cfs manipulate nice levels? If you set constant nice levels, and  
remove the filter, things will work more as expected. High nice value,  
should be short slice. That is your "group", for instance "low priority  
stuff".

That a filter filters out the initial cpu spike, only to starve it and  
elevate the priority later, is silly. That is delayed execution.

Now I haven`t looked that close at the whole scheduler yet, but no.. I  
can`t possibly think what a filter does in there, that smears at 100uS  
spike, over 10ms.

Btw, I did set it to 1ns, and it only improved things. So that it should  
have some function seems odd to me.

Are you sure this isn`t just a design-philosophy that was done, without  
much knowledge of filters?

Peace Be With You.

  parent reply	other threads:[~2012-10-01 15:24 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-30 11:44 Uwaysi Bin Kareem
2012-09-30 19:18 ` Uwaysi Bin Kareem
     [not found] ` <1349064397.6957.26.camel@marge.simpson.net>
2012-10-01 13:24   ` Uwaysi Bin Kareem [this message]
     [not found]     ` <1349146202.7086.23.camel@marge.simpson.net>
2012-10-02  7:04       ` Uwaysi Bin Kareem
2012-10-02 17:25       ` The 10ms averager in fair.c + granularity Uwaysi Bin Kareem
2012-10-03 15:14       ` Uwaysi Bin Kareem
2012-10-04 12:20         ` el es
2012-10-03 20:26       ` Doom 3 perfect on linux Uwaysi Bin Kareem
2012-10-04 10:23       ` RME Fireface UCX in Classcompliant USB-mode, is not working? Uwaysi Bin Kareem
2012-10-05 19:54       ` The 10ms averager in fair.c Uwaysi Bin Kareem
2012-10-02  6:56   ` Uwaysi Bin Kareem
     [not found]     ` <1349169555.7086.48.camel@marge.simpson.net>
2012-10-02  8:07       ` Uwaysi Bin Kareem
     [not found]         ` <1349176973.7086.96.camel@marge.simpson.net>
2012-10-02  9:28           ` Uwaysi Bin Kareem
2012-09-30 12:16 Uwaysi Bin Kareem

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=op.wlh4u2xj6426ze@millennium.getinternet.no \
    --to=uwaysi.bin.kareem@paradoxuncreated.com \
    --cc=efault@gmx.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®