Hi again, Here it comes v2, redone following Jan's suggestion, which allowed to get rid of patch 2, and do everything in sched_credit.c.
So, in summary, because of the fact that vcpu_migrate() forces the vcpus into a sleep+wakeup cycle, vcpus being migrated to a new pcpu, were also being granted BOOST priority, inside Credit1, and that is not correct. More info on v1's cover letter, which is here: http://lists.xenproject.org/archives/html/xen-devel/2016-02/msg01620.html I re-run the same set of benchmarks described in v1, and the result for this rework of the series are basically the same as there. Thanks and Regards, Dario --- Dario Faggioli (2): xen: credit1: trace vCPU boost/unboost xen: credit1: avoid boosting vCPUs being "just" migrated xen/common/sched_credit.c | 34 ++++++++++++++++++++++++++++++---- xen/include/xen/perfc_defn.h | 1 + 2 files changed, 31 insertions(+), 4 deletions(-) -- <<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) _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel