From: "Edwin Török" <edvin.torok@citrix.com>
To: <xen-devel@lists.xenproject.org>
Cc: "Edwin Török" <edvin.torok@citrix.com>,
"Doug Goldstein" <cardoe@cardoe.com>,
"Andrew Cooper" <andrew.cooper3@citrix.com>,
"George Dunlap" <george.dunlap@citrix.com>,
"Ian Jackson" <iwj@xenproject.org>,
"Jan Beulich" <jbeulich@suse.com>,
"Julien Grall" <julien@xen.org>,
"Stefano Stabellini" <sstabellini@kernel.org>,
"Wei Liu" <wl@xen.org>,
"Christian Lindig" <christian.lindig@citrix.com>,
"David Scott" <dave@recoil.org>
Subject: [PATCH v1 0/4] tools/ocaml/libs/xc: domid control at domain creation time
Date: Tue, 17 Nov 2020 18:24:08 +0000 [thread overview]
Message-ID: <cover.1605636799.git.edvin.torok@citrix.com> (raw)
The xl toolstack allows some control over the domid at VM creation time,
allow xenopsd similar control by exposing the appropriate domid field in the OCaml xenctrl bindings.
A new API function is introduced to preserve backwards compatibility without merge ordering
requirements between the Xen and xenopsd patches: Xen can merge the patch and xenopsd will keep
building with the old function, and a new version of xenopsd will start using the new function.
I've also included some build system fixes to allow me to test the build
in an upstream build environment:
```
cd automation/build
podman build -t registry.gitlab.com/xen-project/xen/ubuntu:focal -f ubuntu/focal.dockerfile ubuntu
DOCKER_CMD=podman CONTAINER_NO_PULL=1 CONTAINER=registry.gitlab.com/xen-project/xen/ubuntu:focal automation/scripts/containerize make build-tools-oxenstored
```
It'd be good if someone could test whether containerize still works on non-SELinux systems now, or
whether we need more detection logic in the script.
This works around bugs in the OCaml makefiles that end up in "inconsistent assumptions" by doing a
'make clean' before building the OCaml files every time. This is inefficient, but works.
Long term it would be beneficial to switch to Dune as build system,
which can do correct incremental builds with minimal configuration.
I'll send a separate patch series for that.
Edwin Török (4):
automation/scripts/containerize: fix DOCKER_CMD=podman
automation/: add Ubuntu:focal container
Makefile: add build-tools-oxenstored
tools/ocaml/libs/xc: backward compatible domid control at domain
creation time
Makefile | 6 +++
automation/build/ubuntu/focal.dockerfile | 50 ++++++++++++++++++++++++
automation/scripts/containerize | 7 ++--
tools/ocaml/Makefile | 8 ++++
tools/ocaml/libs/xc/xenctrl.ml | 3 ++
tools/ocaml/libs/xc/xenctrl.mli | 2 +
tools/ocaml/libs/xc/xenctrl_stubs.c | 9 ++++-
7 files changed, 80 insertions(+), 5 deletions(-)
create mode 100644 automation/build/ubuntu/focal.dockerfile
--
2.18.4
next reply other threads:[~2020-11-17 18:24 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-17 18:24 Edwin Török [this message]
2020-11-17 18:24 ` [PATCH v1 1/4] automation/scripts/containerize: fix DOCKER_CMD=podman Edwin Török
2020-11-18 16:39 ` Doug Goldstein
2020-11-17 18:24 ` [PATCH v1 2/4] automation/: add Ubuntu:focal container Edwin Török
2020-11-18 16:40 ` Doug Goldstein
2020-11-18 16:43 ` Edwin Torok
2020-11-18 17:57 ` Andrew Cooper
2020-11-17 18:24 ` [PATCH v1 3/4] Makefile: add build-tools-oxenstored Edwin Török
2020-12-07 15:42 ` Wei Liu
2020-11-17 18:24 ` [PATCH v1 4/4] tools/ocaml/libs/xc: backward compatible domid control at domain creation time Edwin Török
2020-11-18 18:13 ` Andrew Cooper
2020-11-19 9:13 ` Edwin Torok
2020-11-18 10:46 ` [PATCH v1 0/4] tools/ocaml/libs/xc: " Christian Lindig
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=cover.1605636799.git.edvin.torok@citrix.com \
--to=edvin.torok@citrix.com \
--cc=andrew.cooper3@citrix.com \
--cc=cardoe@cardoe.com \
--cc=christian.lindig@citrix.com \
--cc=dave@recoil.org \
--cc=george.dunlap@citrix.com \
--cc=iwj@xenproject.org \
--cc=jbeulich@suse.com \
--cc=julien@xen.org \
--cc=sstabellini@kernel.org \
--cc=wl@xen.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 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.