From: Ian Campbell <ian.campbell@citrix.com>
To: xen-devel@lists.xensource.com
Cc: ian.jackson@citrix.com
Subject: [PATCH 00 of 32 RFC] libxl: drop device_model_info, better defaults support, stubdoms by default
Date: Mon, 16 Jan 2012 12:14:58 +0000 [thread overview]
Message-ID: <patchbomb.1326716098@cosworth.uk.xensource.com> (raw)
My original intention with this series was to introduce the ability
for users of libxl to explicitly say "pick the default". Mainly by
introducing the libxl_defbool type but also by arranging more
generally that leaving a value set to zero means "default".
However it turned into something of a yakk shaving excercise and so
contains the following:
* Removes support for comments in the IDL language itself. Generating
comments in the generated files is not all that useful so lets just
use source level comments in the .idl instead.
* Switches to using keyword arguments for Fields in Aggregate types
instead of a variadic tuple.
* Removes libxl_device_model_info from the library API. The use of
a device model to supply specific features is really an
implementation detail within the library and not something users of
the library should need to be concerned with.
Many of the fields of the struct we really guest configuration
details, e.g. VNC setup and therefore belong in
libxl_domain_build_info.
This struct also contained a number of things which were duplicated
from the build_info and which could never sensibly differ in this
context (e.g. guest total RAM, ACPI on off etc).
I have tried to make a determination whether something belongs in
the global part of libxl_domain_build_info or whether it is HVM
specific.
This forms a big part of the series.
* Introduce libxl_defbool and switch to using it.
* Some other changes to arrange that the 0 value means "default". At
least one instance of this (the timer_mode enum) introduces a wierd
disconnect between the underlying values and the libxl exposed
values. I'm not sure that I like this.
* Use device model stubdoms by default when possible.
One question which arised from this change is how we deal with
per-arch options in the future. Do we need to define
libxl_domain_build_info_x86? We could also tag fields with a list of
architectures.
This series has been lightly tested with PV domains (with and without
PVFB) and HVM domains (with and without stubdom).
next reply other threads:[~2012-01-16 12:14 UTC|newest]
Thread overview: 83+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-16 12:14 Ian Campbell [this message]
2012-01-16 12:14 ` [PATCH 01 of 32 RFC] libxl: remove comment support from IDL Ian Campbell
2012-01-16 16:23 ` Ian Jackson
2012-01-16 16:29 ` Ian Campbell
2012-01-16 16:30 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 02 of 32 RFC] libxl: use keyword arguments for field definitions in aggregate types Ian Campbell
2012-01-16 16:03 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 03 of 32 RFC] ocaml: use libxl IDL type helpers for C argument passing Ian Campbell
2012-01-16 12:15 ` [PATCH 04 of 32 RFC] ocaml: Correct ocaml type name for Aggregate types Ian Campbell
2012-01-16 16:49 ` [PATCH 04 of 32] " Ian Jackson
2012-01-16 12:15 ` [PATCH 05 of 32 RFC] libxl: define libxl_vnc_info to hold all info about the vnc info Ian Campbell
2012-01-16 16:16 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 06 of 32 RFC] libxl: define libxl_spice_info to hold all info about the spice server Ian Campbell
2012-01-16 16:04 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 07 of 32 RFC] libxl: define libxl_sdl_info to hold all info about the SDL config Ian Campbell
2012-01-16 16:10 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 08 of 32 RFC] libxl: plumb libxl_domain_config down into device model creation Ian Campbell
2012-01-16 16:21 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 09 of 32 RFC] libxl: now that dm creation takes domain_config stop passing down devices Ian Campbell
2012-01-16 12:15 ` [PATCH 10 of 32 RFC] libxl: remove redundant info from dm info Ian Campbell
2012-01-16 16:22 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 11 of 32 RFC] libxl: drop dm_info.dom_name Ian Campbell
2012-01-16 16:14 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 12 of 32 RFC] libxl: use vfb[0] directly for xenpv device model Ian Campbell
2012-01-16 16:13 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 13 of 32 RFC] libxl: move HVM emulated GFX support into b_info->u.hvm Ian Campbell
2012-01-16 16:27 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 14 of 32 RFC] libxl: HVM device configuration info build_info->u.hvm Ian Campbell
2012-01-16 16:29 ` Ian Jackson
2012-01-16 16:46 ` Ian Campbell
2012-01-16 16:53 ` Ian Jackson
2012-01-16 17:45 ` Ian Campbell
2012-01-31 14:48 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 15 of 32 RFC] libxl: move gfx_passthru setting to b_info->u.hvm Ian Campbell
2012-01-16 16:11 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 16 of 32 RFC] libxl: Remove libxl_device_model_info.type Ian Campbell
2012-01-16 16:20 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 17 of 32 RFC] libxl: remove uuid from device model info Ian Campbell
2012-01-16 16:13 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 18 of 32 RFC] libxl: move device model selection variables to b_info Ian Campbell
2012-01-16 12:15 ` [PATCH 19 of 32 RFC] libxl: move "saved_state" to libxl__domain_build_state Ian Campbell
2012-01-16 16:21 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 20 of 32 RFC] libxl: remove libxl_device_model_info Ian Campbell
2012-01-16 16:30 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 21 of 32 RFC] libxl: drop vfs patch -- fsback/front were deleted some time ago Ian Campbell
2012-01-16 16:29 ` Olaf Hering
2012-01-16 16:41 ` Ian Campbell
2012-01-16 16:50 ` [PATCH 21 of 32] libxl: drop vfs patch Ian Jackson
2012-01-16 17:09 ` Ian Campbell
2012-01-16 17:30 ` Ian Campbell
2012-01-16 17:56 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 22 of 32 RFC] libxl: only write "disable_pf" key to xenstore when it makes sense Ian Campbell
2012-01-16 16:04 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 23 of 32 RFC] libxl: use libxl_*_init internally too Ian Campbell
2012-01-16 16:09 ` Ian Jackson
2012-01-16 16:15 ` Ian Campbell
2012-01-16 16:16 ` Ian Jackson
2012-01-16 16:23 ` Ian Campbell
2012-01-16 16:25 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 24 of 32 RFC] libxl: add new "defbool" built in type Ian Campbell
2012-01-16 16:06 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 25 of 32 RFC] libxl: make boolean members of libxl_domain_create_info into libxl_defbool Ian Campbell
2012-01-16 16:31 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 26 " Ian Campbell
2012-01-16 16:28 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 27 of 32 RFC] libxl: use defbool for graphics related options Ian Campbell
2012-01-16 16:26 ` Ian Jackson
2012-01-16 16:40 ` Ian Campbell
2012-01-16 12:15 ` [PATCH 28 of 32 RFC] libxl: initialise NULL==default members of build info later Ian Campbell
2012-01-16 12:15 ` [PATCH 29 of 32 RFC] libxl: add named enum for timer mode Ian Campbell
2012-01-16 16:17 ` Ian Jackson
2012-01-16 16:25 ` Ian Campbell
2012-01-16 16:27 ` Ian Jackson
2012-01-17 11:26 ` Ian Campbell
2012-01-16 12:15 ` [PATCH 30 of 32 RFC] libxl: drop 8M slack for PV guests Ian Campbell
2012-01-16 12:15 ` [PATCH 31 of 32 RFC] libxl: switch device model selection over to libxl_defbool Ian Campbell
2012-01-16 16:12 ` Ian Jackson
2012-01-16 12:15 ` [PATCH 32 of 32 RFC] libxl: Default to stub device model whenever possible Ian Campbell
2012-01-16 15:53 ` Stefano Stabellini
2012-01-16 16:02 ` Ian Campbell
2012-01-16 17:01 ` Stefano Stabellini
2012-01-16 17:14 ` Ian Campbell
2012-01-17 14:11 ` Stefano Stabellini
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=patchbomb.1326716098@cosworth.uk.xensource.com \
--to=ian.campbell@citrix.com \
--cc=ian.jackson@citrix.com \
--cc=xen-devel@lists.xensource.com \
/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;
as well as URLs for NNTP newsgroup(s).