Re: [Xen-devel] [PATCH 00/19] Assorted fixes and improvements to Credit2

[George Dunlap <George.Dunlap@xxxxxxxxxxxxx>]

On Fri, Jun 17, 2016 at 6:32 PM, Dario Faggioli
<dario.faggioli@xxxxxxxxxx> wrote:
> Hi everyone,
>
> Here you go a collection of pseudo-random fixes and improvement to Credit2.
>
> In the process of working on Soft Affinity and Caps support, I stumbled upon
> them, one after the other, and decided to take care.
>
> It's been hard to test and run benchmark, due to the "time goes backwards" bug
> I uncovered [1], and this is at least part of the reason why the code for
> affinity and caps is still missing. I've got it already, but need to refine a
> couple of things, after double checking benchmark results. So, now that we 
> have
> Jan's series [2] (thanks! [*]), and that I managed to indeed run some tests on
> this preliminary set of patches, I decided I better set this first group free,
> while working on finishing the rest.
>
> The various patches do a wide range of different things, so, please, refer to
> Dario Faggioli (19):

I've pushed the following patches:

>       xen: sched: make the 'tickled' perf counter clearer
>       xen: credit2: insert and tickle don't need a cpu parameter
>       xen: credit2: kill useless helper function choose_cpu
>       xen: credit2: do not warn if calling burn_credits more than once
>       xen: credit2: when tickling, check idle cpus first
>       xen: credit2: avoid calling __update_svc_load() multiple times on the 
> same vcpu
>       xen: credit2: use non-atomic cpumask and bit operations

The ones below either have outstanding comments, or don't apply
without patches which haven't been applied.

>       xen: sched: leave CPUs doing tasklet work alone.
>       xen: credit2: read NOW() with the proper runq lock held
>       xen: credit2: prevent load balancing to go mad if time goes backwards
>       xen: credit2: rework load tracking logic
>       tools: tracing: adapt Credit2 load tracking events to new format
>       xen: credit2: make the code less experimental
>       xen: credit2: add yet some more tracing
>       xen: credit2: only marshall trace point arguments if tracing enabled
>       tools: tracing: deal with new Credit2 events
>       xen: credit2: the private scheduler lock can be an rwlock.
>       xen: credit2: implement SMT support independent runq arrangement
>       xen: credit2: use cpumask_first instead of cpumask_any when choosing cpu

 -George

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