From: George Dunlap <george.dunlap@citrix.com>
To: Eunbyung Park <silverbottlep@gmail.com>
Cc: George Dunlap <George.Dunlap@eu.citrix.com>,
"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Subject: Re: cap and reservation functionality in the credit2 scheduler?
Date: Thu, 6 Oct 2011 14:57:19 +0100 [thread overview]
Message-ID: <1317909439.15843.33.camel@elijah> (raw)
In-Reply-To: <4E8C3AAC.3070206@gmail.com>
On Wed, 2011-10-05 at 12:08 +0100, Eunbyung Park wrote:
> In addition, I found that adjusting dom0 weight have failed to work,
> but
> except for the dom0, it worked well.
> I found that this problem had been on xen-devel lists long time ago, but
> it seems not to be solved yet.
> I think this may be related with spinlock. When I remove the
> vcpu_schedule_lock_irq() in csched_dom_cntl(), it worked well.
> I'm not sure what is the real problem because I didn't completely
> understand the mechanism about spinlocks in xen scheduler.
So I'm afraid that instead of telling you how the spinlocks are supposed
to work, I'm going to let you figure out what the locks should be doing
on your own. If you can't do that, then you may not be ready yet to work
on the scheduler.
I hope that doesn't sound too harsh, but the fact is that the scheduler
is probably one of the most deceptively simple parts of the Xen code.
It requires a lot of expertise to know even how to avoid race conditions
and deadlock, not to mention making a simple algorithm which has the
desired effects without undesirable side effects. (The shadow code is
probably more complicated, for example, but it *looks* more complicated,
so people don't end up in over their head.) I wouldn't be doing either
you or me a favor by encouraging you to work on something past your
capability level.
> 2011-10-05 오후 7:24, George Dunlap 쓴 글:
> Actually, I don't have any idea in mind yet.
> however, I will try to devise a method to support them and make a patch.
> So, If you can afford to review, I will appreciate it :)
I'll be happy to review whatever you can send to the list. But like I
said above, beware of getting in over your head. :-)
> And, I think that there are still some to-do lists for the credit2 and
> xl tools needs to include credit2 support.
This should be fairly straightforward to do, and would be much
appreciated!
-George
next prev parent reply other threads:[~2011-10-06 13:57 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-05 4:43 cap and reservation functionality in the credit2 scheduler? Eunbyung Park
2011-10-05 10:24 ` George Dunlap
2011-10-05 11:08 ` Eunbyung Park
2011-10-06 13:57 ` George Dunlap [this message]
2011-10-08 12:59 ` Eunbyung Park
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=1317909439.15843.33.camel@elijah \
--to=george.dunlap@citrix.com \
--cc=George.Dunlap@eu.citrix.com \
--cc=silverbottlep@gmail.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.