iptables resides in /usr/sbin on SuSE distros. Add it to path when
searching for iptables.
Regards,
Jim
diff -ur a/configure.in b/configure.in
--- a/configure.in 2008-05-05 13:46:20.0 -0600
+++ b/configure.in 2008-05-05 13:43:14.0 -0600
@@ -217,7 +217,7 @@
Hi All!
I came across some problems trying to create a new LVM-based storage
pool using this config
pool type=logical
nametest_vg/name
source
device path=/dev/sdb1/
/source
target
path/dev/test_vg/path
/target
/pool
Volume group did not previously exist so I did
virsh
Daniel Veillard wrote:
On Fri, Mar 31, 2006 at 04:47:38PM -0500, Daniel Veillard wrote:
I just added the following:
/**
* virDomainReboot:
* @domain: a domain object
* @flags: extra flags for the reboot operation, not used yet
*
* Reboot a domain, the domain object is still usable there
Daniel Veillard wrote:
On Thu, Apr 20, 2006 at 04:24:28PM -0600, Jim Fehlig wrote:
FYI,
I have managed to get one of the LTC providers (Xen_ComputerSystem)
working on SLES using openwbem cimom, the xm shim, and libvirt. The
following changes were made:
libvirt
- changed
Daniel P. Berrange wrote:
On Tue, Apr 18, 2006 at 05:40:07PM -0400, Daniel Veillard wrote:
On Tue, Apr 18, 2006 at 04:40:13PM -0400, Daniel Veillard wrote:
So we have 2 more APIs which allows to define the XML for a domain
and name it. That then allow to reserve that name, and the
Daniel Veillard wrote:
On Fri, Apr 07, 2006 at 04:10:36PM -0600, Jim Fehlig wrote:
Daniel,
FYI, I'm working on a patch to support hvm guests.
[snip]
Calling virsh dumpxml some_hvm_guest will (almost)
correctly dump xml config info on hvm guests. Needs to handle
type=ioemu
Has any progress been made on this proposal? I don't see anything in
current CVS. This functionality would be useful for Xen-CIM project.
Regards,
Jim
[EMAIL PROTECTED] wrote:
Corrections on proposal:
1) PinVcpus
Replace:
* @cpumap: pointer to a bit map of real CPUs (format in
Philippe Berthault wrote:
Daniel Veillard a écrit :
On Tue, Aug 01, 2006 at 05:06:59PM +0200, Philippe Berthault wrote:
[snip]
I wonder how people are most likely to use those APIs. Building scenarios
like:
- physical CPU is to be locked to serve only VCPU N in domain D
-
Daniel Veillard wrote:
On Wed, Aug 23, 2006 at 07:16:45PM +0100, Daniel P. Berrange wrote:
So I was thinking a little about lifecycle support for passive domains in
libvirt and wanted to clarify the intended semantics of the two methods:
virConnectListDomains
Stumbled across a problem trying to list domains with libvirt-0.1.5.
Using virsh I get errors such as
xen81:/tests/jim # virsh list
Id Name State
--
0 Domain-0 running
libvir: Xen Daemon error : GET operation failed: No such domain
Daniel P. Berrange wrote:
We currently have logic in the remote driver so that it handles the local
QEMU driver URIs, so they get re-directed to the daemon. It also handles
networking APIs for Xen driver. For normal APIs, Xen has the auto-spawned
setuid proxy daemon. This was very useful at
David Lutterkort wrote:
On Fri, 2007-10-26 at 09:08 -0400, Daniel Veillard wrote:
I can understand the need to make it easy for an user, I still don't
think this means those tuning informations need to be associated to the
domain definition, to me it is somehow orthogonal to the domain
Katti, Vadiraj (STSD-Openview) wrote:
Hi,
Currently the latest version of libvirt available is 0.3.3. But
SLES 10 ships 0.2.0 version of libvirt (not sure what version RHEL 5.1
ships). Any idea when and on what basis SLES decides to ship the
upgraded version of libvirt.
libvirt will be
Jim Fehlig wrote:
[...]
I have latest libvirt packages for 10.3 and SLE in my home project:
https://build.opensuse.org/package/show?project=home%3Ajfehligpackage=libvirt
Oops, sorry - I was logged in when I copied that link. For SLE you can
get rpms here
http://download.opensuse.org
This patch fixes some build error I've encountered, but would prefer an
ACK before pushing.
Thanks,
Jim
From ff81637b66793fd4b2b46e8e04357e8d406dde31 Mon Sep 17 00:00:00 2001
From: Jim Fehlig jfeh...@suse.com
Date: Mon, 5 Mar 2012 12:08:54 -0700
Subject: [PATCH] Fix build after commit e3ba4025
Eric Blake wrote:
On 03/05/2012 12:17 PM, Jim Fehlig wrote:
+++ b/src/util/virnetlink.h
@@ -55,7 +55,7 @@ int virNetlinkEventServiceStart(void);
/**
* virNetlinkEventServiceIsRunning: returns if the netlink event service is
running.
*/
-bool virNetlinkEventServiceIsRunning(void
Chunyan Liu wrote:
Add migration APIs for libxl driver.
Implemented in migration version 3. Based on xen 4.1.
I didn't get a chance to test this yet, but have some initial review
comments.
Signed-off-by: Chunyan Liu cy...@suse.com
---
src/libxl/libxl_driver.c | 617
Chunyan Liu wrote:
Chun Yan Liu cy...@suse.com 3/6/2012 2:29 PM
I didn't get a chance to test this yet, but have some initial review
comments.
Signed-off-by: Chunyan Liu
---
src/libxl/libxl_driver.c | 617
++
Chunyan Liu wrote:
Hi, Jim,
I made some changes to the patch according to your comments:
a. support concurrent migrations, add virBitmapPtr for probing migration ports
b. update doParseURI:
use virAsprintf instead of strdup and snprintf,
support migration URI syntax hostname[:port],
Chunyan Liu wrote:
2012/3/15 Jim Fehlig jfeh...@suse.com mailto:jfeh...@suse.com
+/* Create socket connection to receive migration data */
+if (!uri_in) {
+hostname = virGetHostname(dconn);
+if (hostname == NULL)
+goto cleanup
Jim Fehlig wrote:
+static int
+libxlDomainMigratePerform3(virDomainPtr dom,
+const char *xmlin ATTRIBUTE_UNUSED,
+const char *cookiein ATTRIBUTE_UNUSED,
+int cookieinlen ATTRIBUTE_UNUSED
Daniel Veillard wrote:
As scheduled, we are entering the freeze for 0.9.11.
I think most API additions are now commited to git upstream (please
raise your voice quickly if you see something missing !)
I have made a release candidate 1 tarball (and associated rpms) at
Jim Fehlig wrote:
Daniel Veillard wrote:
As scheduled, we are entering the freeze for 0.9.11.
I think most API additions are now commited to git upstream (please
raise your voice quickly if you see something missing !)
I have made a release candidate 1 tarball (and associated rpms
Ian Campbell wrote:
if (l_disk-driverName) {
...
} else {
/* No driverName - default to raw/tap?? */
x_disk-format = LIBXL_DISK_FORMAT_RAW;
x_disk-backend = LIBXL_DISK_BACKEND_TAP;
}
I recall some discussion about the choice of these defaults,
The libxl driver was setting the backend field of libxl_device_disk
structure to LIBXL_DISK_BACKEND_TAP when the driver element of disk
configuration was not specified. This needlessly forces the use of
blktap driver, which may not be loaded in dom0
Eric Blake wrote:
On 02/20/2013 01:31 PM, Jim Fehlig wrote:
The libxl driver was setting the backend field of libxl_device_disk
structure to LIBXL_DISK_BACKEND_TAP when the driver element of disk
configuration was not specified. This needlessly forces the use of
blktap driver, which may
With the apparmor security driver enabled, qemu instances fail
to start
# grep ^security_driver /etc/libvirt/qemu.conf
security_driver = apparmor
# virsh start test-kvm
error: Failed to start domain test-kvm
error: internal error security label already defined for VM
The model field of
Eric Blake wrote:
On 02/27/2013 04:51 PM, Jim Fehlig wrote:
With the apparmor security driver enabled, qemu instances fail
to start
# grep ^security_driver /etc/libvirt/qemu.conf
security_driver = apparmor
# virsh start test-kvm
error: Failed to start domain test-kvm
error: internal
Guannan Ren wrote:
On 03/01/2013 08:37 AM, Jim Fehlig wrote:
Eric Blake wrote:
On 02/27/2013 04:51 PM, Jim Fehlig wrote:
With the apparmor security driver enabled, qemu instances fail
to start
# grep ^security_driver /etc/libvirt/qemu.conf
security_driver = apparmor
# virsh start test
Eric Blake wrote:
On 03/05/2013 05:43 AM, John Ferlan wrote:
1. The virObjectLock() call was unconditional, but Unlock was conditional
on vm being valid. Removed the check
2. A call to virDomainEventNewFromObj() isn't guaranteed to return an
event - that check needs to be made
Guannan Ren wrote:
On 03/02/2013 12:41 AM, Jim Fehlig wrote:
Guannan Ren wrote:
Hi Jim
In selinux, libvirt added a label for tapfd.
Do you think this patch makes sense for apparmor?
https://www.redhat.com/archives/libvir-list/2012-October/msg01461.html
Hi Gunannan
Guannan Ren wrote:
Rename AppArmorSetImageFDLabel to AppArmorSetFDLabel which could
be used as a common function for *ALL* fd relabelling in Linux.
In apparmor profile for specific vm with uuid
cdbebdfa-1d6d-65c3-be0f-fd74b978a773
Path:
AL13N wrote:
Thread 1 (Thread 0x7fdef683b800 (LWP 20522)):
#0 0x in ?? ()
#1 0x7fdee9a72dc7 in libxl_osevent_occurred_timeout (ctx=optimized
out,
for_libxl=0x7fdedc001608) at libxl_event.c:1039
#2 0x7fdee9c9ff87 in libxlTimerCallback (timer=optimized out,
Noticed that parsing bond interface XML containing the miimon element
fails
interface type=bond name=bond0
...
bond mode=active-backup
miimon freq=100 carrier=netif/
...
/bond
/interface
This configuration does not contain the optional updelay and downdelay
Noticed that parsing bond interface XML containing the miimon element
fails
interface type=bond name=bond0
...
bond mode=active-backup
miimon freq=100 carrier=netif/
...
/bond
/interface
This configuration does not contain the optional updelay and downdelay
Eric Blake wrote:
On 03/21/2013 03:50 PM, Jim Fehlig wrote:
Noticed that parsing bond interface XML containing the miimon element
fails
interface type=bond name=bond0
...
bond mode=active-backup
miimon freq=100 carrier=netif/
...
/bond
/interface
Eric Blake wrote:
On 03/21/2013 05:13 PM, Jim Fehlig wrote:
Noticed that parsing bond interface XML containing the miimon element
fails
interface type=bond name=bond0
...
bond mode=active-backup
miimon freq=100 carrier=netif/
...
/bond
/interface
AL13N wrote:
Op woensdag 20 maart 2013 08:42:52 schreef Jim Fehlig:
AL13N wrote:
Thread 1 (Thread 0x7fdef683b800 (LWP 20522)):
#0 0x in ?? ()
#1 0x7fdee9a72dc7 in libxl_osevent_occurred_timeout (ctx=optimized
out, for_libxl=0x7fdedc001608) at libxl_event.c
Laine Stump wrote:
On 04/10/2013 05:10 AM, Daniel P. Berrange wrote:
On Wed, Apr 10, 2013 at 04:44:43AM +0200, Marek Marczykowski wrote:
This can be useful for route or NAT networks, or any other custom
network setup. Especially configuration example in documentation uses
script/
Daniel P. Berrange wrote:
On Wed, Apr 10, 2013 at 04:44:43AM +0200, Marek Marczykowski wrote:
---
src/libxl/libxl_conf.c | 72
++
src/libxl/libxl_conf.h | 2 ++
2 files changed, 74 insertions(+)
This needs todo more than just
Marek Marczykowski wrote:
libxl uses some xenstore entries for hints in memory management
(especially when starting new domain). This includes dom0 memory limit
and Xen free memory margin, based on current system state. Entries are
created at first usage, so force such usage at daemon startup,
On 04/11/2013 07:09 AM, Ian Campbell wrote:
On Thu, 2013-04-11 at 13:46 +0100, Ian Campbell wrote:
Hi Jim,
I don't see any calls to libxl_event_dispose in libvirt.git,
actually I think I meant libxl_event_free, but I can't see that either.
Right. I confirmed that was needed with Ian J. a
Carlos Rodrigues wrote:
Hello everybody,
I try to doing USB pass-through with XEN hypervisor using libvirt and i
get the following error:
# virsh attach-device c6test /tmp/usb_device.xml
error: Failed to attach device from /tmp/usb_device.xml
error: unsupported configuration: unsupported
Nearly a question for the users list, but I might be encountering a bug too.
I'm using virtlockd with 'auto_disk_leases = 0' and struggle with the
correct configuration for a lease in the domXML. Using
lease
lockspacetest-ls/lockspace
keytest-lock/key
target
Laine Stump wrote:
This was the only hypervisor driver other than qemu that implemented
virNodeDeviceDettach. It doesn't currently support multiple pci device
assignment driver backends, but it is simple to plug in this new API,
which will make it easier for Xen people to fill it in later when
Bamvor Jian Zhang wrote:
this patch fix the wrong sequence for fd and timeout register. the sequence
was right in dfa1e1dd for fd register, but it changed in e0622ca2.
in this patch, set priv, xl_priv in info and increase info-priv ref count
before virEventAddHandle. if do this after
AL13N wrote:
AL13N wrote:
Op woensdag 20 maart 2013 08:42:52 schreef Jim Fehlig:
AL13N wrote:
Thread 1 (Thread 0x7fdef683b800 (LWP 20522)):
#0 0x in ?? ()
#1 0x7fdee9a72dc7 in libxl_osevent_occurred_timeout
(ctx=optimized
out, for_libxl
Jim Fehlig wrote:
Laine Stump wrote:
On 04/10/2013 05:10 AM, Daniel P. Berrange wrote:
On Wed, Apr 10, 2013 at 04:44:43AM +0200, Marek Marczykowski wrote:
This can be useful for route or NAT networks, or any other custom
network setup. Especially configuration
David Scott wrote:
Something like the attached, which seems to work well for me when
specifying driverName = qemu, e.g.
disk type='file' device='disk'
driver name='qemu'/
source file='/var/lib/xen/images/sles11sp2-pv/disk0.raw'/
target dev='xvda' bus='xen'/
David Scott wrote:
libxl allows users to choose between two standard emulators:
1. (default in xen-4.2): qemu traditional (aka qemu-dm)
2. (default in xen-4.3): qemu upstream (aka qemu-system-i386)
The person who builds and packages xen gets to choose which
emulators are built. We examine
David Scott wrote:
We cross-check the given path against the capabilties, and translate
it into a libxl_device_model_version.
Signed-off-by: David Scott dave.sc...@eu.citrix.com
---
src/libxl/libxl_conf.c | 41 +
1 files changed, 41 insertions(+),
Daniel P. Berrange wrote:
On Mon, Apr 29, 2013 at 12:18:56PM -0600, Jim Fehlig wrote:
David Scott wrote:
libxl allows users to choose between two standard emulators:
1. (default in xen-4.2): qemu traditional (aka qemu-dm)
2. (default in xen-4.3): qemu upstream (aka qemu-system-i386
Jim Fehlig wrote:
David Scott wrote:
We cross-check the given path against the capabilties, and translate
it into a libxl_device_model_version.
Signed-off-by: David Scott dave.sc...@eu.citrix.com
---
src/libxl/libxl_conf.c | 41 +
1 files
libxlBuildDomainConfig() was disposing the libxl_domain_config object
on error, only to have it disposed again by libxlBuildDomainConfig()'s
caller, which resulted in a segfault. Leave disposing of the config
object to it's owner.
---
src/libxl/libxl_conf.c | 12
1 file changed, 4
disk format. Fix
this with the first patch, including adding format checks in the other
supported disk backends.
Jim Fehlig (2):
libxl: Fix disk format error message
libxl: support qdisk backend
src/libxl/libxl_conf.c | 49 +++--
1 file changed, 47
libxl supports the LIBXL_DISK_BACKEND_QDISK disk backend, where qemu
is used to provide the disk backend. This patch simply maps the
existing driver name='qemu'/ to LIBXL_DISK_BACKEND_QDISK.
---
src/libxl/libxl_conf.c | 25 +
1 file changed, 25 insertions(+)
diff --git
Specifying an unsupported disk format with the tap driver resulted in
a less than helpful error message
error: Failed to start domain test-hvm
error: internal error libxenlight does not support disk driver qed
Change the message to state that the qed format is not supported by
the tap driver,
Daniel P. Berrange wrote:
On Mon, Apr 29, 2013 at 12:18:56PM -0600, Jim Fehlig wrote:
David Scott wrote:
libxl allows users to choose between two standard emulators:
1. (default in xen-4.2): qemu traditional (aka qemu-dm)
2. (default in xen-4.3): qemu upstream (aka qemu-system-i386
Daniel P. Berrange wrote:
On Mon, Apr 29, 2013 at 11:01:04PM -0600, Jim Fehlig wrote:
Daniel P. Berrange wrote:
On Mon, Apr 29, 2013 at 12:18:56PM -0600, Jim Fehlig wrote:
David Scott wrote:
libxl allows users to choose between two standard emulators:
1
Eric Blake wrote:
On 04/29/2013 05:20 PM, Jim Fehlig wrote:
libxlBuildDomainConfig() was disposing the libxl_domain_config object
on error, only to have it disposed again by libxlBuildDomainConfig()'s
caller, which resulted in a segfault. Leave disposing of the config
object to it's
David Scott wrote:
The emulator path supplied can be any valid path on the system.
Note that when setting a device_model, libxl needs us to set the
device_model_version too. The device_model_version can be either
...QEMU_XEN: meaning upstream qemu, the default in xen-4.3 onwards
David Scott wrote:
Hi,
[added xen-devel: FYI this is about how to properly set the libxl
device_model_version when the user has provided a manual device_model
override (aka a path to a qemu) in the libvirt domain XML.]
On 30/04/13 16:10, Jim Fehlig wrote:
David Scott wrote:
The emulator
Ian Campbell wrote:
On Wed, 2013-05-01 at 04:42 +0100, Jim Fehlig wrote:
David Scott wrote:
Hi,
[added xen-devel: FYI this is about how to properly set the libxl
device_model_version when the user has provided a manual device_model
override (aka a path to a qemu) in the libvirt
Ian Campbell wrote:
On Wed, 2013-05-01 at 15:31 +0100, Jim Fehlig wrote:
Ian Campbell wrote:
On Wed, 2013-05-01 at 04:42 +0100, Jim Fehlig wrote:
David Scott wrote:
Hi,
[added xen-devel: FYI this is about how to properly set the libxl
Commit 7c9a2d88 missed inclusion of virstring.h in a few places
when WITH_HAL is defined, causing build failures.
---
Pushing under build-breaker rule.
src/node_device/node_device_driver.c | 1 +
src/node_device/node_device_hal.c| 1 +
2 files changed, 2 insertions(+)
diff --git
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
The Xen hypervisor driver checks for 'priv-handle 0' and
returns -1, but without raising any error. Fortunately this
code will never be executed, since the main Xen driver always
checks
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Some of the Xen sub-drivers have checks against the
VIR_CONNECT_RO flag. This is not required, since such
checks are done at the top level before the driver
methods are invoked
Signed-off-by: Daniel P. Berrange
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
The Xen driver uses a macro GET_PRIVATE as a supposed shorthand
for 'xenUnifiedPrivatePtr priv = (xenUnifiedPrivatePtr) (conn)-privateData'.
It does not in fact save any lines of code, and obscures what is
happening.
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Since the Xen driver was changed to only execute inside libvirtd,
there is no scenario in which it will be opened from a non-privileged
context. This all the code dealing with opening the sub-drivers can
s/This/Thus/
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
There is no point iterating over sub-drivers since the user
would not have a virConnectPtr instance at all if opening
the drivers failed. Just return 'Xen' immediately.
Signed-off-by: Daniel P. Berrange
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
The hypervisor driver is mandatory, so the the call to
xenHypervisorGetVersion must always succeed. Thus there
is no need to ever run xenDaemonGetVersion
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Unconditionally call into xenHypervisorGetMaxVcpus and
xenDaemonNodeGetInfo respectively, since those drivers
are both mandatory
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c | 15
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
The XenStore driver is mandatory, so it can be used unconditonally
for the xenUnifiedConnectListDomains xenUnifiedConnectNumOfDomains
drivers. Delete the unused XenD and Hypervisor driver code for
listing / counting
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Unconditionally call xenDaemonCreateXML in the
xenUnifiedDomainCreateXML driver, since the XenD
driver is always present.
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c| 6 ++
Jim Fehlig wrote:
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
The XenStore driver is mandatory, so it can be used unconditonally
for the xenUnifiedConnectListDomains xenUnifiedConnectNumOfDomains
drivers. Delete the unused XenD and Hypervisor driver code
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Unconditionally invoke the xenHypervisorLookupDomainByID,
xenHypervisorLookupDomainByUUID or xenDaemonLookupByName
for looking up domains. Fallback to xenXMDomainLookupByUUID
and xenXMDomainLookupByName for legacy XenD
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Unconditionally call xenDaemonLookupByUUID, since the XenD
driver must always be present.
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c | 41 +++--
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Update xenUnifiedDomainSuspend and xenUnifiedDomainResume to
unconditionally invoke the XenD APIs for suspend/resume. Delete
the impls in the hypervisor driver which was unreachable.
Signed-off-by: Daniel P. Berrange
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Unconditionally call the xenDaemonDomainDestroyFlags API
since the XenD driver is always available.
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c | 19 +---
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Make xenUnifiedDomainGetOSType directly call either the
xenHypervisorDomainGetOSType or xenDaemonDomainGetOSType
method depending on whether the domain is active or not.
Useful to add a note about removing the unused
Jim Fehlig wrote:
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Make xenUnifiedDomainGetOSType directly call either the
xenHypervisorDomainGetOSType or xenDaemonDomainGetOSType
method depending on whether the domain is active or not.
Useful to add
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
The xenGetHostname entry point in the xenUnifiedDriver table
was unused.
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.h | 1 -
1 file changed, 1 deletion(-)
ACK.
Regards,
Jim
Jim Fehlig wrote:
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Unconditionally invoke the xenHypervisorLookupDomainByID,
xenHypervisorLookupDomainByUUID or xenDaemonLookupByName
for looking up domains. Fallback to xenXMDomainLookupByUUID
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Simplify the Xen memory limit driver methods to directly call
the most appropriate sub-driver
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c | 50 ++-
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Make the xenUnifiedDomainGetInfo and xenUnifiedDomainGetState drivers
call the correct sub-driver APIs directly.
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c | 52
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Unconditionally call the XenD APIs for save/restore, since that
driver will always be open.
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c| 21 -
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c | 91
src/xen/xen_driver.h | 2 --
src/xen/xen_hypervisor.c | 90
Daniel P. Berrange wrote:
On Wed, May 08, 2013 at 11:37:44AM +0100, Daniel P. Berrange wrote:
On Mon, May 06, 2013 at 09:40:43PM -0600, Jim Fehlig wrote:
Jim Fehlig wrote:
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Unconditionally
Daniel P. Berrange wrote:
On Mon, May 06, 2013 at 03:01:25PM -0600, Jim Fehlig wrote:
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Unconditionally invoke the xenHypervisorLookupDomainByID,
xenHypervisorLookupDomainByUUID or xenDaemonLookupByName
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
The xenUnifiedDomainGetXMLDesc driver can assume that
the XM and XenD drivers are always present
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c| 24 +---
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
All the migration code is done by the XenD subdriver which
can be assumed to always be present
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c | 32 +---
1
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Directly call either XenD or the XM driver for handling
domain define operations.
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c| 28 ++--
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Make the domain define/undefine driver methods directly call
into either the XenD or XM drivers
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c| 24
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Directly call either the XenD or XM driver when starting
a persistent domain
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c| 20 +++-
src/xen/xen_driver.h| 1 -
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Make the domain attach/dettach driver methods directly call
into either the XenD or XM drivers
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c| 55
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Make the Xen domain scheduler parameter methods directly
call into XenD or Xen hypervisor drivers
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c | 54
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Unconditionally call into the XenD or XM drivers for autostart
handling, since they are guaranteed to be open
---
src/xen/xen_driver.c| 18 --
src/xen/xend_internal.c | 14 --
2 files
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Make the Xen domain stats / peek and node memory driver
methods unconditionally call the sub-drivers which are
guaranteed to be open.
Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
src/xen/xen_driver.c|
Daniel P. Berrange wrote:
From: Daniel P. Berrange berra...@redhat.com
Introduce use of a virDomainDefPtr in the domain lookup
APIs to simplify introduction of ACL security checks.
The virDomainPtr cannot be safely used, since the app
may have supplied mis-matching name/uuid/id fields. eg
Daniel P. Berrange wrote:
Thanks for all the reviews so far ! I'm going to push the first 29
patches now. I've tested basic operations and things seem to be
working sanely to me.
Nice. I've done a fair bit of testing with the first 30 patches applied
(including the tweeks to 10 and 24)
1 - 100 of 2571 matches
Mail list logo