If we use p2p migration and the vm is transient, vm will be deleted in
qemuDomainMigrateConfirm3(). But the caller(doPeer2PeerMigrate() and
qemuMigrationPerform()) still access it, and it will cause libvirtd crashed.
---
src/qemu/qemu_migration.c |7 ++-
1 files changed, 6 insertions(+),
If we do not performe a live migration, the vm is always paused when we go to
doPeer2PeerMigrate2() and doPeer2PeerMigrate3(), because we stop the vm in
qemuMigrationPerform(). So we should check it before setting it offline.
---
src/qemu/qemu_migration.c |9 +++--
1 files changed, 3
Wen Congyang (2):
avoid vm is deleted in qemuDomainMigrateConfirm3()
check whether the vm is paused before setting it offline
src/qemu/qemu_migration.c | 16 +---
1 files changed, 9 insertions(+), 7 deletions(-)
--
libvir-list mailing list
libvir-list@redhat.com
于 2011年05月19日 15:34, Daniel Veillard 写道:
On Sun, May 15, 2011 at 09:37:21PM -0400, Mark Wagner wrote:
On 05/12/2011 06:45 AM, Daniel P. Berrange wrote:
On Thu, May 12, 2011 at 06:22:49PM +0800, Osier Yang wrote:
Hi, All
This series adopts Daniel's suggestion on v1, using libnuma but
not
On Fri, May 20, 2011 at 03:22:10PM +0800, Osier Yang wrote:
于 2011年05月19日 15:34, Daniel Veillard 写道:
On Sun, May 15, 2011 at 09:37:21PM -0400, Mark Wagner wrote:
On 05/12/2011 06:45 AM, Daniel P. Berrange wrote:
On Thu, May 12, 2011 at 06:22:49PM +0800, Osier Yang wrote:
Hi, All
This
On Thu, May 19, 2011 at 05:14:54PM -0500, Christian Benvenuti (benve) wrote:
I have one generic question.
I know it is not a new problem, and it is not necessarily something
worth investing time and code on, but what if the admin
(directly or indirectly by activating some features)
changes
On Fri, May 20, 2011 at 03:09:42PM +0800, Wen Congyang wrote:
If we do not performe a live migration, the vm is always paused when we go to
doPeer2PeerMigrate2() and doPeer2PeerMigrate3(), because we stop the vm in
qemuMigrationPerform(). So we should check it before setting it offline.
The
On Fri, May 20, 2011 at 03:09:09PM +0800, Wen Congyang wrote:
If we use p2p migration and the vm is transient, vm will be deleted in
qemuDomainMigrateConfirm3(). But the caller(doPeer2PeerMigrate() and
qemuMigrationPerform()) still access it, and it will cause libvirtd crashed.
---
Hi all,
This patchset enables us to configure inactive domains' CPU affinity setting.
v3 - v4:
- Rebase
*[PATCHv4 1/4] vcpupin: inroduce a new libvir API (virDomainPinVcpuFlags)
*[PATCHv4 2/4] vcpupin: implement the code to address the new API in the qemu
driver
*[PATCHv4 3/4] vcpupin:
On Fri, May 13, 2011 at 10:14:20AM +0200, Paolo Bonzini wrote:
On 05/11/2011 09:55 AM, Daniel P. Berrange wrote:
Hmm, that's kinda wierd i'm suprised it works, particularly for LSI
since I thought guest drivers would need support for multifunction too.
For well-behaved
This patch introduces a new libvirt API (virDomainPinVcpuFlags)
Signd-off-by: Taku Izumi izumi.t...@jp.fujitsu.com
---
include/libvirt/libvirt.h.in |6 +++
src/driver.h |7 +++
src/libvirt.c| 83 +++
This patch implements the code to address the new API (virDomainPinVcpuFlags)
in the qemu driver.
Signed-off-by: Taku Izumi izumi.t...@jp.fujitsu.com
---
src/qemu/qemu_driver.c | 99 +
1 file changed, 76 insertions(+), 23 deletions(-)
Index:
This patch implements the remote protocol to address the new API
(virDomainPinVcpuFlags).
Signd-off-by: Taku Izumi izumi.t...@jp.fujitsu.com
---
daemon/remote.c | 42 ++
src/remote/remote_driver.c |1 +
src/remote/remote_protocol.x |
This patch adds the new option (--live, --config and --current) to
virsh vcpupin command. The behavior of above aption is the same as
that of virsh setmem, virsh setvcpus, and whatnot.
When the --config option is specified, the command affects a persistent domain,
while --live option is
On Fri, May 13, 2011 at 11:32:43AM +0900, KAMEZAWA Hiroyuki wrote:
On Wed, 11 May 2011 08:55:56 +0100
Daniel P. Berrange berra...@redhat.com wrote:
On Wed, May 11, 2011 at 10:30:43AM +0800, Wen Congyang wrote:
3. After hot plug multi function PCI devices:
# lspci
00:00.0 Host
Change log level order so that messages at all other levels get logged
for DEBUG level. Replace magic numbers with corresponding VSH_ERR_xx.
Signed-off-by: Supriya Kannery supri...@in.ibm.com
---
tools/virsh.c | 119
--
1 file changed,
On Fri, May 20, 2011 at 03:05:28PM +0530, Supriya Kannery wrote:
Change log level order so that messages at all other levels get logged
for DEBUG level. Replace magic numbers with corresponding VSH_ERR_xx.
Signed-off-by: Supriya Kannery supri...@in.ibm.com
---
tools/virsh.c | 119
On 05/20/2011 05:13 AM, Eric Blake wrote:
On 05/09/2011 01:08 AM, Supriya Kannery wrote:
@@ -2103,7 +2103,7 @@ cmdDominfo(vshControl *ctl, const vshCmd
/* Check and display whether the domain is persistent or not */
persistent = virDomainIsPersistent(dom);
-vshDebug(ctl, 5,
Hi,
Has the git snapshot tar service stopped working?
The file is dated 6th May and is very out of date compared to git head.
Rgs
Neil
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
On Thu, May 19, 2011 at 12:19:11PM -0600, Eric Blake wrote:
On 05/09/2011 01:07 AM, Supriya Kannery wrote:
Defining environment variables for debug log-level and log-file will avoid
specifying the same in each virsh command. Following is a patchset
for enabling env variable support for
The qemuMigrationConfirm method shouldn't deal with final VM
cleanup, since it can be called from the peer2peer migration,
which expects to still use the 'vm' object afterwards.
Push the cleanup code out of qemuMigrationConfirm, into its
caller, qemuDomainMigrateConfirm3
*
On Fri, May 20, 2011 at 09:36:12AM +0100, Daniel P. Berrange wrote:
On Fri, May 20, 2011 at 03:09:09PM +0800, Wen Congyang wrote:
If we use p2p migration and the vm is transient, vm will be deleted in
qemuDomainMigrateConfirm3(). But the caller(doPeer2PeerMigrate() and
On Tue, May 17, 2011 at 04:01:36PM +0900, Minoru Usui wrote:
virNodeGetCPUTimeParameters: Expose new API
Signed-off-by: Minoru Usui u...@mxm.nes.nec.co.jp
---
include/libvirt/libvirt.h.in | 65
++
src/libvirt_public.syms |1 +
2 files
On Tue, May 17, 2011 at 04:01:46PM +0900, Minoru Usui wrote:
virNodeGetCPUTimeParameters: Define internal driver API
Signed-off-by: Minoru Usui u...@mxm.nes.nec.co.jp
---
src/driver.h |8
1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/src/driver.h
On Tue, May 17, 2011 at 04:02:02PM +0900, Minoru Usui wrote:
virNodeGetCPUTime: Implement public API
Signed-off-by: Minoru Usui u...@mxm.nes.nec.co.jp
---
src/libvirt.c | 87
+
1 files changed, 87 insertions(+), 0 deletions(-)
On 05/20/2011 03:19 PM, Daniel P. Berrange wrote:
On Thu, May 19, 2011 at 12:19:11PM -0600, Eric Blake wrote:
On 05/09/2011 01:07 AM, Supriya Kannery wrote:
a. VIRSH_DEBUG=log_level
log_level can be a value between 0 - 4, where
0 - ERROR
1 - WARNING
2 - NOTICE
3 - INFO
On Tue, May 17, 2011 at 04:02:21PM +0900, Minoru Usui wrote:
virNodeGetCPUTimeParameters: Implement remote protocol
Signed-off-by: Minoru Usui u...@mxm.nes.nec.co.jp
---
daemon/remote.c | 76
++
src/remote/remote_driver.c | 65
On Tue, May 17, 2011 at 04:01:10PM +0900, Minoru Usui wrote:
virNodeGetCPUTimeParameters: Implement linux support
Signed-off-by: Minoru Usui u...@mxm.nes.nec.co.jp
---
src/libvirt_private.syms |1 +
src/lxc/lxc_driver.c |1 +
src/nodeinfo.c | 127
On Tue, May 17, 2011 at 04:00:07PM +0900, Minoru Usui wrote:
virNodeGetCPUTimeParameters: Implement virsh support
Signed-off-by: Minoru Usui u...@mxm.nes.nec.co.jp
---
tools/virsh.c | 107
+++
tools/virsh.pod |4 ++
2 files
On Fri, May 20, 2011 at 04:17:12PM +0530, Supriya Kannery wrote:
On 05/20/2011 03:19 PM, Daniel P. Berrange wrote:
On Thu, May 19, 2011 at 12:19:11PM -0600, Eric Blake wrote:
On 05/09/2011 01:07 AM, Supriya Kannery wrote:
a. VIRSH_DEBUG=log_level
log_level can be a value between 0 - 4,
On 05/20/2011 11:06 AM, Daniel P. Berrange wrote:
So if I'm understanding correctly, the only difference we'll see with
assigning devices to functions, instead of slots, is that hotplug of
individual devices is not possible. From a guest (driver) POV everything
else is 100% functionally
At 05/20/2011 04:34 PM, Daniel P. Berrange Write:
On Fri, May 20, 2011 at 03:09:42PM +0800, Wen Congyang wrote:
If we do not performe a live migration, the vm is always paused when we go to
doPeer2PeerMigrate2() and doPeer2PeerMigrate3(), because we stop the vm in
qemuMigrationPerform(). So we
On 05/19/2011 03:34 AM, Daniel Veillard wrote:
On Sun, May 15, 2011 at 09:37:21PM -0400, Mark Wagner wrote:
On 05/12/2011 06:45 AM, Daniel P. Berrange wrote:
On Thu, May 12, 2011 at 06:22:49PM +0800, Osier Yang wrote:
Hi, All
This series adopts Daniel's suggestion on v1, using libnuma but
On Wed, May 18, 2011 at 02:08:12PM -0500, Adam Litke wrote:
Version 2 of the block device streaming API diverged
significantly from the original API provided by Qemu.
In an effort to provide an API that is designed to work
well for the specific task of streaming data to a local
device from
On Fri, May 20, 2011 at 07:00:54PM +0800, Wen Congyang wrote:
At 05/20/2011 04:34 PM, Daniel P. Berrange Write:
On Fri, May 20, 2011 at 03:09:42PM +0800, Wen Congyang wrote:
If we do not performe a live migration, the vm is always paused when we go
to
doPeer2PeerMigrate2() and
On 05/19/2011 06:15 PM, Christian Benvenuti (benve) wrote:
-Original Message-
From: libvir-list-boun...@redhat.com [mailto:libvir-list-
boun...@redhat.com] On Behalf Of Laine Stump
Sent: Thursday, May 19, 2011 1:51 PM
To: libvir-list@redhat.com
Subject: [libvirt] [PATCHv2 1/7] interface:
2011/5/20 Daniel P. Berrange berra...@redhat.com:
On Tue, May 17, 2011 at 04:01:36PM +0900, Minoru Usui wrote:
virNodeGetCPUTimeParameters: Expose new API
Signed-off-by: Minoru Usui u...@mxm.nes.nec.co.jp
---
include/libvirt/libvirt.h.in | 65
++
On Fri, May 20, 2011 at 07:06:46AM -0400, Mark Wagner wrote:
On 05/19/2011 03:34 AM, Daniel Veillard wrote:
Hi Mark,
I think that maybe I didn't fully understand the entire context.
Well I wasn't sure either if the problem was lack
of expressivity or some problem in the syntax :-)
My
Hi Dan, thank you for another round of review. Stefan and Anthony,
could you respond to Dan's suggestion of adding qemu events for stream
completion and errors at the bottom of this message?
On 05/20/2011 06:07 AM, Daniel P. Berrange wrote:
On Wed, May 18, 2011 at 02:08:12PM -0500, Adam Litke
On Fri, May 20, 2011 at 07:42:26AM -0500, Adam Litke wrote:
Hi Dan, thank you for another round of review. Stefan and Anthony,
could you respond to Dan's suggestion of adding qemu events for stream
completion and errors at the bottom of this message?
On 05/20/2011 06:07 AM, Daniel P.
On Fri, May 20, 2011 at 01:25:46PM +0800, Guangya Liu wrote:
Hi libvirt support,
Found an interesting problem for Rhel, libvirt 0.8.2 is shipped with the
distribution of Rhel5u6; libvirt 0.8.1 is shipped with the distributions
of Rhel6, do you know why, why did we downgrade the version for
On Fri, May 20, 2011 at 07:42:26AM -0500, Adam Litke wrote:
In the VIR_DOMAIN_BLOCK_STREAM_CONTINUOUS case, I think it is desirable
to have an async event emitted when the streaming is completed, so we
don't force apps to poll on the GetBlockStreamInfo if they only care
about completion.
Thanks Daniel for the quick response!
OK, I will post to Rhel Support to see if I can get some help.
Cheers,
Guangya
-Original Message-
From: Daniel P. Berrange [mailto:berra...@redhat.com]
Sent: Friday, May 20, 2011 10:15 PM
To: Guangya Liu
Cc: libvir-list@redhat.com
Subject: Re:
This attached patch resolves a fairly obvious problem which stops
qemuDomainMemoryPeek from working at all.
However, it's not the whole story. Read on ...
(1) qemu still fails unless I manually set the mode of
/var/cache/libvirt to 0755 (it is normally 0700).
Owner
On Wed, May 18, 2011 at 04:57:53PM +0800, Osier Yang wrote:
This patch seperate the domain config loading just as qemu driver
does, first loading config of running or trasient domains, then
of persistent inactive domains. And only try to reconnect the
monitor of running domains, so that it
On Fri, May 20, 2011 at 03:28:14PM +0100, Richard W.M. Jones wrote:
This attached patch resolves a fairly obvious problem which stops
qemuDomainMemoryPeek from working at all.
However, it's not the whole story. Read on ...
(1) qemu still fails unless I manually set the mode of
On Fri, May 20, 2011 at 03:40:35PM +0100, Daniel P. Berrange wrote:
I think we likely need /var/cache/libvirt to be 0711 so that
QEMU can access directories below it, but not actually read it.
0711 does indeed work fine. However, where/what sets this?
Oh, there is a bogus 'if (dom)
Hi,
something like disk type='block' driver name='phy'/ source dev='%s'/
target dev='sdb'/ /disk used to be allowed. But now it appears (looking at
git history) only driver name='qemu' is supported. Is that correct? Should
tools which are spitting out libvirt xml be updated to use 'qemu', or
On Fri, May 20, 2011 at 03:57:03PM +0100, Richard W.M. Jones wrote:
On Fri, May 20, 2011 at 03:40:35PM +0100, Daniel P. Berrange wrote:
I think we likely need /var/cache/libvirt to be 0711 so that
QEMU can access directories below it, but not actually read it.
0711 does indeed work fine.
2011/5/16 Eric Blake ebl...@redhat.com:
On 05/15/2011 12:23 AM, Matthias Bolte wrote:
Replace $calls{$_} with $call in the dispatch bodies generator function.
No functional change included.
---
daemon/remote_generator.pl | 100
++-
1 files
On Fri, May 20, 2011 at 10:00:55AM -0500, Serge Hallyn wrote:
Hi,
something like disk type='block' driver name='phy'/ source dev='%s'/
target dev='sdb'/ /disk used to be allowed. But now it appears (looking
at
git history) only driver name='qemu' is supported. Is that correct? Should
On Fri, May 20, 2011 at 04:19:55PM +0100, Richard W.M. Jones wrote:
On Fri, May 20, 2011 at 04:02:41PM +0100, Daniel P. Berrange wrote:
On Fri, May 20, 2011 at 03:57:03PM +0100, Richard W.M. Jones wrote:
On Fri, May 20, 2011 at 03:40:35PM +0100, Daniel P. Berrange wrote:
I think we
On Fri, May 20, 2011 at 04:02:41PM +0100, Daniel P. Berrange wrote:
On Fri, May 20, 2011 at 03:57:03PM +0100, Richard W.M. Jones wrote:
On Fri, May 20, 2011 at 03:40:35PM +0100, Daniel P. Berrange wrote:
I think we likely need /var/cache/libvirt to be 0711 so that
QEMU can access
2011/5/16 Eric Blake ebl...@redhat.com:
On 05/16/2011 12:14 PM, Matthias Bolte wrote:
This patch depends on this unapplied series:
https://www.redhat.com/archives/libvir-list/2011-May/msg00987.html
which is now reviewed.
Subject: [PATCH] remote generator: Add special case for
Hi everyone,
I'm trying to test out the virtio driver's tx=bh option, or iothread in
libvirt. When I create a machine, I get the unsupported error message:
# virsh create web101.xml
error: Failed to create domain from web101.xml
error: unsupported configuration: vhost-net is not supported with
I should also add that I did compile qemu with vhost-net enabled:
# /usr/local/bin/qemu-system-x86_64 -device virtio-net-pci,?
virtio-net-pci.ioeventfd=on/off
virtio-net-pci.vectors=uint32
virtio-net-pci.indirect_desc=on/off
virtio-net-pci.csum=on/off
virtio-net-pci.guest_csum=on/off
On 05/20/2011 12:15 PM, Cal Heldenbrand wrote:
Hi everyone,
I'm trying to test out the virtio driver's tx=bh option, or iothread
in libvirt. When I create a machine, I get the unsupported error
message:
# virsh create web101.xml
error: Failed to create domain from web101.xml
error:
On 05/19/2011 07:29 PM, Eric Blake wrote:
On 05/10/2011 09:52 AM, Laine Stump wrote:
(I want to make sure that nobody thinks opening a transaction means that
all the changes are saved into a staging area and then later committed
to the live config all at once when the transaction is committed -
Port was committed to the FreeBSD portstree!
Many thanks,
Jason
--
Jason Helfman
System Administrator
experts-exchange.com
http://www.experts-exchange.com/M_4830110.html
E4AD 7CF1 1396 27F6 79DD 4342 5E92 AD66 8C8C FBA5
---BeginMessage---
Synopsis: [new port] devel/libvirt: Toolkit to interact
This thread may be of interest:
http://lists.gnu.org/archive/html/qemu-devel/2011-05/threads.html#02162
Currently, qemu silently clips any JSON integer in the range
0x8000___ - 0x___ (all numbers in this
range will be clipped to 0x7fff___ == LLONG_MAX).
On Thu, May 19, 2011 at 6:24 AM, Daniel P. Berrange berra...@redhat.com wrote:
Sanlock is a project that implements a disk-paxos locking
algorithm. This is suitable for cluster deployments with
shared storage.
* src/Makefile.am: Add dlopen plugin for sanlock
*
When using TLS authentication and operating as the non-root user,
initially attempt to use that specific user's TLS certificates before
attempting to use the system wide TLS certificates.
Signed-off-by: Doug Goldstein car...@cardoe.com
---
src/remote/remote_driver.c |2 +-
1 files changed, 1
On 05/20/2011 01:12 PM, Cal Heldenbrand wrote:
First look in /var/log/libvirt/qemu/domainname.log and see which
qemu binary is being executed by libvirt - it may not be the one
you've built.
It looks correct... I pulled out the driver tag from my config and
fired it up. Here
Eric Blake ebl...@redhat.com writes:
I'm getting this when trying to build libvirt on rawhide (using the
package gnutls-devel-2.12.3-1.fc16.x86_64):
remote/remote_driver.c: In function 'negotiate_gnutls_on_connection':
remote/remote_driver.c:1361:9: error:
On 05/20/2011 04:08 AM, Taku Izumi wrote:
/**
+ * virDomainPinVcpuFlags:
+ * @domain: pointer to domain object, or NULL for Domain0
+ * @vcpu: virtual CPU number
+ * @cpumap: pointer to a bit map of real CPUs (in 8-bit bytes) (IN)
+ * Each bit set to 1 means that corresponding CPU is
On 05/20/2011 04:09 AM, Taku Izumi wrote:
--- libvirt.orig/src/qemu/qemu_driver.c
+++ libvirt/src/qemu/qemu_driver.c
@@ -2859,17 +2859,24 @@ qemudDomainSetVcpus(virDomainPtr dom, un
static int
-qemudDomainPinVcpu(virDomainPtr dom,
- unsigned int vcpu,
-
On 05/20/2011 04:10 AM, Taku Izumi wrote:
--- libvirt.orig/tools/virsh.c
+++ libvirt/tools/virsh.c
@@ -2757,6 +2757,9 @@ static const vshCmdOptDef opts_vcpupin[]
{domain, VSH_OT_DATA, VSH_OFLAG_REQ, N_(domain name, id or uuid)},
{vcpu, VSH_OT_INT, VSH_OFLAG_REQ, N_(vcpu number)},
On 05/16/2011 07:30 PM, Eric Blake wrote:
+while (priv-jobSignals QEMU_JOB_SIGNAL_BLKSTAT)
+ignore_value(virCondWait(priv-signalCond,vm-lock));
+
+priv-jobSignalsData.statDevName = disk-info.alias;
+priv-jobSignalsData.blockStat = stats;
+
On 05/20/2011 03:11 PM, Paolo Bonzini wrote:
On 05/16/2011 07:30 PM, Eric Blake wrote:
+while (priv-jobSignals QEMU_JOB_SIGNAL_BLKSTAT)
+ignore_value(virCondWait(priv-signalCond,vm-lock));
+
+priv-jobSignalsData.statDevName = disk-info.alias;
+
On 05/20/2011 02:54 PM, Adam Litke wrote:
I am really confused about what the default flags
(VIR_DOMAIN_VCPU_CURRENT) actually means. It doesn't seem like I can
ever know what it means from an API perspective (since it's behavior is
arbitrary depending on the hypervisor). Wouldn't it be
On 05/20/2011 03:11 PM, Adam Litke wrote:
On 05/20/2011 04:10 AM, Taku Izumi wrote:
--- libvirt.orig/tools/virsh.c
+++ libvirt/tools/virsh.c
@@ -2757,6 +2757,9 @@ static const vshCmdOptDef opts_vcpupin[]
{domain, VSH_OT_DATA, VSH_OFLAG_REQ, N_(domain name, id or uuid)},
{vcpu,
Hi Laine,
It appears that patch worked! I grabbed the latest git source for both
libvirt and qemu, applied your patch and reinstalled. After running through
a few more setup stuff, I managed to get my web machine up and running with
tx=bh.
However, I didn't notice any latency improvements
72 matches
Mail list logo