[libvirt] [RFC]How to improve KVM VM resource assignment and per-vm process/thread scheduling.

2014-04-24 Thread Huangpeng (Peter)
Hi, ALL Currently kvm hypervisor have lots of features depend on linux standard apis, like vcpupin/mempin/processpin etc. But in the real production environment, we need an automated resource assign and/or scheduling, is there any plan to implement it? resource assignment requirements like: cpu

[libvirt] [PATCH] virsh: Move 'cpu-baseline' to host command group

2014-04-24 Thread Li Yang
As manual said, 'cpu-baseline' isn't specific to a domain, it should not belong to domain command group, and it's used for host usually, so move it to host command group. Signed-off-by: Li Yang --- tools/virsh-domain.c | 115 - tools/virsh-host.c

Re: [libvirt] [PATCH V3] libxl: add migration support

2014-04-24 Thread Jim Fehlig
Opps, forgot --subject-prefix="PATCH V3" when sending this patch... Regards, Jim Jim Fehlig wrote: > This patch adds initial migration support to the libxl driver, > using the VIR_DRV_FEATURE_MIGRATION_PARAMS family of migration > functions. > > Signed-off-by: Jim Fehlig > --- > > V3 of patch to

[libvirt] [PATCH] libxl: add migration support

2014-04-24 Thread Jim Fehlig
This patch adds initial migration support to the libxl driver, using the VIR_DRV_FEATURE_MIGRATION_PARAMS family of migration functions. Signed-off-by: Jim Fehlig --- V3 of patch to add migration support to the libxl driver. V2 is here https://www.redhat.com/archives/libvir-list/2014-March/msg

Re: [libvirt] [PATCH 9/9] Use virFileFindResource to locate CPU map XML

2014-04-24 Thread Eric Blake
On 04/24/2014 10:05 AM, Daniel P. Berrange wrote: > Replace use of cpuMapOverride with virFileFindResource > to locate CPU map from build dir. > > Signed-off-by: Daniel P. Berrange > --- > daemon/libvirtd.c| 19 --- > src/cpu/cpu_map.c| 31 ++--

Re: [libvirt] [PATCH 8/9] Use virFileFindResource to locate driver plugins

2014-04-24 Thread Eric Blake
On 04/24/2014 04:11 PM, Eric Blake wrote: > On 04/24/2014 10:05 AM, Daniel P. Berrange wrote: >> Replace virDriverModuleInitialize with virFileFindResource >> usage. >> >> Signed-off-by: Daniel P. Berrange >> --- >> daemon/libvirtd.c | 13 + >> src/driver.c|

Re: [libvirt] [PATCH 8/9] Use virFileFindResource to locate driver plugins

2014-04-24 Thread Eric Blake
On 04/24/2014 10:05 AM, Daniel P. Berrange wrote: > Replace virDriverModuleInitialize with virFileFindResource > usage. > > Signed-off-by: Daniel P. Berrange > --- > daemon/libvirtd.c | 13 + > src/driver.c| 26 +++--- > src/driver.h

Re: [libvirt] [PATCH 7/9] Use virFileFindResource to locate lock manager plugins

2014-04-24 Thread Eric Blake
On 04/24/2014 10:05 AM, Daniel P. Berrange wrote: > Replace virLockManagerSetPluginDir with virFileFindResource > usage. > > Signed-off-by: Daniel P. Berrange > --- > daemon/libvirtd.c | 1 - > src/locking/lock_manager.c | 28 +--- > src/locking/lock_manager.h |

Re: [libvirt] [PATCH 3/9] Use virFileFindResource to locate iohelper for virFileWrapperFdNew

2014-04-24 Thread Eric Blake
On 04/24/2014 10:05 AM, Daniel P. Berrange wrote: > From: Nehal J Wani > > Instead of hardcoding LIBEXECDIR as the location of the libvirt_iohelper > binary, use virFileFindResource to optionally find it in the current > build directory. > > Signed-off-by: Daniel P. Berrange > --- > src/util/v

Re: [libvirt] [PATCH] Add test suite for viralloc APIs

2014-04-24 Thread Eric Blake
On 04/24/2014 02:25 PM, Nehal J Wani wrote: +static int +testAllocScalar(const void *opaque ATTRIBUTE_UNUSED) +{ +testDummyStruct *t; +int ret = -1; + +if (VIR_ALLOC(t) < 0) +return -1; + +if (t == NULL) { +fprin

[libvirt] [PATCH] build: avoid 'index' as variable name

2014-04-24 Thread Eric Blake
Once again, gcc 4.4.7 (hello RHEL) rears its ugly head: conf/domain_conf.c: In function 'virDomainDiskBackingStoreFormat': conf/domain_conf.c:14940: error: declaration of 'index' shadows a global declaration [-Wshadow] /usr/include/string.h:489: error: shadowed declaration is here [-Wshadow] * s

Re: [libvirt] [PATCH v5 1/5] Introduce virDomainFSFreeze() and virDomainFSThaw() public API

2014-04-24 Thread Tomoki Sekiyama
On 4/24/14 4:58 , "Daniel P. Berrange" wrote: >On Thu, Apr 24, 2014 at 12:16:00AM +, Tomoki Sekiyama wrote: >> Hi Daniel, >> >> >> On 4/23/14 5:55 , "Daniel P. Berrange" wrote: >> >On Tue, Apr 22, 2014 at 06:22:18PM +, Tomoki Sekiyama wrote: >> >> Hi Daniel, >> >> thanks for your comme

Re: [libvirt] [PATCH 1/9] Add helpers for resolving path to resources in build tree

2014-04-24 Thread Eric Blake
On 04/24/2014 10:05 AM, Daniel P. Berrange wrote: > Add virFileFindResource which will try to locate files > in the local build tree if the calling binary (eg libvirtd or > test suite) is being run from the build tree. The corresponding > virFileActivateDirOverride should be called at startup passi

Re: [libvirt] [PATCH 2/9] Activate build dir overrides in libvirtd, virtlockd & tests

2014-04-24 Thread Eric Blake
On 04/24/2014 10:05 AM, Daniel P. Berrange wrote: > Add calls to virFileActivateDirOverride so that the build dir > overrides are activated. > > Signed-off-by: Daniel P. Berrange > --- > daemon/libvirtd.c | 2 ++ > src/locking/lock_daemon.c | 2 ++ > tests/testutils.c | 2 ++ > 3

Re: [libvirt] [PATCH] Add test suite for viralloc APIs

2014-04-24 Thread Nehal J Wani
>>> +static int >>> +testAllocScalar(const void *opaque ATTRIBUTE_UNUSED) >>> +{ >>> +testDummyStruct *t; >>> +int ret = -1; >>> + >>> +if (VIR_ALLOC(t) < 0) >>> +return -1; >>> + >>> +if (t == NULL) { >>> +fprintf(stderr, "Allocation succeeded by pointer is NULL\n")

Re: [libvirt] [PATCH 1/9] Add helpers for resolving path to resources in build tree

2014-04-24 Thread Eric Blake
On 04/24/2014 01:21 PM, Nehal J Wani wrote: >> + * virFileFindResourceFull: >> + * @filename: libvirt distributed filename without any path >> + * @prefix: optional string to prepend to filename >> + * @suffix: optional string to append to filename >> + * @builddir: location of the binary in the so

Re: [libvirt] [PATCH 5/4] qemuDomainBlockPull: Validate base in on the backing chain

2014-04-24 Thread Eric Blake
On 04/24/2014 07:15 AM, Jiri Denemark wrote: > This patch also adds support for addressing backing stores by index for > this API. > > Signed-off-by: Jiri Denemark > --- > > Notes: > - should probably be just squashed into the previous patch Yeah, squashing makes sense. > > src/qemu/qemu

Re: [libvirt] [PATCH 4/4] Add support for addressing backing stores by index

2014-04-24 Thread Eric Blake
On 04/22/2014 06:49 AM, Jiri Denemark wrote: > Each backing store of a given disk is associated with a unique index > (which is also formated in domain XML) for easier addressing of any s/formated/formatted/ > particular backing store. With this patch, any backing store can be > addressed by its

Re: [libvirt] [PATCH 3/4] virStorageFileChainLookup: Return virStorageSourcePtr

2014-04-24 Thread Eric Blake
On 04/22/2014 06:49 AM, Jiri Denemark wrote: > Returning both virStorageSourcePtr and its path member does not make a > lot of sense. Yep - artifacts from pre-refactoring that are now pointless. > > Signed-off-by: Jiri Denemark > --- > src/qemu/qemu_driver.c| 7 +++ > src/util/virstor

Re: [libvirt] [PATCH 1/9] Add helpers for resolving path to resources in build tree

2014-04-24 Thread Nehal J Wani
> + * virFileFindResourceFull: > + * @filename: libvirt distributed filename without any path > + * @prefix: optional string to prepend to filename > + * @suffix: optional string to append to filename > + * @builddir: location of the binary in the source tree build tree > + * @installdir: location

Re: [libvirt] [PATCH 2/4] qemuDomainBlockCommit: Track virStorageSourcePtr for base

2014-04-24 Thread Eric Blake
On 04/22/2014 06:49 AM, Jiri Denemark wrote: > virStorageFileChainLookup is able to give use virStorageSourcePtr which > contains the pointer to its canonical path. Let's use a more general > virStorageSourcePtr instead of just canonical path. > > Former base_canon maps to baseSource->path. > > S

Re: [libvirt] [PATCH 1/4] qemuDomainBlockCommit: Don't track top_canon path separately

2014-04-24 Thread Jiri Denemark
On Thu, Apr 24, 2014 at 08:47:34 -0600, Eric Blake wrote: > On 04/22/2014 06:49 AM, Jiri Denemark wrote: > > virStorageFileChainLookup is able to give use virStorageSourcePtr which > > contains the pointer to its canonical path. There's no need for the > > caller to store both of them. > > > > For

Re: [libvirt] KVM Connection issue

2014-04-24 Thread Eric Blake
On 04/24/2014 10:57 AM, Vikas Kokare wrote: > > *this function is not supported by the connection driver: > qemuGetSchedulerType Stack trace: > org.libvirt.ErrorHandler.processError(Unknown Source) > org.libvirt.Connect.processError(Unknown Source) > org.libvirt.Domain.processError(Unknown Source

[libvirt] KVM Connection issue

2014-04-24 Thread Vikas Kokare
We are using Libvirt Java API version 0.5.0 to connect RHEL 5 KVM, and fetch KVM environment attributes. While doing it, we are seeing the following exception: *this function is not supported by the connection driver: qemuGetSchedulerType Stack trace: org.libvirt.ErrorHandler.processError(Un

Re: [libvirt] [PATCH 1/1] Enable QEMU_CAPS_PCI_MULTIBUS capability for QEMU2.0 forward.

2014-04-24 Thread Daniel P. Berrange
On Wed, Apr 09, 2014 at 10:03:32AM +0800, Li Zhang wrote: > From: Li Zhang > > For QEMU2.0 forward version, it supports PCI multiBUS. > Currently, libvirt still disables it which causes an error > "Bus 'pci' not found". > > Signed-off-by: Li Zhang > --- > src/qemu/qemu_capabilities.c | 3 +++ >

[libvirt] [PATCH 9/9] Use virFileFindResource to locate CPU map XML

2014-04-24 Thread Daniel P. Berrange
Replace use of cpuMapOverride with virFileFindResource to locate CPU map from build dir. Signed-off-by: Daniel P. Berrange --- daemon/libvirtd.c| 19 --- src/cpu/cpu_map.c| 31 ++- src/cpu/cpu_map.h| 3 --- tests/cputest.c

[libvirt] [PATCH 3/9] Use virFileFindResource to locate iohelper for virFileWrapperFdNew

2014-04-24 Thread Daniel P. Berrange
From: Nehal J Wani Instead of hardcoding LIBEXECDIR as the location of the libvirt_iohelper binary, use virFileFindResource to optionally find it in the current build directory. Signed-off-by: Daniel P. Berrange --- src/util/virfile.c | 13 +++-- 1 file changed, 11 insertions(+), 2 del

[libvirt] [PATCH 5/9] Use virFileFindResource to locate parthelper for storage backend

2014-04-24 Thread Daniel P. Berrange
From: Nehal J Wani Instead of hardcoding LIBEXECDIR as the location of the libvirt_parthelper binary, use virFileFindResource to optionally find it in the current build directory. Signed-off-by: Daniel P. Berrange --- src/storage/storage_backend_disk.c | 31 --- 1 f

[libvirt] [PATCH 6/9] Use virFileFindResource to locate iohelper for fdstream

2014-04-24 Thread Daniel P. Berrange
From: Nehal J Wani Instead of hardcoding LIBEXECDIR as the location of the libvirt_iohelper binary, use virFileFindResource to optionally find it in the current build directory. Signed-off-by: Daniel P. Berrange --- src/fdstream.c | 21 ++--- src/fdstream.h

[libvirt] [PATCH 8/9] Use virFileFindResource to locate driver plugins

2014-04-24 Thread Daniel P. Berrange
Replace virDriverModuleInitialize with virFileFindResource usage. Signed-off-by: Daniel P. Berrange --- daemon/libvirtd.c | 13 + src/driver.c| 26 +++--- src/driver.h| 1 - tests/virdrivermoduletest.c | 2 -- 4 files ch

[libvirt] [PATCH 4/9] Use virFileFindResource to locate libvirt_lxc for capabilities

2014-04-24 Thread Daniel P. Berrange
From: Nehal J Wani Instead of hardcoding LIBEXECDIR as the location of the libvirt_lxc binary set in the LXC driver capabilities, use virFileFindResource to optionally find it in the current build directory. Signed-off-by: Daniel P. Berrange --- src/lxc/lxc_conf.c | 14 -- 1 file c

Re: [libvirt] [PATCHv2 0.5/4] network: fix virNetworkObjAssignDef and persistence

2014-04-24 Thread John Ferlan
On 04/23/2014 09:49 AM, Laine Stump wrote: > Experimentation showed that if virNetworkCreateXML() was called for a > network that was already defined, and then the network was > subsequently shutdown, the network would continue to be persistent > after the shutdown (expected/desired), but the ori

[libvirt] [PATCH 0/9] Standardize on lookup of resources in build dir

2014-04-24 Thread Daniel P. Berrange
Instead of having a bunch of custom override functions for each different area of code, standardize on one set of helper APIs for loading resources. This is derived from Nehal's proposal, based on my / Eric's feedback https://www.redhat.com/archives/libvir-list/2014-March/msg01544.html Daniel

[libvirt] [PATCH 7/9] Use virFileFindResource to locate lock manager plugins

2014-04-24 Thread Daniel P. Berrange
Replace virLockManagerSetPluginDir with virFileFindResource usage. Signed-off-by: Daniel P. Berrange --- daemon/libvirtd.c | 1 - src/locking/lock_manager.c | 28 +--- src/locking/lock_manager.h | 1 - 3 files changed, 9 insertions(+), 21 deletions(-) diff --g

[libvirt] [PATCH 2/9] Activate build dir overrides in libvirtd, virtlockd & tests

2014-04-24 Thread Daniel P. Berrange
Add calls to virFileActivateDirOverride so that the build dir overrides are activated. Signed-off-by: Daniel P. Berrange --- daemon/libvirtd.c | 2 ++ src/locking/lock_daemon.c | 2 ++ tests/testutils.c | 2 ++ 3 files changed, 6 insertions(+) diff --git a/daemon/libvirtd.c b/da

[libvirt] [PATCH 1/9] Add helpers for resolving path to resources in build tree

2014-04-24 Thread Daniel P. Berrange
Add virFileFindResource which will try to locate files in the local build tree if the calling binary (eg libvirtd or test suite) is being run from the build tree. The corresponding virFileActivateDirOverride should be called at startup passing in argv[0]. This will be examined for evidence of libto

Re: [libvirt] [PATCH] docs: document nmdm type console

2014-04-24 Thread Roman Bogorodskiy
Ján Tomko wrote: > On 04/21/2014 03:19 PM, Roman Bogorodskiy wrote: > > * Add nmdm type device to domain format documnetation > > * Add a section about nmdm console usage to the bhyve driver > > documentation > > --- > > docs/drvbhyve.html.in | 32 > > doc

Re: [libvirt] [PATCH] Add test suite for viralloc APIs

2014-04-24 Thread Eric Blake
On 04/09/2014 12:17 PM, Nehal J Wani wrote: > On Tue, Apr 8, 2014 at 8:18 PM, Daniel P. Berrange > wrote: >> In debugging a crash on OOM, I thought that the virInsert APIs >> might be at fault, but couldn't isolate them as a cause. While >> the viralloc APIs are used in many test suites, this is

[libvirt] [PATCH] qemu: properly quit migration with abort_on_error

2014-04-24 Thread Martin Kletzander
When EIO comes to qemu while it's replying to qemuMigrationUpdateJobStatus(), qemu blocks, the migration of RAM can complete in the meantime, and when qemu unblocks, it sends us BLOCK_IO_ERROR plus migrations "status": "complete". Even though we act upon the BLOCK_IO_ERROR by setting the proper st

Re: [libvirt] [PATCH v3] qemu: don't check for backing chains for formats w/o snapshot support

2014-04-24 Thread Martin Kletzander
On Thu, Apr 24, 2014 at 08:36:30AM -0600, Eric Blake wrote: On 04/24/2014 07:41 AM, Martin Kletzander wrote: Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1019926 Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=868673 Signed-off-by: Martin Kletzander --- +++ b/src/qemu/qemu_d

Re: [libvirt] [PATCH 1/4] qemuDomainBlockCommit: Don't track top_canon path separately

2014-04-24 Thread Eric Blake
On 04/22/2014 06:49 AM, Jiri Denemark wrote: > virStorageFileChainLookup is able to give use virStorageSourcePtr which > contains the pointer to its canonical path. There's no need for the > caller to store both of them. > > Former top_meta maps to topSource and top_canon maps to topSource->path.

Re: [libvirt] [PATCHv2 4/6] Use virGetUninstalledDir() in src/storage/storage_backend_disk.c

2014-04-24 Thread Eric Blake
On 04/24/2014 07:51 AM, Daniel P. Berrange wrote: > On Tue, Mar 25, 2014 at 01:53:14PM +0530, Nehal J Wani wrote: >> src/storage/storage_backend_disk.c: >>*Check if libvirtd is running uninstalled from a build tree and change >> parthelper_path accordingly >> > Hmm, it getting rather ugly

Re: [libvirt] [PATCH v3] qemu: don't check for backing chains for formats w/o snapshot support

2014-04-24 Thread Eric Blake
On 04/24/2014 07:41 AM, Martin Kletzander wrote: > Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1019926 > Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=868673 > > Signed-off-by: Martin Kletzander > --- > > +++ b/src/qemu/qemu_domain.c > @@ -2268,8 +2268,18 @@ qemuDomainCheckDis

Re: [libvirt] [PATCH 1/4] conf: Output disk backing store details in domain XML

2014-04-24 Thread Jiri Denemark
On Tue, Apr 22, 2014 at 13:02:17 +0200, Peter Krempa wrote: > On 04/21/14 10:32, Jiri Denemark wrote: > > The XML for quite a longish backing chain is shown below: ... > > s/driver/disk/ ? > > > +or network location contains the data of the described backing > > +store. >

Re: [libvirt] [PATCHv2 4/6] Use virGetUninstalledDir() in src/storage/storage_backend_disk.c

2014-04-24 Thread Daniel P. Berrange
On Tue, Mar 25, 2014 at 01:53:14PM +0530, Nehal J Wani wrote: > src/storage/storage_backend_disk.c: >*Check if libvirtd is running uninstalled from a build tree and change > parthelper_path accordingly > > --- > src/storage/storage_backend_disk.c | 28 ++-- > 1 f

Re: [libvirt] [PATCHv2 3/6] Use virGetUninstalledDir() in src/lxc/lxc_conf.c

2014-04-24 Thread Daniel P. Berrange
On Tue, Mar 25, 2014 at 01:53:13PM +0530, Nehal J Wani wrote: > src/lxc/lxc_conf.c: >*Check if libvirtd is running uninstalled from a build tree and change > lxc_path accordingly > > --- > src/lxc/lxc_conf.c | 19 +-- > 1 files changed, 17 insertions(+), 2 deletions(-) >

[libvirt] [PATCH v3] qemu: don't check for backing chains for formats w/o snapshot support

2014-04-24 Thread Martin Kletzander
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1019926 Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=868673 Signed-off-by: Martin Kletzander --- Notes: v3: - rebased on current master and this time it doesn't break startupPolicy for RAW disks v2: - sm

Re: [libvirt] [PATCHv2 2/6] Use virGetUninstalledDir() in src/util/virfile.c

2014-04-24 Thread Daniel P. Berrange
On Tue, Mar 25, 2014 at 01:53:12PM +0530, Nehal J Wani wrote: > src/util/virfile.c >*Check if libvirtd is running uninstalled from a build tree and change > iohelper_path accordingly > > --- > src/util/virfile.c | 19 +-- > 1 files changed, 17 insertions(+), 2 deletions(

Re: [libvirt] [PATCHv2 1/6] Add utility functions for storing uninstalled location

2014-04-24 Thread Daniel P. Berrange
On Tue, Mar 25, 2014 at 01:53:11PM +0530, Nehal J Wani wrote: > diff --git a/daemon/libvirtd.c b/daemon/libvirtd.c > index 4179147..dc3da2a 100644 > --- a/daemon/libvirtd.c > +++ b/daemon/libvirtd.c > @@ -1165,6 +1165,7 @@ int main(int argc, char **argv) { > exit(EXIT_FAILURE); >

[libvirt] [PATCH] Fix vlan ID detection in udev interface driver

2014-04-24 Thread Ján Tomko
Instead of guessing it from the interface name, look into /proc/net/vlan/. This works for devices not named ., avoiding an error flood when virt-manager keeps asking about them every second: https://bugzilla.redhat.com/show_bug.cgi?id=966329 --- src/interface/interface_backend_udev.c | 67 ++

[libvirt] [PATCH 5/4] qemuDomainBlockPull: Validate base in on the backing chain

2014-04-24 Thread Jiri Denemark
This patch also adds support for addressing backing stores by index for this API. Signed-off-by: Jiri Denemark --- Notes: - should probably be just squashed into the previous patch src/qemu/qemu_driver.c | 17 - 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a

Re: [libvirt] [PATCH v2] qemu: don't check for backing chains for formats w/o snapshot support

2014-04-24 Thread Martin Kletzander
On Thu, Apr 24, 2014 at 12:45:52PM +0200, Jiri Denemark wrote: On Thu, Apr 24, 2014 at 12:32:01 +0200, Martin Kletzander wrote: Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1019926 Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=868673 Signed-off-by: Martin Kletzander --- Notes

Re: [libvirt] [PATCHv2] util: storage: Invert the way recursive metadata retrieval works

2014-04-24 Thread Peter Krempa
On 04/24/14 14:19, Eric Blake wrote: > On 04/23/2014 03:19 PM, Peter Krempa wrote: >> To avoid having the root of a backing chain present twice in the list we >> need to invert the working of virStorageFileGetMetadataRecurse. >> >> Until now the recursive worker created a new backing chain element

Re: [libvirt] [PATCHv2 2/4] network: change location of network state xml files

2014-04-24 Thread Eric Blake
On 04/24/2014 04:30 AM, Laine Stump wrote: >>> >> ACK; however... Your call if you want to wait for virDirRead() from: >> >> http://www.redhat.com/archives/libvir-list/2014-April/msg00745.html >> >> Then refactor your for (;;) loop to use it... > > I think I would rather push it as-is, then send

Re: [libvirt] [PATCHv2] util: storage: Invert the way recursive metadata retrieval works

2014-04-24 Thread Eric Blake
On 04/23/2014 03:19 PM, Peter Krempa wrote: > To avoid having the root of a backing chain present twice in the list we > need to invert the working of virStorageFileGetMetadataRecurse. > > Until now the recursive worker created a new backing chain element from > the name and other information pass

Re: [libvirt] [PATCH v2] qemu: don't check for backing chains for formats w/o snapshot support

2014-04-24 Thread Jiri Denemark
On Thu, Apr 24, 2014 at 12:32:01 +0200, Martin Kletzander wrote: > Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1019926 > Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=868673 > > Signed-off-by: Martin Kletzander > --- > > Notes: > Smaller and cleaner variant of v1 [1] after

Re: [libvirt] [PATCHv2 3/4] network: set macvtap/hostdev networks active if their state file exists

2014-04-24 Thread Laine Stump
[Did you miss Patch 0.5/4? Or just get to the end of the day first?] On 04/23/2014 10:00 PM, John Ferlan wrote: > > On 04/17/2014 07:43 AM, Laine Stump wrote: >> libvirt attempts to determine at startup time which networks are >> already active, and set their active flags. Previously it has done >

[libvirt] [PATCH v2] qemu: don't check for backing chains for formats w/o snapshot support

2014-04-24 Thread Martin Kletzander
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1019926 Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=868673 Signed-off-by: Martin Kletzander --- Notes: Smaller and cleaner variant of v1 [1] after discussion with Eric [1] https://www.redhat.com/archives/libvir-list/201

Re: [libvirt] [PATCHv2 2/4] network: change location of network state xml files

2014-04-24 Thread Laine Stump
On 04/23/2014 09:52 PM, John Ferlan wrote: > > On 04/17/2014 07:43 AM, Laine Stump wrote: >> For some reason these have been stored in /var/lib, although other >> drivers (e.g. qemu and lxc) store their state files in /var/run. >> >> It's much nicer to store state files in /var/run because it is >>

Re: [libvirt] [PATCH] bhyve: implement domainGetCPUStats

2014-04-24 Thread Ján Tomko
On 04/24/2014 11:14 AM, Roman Bogorodskiy wrote: > Ján Tomko wrote: > >> On 04/22/2014 07:44 PM, Roman Bogorodskiy wrote: >>> For per CPU stats, implement virBhyveGetDomainPercpuStats() that >>> uses bhyvectl tool to obtain the guest's vcpu stats. >>> >> >> In virDomainGetCPUStats API, the start

Re: [libvirt] [PATCH] bhyve: implement domainGetCPUStats

2014-04-24 Thread Roman Bogorodskiy
Ján Tomko wrote: > On 04/22/2014 07:44 PM, Roman Bogorodskiy wrote: > > For per CPU stats, implement virBhyveGetDomainPercpuStats() that > > uses bhyvectl tool to obtain the guest's vcpu stats. > > > > In virDomainGetCPUStats API, the start_cpu and ncpus parameters refer to host > CPUs, not th

[libvirt] KVM connection issue

2014-04-24 Thread Vikas Kokare
We are using Libvirt Java API version 0.5.0 to connect RHEL 5 KVM, and fetch KVM environment attributes. While doing it, we are seeing the following exception: *this function is not supported by the connection driver: qemuGetSchedulerTypeStack trace: org.libvirt.ErrorHandler.processError(Unk

Re: [libvirt] [PATCH v5 1/5] Introduce virDomainFSFreeze() and virDomainFSThaw() public API

2014-04-24 Thread Daniel P. Berrange
On Thu, Apr 24, 2014 at 12:16:00AM +, Tomoki Sekiyama wrote: > Hi Daniel, > > > On 4/23/14 5:55 , "Daniel P. Berrange" wrote: > >On Tue, Apr 22, 2014 at 06:22:18PM +, Tomoki Sekiyama wrote: > >> Hi Daniel, > >> thanks for your comment. > >> > >> On 4/22/14 11:39 , "Daniel P. Berrange"

Re: [libvirt] [PATCH] docs: document nmdm type console

2014-04-24 Thread Ján Tomko
On 04/21/2014 03:19 PM, Roman Bogorodskiy wrote: > * Add nmdm type device to domain format documnetation > * Add a section about nmdm console usage to the bhyve driver > documentation > --- > docs/drvbhyve.html.in | 32 > docs/formatdomain.html.in | 29 ++

Re: [libvirt] [PATCH] Restore skipping of setting capacity

2014-04-24 Thread Daniel P. Berrange
On Wed, Apr 23, 2014 at 11:58:44AM -0600, Eric Blake wrote: > On 04/23/2014 07:28 AM, John Ferlan wrote: > > Commit id 'ac9a0963' refactored out the 'withCapacity' for the > > virStorageBackendUpdateVolInfo() API. See: > > Fortunately, we haven't released this regression of mine :) > > > > > ht

Re: [libvirt] [PATCH] bhyve: implement domainGetCPUStats

2014-04-24 Thread Ján Tomko
On 04/22/2014 07:44 PM, Roman Bogorodskiy wrote: > For per CPU stats, implement virBhyveGetDomainPercpuStats() that > uses bhyvectl tool to obtain the guest's vcpu stats. > In virDomainGetCPUStats API, the start_cpu and ncpus parameters refer to host CPUs, not the vcpus. Jan signature.asc Des

Re: [libvirt] [PATCHv2 1/1] Set pci-ohci as the USB default controller for PPC64.

2014-04-24 Thread Li Zhang
On 2014年04月23日 19:45, Daniel P. Berrange wrote: ACK and pushed. For older QEMU versions this change has no functional effect, but for 2.0.0 it is needed to avoid duplicate mouse/kbd as you describe. Got it, thanks. :) -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/ma