On 08/15/2011 07:33 PM, Eric Blake wrote:
Call me lazy, but:
virsh qemu-monitor-command dom --hmp info status
is nicer than:
virsh qemu-monitor-command dom --hmp 'info status'
* tools/virsh.c (cmdQemuMonitorCommand): Allow multiple arguments,
for convenience.
Looks like a nice backward-comp
Hi Daniel,
The last version of my patch fixed bug #611823 prohibit pools with duplicate
storage,
you gave some comments that should do check by source info instead of target
path I used.
These days I view the code and existing documents about storage pool, and tried
to
code based on your comme
于 2011年08月16日 22:58, Eric Blake 写道:
On 08/16/2011 09:05 AM, Osier Yang wrote:
We don't allow to define domain with same name and different UUID,
or with same UUID, so it's reasonable to not load the domain config
if there is domain with same name already exists.
Otherwise it can cause problem l
On Tue, Aug 16, 2011 at 11:39 PM, Daniel P. Berrange
wrote:
> On Tue, Aug 16, 2011 at 09:28:27AM +0800, Zhi Yong Wu wrote:
>> On Tue, Aug 16, 2011 at 6:52 AM, Stefan Hajnoczi wrote:
>> > On Mon, Aug 15, 2011 at 1:36 PM, Adam Litke wrote:
>> >> On 08/14/2011 11:40 PM, Zhi Yong Wu wrote:
>> >>> HI
On Tue, 2011-08-16 at 09:31 -0500, Adam Litke wrote:
> Stefan has a git repo with QED block streaming support here:
>
> git://repo.or.cz/qemu/stefanha.git stream-command
OK. thanks.
>
>
> On 08/15/2011 08:25 PM, Zhi Yong Wu wrote:
> > On Mon, Aug 15, 2011 at 8:36 PM, Adam Litke wrote:
> >> On 0
On Tue, Aug 16, 2011 at 04:26:03PM -0700, Daniel P. Berrange wrote:
> On Tue, Aug 16, 2011 at 02:52:07PM -0400, Dave Allan wrote:
> > On Tue, Aug 16, 2011 at 11:15:04AM -0700, Daniel P. Berrange wrote:
> > > On Mon, Aug 15, 2011 at 02:19:27PM -0400, Dave Allan wrote:
> > > > On Mon, Aug 15, 2011 at
On Tue, Aug 16, 2011 at 05:49:27PM -0600, Eric Blake wrote:
> On 08/16/2011 05:25 PM, Daniel P. Berrange wrote:
> >From: "Daniel P. Berrange"
> >
> >If a client had initiated a stream abort, it will have a call
> >waiting for a reply in the queue. If more data continues to
> >arrive on the stream,
On 08/16/2011 05:25 PM, Daniel P. Berrange wrote:
From: "Daniel P. Berrange"
If a client had initiated a stream abort, it will have a call
waiting for a reply in the queue. If more data continues to
arrive on the stream, the abort command could mistakenly get
signalled as complete. Remove the co
On 08/16/2011 05:25 PM, Daniel P. Berrange wrote:
From: "Daniel P. Berrange"
The I/O event callback processes incoming packets first, and then
does outgoing packets. If the incoming packet caused the stream to
close, then the attempt to process outgoing data resulted in an
error. This caused lib
On Tue, Aug 16, 2011 at 02:52:07PM -0400, Dave Allan wrote:
> On Tue, Aug 16, 2011 at 11:15:04AM -0700, Daniel P. Berrange wrote:
> > On Mon, Aug 15, 2011 at 02:19:27PM -0400, Dave Allan wrote:
> > > On Mon, Aug 15, 2011 at 10:12:54AM -0700, Daniel P. Berrange wrote:
> > > > From: "Daniel P. Berran
From: "Daniel P. Berrange"
The I/O event callback processes incoming packets first, and then
does outgoing packets. If the incoming packet caused the stream to
close, then the attempt to process outgoing data resulted in an
error. This caused libvirt to then send an error back to the client,
but
From: "Daniel P. Berrange"
If a client had initiated a stream abort, it will have a call
waiting for a reply in the queue. If more data continues to
arrive on the stream, the abort command could mistakenly get
signalled as complete. Remove the code from async data processing
that looked for waiti
On Mon, Aug 08, 2011 at 01:48:59PM +0200, Jiri Denemark wrote:
> Hi,
>
> AFAIK this topic is not new but I think we still do not have a good solution
> for it. Libvirt already supports specifying what CPU and its features a guest
> should see but imagine one wants to run a guest on the best possib
Fixes confusion introduced in commit 98369d3.
* tools/virsh.c (cmdSnapshotParent): Operates on named snapshot,
not current.
---
virsh.pod got it right. I'm pushing this under the trivial rule.
tools/virsh.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/tools/virsh.c
It would technically be possible to have virsh compute the list
of descendants of a given snapshot, then delete those one at
a time. But it's complex, and not worth writing for a first
cut at implementing the new flags.
* tools/virsh.c (cmdSnapshotDelete): Add --children-only,
--metadata.
* tools
On Tue, Aug 16, 2011 at 03:46:23PM -0700, Daniel P. Berrange wrote:
> On Tue, Aug 16, 2011 at 05:46:43PM -0400, Dave Allan wrote:
> > On Tue, Aug 16, 2011 at 10:55:32PM +0200, Jiri Denemark wrote:
> > > On Mon, Aug 08, 2011 at 13:48:59 +0200, Jiri Denemark wrote:
> > > > Hi,
> > > >
> > > > AFAIK
On Tue, Aug 16, 2011 at 05:46:43PM -0400, Dave Allan wrote:
> On Tue, Aug 16, 2011 at 10:55:32PM +0200, Jiri Denemark wrote:
> > On Mon, Aug 08, 2011 at 13:48:59 +0200, Jiri Denemark wrote:
> > > Hi,
> > >
> > > AFAIK this topic is not new but I think we still do not have a good
> > > solution
>
On Tue, Aug 16, 2011 at 11:53:04PM +0200, Jiri Denemark wrote:
> Hi all,
>
> Currently when we start a non-tunneled migration, data go straight from
> source qemu to destination qemu. This is nice in that there is no additional
> overhead but it also has several disadvantages. If the communication
Reverting to a state prior to an external snapshot risks
corrupting any other branches in the snapshot hierarchy that
were using the snapshot as a read-only backing file. So
disk snapshot code will default to preventing reverting to
a snapshot that has any children, meaning that deleting just
the
On Tue, Aug 16, 2011 at 17:46:43 -0400, Dave Allan wrote:
> On Tue, Aug 16, 2011 at 10:55:32PM +0200, Jiri Denemark wrote:
> > On Mon, Aug 08, 2011 at 13:48:59 +0200, Jiri Denemark wrote:
> > > Hi,
> > >
> > > AFAIK this topic is not new but I think we still do not have a good
> > > solution
> >
Hi all,
Currently when we start a non-tunneled migration, data go straight from
source qemu to destination qemu. This is nice in that there is no additional
overhead but it also has several disadvantages. If the communication between
source and destination qemu breaks, we only get unexpected error
On Tue, Aug 16, 2011 at 10:55:32PM +0200, Jiri Denemark wrote:
> On Mon, Aug 08, 2011 at 13:48:59 +0200, Jiri Denemark wrote:
> > Hi,
> >
> > AFAIK this topic is not new but I think we still do not have a good solution
> > for it. Libvirt already supports specifying what CPU and its features a
>
On Tue, Aug 16, 2011 at 18:39:12 +0200, Michal Privoznik wrote:
> If libvirt daemon gets restarted and there is (at least) one
> unresponsive qemu, the startup procedure hangs up. This patch creates
> one thread per vm in which we try to reconnect to monitor. Therefore,
> blocking in one thread wil
On 08/15/2011 05:33 PM, Eric Blake wrote:
Audit all changes to the qemu vm->current_snapshot, and make them
update the saved xml file for both the previous and the new
snapshot, so that there is always at most one snapshot with
1 in the xml, and that snapshot is used as the
current snapshot even
On Tue, Aug 16, 2011 at 03:29:04PM -0400, Laine Stump wrote:
> On 08/16/2011 11:47 AM, Daniel P. Berrange wrote:
> >On Tue, Aug 16, 2011 at 04:44:42AM -0400, Laine Stump wrote:
> >>This is related to: https://bugzilla.redhat.com/show_bug.cgi?id=638633#c14
> >>
> >>I had started to reply to it in th
On Mon, Aug 08, 2011 at 13:48:59 +0200, Jiri Denemark wrote:
> Hi,
>
> AFAIK this topic is not new but I think we still do not have a good solution
> for it. Libvirt already supports specifying what CPU and its features a guest
> should see but imagine one wants to run a guest on the best possible
On 08/15/2011 05:33 PM, Eric Blake wrote:
Just as leaving managed save metadata behind can cause problems
when creating a new domain that happens to collide with the name
of the just-deleted domain, the same is true of leaving any
snapshot metadata behind. For safety sake, extend the semantic
ch
New flag bits are worth exposing via virsh. Additionally, even
though I recently added 'virsh snapshot-parent', doing it one
snapshot at a time is painful, so make it possible to expand the
snapshot-list table at once. In the case of snapshot-list --roots,
it's possible to emulate this even when
On 08/15/2011 05:33 PM, Eric Blake wrote:
Filtering for roots is pretty easy to do.
@@ -11180,6 +11181,8 @@ static void virDomainSnapshotObjListCopyNames(void
*payload,
if (data->oom)
return;
+if ((data->flags& VIR_DOMAIN_SNAPSHOT_LIST_ROOTS)&& !obj->def->parent)
+
I was testing a virsh patch, and wanted to see if I had passed the
flags I thought. But with LIBVIRT_DEBUG in the environment, I just
saw:
14:24:52.359: 15022: debug : virDomainSnapshotNum:15586 : dom=0xc9c180, (VM:
name=rhel_6-64, uuid=48f8e8e7-e14f-0e14-02f0-ce71997bdcab),
including a trailin
On 08/16/2011 03:25 PM, Eric Blake wrote:
On 08/16/2011 01:01 PM, Stefan Berger wrote:
Following the latter couldn't it just be handled during runtime
altogether?
That actually sounds better.
Stefan
Along those lines I propose this patch below:
Signed-off-by: Stefan Berger
+int virFile
On Tue, Aug 16, 2011 at 08:33:04 -0600, Eric Blake wrote:
> On 08/16/2011 04:44 AM, Jiri Denemark wrote:
> > If migration failed on source daemon, the migration is automatically
> > canceled by the daemon itself. Thus we don't need to call
> > virDomainMigrateConfirm3(cancelled=1). Calling it doesn
On 08/16/2011 11:47 AM, Daniel P. Berrange wrote:
On Tue, Aug 16, 2011 at 04:44:42AM -0400, Laine Stump wrote:
This is related to: https://bugzilla.redhat.com/show_bug.cgi?id=638633#c14
I had started to reply to it in the comments of the bug, but my
reply became too long, and expanded into an i
On 08/16/2011 02:00 PM, Matthias Bolte wrote:
2011/8/16 Zdenek Styblik:
On 08/16/11 18:07, Zdenek Styblik wrote:
[...]
Well, libvirt.spec has had Require: dnsmasq for at least a couple
years, so any install of a libvirt rpm should fail when dnsmasq
isn't present. I don't know for certain how lo
On 08/16/2011 01:01 PM, Stefan Berger wrote:
Following the latter couldn't it just be handled during runtime
altogether?
That actually sounds better.
Stefan
Along those lines I propose this patch below:
Signed-off-by: Stefan Berger
+int virFileIsLink(const char *linkpath)
+{
+ struct st
On 08/16/2011 02:34 PM, Stefan Berger wrote:
On 08/16/2011 02:14 PM, Eric Blake wrote:
On 08/16/2011 12:07 PM, Stefan Berger wrote:
I split this off and pushed the rest.
If procfs's are different, then maybe we should use #if
PROCFS_PID_EXE_LINK_AVAIL here.
Seems like a reasonable name.
Wha
On Tue, Aug 16, 2011 at 11:15:04AM -0700, Daniel P. Berrange wrote:
> On Mon, Aug 15, 2011 at 02:19:27PM -0400, Dave Allan wrote:
> > On Mon, Aug 15, 2011 at 10:12:54AM -0700, Daniel P. Berrange wrote:
> > > From: "Daniel P. Berrange"
> > >
> > > After running 'virsh console' in interactive mode,
On 08/16/11 20:00, Matthias Bolte wrote:
> 2011/8/16 Zdenek Styblik :
>> On 08/16/11 18:07, Zdenek Styblik wrote:
>> [...]
> Well, libvirt.spec has had Require: dnsmasq for at least a couple
> years, so any install of a libvirt rpm should fail when dnsmasq
> isn't present. I don't know
On 08/16/2011 02:14 PM, Eric Blake wrote:
On 08/16/2011 12:07 PM, Stefan Berger wrote:
I split this off and pushed the rest.
If procfs's are different, then maybe we should use #if
PROCFS_PID_EXE_LINK_AVAIL here.
Seems like a reasonable name.
What is needed in this case seems to be that /pro
On Mon, Aug 15, 2011 at 02:19:27PM -0400, Dave Allan wrote:
> On Mon, Aug 15, 2011 at 10:12:54AM -0700, Daniel P. Berrange wrote:
> > From: "Daniel P. Berrange"
> >
> > After running 'virsh console' in interactive mode, there was a
> > missing call to virStreamAbort, which meant the server kept t
On 08/16/2011 12:07 PM, Stefan Berger wrote:
I split this off and pushed the rest.
If procfs's are different, then maybe we should use #if
PROCFS_PID_EXE_LINK_AVAIL here.
Seems like a reasonable name.
What is needed in this case seems to be that /proc//exe is a
symbolic link to the executable
On Tue, Aug 16, 2011 at 11:02:23AM -0600, Eric Blake wrote:
> On 08/12/2011 04:31 AM, Radek Hladik wrote:
> >Hi,
> >I've noticed newer version of curl in last qemu changelog. It seems that
> >its needed for curl block device. Unfortunatelly, I was not able to find
> >out more. I tried to look into
On 08/16/2011 12:04 PM, Daniel P. Berrange wrote:
From: "Daniel P. Berrange"
If a stream gets a server initiated abort, the client may still
send an abort request before it receives the server side abort.
This causes the server to send back another abort for the
stream. Since the protocol define
On 08/16/2011 01:35 PM, Eric Blake wrote:
On 08/16/2011 11:29 AM, Eric Blake wrote:
+++ b/src/util/virpidfile.c
@@ -213,6 +213,8 @@ int virPidFileReadPathIfAlive(const char *path,
#ifdef __linux__
if (virFileLinkPointsTo(procpath, binpath) == 0)
*pid = -1;
+#else
+ (void)binpath;
#endif
Here,
From: "Daniel P. Berrange"
If a stream gets a server initiated abort, the client may still
send an abort request before it receives the server side abort.
This causes the server to send back another abort for the
stream. Since the protocol defines that abort is the last thing
to be sent, the clie
2011/8/16 Zdenek Styblik :
> On 08/16/11 18:07, Zdenek Styblik wrote:
> [...]
Well, libvirt.spec has had Require: dnsmasq for at least a couple
years, so any install of a libvirt rpm should fail when dnsmasq
isn't present. I don't know for certain how long the BuildRequires:
dns
On 08/16/2011 11:29 AM, Eric Blake wrote:
+++ b/src/util/virpidfile.c
@@ -213,6 +213,8 @@ int virPidFileReadPathIfAlive(const char *path,
#ifdef __linux__
if (virFileLinkPointsTo(procpath, binpath) == 0)
*pid = -1;
+#else
+ (void)binpath;
#endif
Here, I'd rather mark binpath as ATTRIBUTE_UNUSED
On 08/16/2011 11:19 AM, Stefan Berger wrote:
s/main/maint/ in the subject line.
This patch fixes *some* compilation issues on non-Linux platforms (cygwin).
Signed-off-by: Stefan Berger
int
-pciDeviceNetName(char *device_link_sysfs_path, char **netname)
+pciDeviceNetName(char *device_link_sys
On 8/16/11 9:02 AM, "Stefan Berger" wrote:
> On 08/16/2011 12:28 AM, Roopa Prabhu wrote:
>> This patch tries to fix getPhysFn in macvtap.c to get the physical
>> function(PF) of the direct attach interface, if the interface is a SR-IOV VF.
>>
>> It moves some of the sriov pci device handling
This patch fixes *some* compilation issues on non-Linux platforms (cygwin).
Signed-off-by: Stefan Berger
diff --git a/src/util/interface.c b/src/util/interface.c
index aec12f5..5d473b7 100644
--- a/src/util/interface.c
+++ b/src/util/interface.c
@@ -1317,7 +1317,7 @@ ifaceGetPhysicalFunction(c
On 08/16/11 18:07, Zdenek Styblik wrote:
[...]
>>> Well, libvirt.spec has had Require: dnsmasq for at least a couple
>>> years, so any install of a libvirt rpm should fail when dnsmasq
>>> isn't present. I don't know for certain how long the BuildRequires:
>>> dnsmasq has been there, but certainly
On 08/12/2011 05:41 AM, Peter Krempa wrote:
Early errors during start of libvirtd didn't have
an error reporting mechanism and caused libvirtd
to exit silently (only the return value indicated
an error).
Libvirt logging is initialized very early using
enviroment variables and the internal error
On 08/12/2011 04:31 AM, Radek Hladik wrote:
Hi,
I've noticed newer version of curl in last qemu changelog. It seems that
its needed for curl block device. Unfortunatelly, I was not able to find
out more. I tried to look into qemu source code and to some patches but
I am not much wiser.
As I under
On 08/13/2011 02:54 AM, Tom Vijlbrief wrote:
A virsh command like:
migrate --live --copy-storage-all Guest qemu+ssh://user@host/system
--persistent --verbose
shows
Migration: [ 0 %]
during the storage copy and does not start counting
untill the ram transfer starts
Patch for including the st
This patch annotates APIs with low or high priority.
In low set MUST be all APIs which might eventually access monitor
(and thus block indefinitely). Other APIs may be marked as high
priority. However, some must be (e.g. domainDestroy).
For high priority calls (HPC), there is new thread pool creat
If libvirt daemon gets restarted and there is (at least) one
unresponsive qemu, the startup procedure hangs up. This patch creates
one thread per vm in which we try to reconnect to monitor. Therefore,
blocking in one thread will not affect other APIs.
---
src/qemu/qemu_driver.c | 23 +++
If there is an unresponsive qemu process and libvirt access
it's monitor, it will not get any response and this thread will
block indefinitely, until the qemu process resumes or it's destroyed.
If users continues executing APIs against that domain, libvirt will
run out of worker threads and hangs (
This patch creates an optional BeginJob queue size limit. When
active, all other attempts above level will fail. To set this
feature assign desired value to max_queued variable in qemu.conf.
Setting it to 0 turns it off.
---
src/qemu/libvirtd_qemu.aug |1 +
src/qemu/qemu.conf |7 ++
On 08/16/2011 10:16 AM, Stefan Berger wrote:
Add missing symbols to libvirt_private.syms.
Signed-off-by: Stefan Berger
---
src/libvirt_private.syms | 8
1 file changed, 8 insertions(+)
ACK.
--
Eric Blake ebl...@redhat.com+1-801-349-2682
Libvirt virtualization library http://li
Add missing symbols to libvirt_private.syms.
Signed-off-by: Stefan Berger
---
src/libvirt_private.syms |8
1 file changed, 8 insertions(+)
Index: libvirt-acl/src/libvirt_private.syms
===
--- libvirt-acl.orig/src/libv
On 08/16/11 17:38, Daniel P. Berrange wrote:
> On Tue, Aug 16, 2011 at 09:15:12AM -0400, Laine Stump wrote:
>> On 08/14/2011 10:49 AM, Zdenek Styblik wrote:
>>> On 08/14/11 16:17, Zdenek Styblik wrote:
Hi,
I've noticed libvirt is persisting to have dnsmasq present and use it.
Ve
On 08/16/11 15:15, Laine Stump wrote:
> On 08/14/2011 10:49 AM, Zdenek Styblik wrote:
>> On 08/14/11 16:17, Zdenek Styblik wrote:
>>> Hi,
>>>
>>> I've noticed libvirt is persisting to have dnsmasq present and
>>> use it. Version of libvirt in question is 0.9.4.
>>>
>>> There is no dnsmasq present
On 08/16/2011 12:28 AM, Roopa Prabhu wrote:
This patch tries to fix getPhysFn in macvtap.c to get the physical
function(PF) of the direct attach interface, if the interface is a SR-IOV VF.
It moves some of the sriov pci device handling code from node_device_driver
to src/util/pci.[ch].
This pat
Visit it! Its cool and fantastic!.
http://hosting.nborges.net/com.page.php?gusiteid=65o5
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
On 08/12/11 16:18, Christophe Fergeau wrote:
On Fri, Aug 12, 2011 at 04:53:04PM +0300, Alon Levy wrote:
I don't know when libvirt started adding mf=on.
For what it's worth, this was added in
[ adding libvirt list to cc: ]
commit 9f8baf646ea44aa2de47cba5002af39a7ab1ac08
Author: Wen Congyang
On Tue, Aug 16, 2011 at 04:44:42AM -0400, Laine Stump wrote:
> This is related to: https://bugzilla.redhat.com/show_bug.cgi?id=638633#c14
>
> I had started to reply to it in the comments of the bug, but my
> reply became too long, and expanded into an issue wider than that
> single bug, so I figur
On Tue, Aug 16, 2011 at 08:13:07AM -0600, Eric Blake wrote:
> On 08/16/2011 07:34 AM, Justin Clift wrote:
> >On 16/08/2011, at 1:04 PM, Hu Tao wrote:
> >>On Tue, Jul 26, 2011 at 11:03:52AM +0800, Hu Tao wrote:
> >>>Hi,
> >>>
> >>> We now have libvirt advanced at 0.9.3, but there is almost no
> >>
On Tue, Aug 16, 2011 at 09:28:27AM +0800, Zhi Yong Wu wrote:
> On Tue, Aug 16, 2011 at 6:52 AM, Stefan Hajnoczi wrote:
> > On Mon, Aug 15, 2011 at 1:36 PM, Adam Litke wrote:
> >> On 08/14/2011 11:40 PM, Zhi Yong Wu wrote:
> >>> HI, Deniel and Adam.
> >>>
> >>> Have the patchset been merged into l
On Tue, Aug 16, 2011 at 09:15:12AM -0400, Laine Stump wrote:
> On 08/14/2011 10:49 AM, Zdenek Styblik wrote:
> >On 08/14/11 16:17, Zdenek Styblik wrote:
> >>Hi,
> >>
> >>I've noticed libvirt is persisting to have dnsmasq present and use it.
> >>Version of libvirt in question is 0.9.4.
> >>
> >>Ther
On Tue, Aug 16, 2011 at 02:24:53PM +0200, Guido Günther wrote:
> Hi Daniel,
> On Sat, Aug 13, 2011 at 08:57:45PM -0700, Daniel P. Berrange wrote:
> > On Fri, Aug 12, 2011 at 11:54:28PM +0200, Guido Günther wrote:
> [..snip..]
> > In the default libvirt event loop, the 'ff' callback is always invok
This patch prints file name of currently parsed document
if an parse error occurs. While parsing XML documments from
string, user may specify NULL as URL and the original error message
(not containing filename information) is printed.
Fixes BZ #726771
---
src/util/xml.c | 16
1
On 08/16/2011 12:28 AM, Roopa Prabhu wrote:
From: Roopa Prabhu
This patch adds the following functions to get PF/VF relationship of an SRIOV
network interface:
ifaceIsVirtualFunction: Function to check if a network interface is a SRIOV VF
ifaceGetVirtualFunctionIndex: Function to get VF index if
On 08/16/2011 12:28 AM, Roopa Prabhu wrote:
From: Roopa Prabhu
This patch adds the following helper functions:
pciDeviceIsVirtualFunction: Function to check if a pci device is a sriov VF
pciGetVirtualFunctionIndex: Function to get the VF index of a sriov VF
pciDeviceNetName: Function to get the
On 08/16/2011 12:28 AM, Roopa Prabhu wrote:
From: Roopa Prabhu
This patch renames getPhysfn to getPhysfnDev and adds code to get the
Physical function and Virtual Function index of the direct attach linkdev (if
the direct attach interface is a SRIOV VF). The idea is to send the port
profile mess
On 08/16/2011 04:53 PM, Eric Blake wrote:
On 08/16/2011 06:49 AM, Peter Krempa wrote:
Patch a53ab1094e93f1b6d93ad9be63d8ccc5fd19a2a9 introduces printing
file name on XML errors. This corrects the URL string to be NULL and
therefrore to print an error message not containing bogus filename
s/the
Patch facb914fead15fccb43122db3357dd62ad909522 introduces printing
file name on XML errors. This corrects the URL string to be NULL and
therefore to print an error message not containing bogus filename
"domain.xml".
---
src/conf/domain_conf.c|2 +-
src/security/virt-aa-helper.c |2
On 08/16/2011 12:28 AM, Roopa Prabhu wrote:
From: Roopa Prabhu
This patch moves some of the sriov related pci code from node_device driver
to src/util/pci.[ch]. Some functions had to go thru name and argument list
change to accommodate the move.
Signed-off-by: Roopa Prabhu
Signed-off-by: Christ
On 08/16/2011 04:52 PM, Peter Krempa wrote:
This patch prints file name of currently parsed document
if an parse error occurs. While parsing XML documments from
string, user may specify NULL as URL and the original error message
(not containing filename information) is printed.
Fixes BZ #726771
On 08/16/2011 09:05 AM, Osier Yang wrote:
We don't allow to define domain with same name and different UUID,
or with same UUID, so it's reasonable to not load the domain config
if there is domain with same name already exists.
Otherwise it can cause problem like:
1) % cp /etc/libvirt/qemu/dom.x
Patch facb914fead15fccb43122db3357dd62ad909522 introduces printing
file name on XML errors. This corrects the URL string to be NULL and
therefrore to print an error message not containing bogus filename
"domain.xml".
---
src/conf/domain_conf.c|2 +-
src/security/virt-aa-helper.c |2
On 08/16/2011 06:49 AM, Peter Krempa wrote:
Patch a53ab1094e93f1b6d93ad9be63d8ccc5fd19a2a9 introduces printing
file name on XML errors. This corrects the URL string to be NULL and
therefrore to print an error message not containing bogus filename
s/therefrore/therefore/
"domain.xml".
---
sr
This patch prints file name of currently parsed document
if an parse error occurs. While parsing XML documments from
string, user may specify NULL as URL and the original error message
(not containing filename information) is printed.
Fixes BZ #726771
---
src/util/xml.c | 16
1
On 08/16/2011 04:37 PM, Eric Blake wrote:
On 08/16/2011 06:49 AM, Peter Krempa wrote:
This patch prints file name of currently parsed document
if an parse error occurs. While parsing XML documments from
string, user may specify NULL as URL and the original error message
(not containing filename
On 08/16/2011 06:49 AM, Peter Krempa wrote:
This patch prints file name of currently parsed document
if an parse error occurs. While parsing XML documments from
string, user may specify NULL as URL and the original error message
(not containing filename information) is printed.
Fixes BZ #726771
Stefan has a git repo with QED block streaming support here:
git://repo.or.cz/qemu/stefanha.git stream-command
On 08/15/2011 08:25 PM, Zhi Yong Wu wrote:
> On Mon, Aug 15, 2011 at 8:36 PM, Adam Litke wrote:
>> On 08/14/2011 11:40 PM, Zhi Yong Wu wrote:
>>> HI, Deniel and Adam.
>>>
>>> Have the
On 08/16/2011 04:02 AM, Osier Yang wrote:
+++ b/src/qemu/qemu_migration.c
@@ -2050,6 +2050,9 @@ finish:
* do, but fortunately worst case is that there is a
* domain left in 'paused' state on source.
*/
+ if (ret< 0)
+ VIR_WARN("Guest %s probably left in 'paused' state on source",
+ vm->def->name)
On 08/16/2011 04:44 AM, Jiri Denemark wrote:
If migration failed on source daemon, the migration is automatically
canceled by the daemon itself. Thus we don't need to call
virDomainMigrateConfirm3(cancelled=1). Calling it doesn't cause any harm
but the resulting error message printed in logs may
On 08/16/2011 04:44 AM, Jiri Denemark wrote:
When virStreamAbort is called on a stream that has not been used yet,
quite confusing error is returned: "this function is not supported by
the connection driver". Let's just ignore such streams as there's
nothing to abort anyway.
---
src/libvirt.c |
We don't allow to define domain with same name and different UUID,
or with same UUID, so it's reasonable to not load the domain config
if there is domain with same name already exists.
Otherwise it can cause problem like:
1) % cp /etc/libvirt/qemu/dom.xml /etc/libvirt/qemu/dom_diffuuid.xml
2) rem
On 08/15/2011 10:28 PM, Osier Yang wrote:
> 于 2011年08月16日 00:40, Adam Litke 写道:
>> Hi Osier,
>>
>> Just to be clear, this is a cleanup not a bugfix right? The current
>> code should be working properly as written.
>
> No, "virDomainMemoryStats" will return empty result if libvirt
> communicates t
On 08/16/2011 03:57 AM, Daniel Veillard wrote:
I just tred compiling 0.9.4 and git head on Centos 6.0, for the
possible libvirt.org server replacement and it fails in both cases
with:
CC libvirtd-remote.o
remote.c: In function 'remoteDispatchAuthPolkit':
remote.c:2161: error: invalid i
On 08/16/2011 07:34 AM, Justin Clift wrote:
On 16/08/2011, at 1:04 PM, Hu Tao wrote:
On Tue, Jul 26, 2011 at 11:03:52AM +0800, Hu Tao wrote:
Hi,
We now have libvirt advanced at 0.9.3, but there is almost no
progress has been made on Virsh Command Reference.
For future reference, it wou
On 16/08/2011, at 1:04 PM, Hu Tao wrote:
> On Tue, Jul 26, 2011 at 11:03:52AM +0800, Hu Tao wrote:
>> Hi,
>>
>> We now have libvirt advanced at 0.9.3, but there is almost no
>> progress has been made on Virsh Command Reference. I'm now
>> starting on updating Virsh Command Reference to get i
Hi Daniel,
On Sat, Aug 13, 2011 at 08:57:45PM -0700, Daniel P. Berrange wrote:
> On Fri, Aug 12, 2011 at 11:54:28PM +0200, Guido Günther wrote:
[..snip..]
> In the default libvirt event loop, the 'ff' callback is always invoked
> from a "clean" stack in the event loop, so you never have this probl
On 08/11/2011 04:08 PM, Eric Blake wrote:
On 08/11/2011 09:27 AM, Peter Krempa wrote:
---
src/libvirt.c | 122
+
1 files changed, 122 insertions(+), 0 deletions(-)
For new API, I've lately been squashing the definition (libvirt.h.in
On 08/12/2011 07:25 AM, Michal Novotny wrote:
Hi,
this is the third version of my patch to the bridge driver and
libvirt XML file to include support for the SRV records in the DNS.
The syntax is based on DNSMasq man page and tests for both xml2xml
and xml2argv were added as well.
Differences bet
On 08/14/2011 10:49 AM, Zdenek Styblik wrote:
On 08/14/11 16:17, Zdenek Styblik wrote:
Hi,
I've noticed libvirt is persisting to have dnsmasq present and use it.
Version of libvirt in question is 0.9.4.
There is no dnsmasq present on the Host during libvirt compilation, yet
it seems to me ./co
On 08/15/2011 09:21 AM, Daniel Veillard wrote:
On Sun, Aug 14, 2011 at 03:43:52AM -0400, Laine Stump wrote:
In some versions of qemu, both virtio-blk-pci and virtio-net-pci
devices can have an event_idx setting that determines some details of
event processing. When it is enabled, it "reduces the
Patch a53ab1094e93f1b6d93ad9be63d8ccc5fd19a2a9 introduces printing
file name on XML errors. This corrects the URL string to be NULL and
therefrore to print an error message not containing bogus filename
"domain.xml".
---
src/conf/domain_conf.c|2 +-
src/security/virt-aa-helper.c |2
This patch prints file name of currently parsed document
if an parse error occurs. While parsing XML documments from
string, user may specify NULL as URL and the original error message
(not containing filename information) is printed.
Fixes BZ #726771
---
src/util/xml.c | 16
1
1 - 100 of 111 matches
Mail list logo