From: Roger Pau Monne <roger.pau@entel.upc.edu>
To: xen-devel@lists.xensource.com
Subject: [PATCH 00 of 11 v3] libxl: add support for hotplug script calling from libxl
Date: Thu, 01 Dec 2011 16:08:07 +0100 [thread overview]
Message-ID: <patchbomb.1322752087@loki.upc.es> (raw)
This patch series adds support for hotplug script calling directly
from libxl, instead of relying on xenbackendd. Also some patches
contain general bug fixes.
Currently Linux hotplug script call functions are empty, so Linux
continues to use udev rules to call hotplug scripts.
Patches 1, 7, 8, 9, 10 are NetBSD specific, and basicaly pave the way
for the application of the bigger changes present in this series.
Patch 11 is a trivial update for an error message.
Patch 2 adds support for mounting raw image files using the vnd
device. Since NetBSD doesn't have qdisk or blktap support, the only
way to use raw images with guests is to mount the image and pass the
block device as a "PHY" backend. To check wheter an OS supports raw
images as "PHY" backends two new files are added to the project, to
avoid using #ifdefs, that contain a helper function. The file
to be included is decided during the compilation process.
Patch 3 adds a generic function to fork the current process and
execute a given file, which will be later used to execute hotplug
scripts synchronously.
Patches 4 and 5 add a new function to wait for a device to reach a
certain state, and replace wait_for_dev_destroy with this more generic
implementation. This function is also used to wait for device
initialization before calling hotplug scripts. The added function will
benefit from a rework after event support is added to libxl.
Finally patch 6 adds the calling of hotplug scripts when devices are
initializated and removed. Two new files are also added to support
hotplug scripts, since Linux and NetBSD hotplug scripts have different
call parameters. The path of the script to execute is retrieved
from xenstore. The file to include is also decided at compile
time.
Changes since v2:
* Use libxl_linux.c and libxl_netbsd.c to place OS specific
functions.
* Fixed libxl__forkexec comment, adding the need to pass NULL as the
last array member.
* Use XenbusState instead of state numbers.
* Use libxl_report_child_exit and return the exit status of the
process in libxl__forkexec.
* Pass action to execute to hotplug scripts, either CONNECT or
DISCONNECT.
Please review, Roger.
next reply other threads:[~2011-12-01 15:08 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-01 15:08 Roger Pau Monne [this message]
2011-12-01 15:08 ` [PATCH 01 of 11 v3] xenbackendd: pass type of block device to hotplug script Roger Pau Monne
2011-12-01 15:08 ` [PATCH 02 of 11 v3] libxl: add support for image files for NetBSD Roger Pau Monne
2011-12-01 15:08 ` [PATCH 03 of 11 v3] libxl: add libxl__forkexec function to libxl_exec Roger Pau Monne
2011-12-01 15:08 ` [PATCH 04 of 11 v3] libxl: introduce libxl__wait_for_device_state Roger Pau Monne
2011-12-01 15:08 ` [PATCH 05 of 11 v3] libxl: wait for devices to initialize upon addition to the domain Roger Pau Monne
2011-12-01 15:08 ` [PATCH 06 of 11 v3] libxl: execute hotplug scripts directly from libxl Roger Pau Monne
2011-12-01 15:08 ` [PATCH 07 of 11 v3] hotplug NetBSD: pass an action instead of a state to hotplug scripts Roger Pau Monne
2011-12-01 15:08 ` [PATCH 08 of 11 v3] xenbackendd: pass action to hotplug script Roger Pau Monne
2011-12-01 15:08 ` [PATCH 09 of 11 v3] hotplug: remove debug messages from NetBSD hotplug scripts Roger Pau Monne
2011-12-01 15:08 ` [PATCH 10 of 11 v3] rc.d NetBSD: don't start xenbackendd by default, only when xend needs it Roger Pau Monne
2011-12-01 15:08 ` [PATCH 11 of 11 v3] libxl: fix incorrect log message in libxl_domain_destroy Roger Pau Monne
2011-12-12 18:03 ` [PATCH 00 of 11 v3] libxl: add support for hotplug script calling from libxl Ian Jackson
2011-12-12 19:31 ` Roger Pau Monné
2011-12-13 11:24 ` Ian Jackson
2011-12-13 13:26 ` Roger Pau Monné
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.1322752087@loki.upc.es \
--to=roger.pau@entel.upc.edu \
--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 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.