On Mon, Apr 22, 2013 at 10:43:35AM -0400, Laine Stump wrote:
> On 04/22/2013 10:13 AM, Laine Stump wrote:
> > Of course in this case, we *could* avoid the camelCase vs underscore
> > entirely by just naming the attribute "queues" instead of numQueues or
> > num_queues (the fact that the attribute i
hi,all:
if xml configuration is not right?
Nehalem
thanks--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
On Mon, Apr 22, 2013 at 10:33 PM, Eric Blake wrote:
[...]
> It's capital 'I', not lower case 'l'. That's shorthand using sed syntax
> for an in-place replacement you should perform on your text. It means
> you should have written:
> After I read libvirt/HACKING file, I find we should
> for that
1.Lost soapAction in http request header
When I tried to retrieve the property of storage.perDatastoreUsage under
specify virtual machine and got a invalidProperty error but other
properties(e.g. summary.storage, guest) under the same virtual machine are ok.
Then I write a same program u
On 04/22/2013 02:37 PM, Laine Stump wrote:
> On 04/22/2013 02:43 PM, Ján Tomko wrote:
>>
>> is auto-added to pc* machine types.
>> Without this controller PCI bus 0 is not available and
>> no PCI addresses are assigned by default.
>>
>> Since older libvirt supported PCI bus 0 even without
>> this
On 04/22/2013 12:43 PM, Ján Tomko wrote:
> Add a "dry run" address allocation to figure out how many bridges
> will be needed for all the devices without explicit addresses.
>
> Auto-add just enough bridges to put all the devices on, or up to the
> bridge with the largest specified index.
> ---
I
(I'm not sure what you did differently when you sent this mail, but
somehow your mailer botched the "In-Reply-To:" header, which broke the
threaded display in Thunderbird. No big deal, but I thought you might
want to know.)
On 04/22/2013 12:51 PM, james robson wrote:
>> Date: Thu, 18 Apr 2013 14:1
(before anything else - you committed an unresolved merge conflict in
qemu_command.h. You'll need to remove the extra "> blah" text.)
Hopefully Eric can once again review the logic of the code that
determines what bridges need to be auto-added, and assign PCI addresses
to devices, since he
On 04/22/2013 02:43 PM, Ján Tomko wrote:
>
> is auto-added to pc* machine types.
> Without this controller PCI bus 0 is not available and
> no PCI addresses are assigned by default.
>
> Since older libvirt supported PCI bus 0 even without
> this controller, it is removed from the XML when migratin
On 04/22/2013 01:00 PM, Eduardo Habkost wrote:
> This property will be useful for libvirt, as libvirt already has logic
> based on low-level feature bits (not feature names), so it will be
> really easy to convert the current libvirt logic to something using the
> "feature-words" property.
>
> The
On 04/22/2013 02:43 PM, Ján Tomko wrote:
> From: liguang
Add a "Signed-off-by:" line for yourself.
>
> ---
> src/qemu/qemu_capabilities.c | 3 +++
> src/qemu/qemu_capabilities.h | 1 +
> src/qemu/qemu_command.c | 30 +-
> tests/qemuhelptest.c | 21 ++
On 04/22/2013 02:43 PM, Ján Tomko wrote:
> Add new controller type 'pci' with models 'pci-root' and 'pci-bridge'.
> ---
> docs/formatdomain.html.in | 22 +-
> docs/schemas/domaincommon.rng | 12
> src/conf/domain_conf.c| 21 -
> src/
On 04/22/2013 02:43 PM, Ján Tomko wrote:
> Assume format type is 'auto' when none is specified on
> qemu command line.
> ---
> src/conf/domain_conf.c | 2 +-
> src/conf/domain_conf.h | 5 +
> src/libvirt_
This fixes make distcheck
---
I've pushed this under the trivial rule.
Christophe
Makefile.am | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Makefile.am b/Makefile.am
index 9ef319a..726d475 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -10,6 +10,8 @@ EXTRA_DIST = \
$(PACKAGE).spec
This field will contain the feature bits that were filtered out because
of missing host support.
Signed-off-by: Eduardo Habkost
---
target-i386/cpu-qom.h | 3 +++
target-i386/cpu.c | 9 ++---
2 files changed, 9 insertions(+), 3 deletions(-)
diff --git a/target-i386/cpu-qom.h b/target-i3
Break lines on kvm_check_features_against_host(), kvm_cpu_fill_host(),
and builtin_x86_defs, so they don't get too long once the *_features
fields are replaced by an array.
Signed-off-by: Eduardo Habkost
Reviewed-By: Igor Mammedov
---
Changes v9:
* Merged all "Break lines" patches from previous
Add appropriate spaces around operators, and break line where it needs
to be broken to allow feature-words array to be introduced without
having too-long lines.
Signed-off-by: Eduardo Habkost
Reviewed-By: Igor Mammedov
---
Changes v9:
* 1-char alignment change: keep the opening parenthesis of b
Consolidate level, xlevel, xlevel2 fields in x86_def_t and CPUX86State.
Signed-off-by: Eduardo Habkost
Reviewed-By: Igor Mammedov
---
Changes v9:
* Merged "target-i386: Move cpuid_xlevel, cpuid_xlevel2 fields in X86CPU"
and "target-i386: Move xlevel/xlevel2 in struct x86_def_t"
in a singl
This replaces the feature-bit fields on both X86CPU and x86_def_t
structs with an array.
With this, we will be able to simplify code that simply does the same
operation on all feature words (e.g. kvm_check_features_against_host(),
filter_features_for_kvm(), add_flagname_to_bitmaps(), CPU feature-b
FEAT_7_0_EBX uses ECX as input, so we have to take that into account
when reporting feature word values.
Signed-off-by: Eduardo Habkost
---
target-i386/cpu.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index 110ef98..314931
This property will contain all the features that were removed from the
CPU because they are not supported by the host.
This way, libvirt or other management tools can emulate the
check/enforce behavior by checking if filtered-properties is all zeroes,
before starting the guest.
Example output whe
This property will be useful for libvirt, as libvirt already has logic
based on low-level feature bits (not feature names), so it will be
really easy to convert the current libvirt logic to something using the
"feature-words" property.
The property will have two main use cases:
- Checking host ca
This series includes the previous "replace cpuid_*features fields with a feature
word array" series.
The first 4 patches already have a Reviewed-by from Igor, they correspond to v10
plus a small indent fix requested by him.
As the cpuid_*features series was holding my "feature-words"/"filtered-fe
Instead of open-coding the filtering code for each feature word, change
the existing code to use the feature_word_info array, that have exactly
the same CPUID eax/ecx/register values for each feature word.
Signed-off-by: Eduardo Habkost
---
target-i386/cpu.c | 24 +++-
1 file
Add new controller type 'pci' with models 'pci-root' and 'pci-bridge'.
---
docs/formatdomain.html.in | 22 +-
docs/schemas/domaincommon.rng | 12
src/conf/domain_conf.c| 21 -
src/conf/domain_conf.h| 9 +
4 files cha
Add new 'pci' controller type with two models:
pci-root - auto-added to machines with implicit pci bus
pci-bridge - auto-added if the devices would not leave
at least one slot empty on bus 0 or bus >0 is specified
v3:
moved the implicit PCI root addition to qemu's post parse
callback,
added an xml
From: liguang
---
src/qemu/qemu_capabilities.c | 3 +++
src/qemu/qemu_capabilities.h | 1 +
src/qemu/qemu_command.c | 30 +-
tests/qemuhelptest.c | 21 ++---
4 files changed, 47 insertions(+), 8 deletions(-)
diff --git a/src/qemu/qemu_c
Add a "dry run" address allocation to figure out how many bridges
will be needed for all the devices without explicit addresses.
Auto-add just enough bridges to put all the devices on, or up to the
bridge with the largest specified index.
---
src/conf/domain_conf.c |
Assume format type is 'auto' when none is specified on
qemu command line.
---
src/conf/domain_conf.c | 2 +-
src/conf/domain_conf.h | 5 +
src/libvirt_private.syms
On Mon, Apr 22, 2013 at 18:40:07 +0200, Jiri Denemark wrote:
> Jiri Denemark (3):
> qemu: Move QEMU log reading into a separate function
> qemu: Ignore libvirt logs when reading QEMU error output
> logging: Make log regexp more compact (and readable)
>
> src/qemu/qemu_process.c | 81
>
On 04/21/2013 10:34 AM, Gene Czarcinski wrote:
> 1. Handle invalid ULong prefix specified.
> When parsing for @prefix as a ULong, a -2 can be returned
> if the specification is not a valid ULong.
>
> 2. Error out if address= is not specified.
>
> 3. Merge netmask process/tests under family tests.
On Thu, Apr 18, 2013 at 06:12:33PM +0200, Christophe Fergeau wrote:
> diff --git a/libvirt-designer/libvirt-designer-domain.c
> b/libvirt-designer/libvirt-designer-domain.c
> index 7aec002..73dbc8d 100644
> --- a/libvirt-designer/libvirt-designer-domain.c
> +++ b/libvirt-designer/libvirt-designer-
On Fri, Apr 19, 2013 at 11:46:21AM +0200, Michal Privoznik wrote:
> On 18.04.2013 18:08, Christophe Fergeau wrote:
> > +static OsinfoDevice *
> > +gvir_designer_domain_get_fallback_disk_controller(GVirDesignerDomain
> > *design,
> > + GError **error
On Mon, Apr 22, 2013 at 06:57:11PM +0200, Christophe Fergeau wrote:
> Minimum libvirt-gconfig and libosinfo versions appear in both
> README and configure.ac, which means they easily get out of sync.
> This commit renames README to README.in so that we can substitute
> the configure.ac version in t
On 23/04/13 00:40, Jiri Denemark wrote:
---
src/qemu/qemu_process.c | 61 ++---
1 file changed, 38 insertions(+), 23 deletions(-)
diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index ce9f501..a1fbd5f 100644
--- a/src/qemu/qemu_proces
On 23/04/13 00:40, Jiri Denemark wrote:
When QEMU fails to start, libvirt read its error output and reports it
back in an error message. However, when libvirtd is configured to log
debug messages, one would get the following unhelpful garbage:
virsh # start cd
error: Failed to start do
On 23/04/13 00:40, Jiri Denemark wrote:
---
src/util/virlog.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/util/virlog.c b/src/util/virlog.c
index 721c9bd..e31abb0 100644
--- a/src/util/virlog.c
+++ b/src/util/virlog.c
@@ -79,8 +79,8 @@ static int virLogEnd = 0;
On Mon, Apr 22, 2013 at 06:57:12PM +0200, Christophe Fergeau wrote:
> Minimum libvirt-gobject, libvirt-gconfig and libosinfo versions
> are set in configure.ac, it's better not to duplicate them in
> libvirt-designer.spec.in, it's too easy to get them out of sync.
> ---
> configure.ac
On Mon, Apr 22, 2013 at 06:57:10PM +0200, Christophe Fergeau wrote:
> ---
> README | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/README b/README
> index ed39b9d..b7ca356 100644
> --- a/README
> +++ b/README
> @@ -24,7 +24,7 @@ Or to install into a private user specific
On Fri, Apr 19, 2013 at 11:46:22AM +0200, Michal Privoznik wrote:
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -10,7 +10,7 @@ AC_CANONICAL_HOST
> >
> > AM_SILENT_RULES([yes])
> >
> > -LIBOSINFO_REQUIRED=0.2.3
> > +LIBOSINFO_REQUIRED=0.2.6
> > LIBVIRT_GCONFIG_REQUIRED=0.0.9
> > LIBVIR
Minimum libvirt-gconfig and libosinfo versions appear in both
README and configure.ac, which means they easily get out of sync.
This commit renames README to README.in so that we can substitute
the configure.ac version in the README file. This way they are always
in sync.
---
Makefile.am |
Minimum libvirt-gobject, libvirt-gconfig and libosinfo versions
are set in configure.ac, it's better not to duplicate them in
libvirt-designer.spec.in, it's too easy to get them out of sync.
---
configure.ac | 2 +-
libvirt-designer.spec.in | 8
2 files changed, 5 insertions(+
---
README | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/README b/README
index ed39b9d..b7ca356 100644
--- a/README
+++ b/README
@@ -24,7 +24,7 @@ Or to install into a private user specific location
make
make install
-The following mandatory dependancies are required
On 04/19/2013 04:01 AM, Peter Krempa wrote:
> Refactoring done in 19c6ad9ac7e7eb2fd3c8262bff5f087b508ad07f didn't
> correctly take into account the order cgroup limit modification needs to
> be done in. This resulted into errors when decreasing the limits.
>
> The operations need to take place in
> Date: Thu, 18 Apr 2013 14:14:32 -0400
> From: Laine Stump
> To: libvir-list@redhat.com
> Subject: Re: [libvirt] [PATCHv2] Configure native vlan modes on Open
> vSwitch ports
> Message-ID: <51703808.2000...@laine.org>
> Content-Type: text/plain; charset=ISO-8859-1
>
> On 04/18/2013 01:44
---
src/qemu/qemu_process.c | 61 ++---
1 file changed, 38 insertions(+), 23 deletions(-)
diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index ce9f501..a1fbd5f 100644
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -1346,6
From: Dan Walsh
Openshift Containers will be run with a unique UID and GID
---
bin/virt-sandbox-service| 43 +--
bin/virt-sandbox-service-bash-completion.sh | 8 +++--
bin/virt-sandbox-service-create.pod | 53 -
3 files
On Mon, Apr 22, 2013 at 12:26:31PM -0400, dwa...@redhat.com wrote:
> From: Dan Walsh
>
> We want to make sure we use as little overhead as possible.
> If a user connects to a lxc container, it will be the same as executing
> a shell within the container.
> ---
> bin/virt-sandbox-service | 15 +++
On Mon, Apr 22, 2013 at 12:26:30PM -0400, dwa...@redhat.com wrote:
> From: Dan Walsh
>
> OpenShift will be creating the path within its management layer.
> ---
> bin/virt-sandbox-service | 6 +-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/bin/virt-sandbox-service b/bin
Jiri Denemark (3):
qemu: Move QEMU log reading into a separate function
qemu: Ignore libvirt logs when reading QEMU error output
logging: Make log regexp more compact (and readable)
src/qemu/qemu_process.c | 81 +
src/util/virlog.c | 4
On Mon, Apr 22, 2013 at 12:26:29PM -0400, dwa...@redhat.com wrote:
> From: Dan Walsh
>
> Openshift Containers will be run with a unique UID and GID
> ---
> bin/virt-sandbox-service| 43 +--
> bin/virt-sandbox-service-bash-completion.sh | 8 +++--
> bin/vi
This patch set is adding support for UID/GID/USERNAME/USERDIR for use with
openshift
containers
Also fixes virt-sandbox-service to not complain if the destdir has been
precreated.
Finally we also do not want excess processes running withing containers
(/bin/sh).
[sandbox PATCH 1/3] Add UID/GI
From: Dan Walsh
We want to make sure we use as little overhead as possible.
If a user connects to a lxc container, it will be the same as executing
a shell within the container.
---
bin/virt-sandbox-service | 15 ++-
1 file changed, 14 insertions(+), 1 deletion(-)
diff --git a/bin/v
From: Dan Walsh
OpenShift will be creating the path within its management layer.
---
bin/virt-sandbox-service | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/bin/virt-sandbox-service b/bin/virt-sandbox-service
index 31aa6a1..dd30993 100755
--- a/bin/virt-sandbox-service
---
src/util/virlog.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/util/virlog.c b/src/util/virlog.c
index 721c9bd..e31abb0 100644
--- a/src/util/virlog.c
+++ b/src/util/virlog.c
@@ -79,8 +79,8 @@ static int virLogEnd = 0;
static regex_t *virLogRegex = NULL;
-#d
When QEMU fails to start, libvirt read its error output and reports it
back in an error message. However, when libvirtd is configured to log
debug messages, one would get the following unhelpful garbage:
virsh # start cd
error: Failed to start domain cd
error: internal error process ex
On Mon, Apr 22, 2013 at 06:18:04PM +0200, Viktor Mihajlovski wrote:
> On 04/22/2013 04:34 PM, Daniel P. Berrange wrote:
> >On Mon, Apr 22, 2013 at 04:22:26PM +0200, Viktor Mihajlovski wrote:
> >I'm not against this in general, but before we enlarge this so much
> >there needs to be some code work t
From: "Daniel P. Berrange"
The change in commit aed4986322fe77bdf718e31a0587d00f04f3d97a
was incomplete, missing a couple of cases of /system. This
caused failure to start VMs.
Pushed under trivial rule
Signed-off-by: Daniel P. Berrange
---
src/lxc/lxc_cgroup.c | 2 +-
src/qemu/qemu_cgroup.
On 04/22/2013 04:34 PM, Daniel P. Berrange wrote:
On Mon, Apr 22, 2013 at 04:22:26PM +0200, Viktor Mihajlovski wrote:
I'm not against this in general, but before we enlarge this so much
there needs to be some code work to make RPC message encoding more
efficient. Currently virNetMessageEncodeHead
(I had thought there would be just a few tweaks to the documentation
that I could squash in and push, but as I reviewed I found more issues,
and don't have time to fix them myself. It is *very* close though, and
the freeze for 1.0.5 is coming up (within a couple days, I think?), so
if you can make
On 04/22/2013 10:20 AM, Laine Stump wrote:
> On 04/20/2013 03:45 PM, Gene Czarcinski wrote:
>> Create the utility function virSocketAddrGetIpPrefix() to
>> determine the prefix for this network. The code in this
>> function was adapted from virNetworkIpDefPrefix().
>>
>> Update virNetworkIpDefPref
On 04/19/2013 01:32 AM, wangxiaojun wrote:
> recent qemu (git resp) with param like:
> #>qemu-kvm -S -no-user-config -nodefaults -nographic -M none -qmp
> monarg -pidfile pidfile
>
> will just hangs forever. no any output or errors .
> and when libvirt Try to get caps via QMP qemuCap
On 04/22/2013 08:00 AM, Laine Stump wrote:
> On 04/19/2013 11:25 AM, Laine Stump wrote:
>>> On 04/18/2013 06:36 AM, Laine Stump wrote:
Trimming a rather deep nesting...
>
> Okay, I misunderstood what you said - you weren't saying that you had
> put num_queues in the element (obviously
On 22/04/13 22:43, Laine Stump wrote:
On 04/22/2013 10:13 AM, Laine Stump wrote:
Of course in this case, we *could* avoid the camelCase vs underscore
entirely by just naming the attribute "queues" instead of numQueues or
num_queues (the fact that the attribute is a number makes it pretty
obvious
On 22/04/13 22:00, Laine Stump wrote:
On 04/19/2013 11:25 AM, Laine Stump wrote:
On 04/19/2013 04:32 AM, Osier Yang wrote:
On 18/04/13 19:59, Laine Stump wrote:
On 04/18/2013 07:27 AM, Osier Yang wrote:
On 18/04/13 19:16, Laine Stump wrote:
On 04/18/2013 05:41 AM, Martin Kletzander wrote:
O
On 04/22/2013 10:13 AM, Laine Stump wrote:
> Of course in this case, we *could* avoid the camelCase vs underscore
> entirely by just naming the attribute "queues" instead of numQueues or
> num_queues (the fact that the attribute is a number makes it pretty
> obvious that it is the "number of queues
On 22.04.2013 16:22, Viktor Mihajlovski wrote:
> From: Daniel Hansel
>
> We have seen an issue on s390x platform where domain XMLs larger than 1MB
> were used. The define command was finished successfully. The dumpxml command
> was not successful (i.e. could not encode message payload).
>
> Enla
On 04/22/2013 07:40 AM, harryxiyou wrote:
> On Mon, Apr 22, 2013 at 9:13 PM, Osier Yang wrote:
>> On 22/04/13 20:33, harryxi...@gmail.com wrote:
>>>
>>> From: Harry Wei
>>>
>>> After i read libvirt/HACKING file, i find we should
>>
>>
>> s/i/I/,
>
> Actually, i wonder what 's/i/l/' means?
It's
On Mon, Apr 22, 2013 at 04:22:26PM +0200, Viktor Mihajlovski wrote:
> From: Daniel Hansel
>
> We have seen an issue on s390x platform where domain XMLs larger than 1MB
> were used. The define command was finished successfully. The dumpxml command
> was not successful (i.e. could not encode messag
With multiqueue network feature, we are advised to pass multiple
vhost-net FDs as well. The ratio should be 1:1. Therefore we must
alter the qemuOpenVhostNet function to allow that.
---
src/qemu/qemu_command.c | 40 ++--
src/qemu/qemu_command.h | 3 ++-
src/qem
Kernel and subsequently QEMU learned multiple transmit queues a while ago. The
feature has a nice advantage, it alloes a single guest to transmit multiple
flows of network data using multiple CPUs simultaneously which increase traffic
bandwidth. A lot.
The documentation how to use this is availabl
Currently, need for use of vhost-net is signalized by returning
zero, and setting passed FD to a value different to negative one.
However, when using multiple vhost-net devices, it is not so easy
so we should use return value for that.
---
src/qemu/qemu_command.c | 22 +-
1 fil
The qemuBuildHostNetStr() function which is responsible for
generating command line for a network interface needs to be aware
of multiqueue network interface as we are required to use:
- fd=%d in case of one FD
- fds=%d:%d:%d:...:%d in case of multiple FDs
These two approaches can't be mixed
---
src/qemu/qemu_command.c | 88 +++--
1 file changed, 64 insertions(+), 24 deletions(-)
diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index dfcfedc..60873c7 100644
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -5816,1
---
src/qemu/qemu_hotplug.c | 96 -
1 file changed, 63 insertions(+), 33 deletions(-)
diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c
index 3505c52..3a00811 100644
--- a/src/qemu/qemu_hotplug.c
+++ b/src/qemu/qemu_hotplug.c
@@ -683,10
Currently, we have one huge function to construct qemu command line.
This is very ineffective esp. if there's a fault somewhere.
---
src/qemu/qemu_command.c | 323 +---
1 file changed, 171 insertions(+), 152 deletions(-)
diff --git a/src/qemu/qemu_comma
Currently, the function knows how to open a TAP device for a single
time. However, in case of multiqueue network we need to open it multiple
times. Moreover, when doing TUNSETIFF ioctl, the IFF_MULTI_QUEUE flag
shall be requested. This commit changes a behaviour slightly as well.
Till now it was po
---
src/qemu/qemu_command.c | 10 ++
src/qemu/qemu_hotplug.c | 5 +++--
2 files changed, 9 insertions(+), 6 deletions(-)
diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index 60873c7..a7d422a 100644
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -5887,12 +5
This attribute is going to represent number of queues for
multique vhost network interface. This commit implements XML
extension part of the feature and add one test as well. For now,
we can only do xml2xml test as qemu command line generation code
is not adapted yet.
---
docs/formatdomain.html.in
Currently, only one tapfd and one vhostfd could be passed.
However, multiqueue network requires several FDs to be passed to
qemu so we must adapt out monitor handling functions to cope with
that.
---
src/qemu/qemu_hotplug.c | 7 +--
src/qemu/qemu_monitor.c | 39 +++
Currently, only one tapfd and one vhostfd could be passed.
However, multiqueue network requires several FDs to be passed to
qemu so we must adapt out monitor handling functions to cope with
that.
---
src/qemu/qemu_hotplug.c | 7 +--
src/qemu/qemu_monitor.c | 39 +++
For future work it's better, if tapfd is passed as pointer.
Moreover, we need to be able to return multiple values now.
---
src/qemu/qemu_command.c | 70 +
src/qemu/qemu_command.h | 4 ++-
src/qemu/qemu_hotplug.c | 4 +--
3 files changed, 41 insert
From: "Daniel P. Berrange"
The element can now accept a
as an alternative to 'loop'. The benefit of NBD is support
for non-raw disk image formats.
Signed-off-by: Daniel P. Berrange
---
docs/formatdomain.html.in | 5 +++--
docs/schemas/domaincommon.rng | 1 +
src/conf/domain_conf.c
On 04/20/2013 03:45 PM, Gene Czarcinski wrote:
> Create the utility function virSocketAddrGetIpPrefix() to
> determine the prefix for this network. The code in this
> function was adapted from virNetworkIpDefPrefix().
>
> Update virNetworkIpDefPrefix() in src/conf/network_conf.c
> to use the new u
From: Daniel Hansel
We have seen an issue on s390x platform where domain XMLs larger than 1MB
were used. The define command was finished successfully. The dumpxml command
was not successful (i.e. could not encode message payload).
Enlarged message related sizes (e.g. maximum string size, message
On 04/22/2013 06:58 AM, Daniel Veillard wrote:
> On Thu, Apr 18, 2013 at 08:27:56AM -0600, Eric Blake wrote:
>> On 04/18/2013 03:02 AM, Martin Kletzander wrote:
>>> In commit d4bf0a9, we used num_queues for an attribute in the XML, but
>>> the consensus is that we use camelCase for that. Since the
From: "Daniel P. Berrange"
Add a virFileNBDDeviceAssociate method, which given a filename
will setup a NBD device, using qemu-nbd as the server.
Signed-off-by: Daniel P. Berrange
---
src/libvirt_private.syms | 1 +
src/util/virfile.c | 151 ++
On 04/19/2013 11:25 AM, Laine Stump wrote:
> On 04/19/2013 04:32 AM, Osier Yang wrote:
>> On 18/04/13 19:59, Laine Stump wrote:
>>> On 04/18/2013 07:27 AM, Osier Yang wrote:
On 18/04/13 19:16, Laine Stump wrote:
> On 04/18/2013 05:41 AM, Martin Kletzander wrote:
>> On 04/18/2013 11:05
From: "Daniel P. Berrange"
The current code for setting up loop devices to LXC disks first
does a switch() based on the disk format, then looks at the
disk driver name. Reverse this so it first looks at the driver
name, and then the disk format. This is more useful since the
list of supported dis
From: "Daniel P. Berrange"
The LXC driver can already configure or
devices to use the loop device. This extends it to also allow
for use of the NBD device, to support non-raw formats.
Signed-off-by: Daniel P. Berrange
---
src/lxc/lxc_controller.c | 80
From: "Daniel P. Berrange"
Extend the element in filesystem devices to
allow a storage format to be set. The new attribute
uses 'format' to reflect the storage format. This is
different from the element in disk devices
which use 'type' to reflect the storage format. This
is because the 'type' a
A second version of:
https://www.redhat.com/archives/libvir-list/2013-March/msg00941.html
Aside from the addressing the previous round of feedbac, this adds
use of the new '--format' flag to qemu-nbd. This is introduced due
to CVE-2013-1922. The upshot is that this code won't work with any
qemu
Add a test case for query-tpm-models QMP command.
Signed-off-by: Stefan Berger
---
tests/qemumonitorjsontest.c | 55
1 file changed, 55 insertions(+)
Index: libvirt/tests/qemumonitorjsontest.c
===
On Mon, Apr 22, 2013 at 9:13 PM, Osier Yang wrote:
> On 22/04/13 20:33, harryxi...@gmail.com wrote:
>>
>> From: Harry Wei
>>
>> After i read libvirt/HACKING file, i find we should
>
>
> s/i/I/,
Actually, i wonder what 's/i/l/' means?
>
>
>> Omit braces with a single-line body. So this patch
>>
On 22/04/13 20:33, harryxi...@gmail.com wrote:
From: Harry Wei
After i read libvirt/HACKING file, i find we should
s/i/I/,
Omit braces with a single-line body. So this patch
fix this coding style problem for Sheepdog storage
s/fix this/fixes the/,
backend driver.
The commit log can be
From: Harry Wei
After i read libvirt/HACKING file, i find we should
Omit braces with a single-line body. So this patch
fix this coding style problem for Sheepdog storage
backend driver.
Signed-off-by: Harry Wei
---
src/storage/storage_backend_sheepdog.c | 11 ---
1 file changed, 4 in
On Sat, Apr 20, 2013 at 11:11:25AM +0200, Paolo Bonzini wrote:
> uses a helper application to do the necessary
> TUN/TAP setup to use an existing network bridge, thus letting
> unprivileged users use TUN/TAP interfaces.
>
> However, libvirt should be preventing QEMU from running any setuid
> prog
On Sat, Apr 20, 2013 at 11:11:24AM +0200, Paolo Bonzini wrote:
> This will be used on a tap file descriptor returned by the bridge helper
> to populate the element, because the helper does not provide
> the interface name.
>
> Signed-off-by: Paolo Bonzini
> ---
> src/libvirt_private.syms | 1 +
ping?
On 04/17/2013 04:15 PM, Michal Novotny wrote:
> Implement check whether (maximum) vCPUs doesn't exceed machine
> type's cpu-max settings.
>
> On older versions of QEMU the check is disabled.
>
> Signed-off-by: Michal Novotny
> ---
> src/qemu/qemu_capabilities.c | 38 +++
On Thu, Apr 18, 2013 at 08:27:56AM -0600, Eric Blake wrote:
> On 04/18/2013 03:02 AM, Martin Kletzander wrote:
> > In commit d4bf0a9, we used num_queues for an attribute in the XML, but
> > the consensus is that we use camelCase for that. Since there was no
> > release yet (the above commit descri
1 - 100 of 106 matches
Mail list logo