Re: [Xen-devel] [PATCH 00 of 11 v3] NUMA aware credit scheduling

[Dario Faggioli <dario.faggioli@xxxxxxxxxx>]

On Fri, 2013-02-01 at 12:01 +0100, Dario Faggioli wrote:
> Hello Everyone,
> 
> V3 of the NUMA aware scheduling series. It is nothing more than v2 with all
> the review comments I got, addressed... Or so I think. :-)
> 
Ping?

There is a minor thing from Jurgen which I really plan to address and
repost, but I was hoping to be able to incorporate other people's
feedback as well while doing it.

In some more details...

> Here are the patches included in the series. I '*'-ed ones already received 
> one
> or more acks during previous rounds. Of course, I retained these Ack-s only 
> for
> the patches that have not been touched, or only underwent minor cluenups. Of
> course, feel free to re-review everything, whatever your Ack is there or not!
> 
>  * [ 1/11] xen, libxc: rename xenctl_cpumap to xenctl_bitmap
>  * [ 2/11] xen, libxc: introduce node maps and masks
>
Jan, are you ok with the changes I made, as a consequence of your
comments to v2 ?

>    [ 3/11] xen: sched_credit: when picking, make sure we get an idle one, if 
> any
>
George ack-ed this.

>    [ 4/11] xen: sched_credit: let the scheduler know about node-affinity
>
Goerge, whenever you're fine, any thoughts on how I addressed the points
you raise at v2 time?

>  * [ 5/11] xen: allow for explicitly specifying node-affinity
>  * [ 6/11] libxc: allow for explicitly specifying node-affinity
>  * [ 7/11] libxl: allow for explicitly specifying node-affinity
>    [ 8/11] libxl: optimize the calculation of how many VCPUs can run on a 
> candidate
>  * [ 9/11] libxl: automatic placement deals with node-affinity
>  * [10/11] xl: add node-affinity to the output of `xl list`
>
These all (apart from 8/11) already received one or more ack(-s), but,
for instance, neither of the Ian's commented, and since I'm touching
[lib]xl, that would be much appreciated. :-)

>    [11/11] docs: rearrange and update NUMA placement documentation
> 
What about this? How bad the English was this time? :-)

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: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel