From: Vitaly Kuznetsov <vkuznets@redhat.com>
To: xen-devel@lists.xenproject.org
Cc: x86@kernel.org, linux-kernel@vger.kernel.org,
Boris Ostrovsky <boris.ostrovsky@oracle.com>,
Juergen Gross <jgross@suse.com>,
Andrew Jones <drjones@redhat.com>
Subject: [PATCH 0/5] x86/xen: untangle PV and PVHVM guest support code
Date: Fri, 24 Feb 2017 17:14:35 +0100 [thread overview]
Message-ID: <20170224161440.2136-1-vkuznets@redhat.com> (raw)
Hi,
it's been a while since my 'RFC' submission in November:
https://lists.xen.org/archives/html/xen-devel/2016-11/msg01044.html
I was advised to wait till PVHv2 stuff lands upstream and as this has already
happened I'm sending the first non-RFC version.
Changes since RFC:
- Rebase
- Use enlighten.c, enlighten_pv.c, enlighten_hvm.c, enlighten_pvh.c (and
similar for other files I split) [David Vrabel]
- Split suspend.c too to get rid of #ifdefs in C code.
- I left XEN_DOM0 intact (rename to XEN_PV_DOM0 was suggested by [Boris
Ostrovsky]) as it occured to me that we may want to have all three when
XEN_PVH_DOM0 comes into play. XEN_DOM0 will be used for common code than so
no need to change #ifdefs twice.
Patches are known to produce checkpatch.pl WARNINGS and a couple of ERRORs,
I fixed a few (mostly in _hvm* code I split) and I refrained from fixing the
rest to make it easier to review. I think that we may leave PV code as it is
as sooner or later it will go away.
Original description:
I have a long-standing idea to separate PV and PVHVM code in kernel and
introduce Kconfig options to make it possible to enable the required
parts only breaking the current 'all or nothing' approach.
Motivation:
- Xen related x86 code in kernel is rather big and it is unclear which
parts of it are required for PV, for HVM or for both. With PVH coming
into picture is becomes even more tangled. It makes it hard to
understand/audit the code.
- In some case we may want to avoid bloating kernel by supporting Xen
guests we don't need. In particular, 90% of the code in arch/x86/xen/ is
required to support PV guests and one may require PVHVM support only.
- PV guests are supposed to go away one day and such code separation would
help us to get ready.
This series adds XEN_PV Kconfig option and makes it possible to build PV-only
and PVHVM-only kernels. It also makes it possible to disable Dom0 support.
Patches are rather big but this is mostly just moving code around, no
functional changes intended. I smoke tested it with PV-only and PVHVM-only
builds, booted and did save/restore test. I also tried the newly introduced
PVHv2 guest, it even worked!
Vitaly Kuznetsov (5):
x86/xen: start untangling PV and PVHVM guest support code
x86/xen: split smp.c for PV and PVHVM guests
x86/xen: put setup.c, mmu.c and p2m.c under CONFIG_XEN_PV
x86/xen: put setup.c, pmu.c and apic.c under CONFIG_XEN_PV
x86/xen: split suspend.c for PV and PVHVM guests
arch/x86/include/asm/hypervisor.h | 3 +-
arch/x86/include/asm/xen/page.h | 44 +-
arch/x86/kernel/cpu/hypervisor.c | 7 +-
arch/x86/kernel/process_64.c | 2 +-
arch/x86/xen/Kconfig | 33 +-
arch/x86/xen/Makefile | 18 +-
arch/x86/xen/enlighten.c | 1904 +------------------------
arch/x86/xen/enlighten_hvm.c | 210 +++
arch/x86/xen/enlighten_pv.c | 1561 +++++++++++++++++++++
arch/x86/xen/enlighten_pvh.c | 114 ++
arch/x86/xen/mmu.c | 2776 +------------------------------------
arch/x86/xen/mmu_hvm.c | 77 +
arch/x86/xen/mmu_pv.c | 2636 +++++++++++++++++++++++++++++++++++
arch/x86/xen/pmu.h | 5 +
arch/x86/xen/smp.c | 523 +------
arch/x86/xen/smp.h | 23 +
arch/x86/xen/smp_hvm.c | 58 +
arch/x86/xen/smp_pv.c | 499 +++++++
arch/x86/xen/suspend.c | 54 -
arch/x86/xen/suspend_hvm.c | 22 +
arch/x86/xen/suspend_pv.c | 44 +
arch/x86/xen/xen-head.S | 4 +
arch/x86/xen/xen-ops.h | 16 +
drivers/xen/balloon.c | 30 +-
include/xen/xen-ops.h | 19 +
25 files changed, 5489 insertions(+), 5193 deletions(-)
create mode 100644 arch/x86/xen/enlighten_hvm.c
create mode 100644 arch/x86/xen/enlighten_pv.c
create mode 100644 arch/x86/xen/enlighten_pvh.c
create mode 100644 arch/x86/xen/mmu_hvm.c
create mode 100644 arch/x86/xen/mmu_pv.c
create mode 100644 arch/x86/xen/smp_hvm.c
create mode 100644 arch/x86/xen/smp_pv.c
create mode 100644 arch/x86/xen/suspend_hvm.c
create mode 100644 arch/x86/xen/suspend_pv.c
--
2.9.3
next reply other threads:[~2017-02-24 16:15 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-24 16:14 Vitaly Kuznetsov [this message]
2017-02-24 16:14 ` [PATCH 1/5] x86/xen: start untangling " Vitaly Kuznetsov
2017-03-01 6:22 ` [Xen-devel] " Juergen Gross
2017-03-01 9:38 ` Vitaly Kuznetsov
2017-02-24 16:14 ` [PATCH 2/5] x86/xen: split smp.c for PV and PVHVM guests Vitaly Kuznetsov
2017-03-01 6:24 ` [Xen-devel] " Juergen Gross
2017-02-24 16:14 ` [PATCH 3/5] x86/xen: put setup.c, mmu.c and p2m.c under CONFIG_XEN_PV Vitaly Kuznetsov
2017-03-01 6:25 ` [Xen-devel] " Juergen Gross
2017-02-24 16:14 ` [PATCH 4/5] x86/xen: put setup.c, pmu.c and apic.c " Vitaly Kuznetsov
2017-03-01 7:10 ` [Xen-devel] " Juergen Gross
2017-02-24 16:14 ` [PATCH 5/5] x86/xen: split suspend.c for PV and PVHVM guests Vitaly Kuznetsov
2017-03-01 7:14 ` [Xen-devel] " Juergen Gross
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=20170224161440.2136-1-vkuznets@redhat.com \
--to=vkuznets@redhat.com \
--cc=boris.ostrovsky@oracle.com \
--cc=drjones@redhat.com \
--cc=jgross@suse.com \
--cc=linux-kernel@vger.kernel.org \
--cc=x86@kernel.org \
--cc=xen-devel@lists.xenproject.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome