All of lore.kernel.org
 help / color / mirror / Atom feed
From: NISHIGUCHI Naoki <nisiguti@jp.fujitsu.com>
To: "Tian, Kevin" <kevin.tian@intel.com>,
	"Su, Disheng" <disheng.su@intel.com>,
	"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Cc: George Dunlap <George.Dunlap@eu.citrix.com>,
	"Ian.Pratt@eu.citrix.com" <Ian.Pratt@eu.citrix.com>,
	"aviv@neocleus.com" <aviv@neocleus.com>,
	"keir.fraser@eu.citrix.com" <keir.fraser@eu.citrix.com>,
	"sakaia@jp.fujitsu.com" <sakaia@jp.fujitsu.com>
Subject: Re: RE: [RFC][PATCH 0/4] Modification of credit	scheduler rev2
Date: Thu, 15 Jan 2009 13:42:52 +0900	[thread overview]
Message-ID: <496EBECC.8020405@jp.fujitsu.com> (raw)
In-Reply-To: <0A882F4D99BBF6449D58E61AAFD7EDD603BB4AB4@pdsmsx502.ccr.corp.intel.com>

Hi, Kevin

Tian, Kevin wrote:
>> From:NISHIGUCHI Naoki
>> Sent: Thursday, January 15, 2009 10:05 AM
>>> 	4. issues left:
>>> 		a. Abrupt glitches are still generated when the 
>> QEMU emulated mouse being used and moving mouse quickly in 
>> guest A. Passing-through USB mouse/keyboard to guest A, then 
>> no glitches.
>>
>> I also noticed that. Though I don't know the precise cause, I 
>> found that 
>> dom0 and guest A would consume largely CPU time (hundreds of 
>> milliseconds) in such situation. In this case, the priority of 
>> dom0 and 
>> guest A falls rapidly, then guest B runs until the priority of 
>> dom0 and 
>> guest A becomes BOOST. In worst case, it will take about 120ms.
> 
> I remember that Disheng once told me that BOOST only happens
> when vcpu is waken up and its current priority is UNDER. In your
> case guest A should be in OVER after running hundreds of ms, 
> and then it waits enough long time to become UNDER and then 
> BOOST. If this is the case, your enhancement on BOOST level
> seems only solving part of the latency issue. Here either assigning
> a static priority, or adding more BOOST source (like event, intr,
> etc) seems more complete solution.

In my case, though the vcpu should be switched to other vcpu in time 
slice, the cpu running the vcpu doesn't schedule during hundreds of ms. 
I don't know why this happens.
In credit scheduler, credit consumed by the vcpu must be subtracted. 
Therefore I think it is correct that dom0 and guest A are OVER because 
my approach is to boost the vcpu within the range of weight.

I think assigning a static priority is one solution. However, I think 
that it affects credit accounting because we don't know how long the 
domain with the static priority (probably highest priority) is run.

About adding more BOOST source, could you explain more to me?

>>> 		b. vcpu migration. As said before, without vcpu 
>> pinned, glitches are obvious.
>>
>> I think that this issue would be solved by adding the condition for 
>> migrating the vcpu.
>> e.g. If the vcpu has boost credit, don't migrate the vcpu.
> 
> Is it over-kill? how about you already get 3 BOOST vcpu in 
> runqueue of current cpu, when other cpus are all running
> OVER vcpus? Boost itself looks not the only determinative 
> factor for migration, and instead what you concern is the 
> relative priority in system wide.

Yes, you are right.
I'll consider about runqueue of each cpu and so on.

Thanks for your advice.

Regards,
Naoki

  reply	other threads:[~2009-01-15  4:42 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-18  2:57 [RFC][PATCH 0/4] Modification of credit scheduler rev2 NISHIGUCHI Naoki
2008-12-18  3:00 ` [RFC][PATCH 1/4] sched: more accurate credit scheduling NISHIGUCHI Naoki
2008-12-18  3:02 ` [RFC][PATCH 2/4] sched: change the handling of credits over upper bound NISHIGUCHI Naoki
2008-12-18  3:04 ` [RFC][PATCH 3/4] sched: balance credits of each vcpu of a domain NISHIGUCHI Naoki
2008-12-18  3:06 ` [RFC][PATCH 4/4] sched: introduce boost credit for latency-sensitive domain NISHIGUCHI Naoki
2009-01-13  8:10 ` [RFC][PATCH 0/4] Modification of credit scheduler rev2 Su, Disheng
2009-01-15  2:04   ` NISHIGUCHI Naoki
2009-01-15  2:56     ` Tian, Kevin
2009-01-15  4:42       ` NISHIGUCHI Naoki [this message]
2009-01-15  5:04         ` Tian, Kevin
2009-01-15  6:05           ` NISHIGUCHI Naoki
2009-01-15  6:41             ` Tian, Kevin
2009-01-15  7:01               ` NISHIGUCHI Naoki
2009-01-15  7:04                 ` Tian, Kevin
2009-01-15  4:55     ` Su, Disheng
2009-01-15  5:19       ` NISHIGUCHI Naoki

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=496EBECC.8020405@jp.fujitsu.com \
    --to=nisiguti@jp.fujitsu.com \
    --cc=George.Dunlap@eu.citrix.com \
    --cc=Ian.Pratt@eu.citrix.com \
    --cc=aviv@neocleus.com \
    --cc=disheng.su@intel.com \
    --cc=keir.fraser@eu.citrix.com \
    --cc=kevin.tian@intel.com \
    --cc=sakaia@jp.fujitsu.com \
    --cc=xen-devel@lists.xensource.com \
    /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.