Am Donnerstag 11 August 2011 21:45:18 schrieb Eric Blake:
> On 08/11/2011 06:44 AM, Philipp Hahn wrote:
> > Currently only tabs and blanks are used for tokenizing the description,
> > which breaks when a term is at the end of a line or has () appended to
> > it.
> > 1. Use also other white space
On Thu, Aug 11, 2011 at 02:44:00PM +0200, Philipp Hahn wrote:
> Currently only tabs and blanks are used for tokenizing the description,
> which breaks when a term is at the end of a line or has () appended to
> it.
> 1. Use also other white space characters such as new-lines and carriage
>retur
On Thu, Aug 11, 2011 at 08:39:23PM -0600, Eric Blake wrote:
> We forgot to add virDomainUndefineFlags for a couple of hypervisors.
> This wires up trivial versions (since neither hypervisor supports
> managed save yet, they do not need to support any flags).
>
> * src/vbox/vbox_tmpl.c (vboxDomainC
On Thu, Aug 11, 2011 at 08:39:22PM -0600, Eric Blake wrote:
> The public API documents that undefine may be used to transition a
> running persistent domain into a transient one. Many drivers still
> do not support this usage, but virsh shouldn't be getting in the
> way of those that do support it
We forgot to add virDomainUndefineFlags for a couple of hypervisors.
This wires up trivial versions (since neither hypervisor supports
managed save yet, they do not need to support any flags).
* src/vbox/vbox_tmpl.c (vboxDomainCreateXML): Update caller.
(vboxDomainUndefine): Move guts...
(vboxDoma
The public API documents that undefine may be used to transition a
running persistent domain into a transient one. Many drivers still
do not support this usage, but virsh shouldn't be getting in the
way of those that do support it.
* tools/virsh.c (cmdUndefine): Allow undefine on active domains;
On 08/11/2011 02:50 PM, Eric Blake wrote:
On 08/10/2011 04:08 PM, Eric Blake wrote:
Undefining
++
Meanwhile, I propose changing the default behavior of
virDomainUndefine[Flags] to reject attempts to undefine a domain with
any defined snapshots, and to add a new flag for virDomainUndefin
On 08/11/2011 02:51 PM, Laine Stump wrote:
+
+ /* Following attributes are optional but we had to make sure their
NULL above */
s/their/they're/
Right, because VIR_REALLOC_N doesn't 0-fill the new memory (it doesn't
know how many there were before).
But VIR_EXPAND_N and VIR_RESIZE_N _do_ k
On 08/11/2011 09:22 AM, Michal Novotny wrote:
Hi,
this is the second 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 be
On 08/10/2011 04:08 PM, Eric Blake wrote:
Undefining
++
In one regards, undefining a domain that has snapshots is just as bad as
undefining a domain with managed save state - since libvirt is
maintaining metadata about snapshot hierarchies, leaving this metadata
behind _will_ interfere w
On 08/11/2011 09:27 AM, Peter Krempa wrote:
Adds support for the new API. While starting a qemu link state
cannot be set with an command line argument and therefore is done
by entering monitor.
---
src/qemu/qemu_driver.c | 228 +++
src/qemu/qemu_pr
On 08/11/2011 09:27 AM, Peter Krempa wrote:
Adds a capability flag based on version number to detect if qemu
supports setting link state.
---
src/qemu/qemu_capabilities.c |5 +
src/qemu/qemu_capabilities.h |1 +
2 files changed, 6 insertions(+), 0 deletions(-)
No testsuite chang
On 08/11/2011 09:27 AM, Peter Krempa wrote:
Adds support for QMP and text monitor access to qemu.
---
src/qemu/qemu_monitor.c | 19 +
src/qemu/qemu_monitor.h |3 ++
src/qemu/qemu_monitor_json.c | 23
src/qemu/qemu_monitor_json.h |4 +
On 08/11/2011 09:27 AM, Peter Krempa wrote:
---
tools/virsh.c | 120 +++
tools/virsh.pod | 11 +
2 files changed, 131 insertions(+), 0 deletions(-)
Again, I think the new virsh commands can be useful, but would need to
map on top
On 08/11/2011 09:27 AM, Peter Krempa wrote:
---
src/conf/domain_conf.c | 19 +++
src/conf/domain_conf.h |1 +
2 files changed, 20 insertions(+), 0 deletions(-)
This patch is useful, even without the new API.
@@ -3077,6 +3081,14 @@ virDomainNetDefParseXML(virCapsPtr c
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,
patch 2/10), backend callback (driver.h 3/
On 08/11/2011 09:27 AM, Peter Krempa wrote:
This API adds methods to modify link state of virtual
interfaces of a domain.
Idea of this API is to enable simulation of unplugging
(virtual) cable from the machine to test networks or
to allow propagation of topology changes on the host.
---
includ
On 08/11/2011 09:27 AM, Peter Krempa wrote:
Adds a optional element to XML definition of domains for
modification of link state of network interfaces.
---
docs/formatdomain.html.in | 21 +
docs/schemas/domain.rng | 11 +++
2 files changed, 32 insertions(+), 0
On 08/11/2011 06:50 AM, Philipp Hahn wrote:
Only the class CParser declares a method warning(), but the other
classes still try to call their self.warning() method, which fails.
Move the warning() functions to its own base classe and inherit from
s/classe/class/
that in all other classes.
S
On Thu, Aug 11, 2011 at 04:06:59PM +0200, Philipp Hahn wrote:
> Hello,
>
> On Thursday 11 August 2011 08:35:25 Philipp Hahn wrote:
> > While testing it I noticed the following behaviour:
> > 1. Start virsh
> > 2. Do "qemu-monitor-command $SOME_VM info\ chardev" to get the path of the
> > pty used
On 08/11/2011 06:40 AM, Philipp Hahn wrote:
When the description of an entry is too long and needs multiple lines,
all other table cells of the same row are currently vertically aligned
on center. Without row borders or different background colors for
alternating rows this is hard to read.
Chang
On 08/11/2011 06:44 AM, Philipp Hahn wrote:
Currently only tabs and blanks are used for tokenizing the description,
which breaks when a term is at the end of a line or has () appended to
it.
1. Use also other white space characters such as new-lines and carriage
return for splitting.
2. Remov
On 08/11/2011 03:37 AM, Peter Krempa wrote:
After adding option to have directiories as storage volumes virsh reported
type of this storage volumes as 'file'. This patch fixes this type in data
structures and fixes the virsh output.
https://bugzilla.redhat.com/show_bug.cgi?id=727088
Peter Kremp
On 08/11/2011 06:18 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). This patch adds error messages printed
to stderr if verbose parameter is specified to the
dae
On 08/11/2011 12:23 AM, Daniel Veillard wrote:
On Wed, Aug 10, 2011 at 10:09:56PM -0600, Eric Blake wrote:
Down the road, I want to add virDomainSnapshotGetParent, and use
the new API rather than xml scraping; but this virsh command can
be implemented even without the new API.
* tools/virsh.c (
On 08/11/2011 08:51 AM, Laine Stump wrote:
On 08/05/2011 11:14 AM, Eric Blake wrote:
}
} else if (flags& VIR_DOMAIN_AFFECT_CONFIG) {
+ /* Clang can't see that if we get here, persistentDef was set. */
+ sa_assert(persistentDef);
+
for (i = 0; i< nparams; i++) {
virTypedParameterPtr param =¶ms[i]
On 08/11/2011 07:38 AM, Michal Novotny wrote:
Please ignore previous mails (done by my sendmail was not working fine
and please review this one instead).
It may have also been an issue of sendmail sending from a different
address, and your patches being stuck in the moderator queue waiting for
On 08/11/2011 09:12 AM, Daniel P. Berrange wrote:
From: "Daniel P. Berrange"
* remote.html.in: Remove obsolete notes about internals of the
RPC protocol
* internals/rpc.html.in: Extensive docs on RPC protocol/API
* sitemap.html.in: Add new page
---
docs/internals/rpc.html.in | 876
On 08/10/2011 04:08 PM, Eric Blake wrote:
I'm not sure I covered this well earlier, but another useful definition is:
"current snapshot" - if this exists, it is the snapshot that would
become the parent if a new snapshot were created; or put another way, it
is the snapshot to which our current
On 08/11/2011 08:11 AM, Kevin Wolf wrote:
I agree with you. It feels a bit backwards for snapshots, but it's
really the only reasonable thing to do if you're using external
snapshots. That you can't rename block devices is actually a very point
point, too.
There's one more point to consider: If
On Thu, Aug 11, 2011 at 05:27:39PM +0200, Peter Krempa wrote:
> Adds a optional element to XML definition of domains for
> modification of link state of network interfaces.
> ---
> docs/formatdomain.html.in | 21 +
> docs/schemas/domain.rng | 11 +++
> 2 files cha
On 07/26/2011 08:51 AM, Corey Bryant wrote:
> +static int raw_open_fd(BlockDriverState *bs, const char *filename,
int flags)
> +{
> +BDRVRawState *s = bs->opaque;
> +const char *fd_str;
> +int fd;
> +
> +/* extract the file descriptor - fail if it's not fd: */
> +if (!strs
On 08/11/2011 02:28 AM, Daniel Veillard wrote:
On Thu, Aug 11, 2011 at 12:15:03AM -0400, Laine Stump wrote:
virNetworkDNSHostsDefParseXML was calling VIR_ALLOC(def->hosts) if
def->nhosts was 0. This is a waste of time, though, since
VIR_REALLOC_N is called a few lines further down, prior to any
Adds support for the new API. While starting a qemu link state
cannot be set with an command line argument and therefore is done
by entering monitor.
---
src/qemu/qemu_driver.c | 228 +++
src/qemu/qemu_process.c | 37
2 files changed, 265 in
Adds a capability flag based on version number to detect if qemu
supports setting link state.
---
src/qemu/qemu_capabilities.c |5 +
src/qemu/qemu_capabilities.h |1 +
2 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabiliti
Adds support for QMP and text monitor access to qemu.
---
src/qemu/qemu_monitor.c | 19 +
src/qemu/qemu_monitor.h |3 ++
src/qemu/qemu_monitor_json.c | 23
src/qemu/qemu_monitor_json.h |4 +++
src/qemu/qemu_monitor_text.c | 47 ++
---
src/libvirt.c | 122 +
1 files changed, 122 insertions(+), 0 deletions(-)
diff --git a/src/libvirt.c b/src/libvirt.c
index c8af3e1..53b7a11 100644
--- a/src/libvirt.c
+++ b/src/libvirt.c
@@ -6421,6 +6421,128 @@ error:
}
/**
+ * virDo
When the description of an entry is too long and needs multiple lines,
all other table cells of the same row are currently vertically aligned
on center. Without row borders or different background colors for
alternating rows this is hard to read.
Change the style-sheet to align the table cells of
Hypervisors provide means for controling link state of virtual NICs. Ability
to control this can be usefully used for various aplications. These patches
add (or propose) an API for controling link states of these nics.
API functions added:
--
This API adds methods to modify link state of virtual
interfaces of a domain.
Idea of this API is to enable simulation of unplugging
(virtual) cable from the machine to test networks or
to allow propagation of topology changes on the host.
---
include/libvirt/libvirt.h.in | 24 +
Adds a optional element to XML definition of domains for
modification of link state of network interfaces.
---
docs/formatdomain.html.in | 21 +
docs/schemas/domain.rng | 11 +++
2 files changed, 32 insertions(+), 0 deletions(-)
diff --git a/docs/formatdomain.htm
---
src/remote/remote_driver.c |2 ++
src/remote/remote_protocol.x | 21 -
src/remote_protocol-structs | 16
3 files changed, 38 insertions(+), 1 deletions(-)
diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c
index e5bfa4b..604b9f
---
src/conf/domain_conf.c | 19 +++
src/conf/domain_conf.h |1 +
2 files changed, 20 insertions(+), 0 deletions(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 010ce57..053f324 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -2748,6
---
src/driver.h | 14 +-
1 files changed, 13 insertions(+), 1 deletions(-)
diff --git a/src/driver.h b/src/driver.h
index 5136fb5..8101df3 100644
--- a/src/driver.h
+++ b/src/driver.h
@@ -351,7 +351,17 @@ typedef int
(virDomainPtr domain,
---
tools/virsh.c | 120 +++
tools/virsh.pod | 11 +
2 files changed, 131 insertions(+), 0 deletions(-)
diff --git a/tools/virsh.c b/tools/virsh.c
index 1d660d0..b4c6e21 100644
--- a/tools/virsh.c
+++ b/tools/virsh.c
@@ -1161,6 +1161,12
Only the class CParser declares a method warning(), but the other
classes still try to call their self.warning() method, which fails.
Move the warning() functions to its own base classe and inherit from
that in all other classes.
Signed-off-by: Philipp Hahn
---
docs/apibuild.py | 48 +
While working with the documentation, I found some issues I'd like to
improve.
Philipp Hahn (3):
Align table cells on top
Improve tokenizing of linkable terms
Fix references to self.warning()
docs/apibuild.py | 48 ++--
docs/libvirt.css |4
Currently only tabs and blanks are used for tokenizing the description,
which breaks when a term is at the end of a line or has () appended to
it.
1. Use also other white space characters such as new-lines and carriage
return for splitting.
2. Remove some common non-word characters from the toke
From: "Daniel P. Berrange"
* remote.html.in: Remove obsolete notes about internals of the
RPC protocol
* internals/rpc.html.in: Extensive docs on RPC protocol/API
* sitemap.html.in: Add new page
---
docs/internals/rpc.html.in | 876
docs/remote.htm
On 08/11/2011 09:54 AM, Upendra Moturi wrote:
Thanks for your clarification.
I am using 2.6.38-8-server kernel.
Will test with 3.0.0 kernel
Also works on 2.6.38.8-35.fc15.x86_64 kernel.
Stefan
On Thu, Aug 11, 2011 at 7:18 PM, Stefan Berger
wrote:
On 08/11/2011 09:39 AM, Stefan Berger w
On 08/05/2011 11:14 AM, Eric Blake wrote:
Our logic throws off analyzer tools:
ptr var = NULL;
if (flags == 0) flags = live ? _LIVE : _CONFIG;
if (flags& _LIVE) do stuff
if (flags& _CONFIG) var = non-null;
if (flags& _LIVE) do more stuff
else if (flags& _CONFIG) use var
the tools keep think
Hi,
this is the second 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 between v1 and v2:
- A minor rewrite of integer pa
[ CCed qemu-devel, just in case someone's interested ]
Am 11.08.2011 15:23, schrieb Eric Blake:
>> [ Okay, some of it is handled later in this document, but I think it's
>> still useful to leave this summary in my mail. External VM state is
>> something that you don't seem to have covered yet - ca
Hello,
On Thursday 11 August 2011 08:35:25 Philipp Hahn wrote:
> While testing it I noticed the following behaviour:
> 1. Start virsh
> 2. Do "qemu-monitor-command $SOME_VM info\ chardev" to get the path of the
> pty used for the console → /dev/pts/Y
> 3. Execute "console $SOME_VM" from within the
Thanks for your clarification.
I am using 2.6.38-8-server kernel.
Will test with 3.0.0 kernel
On Thu, Aug 11, 2011 at 7:18 PM, Stefan Berger
wrote:
> On 08/11/2011 09:39 AM, Stefan Berger wrote:
>>
>> On 08/11/2011 08:07 AM, Upendra Moturi wrote:
>>>
>>> Thanks for your reply.
>>>
>>> The patch
On 08/11/2011 09:39 AM, Stefan Berger wrote:
On 08/11/2011 08:07 AM, Upendra Moturi wrote:
Thanks for your reply.
The patch it appears still has problem.
I have changed only mtu value as 2kb instead of burst value in the
network.c file
With average as 2048 , burst as 2048 and peak 2048
scp cop
On 08/11/2011 08:07 AM, Upendra Moturi wrote:
Thanks for your reply.
The patch it appears still has problem.
I have changed only mtu value as 2kb instead of burst value in the
network.c file
With average as 2048 , burst as 2048 and peak 2048
scp copied at 30-40KB/s
All test I did were with the
Please ignore previous mails (done by my sendmail was not working fine
and please review this one instead).
Thanks,
Michal
On 08/11/2011 03:22 PM, Michal Novotny wrote:
> Hi,
> this is the second version of my patch to the bridge driver and
> libvirt XML file to include support for the SRV record
Oops, sorry. This e-mail is not valid since I did try using git
send-email but I was unable to see it in the archives nor my inbox so I
guess the issue was caused by my sendmail daemon that was not working
properly. Please ignore this e-mail.
Thanks,
Michal
On 08/11/2011 03:21 PM, Michal Novotny
Hi,
this is the second 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 between v1 and v2:
- A minor rewrite of integer pa
Hi,
this is the second 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 between v1 and v2:
- A minor rewrite of integer pa
On 08/11/2011 04:00 AM, Kevin Wolf wrote:
Am 11.08.2011 00:08, schrieb Eric Blake:
After re-reading the feedback received on those threads, I think I've
settled on a pretty robust design for my first round of adding
improvements to the management of snapshots tied to a domain, while
leaving the
Hi,
this is the second 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 between v1 and v2:
- A minor rewrite of integer pa
From: Michal Novotny
Hi,
this is the second 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 between v1 and v2:
- A mino
Thanks for your review, Daniel. I already sent v2 of the patch now ;-)
Michal
On 08/11/2011 04:42 AM, Daniel Veillard wrote:
> On Thu, Aug 11, 2011 at 10:13:34AM +0800, Daniel Veillard wrote:
>> On Tue, Aug 09, 2011 at 05:50:55PM +0200, Michal Novotny wrote:
>>> Hi,
>>> this is the patch to the b
On 08/11/2011 03:17 AM, Avi Kivity wrote:
3) a management tool should be able to query the source and
destination, and then enable xzbrle if both sides support it.
You can argue that (3) could be static. A command could be added to
toggle it dynamically through the monitor.
But no matter what,
On 08/11/2011 03:03 AM, Shribman, Aidan wrote:
From: Anthony Liguori [mailto:anth...@codemonkey.ws]
Sent: Wednesday, August 10, 2011 10:28 PM
To: Avi Kivity
Cc: Blue Swirl; Stefan Hajnoczi; Shribman, Aidan; qemu-devel
Developers; libvir-list@redhat.com
Subject: Re: [Qemu-devel] [PATCH v4] XBZRLE
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). This patch adds error messages printed
to stderr if verbose parameter is specified to the
daemon.
fixes: https://bugzilla.redhat.com/show_
Thanks for your reply.
The patch it appears still has problem.
I have changed only mtu value as 2kb instead of burst value in the
network.c file
With average as 2048 , burst as 2048 and peak 2048
scp copied at 30-40KB/s
With average as 4096 , burst as 4096 and peak 4096
scp copied at same 30-40K
> From: Anthony Liguori [mailto:anth...@codemonkey.ws]
> Sent: Wednesday, August 10, 2011 10:28 PM
> To: Avi Kivity
> Cc: Blue Swirl; Stefan Hajnoczi; Shribman, Aidan; qemu-devel
> Developers; libvir-list@redhat.com
> Subject: Re: [Qemu-devel] [PATCH v4] XBZRLE delta for live migration of
> large m
On 08/10/2011 11:14 AM, Eric Blake wrote:
On 08/10/2011 04:34 AM, Stefan Berger wrote:
When instantiating a filter, a VM may disappear and remove its tap
interfaces. Tolerate this case and don't report an error.
Signed-off-by: Stefan Berger
ACK with one nit:
+ rc = 0;
+ goto exit;
+ }
+
r
Am 11.08.2011 00:08, schrieb Eric Blake:
> [BCC'ing those who have responded to earlier RFC's]
>
> I've posted previous RFCs for improving snapshot support:
>
> ideas on managing a subset of disks:
> https://www.redhat.com/archives/libvir-list/2011-May/msg00042.html
>
> ideas on managing snapsho
Fix of output of detailed volume list. BZ #727088
---
tools/virsh.c | 17 +
1 files changed, 13 insertions(+), 4 deletions(-)
diff --git a/tools/virsh.c b/tools/virsh.c
index 1d660d0..5d88d84 100644
--- a/tools/virsh.c
+++ b/tools/virsh.c
@@ -8989,10 +8989,19 @@ cmdVolList(vshCo
Fix internals of libvirt for new storage volume type.
Libvirt reported an invalid type of the volume.
BZ #727088
---
include/libvirt/libvirt.h.in |1 +
src/storage/storage_backend_fs.c |4
2 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/include/libvirt/libvirt.h.in
After adding option to have directiories as storage volumes virsh reported
type of this storage volumes as 'file'. This patch fixes this type in data
structures and fixes the virsh output.
https://bugzilla.redhat.com/show_bug.cgi?id=727088
Peter Krempa (2):
storage: Directory shouldn't be liste
On Wed, Aug 10, 2011 at 02:27:41PM -0500, Anthony Liguori wrote:
> On 08/10/2011 11:40 AM, Avi Kivity wrote:
> Instead of teaching management tools how to deal with all of these
> things, let's just fix this problem once. It just takes:
>
> a) A query-migration-caps command that returns a dict wi
On 08/11/2011 12:16 PM, Daniel P. Berrange wrote:
On Thu, Aug 11, 2011 at 11:17:09AM +0300, Avi Kivity wrote:
> On 08/10/2011 10:27 PM, Anthony Liguori wrote:
> >>This may be acceptable, wait until the entire migration cluster is
> >>xzbrle capable before enabling it. If not, add a monitor com
On Thu, Aug 11, 2011 at 11:17:09AM +0300, Avi Kivity wrote:
> On 08/10/2011 10:27 PM, Anthony Liguori wrote:
> >>This may be acceptable, wait until the entire migration cluster is
> >>xzbrle capable before enabling it. If not, add a monitor command.
> >
> >
> >1) xzbrle needs to be disabled by defa
On 08/10/2011 10:27 PM, Anthony Liguori wrote:
This may be acceptable, wait until the entire migration cluster is
xzbrle capable before enabling it. If not, add a monitor command.
1) xzbrle needs to be disabled by default. That way management tools
don't unknowingly enable it by not passing
On Thu, Aug 11, 2011 at 12:15:03AM -0400, Laine Stump wrote:
> virNetworkDNSHostsDefParseXML was calling VIR_ALLOC(def->hosts) if
> def->nhosts was 0. This is a waste of time, though, since
> VIR_REALLOC_N is called a few lines further down, prior to any use of
> def->hosts.
> ---
> src/conf/netwo
80 matches
Mail list logo