All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dario Faggioli <dario.faggioli@citrix.com>
To: xen-devel@lists.xen.org
Cc: keir@xen.org, Ian.Campbell@citrix.com, Andrew.Cooper3@citrix.com,
	George.Dunlap@citrix.com, JBeulich@suse.com,
	Ian.Jackson@citrix.com, Wei Liu <wei.liu2@citrix.com>
Subject: [PATCH v8 00/13] Implement vcpu soft affinity for credit1
Date: Fri, 13 Jun 2014 15:09:01 +0200	[thread overview]
Message-ID: <20140613124847.4106.70161.stgit@Solace> (raw)

Hi everyone,

Here it comes v8 of the soft affinity series. This posting addresses all the
comments received during v7 review.

However, there are a few new stuff, which need reviewing or re-reviewing. In
fact, the most notable difference between v8 and v7 is that, I incorporated in
the series a patch from Wei's "JSON infrastructure" series, more specifically
the one about pushing the handling of vcpu pinning from xl, down to libxl.

As explained in the changelog of patch 10 ("libxl/xl: deprecate the
build_info->cpumap field") this has a number of advantages. First of all it
makes both the development (for both of us!) and the reviewing easier.
Moreover, it makes libxl code more straightforward (as compared to what it'd
look like whit mine and Wei's series applied one on top of the other), and it
also makes xl a bit more powerful (see patch 10 for details) when it comes to
specify vcpu hard and soft affinity... for free!! :-D

The only downside is that a few patches in the series are either new, or just
too different from previous versions, and hence I removed all the Reviewed and
Acked tags from there... I guess the review of this round should concentrate on
them.  I am talking about the following patches:

  08/13  xl: enable getting and setting soft affinity
  09/13  libxl/xl: push VCPU affinity pinning down to libxl
  10/13  libxl/xl: deprecate the build_info->cpumap field
  11/13  xl: move the vcpu affinity parsing in a function
  12/13  xl: enable for specifying soft-affinity in the config file


Patch 13 ("libxl: automatic NUMA placement affects soft affinity") is changed a
little bit, but not enough to justify killing the Ack-s... Feel free to re-look
at it, of course.

The other paches have, I think, all the proper Ack-s to be checked in already.

Patches are available here:

  git://xenbits.xen.org/people/dariof/xen.git numa/per-vcpu-affinity-v8

Thanks and Regards,
Dario

---

Dario Faggioli (12):
      xen: sched: rename v->cpu_affinity into v->cpu_hard_affinity
      xen: sched: introduce soft-affinity and use it instead d->node-affinity
      xen: derive NUMA node affinity from hard and soft CPU affinity
      xen/libxc: sched: DOMCTL_*vcpuaffinity works with hard and soft affinity
      libxc/libxl: bump library SONAMEs
      libxc: get and set soft and hard affinity
      libxl: get and set soft affinity
      xl: enable getting and setting soft affinity
      libxl/xl: deprecate the build_info->cpumap field
      xl: move the vcpu affinity parsing in a function
      xl: enable for specifying soft-affinity in the config file
      libxl: automatic NUMA placement affects soft affinity

Wei Liu (1):
      libxl/xl: push VCPU affinity pinning down to libxl


 docs/man/xl.cfg.pod.5                |   50 +++++--
 docs/man/xl.pod.1                    |   32 +++-
 docs/misc/xl-numa-placement.markdown |  162 +++++++++++++++-------
 tools/libxc/Makefile                 |    2 
 tools/libxc/xc_domain.c              |   72 ++++++----
 tools/libxc/xenctrl.h                |   55 +++++++-
 tools/libxl/Makefile                 |    2 
 tools/libxl/libxl.c                  |   97 ++++++++++++-
 tools/libxl/libxl.h                  |   55 +++++++-
 tools/libxl/libxl_create.c           |    6 -
 tools/libxl/libxl_dom.c              |   53 +++++++
 tools/libxl/libxl_types.idl          |   11 +-
 tools/libxl/libxl_utils.h            |   25 +++
 tools/libxl/xl_cmdimpl.c             |  246 ++++++++++++++++++----------------
 tools/libxl/xl_cmdtable.c            |    2 
 tools/ocaml/libs/xc/xenctrl_stubs.c  |    8 +
 tools/python/xen/lowlevel/xc/xc.c    |    6 +
 xen/arch/x86/traps.c                 |   13 +-
 xen/common/domain.c                  |   86 +++++++-----
 xen/common/domctl.c                  |  107 +++++++++++++--
 xen/common/keyhandler.c              |    4 -
 xen/common/sched_credit.c            |  161 +++++++++-------------
 xen/common/sched_sedf.c              |    2 
 xen/common/schedule.c                |   66 ++++++---
 xen/common/wait.c                    |   10 +
 xen/include/public/domctl.h          |   29 ++++
 xen/include/xen/sched-if.h           |    2 
 xen/include/xen/sched.h              |   15 +-
 28 files changed, 936 insertions(+), 443 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)

             reply	other threads:[~2014-06-13 13:09 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-13 13:09 Dario Faggioli [this message]
2014-06-13 13:09 ` [PATCH v8 01/13] xen: sched: rename v->cpu_affinity into v->cpu_hard_affinity Dario Faggioli
2014-06-13 13:09 ` [PATCH v8 02/13] xen: sched: introduce soft-affinity and use it instead d->node-affinity Dario Faggioli
2014-06-13 13:09 ` [PATCH v8 03/13] xen: derive NUMA node affinity from hard and soft CPU affinity Dario Faggioli
2014-06-13 13:09 ` [PATCH v8 04/13] xen/libxc: sched: DOMCTL_*vcpuaffinity works with hard and soft affinity Dario Faggioli
2014-06-13 13:09 ` [PATCH v8 05/13] libxc/libxl: bump library SONAMEs Dario Faggioli
2014-06-13 13:09 ` [PATCH v8 06/13] libxc: get and set soft and hard affinity Dario Faggioli
2014-06-13 13:10 ` [PATCH v8 07/13] libxl: get and set soft affinity Dario Faggioli
2014-06-13 13:10 ` [PATCH v8 08/13] xl: enable getting and setting " Dario Faggioli
2014-06-13 13:10 ` [PATCH v8 09/13] libxl/xl: push VCPU affinity pinning down to libxl Dario Faggioli
2014-06-13 13:25   ` Wei Liu
2014-06-17 10:09     ` Dario Faggioli
2014-06-13 13:10 ` [PATCH v8 10/13] libxl/xl: deprecate the build_info->cpumap field Dario Faggioli
2014-06-13 13:34   ` Wei Liu
2014-06-13 13:38     ` Ian Campbell
2014-06-17 10:11       ` Dario Faggioli
2014-06-13 13:49     ` Wei Liu
2014-06-17  9:59       ` Dario Faggioli
2014-06-17 10:16   ` Wei Liu
2014-06-17 10:29     ` Dario Faggioli
2014-06-17 10:45       ` Wei Liu
2014-06-13 13:10 ` [PATCH v8 11/13] xl: move the vcpu affinity parsing in a function Dario Faggioli
2014-06-13 13:10 ` [PATCH v8 12/13] xl: enable for specifying soft-affinity in the config file Dario Faggioli
2014-06-17 10:34   ` Wei Liu
2014-06-13 13:10 ` [PATCH v8 13/13] libxl: automatic NUMA placement affects soft affinity 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=20140613124847.4106.70161.stgit@Solace \
    --to=dario.faggioli@citrix.com \
    --cc=Andrew.Cooper3@citrix.com \
    --cc=George.Dunlap@citrix.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=Ian.Jackson@citrix.com \
    --cc=JBeulich@suse.com \
    --cc=keir@xen.org \
    --cc=wei.liu2@citrix.com \
    --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.