All of lore.kernel.org
 help / color / mirror / Atom feed
From: Keir Fraser <keir@xensource.com>
To: Emmanuel Ackaouy <ackaouy@gmail.com>,
	Atsushi SAKAI <sakaia@jp.fujitsu.com>
Cc: xen-devel@lists.xensource.com
Subject: Re: [PATCH][RFC] consider vcpu-pin weight on CreditScheduler TAKE2
Date: Wed, 27 Jun 2007 13:31:02 +0100	[thread overview]
Message-ID: <C2A81716.1166C%keir@xensource.com> (raw)
In-Reply-To: <54544386934904fcb51d4ece80ca9db9@gmail.com>

On 27/6/07 13:21, "Emmanuel Ackaouy" <ackaouy@gmail.com> wrote:

>> Which is the best way to solve?
> 
> If you could solve the generic problem in a simpler way, I would
> not be opposed to it. But +365 lines in what is already a fairly
> complex accounting code path doesn't seem right to me.
> 
> I can't even understand what weights mean when every CPU
> has a different pin cpumask. Weights only make sense to me
> when VMs compete for resources.
> 
> In my opinion, adding the concept of dynamic partitioning (or
> segmentation) of the host system would allow a bunch of people
> to no longer have to pin their VCPUs. This is desirable.

Partitioning is a very sensible simplification in many (most?) cases, but
would need plumbing all the way up through xen-api, which is a pain. I still
suspect that the patch could be simplified even without interface changes. I
don't understand the need to add extra complexity on every accounting
period.

 -- Keir

  reply	other threads:[~2007-06-27 12:31 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-27  7:58 [PATCH][RFC] consider vcpu-pin weight on Credit Scheduler TAKE2 Atsushi SAKAI
2007-06-27 11:46 ` Emmanuel Ackaouy
2007-06-27 12:07   ` [PATCH][RFC] consider vcpu-pin weight on CreditScheduler TAKE2 Atsushi SAKAI
2007-06-27 12:21     ` Emmanuel Ackaouy
2007-06-27 12:31       ` Keir Fraser [this message]
2007-06-27 12:52         ` Emmanuel Ackaouy
2007-06-28  9:23         ` [PATCH][RFC] consider vcpu-pin weight onCreditScheduler TAKE2 Atsushi SAKAI

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=C2A81716.1166C%keir@xensource.com \
    --to=keir@xensource.com \
    --cc=ackaouy@gmail.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.