From: Dario Faggioli <dario.faggioli@citrix.com>
To: Ian Jackson <Ian.Jackson@eu.citrix.com>
Cc: "Keir (Xen.org)" <keir@xen.org>,
Ian Campbell <Ian.Campbell@citrix.com>,
George Dunlap <George.Dunlap@eu.citrix.com>,
Juergen Gross <juergen.gross@ts.fujitsu.com>,
"Tim (Xen.org)" <tim@xen.org>,
"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
Jan Beulich <JBeulich@suse.com>
Subject: Re: [PATCH 00 of 11 v5] NUMA aware credit scheduling
Date: Tue, 16 Apr 2013 18:07:31 +0200 [thread overview]
Message-ID: <1366128451.3037.25.camel@Abyss> (raw)
In-Reply-To: <20845.29477.827997.761798@mariner.uk.xensource.com>
[-- Attachment #1.1: Type: text/plain, Size: 1668 bytes --]
On mar, 2013-04-16 at 16:49 +0100, Ian Jackson wrote:
> George Dunlap writes ("Re: [Xen-devel] [PATCH 00 of 11 v5] NUMA aware credit scheduling"):
> > On Tue, Apr 16, 2013 at 2:14 PM, Jan Beulich <JBeulich@suse.com> wrote:
> > > - 1 and 2 touch tools/, but don't have a tools maintainer ack yet
>
> I'm happy for 1 to have my ack provided that Jan's comment on it is
> addressed.
>
I did that already, by replying to his e-mail with an updated patch,
since it was a very minor issue, involving only that very one. Of
course, I can respin the whole series, if you like it better.
> I think 05/11 should explain in a documentation comment what it means
> for a domain to have a node affinity. Specifically it's not clear
> from the docs there whether it's a hint to the hypervisor, or a hard
> requirement, and what semantic restrictions there might be on setting
> affinities.
>
Actually, I tried to do right that in 11/11. I know, we want doc to be
in the same changeset of the change itself, but there is not just one
specific change that enables the new behavior, it's something resulting
from the joint work of 3/11, 4/11, 5/11 6/11 7/11 and 9/11, so it was
unclear to me where to update the doc, and that's why I opted for a new
patch, at the end of the series.
Or was it something different from what I have there that you had in
mind?
Thanks and Regards,
Dario
--
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
[-- Attachment #2: Type: text/plain, Size: 126 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
next prev parent reply other threads:[~2013-04-16 16:07 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-10 16:04 [PATCH 00 of 11 v5] NUMA aware credit scheduling Dario Faggioli
2013-04-10 16:04 ` [PATCH 01 of 11 v5] xen, libxc: rename xenctl_cpumap to xenctl_bitmap Dario Faggioli
2013-04-10 16:05 ` [PATCH 02 of 11 v5] xen, libxc: introduce xc_nodemap_t Dario Faggioli
2013-04-10 16:18 ` Jan Beulich
2013-04-10 16:41 ` Dario Faggioli
2013-04-10 16:05 ` [PATCH 03 of 11 v5] xen: sched_credit: when picking, make sure we get an idle one, if any Dario Faggioli
2013-04-10 16:05 ` [PATCH 04 of 11 v5] xen: sched_credit: let the scheduler know about node-affinity Dario Faggioli
2013-04-10 16:05 ` [PATCH 05 of 11 v5] xen: allow for explicitly specifying node-affinity Dario Faggioli
2013-04-10 16:05 ` [PATCH 06 of 11 v5] libxc: " Dario Faggioli
2013-04-10 16:05 ` [PATCH 07 of 11 v5] libxl: " Dario Faggioli
2013-04-10 16:05 ` [PATCH 08 of 11 v5] libxl: optimize the calculation of how many VCPUs can run on a candidate Dario Faggioli
2013-04-10 16:05 ` [PATCH 09 of 11 v5] libxl: automatic placement deals with node-affinity Dario Faggioli
2013-04-10 16:05 ` [PATCH 10 of 11 v5] xl: add node-affinity to the output of `xl list` Dario Faggioli
2013-04-10 16:05 ` [PATCH 11 of 11 v5] docs: rearrange and update NUMA placement documentation Dario Faggioli
2013-04-11 12:41 ` [PATCH 00 of 11 v5] NUMA aware credit scheduling George Dunlap
2013-04-16 8:44 ` Dario Faggioli
2013-04-16 12:16 ` George Dunlap
2013-04-16 13:14 ` Jan Beulich
2013-04-16 14:47 ` Dario Faggioli
2013-04-16 14:57 ` Ian Campbell
2013-04-16 15:01 ` Dario Faggioli
2013-04-16 15:22 ` George Dunlap
2013-04-16 15:49 ` Ian Jackson
2013-04-16 16:07 ` Dario Faggioli [this message]
2013-04-16 16:12 ` Ian Jackson
2013-04-17 8:53 ` Ian Campbell
2013-04-17 8:56 ` Dario Faggioli
2013-04-17 8:59 ` Ian Campbell
2013-04-17 9:11 ` Dario Faggioli
2013-04-16 16:06 ` Ian Campbell
2013-04-16 16:06 ` Jan Beulich
2013-04-16 16:45 ` Keir Fraser
2013-04-16 17:03 ` Dario Faggioli
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=1366128451.3037.25.camel@Abyss \
--to=dario.faggioli@citrix.com \
--cc=George.Dunlap@eu.citrix.com \
--cc=Ian.Campbell@citrix.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=JBeulich@suse.com \
--cc=juergen.gross@ts.fujitsu.com \
--cc=keir@xen.org \
--cc=tim@xen.org \
--cc=xen-devel@lists.xen.org \
/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.