On 04/28/2015 09:53 AM, Peter Krempa wrote:
> The documentation states that for shallow block copy the image has to
> have the same guest visible content as backing file of the current
> image if the file is being reused. This condition can be achieved also
> with a raw file (or a qcow without a ba
On 04/27/2015 10:08 PM, Shanzhi Yu wrote:
>>> Should libvirt post error when try a shallow blockcopy of file without
>>> backing file, just as shallow blockcommit?
>> I cannot reproduce the error above, could you please post steps to do
>> that? or perhaps debug log from libvirt?
>
> It is easy to
On 04/28/2015 06:46 PM, Cole Robinson wrote:
> Ping... I think this should be safe for the release, since the AIX value has
> never been exposed to the user
ACK, and indeed worth having in the release
>
> - Cole
>
> On 04/22/2015 10:47 AM, Cole Robinson wrote:
>> The phyp driver stuffed it into
On 04/28/2015 05:51 PM, Cole Robinson wrote:
> Many users, who admin their own machines, want to be able to access
> system libvirtd via tools like virt-manager without having to enter
> a root password. Just google 'virt-manager without password' and
> you'll find many hits. I've read at least 5 b
On 04/28/2015 09:36 AM, Peter Krempa wrote:
> On Mon, Apr 27, 2015 at 16:48:44 -0400, Cole Robinson wrote:
>> Only set directory permissions at pool build time, if:
>>
>> - User explicitly requested a mode via the XML
>> - The directory needs to be created
>> - We need to do the crazy NFS root-squa
On 04/28/2015 09:36 AM, Peter Krempa wrote:
> On Mon, Apr 27, 2015 at 16:48:44 -0400, Cole Robinson wrote:
>> Only set directory permissions at pool build time, if:
>>
>> - User explicitly requested a mode via the XML
>> - The directory needs to be created
>> - We need to do the crazy NFS root-squa
On 04/28/2015 09:23 AM, Peter Krempa wrote:
> On Mon, Apr 27, 2015 at 16:48:43 -0400, Cole Robinson wrote:
>> The XML parser sets a default if none is explicitly passed in.
>> This is then used at pool/vol creation time, and unconditionally reported
>> in the XML.
>>
>> The problem with this appro
On 04/28/2015 08:19 AM, Peter Krempa wrote:
> On Mon, Apr 27, 2015 at 16:48:41 -0400, Cole Robinson wrote:
>> The current code attempts to handle this, but it only catches mkdir
>> failing with EEXIST. However if say trying to build /tmp for an
>> unprivileged qemu:///session, mkdir will fail with
On 04/28/2015 08:06 AM, Peter Krempa wrote:
> On Mon, Apr 27, 2015 at 16:48:40 -0400, Cole Robinson wrote:
>> This means pool XML actually reports accurate user/group/mode/label.
>>
>> This uses UpdateVolTargetInfoFD in a bit of a hackish way, but it works
>> ---
>> src/storage/storage_backend_fs.
Ping... I think this should be safe for the release, since the AIX value has
never been exposed to the user
- Cole
On 04/22/2015 10:47 AM, Cole Robinson wrote:
> The phyp driver stuffed it into a DomainDefPtr during its attachdevice
> routine, but the value is never advertised via capabilities so
This can cause permissions failures if qemu.conf user/group is changed.
Fix this by filling in the VNC auto socket path in the same code area
that we fill in default listen address, ports, etc.
https://bugzilla.redhat.com/show_bug.cgi?id=1151762
---
src/qemu/qemu_command.c | 10 ++
src/q
Many users, who admin their own machines, want to be able to access
system libvirtd via tools like virt-manager without having to enter
a root password. Just google 'virt-manager without password' and
you'll find many hits. I've read at least 5 blog posts over the years
describing slightly differen
Jim Fehlig wrote:
> Provide integration with libvirt's lock manager in the libxl driver.
>
> Signed-off-by: Jim Fehlig
> ---
>
> V3 of
>
> https://www.redhat.com/archives/libvir-list/2015-April/msg01006.html
>
> In V3, call virDomainLockProcessStart() with 'paused' parameter
> set to 'true' and ad
Thanks Peter,
It looks good but I did no deep introspection of the code.
Should I rebase my patch [1] on that? I can test your code as well then.
[1] https://github.com/paboldin/libvirt/commits/master
Pavel
On Tue, Apr 28, 2015 at 4:42 PM, Peter Krempa wrote:
> On Tue, Apr 28, 2015 at 11:24:
On Tue, Apr 28, 2015 at 17:37:42 +0200, Peter Krempa wrote:
> Add sanity checks for some unsupported/invalid configs of memory devices.
>
> Peter Krempa (3):
> qemu: conf: Reject memory device if it would exceed configured max
> size
> qemu: command: Validate that memory devices slot ID is
2015-04-24 14:40 GMT+03:00 Michal Privoznik :
> Yeah, it's not as actively developed as libvirt. And it not uncommon for
> our bindings to not have an API implemented. The best would be to post a
> patch to the list and CC Michal Novotny who has done the majority of
> work there.
Thanks, i'm send
This patch add to libvirt php binding libvirt stream support
Also provide resize/upload/download for libvirt volume
Signed-off-by: Vasiliy Tolstov
---
configure.ac | 3 +-
src/libvirt-php.c | 382 +-
src/libvirt-php.h | 17 +++
3 files
This patch add to libvirt php binding libvirt stream support
Also provide resize/upload/download for libvirt volume
Signed-off-by: Vasiliy Tolstov
---
configure.ac | 3 +-
src/libvirt-php.c | 382 +-
src/libvirt-php.h | 17 +++
3 files
libvirt 1.2.13.1 is now available. This is a maintenance release of
libvirt 1.2.13 with additional bugfixes that have accumulated
upstream since the initial release.
This release can be downloaded at:
http://libvirt.org/sources/stable_updates/libvirt-1.2.13.1.tar.gz
Changes in this version:
* F
On 04/23/2015 09:37 PM, Maxim Nestratov wrote:
New functions utilize previosly added prlsdkDelDisk and prlsdkGetDiskIndex
Signed-off-by: Maxim Nestratov
---
src/parallels/parallels_driver.c | 71 ++
src/parallels/parallels_sdk.c| 30
On 04/23/2015 09:37 PM, Maxim Nestratov wrote:
Signed-off-by: Maxim Nestratov
---
src/parallels/parallels_sdk.c | 65 +
1 files changed, 65 insertions(+), 0 deletions(-)
diff --git a/src/parallels/parallels_sdk.c b/src/parallels/parallels_sdk.c
index
libvirt 1.2.9.3 is now available. This is a maintenance release of
libvirt 1.2.9 with additional bugfixes that have accumulated
upstream since the initial release.
This release can be downloaded at:
http://libvirt.org/sources/stable_updates/libvirt-1.2.9.3.tar.gz
Changes in this version:
* stor
The documentation states that for shallow block copy the image has to
have the same guest visible content as backing file of the current
image if the file is being reused. This condition can be achieved also
with a raw file (or a qcow without a backing file) so remove the
condition that would disal
Replace all occurrences of "stream write to differences to"
with "stream to write differences to".
---
tests/testutils.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/tests/testutils.c b/tests/testutils.c
index 6a8fe6a..89026c6 100644
--- a/tests/testutils.c
+++ b/tests
slot id, if specified, has to be less than the slots count.
---
src/qemu/qemu_command.c | 11 ++-
src/qemu/qemu_command.h | 1 +
src/qemu/qemu_hotplug.c | 2 +-
3 files changed, 12 insertions(+), 2 deletions(-)
diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index ba15dc9
Add sanity checks for some unsupported/invalid configs of memory devices.
Peter Krempa (3):
qemu: conf: Reject memory device if it would exceed configured max
size
qemu: command: Validate that memory devices slot ID is in range
qemu: Validate available slot count for memory devices
src
If the added memory device would exceed the maximum memory size, reject
it.
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1216046
---
src/conf/domain_conf.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index fc48ed5..7e4f
While qemu would reject the configuration we can check whether it makes
sense to plug the device upfront.
---
src/qemu/qemu_command.c | 8
src/qemu/qemu_driver.c | 6 ++
src/qemu/qemu_hotplug.c | 6 ++
3 files changed, 20 insertions(+)
diff --git a/src/qemu/qemu_command.c b/src/
On 28.04.2015 03:16, Zhang Bo wrote:
> the free callback should be qemuMonitorChardevInfoFree rather than just 'free'
> when virHashCreate'ing the chardevInfo hash.
>
> Zhang Bo (2):
> qemu: make qemuMonitorChardevInfoFree non-static
> tests: free ChardevInfo correctly in qemumonitorjsontest
>
On 28.04.2015 13:15, Martin Kletzander wrote:
> Signed-off-by: Pavel Fedin
> Signed-off-by: Martin Kletzander
> ---
>
> Notes:
> This was a part of commit 9dc57ce2, but separable; plus the library
> should be in LIBADD and not LDFLAGS. Depends on my previous cleanup [2]
>
> [1]
On 04/28/2015 07:58 AM, Michal Privoznik wrote:
> There's no need to mark a function as inline in the function
> declaration. In fact, it causes a compilation error:
>
> CC xmlgen.lo
> In file included from acl_parsing.h:29:0,
> from xmlgen.h:26,
> from
On 2015-04-26 13:28, r...@openmailbox.org wrote:
Hi. I am trying to get a virtual smartcard attached to a vm but I want
it to use GPG instead of NSS. RedHat focuses on NSS becuase of PKCS#11
requirements and FIPS approval, but for most of the community its GPG
that matters for smartcards.
Is is
On Tue, Apr 28, 2015 at 11:24:37 +0200, Michal Privoznik wrote:
> On 28.04.2015 11:06, Pavel Boldin wrote:
> > Well, actually that seems to be quite a different bug in there.
> >
> > I will start a new thread.
> >
> > In short: migration seems to be broken by commit
> > 1a92c719101e5bfa6fe2b78006
On 04/28/2015 07:50 AM, Ján Tomko wrote:
> The summary says 'EPERM', but the code checks for EACCES.
>
> On Mon, Apr 27, 2015 at 11:57:53AM -0400, Cole Robinson wrote:
>> Trying to use qemu:///session to create a storage pool pointing at
>> /tmp will usually fail with something like:
>>
>> $ virsh
On 04/28/2015 08:07 AM, Ján Tomko wrote:
> On Mon, Apr 27, 2015 at 10:44:54AM -0400, Cole Robinson wrote:
>> First patch fixes a regression introduced with pool state patches
>> Last 3 patches help fix other issues with the state tracking
>>
>> Cole Robinson (4):
>> storage: Fix autostart dir for
On Tue, Apr 28, 2015 at 08:53:28AM -0400, John Ferlan wrote:
>
>
> On 04/28/2015 07:58 AM, Michal Privoznik wrote:
> > There's no need to mark a function as inline in the function
> > declaration. In fact, it causes a compilation error:
> >
> > CC xmlgen.lo
> > In file included from acl_
On Mon, Apr 27, 2015 at 16:48:44 -0400, Cole Robinson wrote:
> Only set directory permissions at pool build time, if:
>
> - User explicitly requested a mode via the XML
> - The directory needs to be created
> - We need to do the crazy NFS root-squash workaround
>
> This allows qemu:///session to
On Mon, Apr 27, 2015 at 16:48:42 -0400, Cole Robinson wrote:
> Currently we try to chown any directory passed to virDirCreate,
> even if the user didn't request any explicit owner/group via the
> pool/vol XML.
>
> This causes issues with qemu:///session: try to build a pool of
> a root owned direc
On Mon, Apr 27, 2015 at 16:48:43 -0400, Cole Robinson wrote:
> The XML parser sets a default if none is explicitly passed in.
> This is then used at pool/vol creation time, and unconditionally reported
> in the XML.
>
> The problem with this approach is that it's impossible for other code
> to de
On Tue, Apr 28, 2015 at 08:53:28AM -0400, John Ferlan wrote:
> FWIW:
>
> libvirt-cim patches have been sent to the libvirt-cim list
> @libvirt-...@redhat.com and not libvir-list...
Given that the libvirt CIM project is in maint mode only aat
this point, with little to no feature work, I'm not sur
On 28.04.2015 14:53, John Ferlan wrote:
>
>
> On 04/28/2015 07:58 AM, Michal Privoznik wrote:
>> There's no need to mark a function as inline in the function
>> declaration. In fact, it causes a compilation error:
>>
>> CC xmlgen.lo
>> In file included from acl_parsing.h:29:0,
>>
On 04/28/2015 07:58 AM, Michal Privoznik wrote:
> There's no need to mark a function as inline in the function
> declaration. In fact, it causes a compilation error:
>
> CC xmlgen.lo
> In file included from acl_parsing.h:29:0,
> from xmlgen.h:26,
> from
On 04/28/2015 08:23 AM, Nehal J Wani wrote:
> The repository link for libvirt-cim at
> http://libvirt.org/CIM/downloads.html seems outdated. Shouldn't it be:
> git clone git://libvirt.org/libvirt-cim
> instead of
> hg clone http://libvirt.org/hg/libvirt-cim
>
yes, those instructions are really
On Tue, Apr 28, 2015 at 07:23:02 -0400, John Ferlan wrote:
>
>
> On 04/28/2015 06:40 AM, John Ferlan wrote:
> > v1 here:
> > http://www.redhat.com/archives/libvir-list/2015-April/msg01382.html
> >
> > v2 changes:
> >
> > Patch 1 - Add a !STRPREFIX(tmp, "iothread") of the returned iothreads
> >
The repository link for libvirt-cim at
http://libvirt.org/CIM/downloads.html seems outdated. Shouldn't it be:
git clone git://libvirt.org/libvirt-cim
instead of
hg clone http://libvirt.org/hg/libvirt-cim
--
Nehal J Wani
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/ma
On Tue, Apr 28, 2015 at 11:28:38AM +0200, Marc-André Lureau wrote:
Hi Martin
On Mon, Apr 27, 2015 at 4:20 PM, Martin Kletzander
wrote:
ACK series with two things to consider mentioned in-line (not hard
requirements, but it would be nice to have).
It seems we missed 1.2.15, so I updated doc
On Mon, Apr 27, 2015 at 16:48:41 -0400, Cole Robinson wrote:
> The current code attempts to handle this, but it only catches mkdir
> failing with EEXIST. However if say trying to build /tmp for an
> unprivileged qemu:///session, mkdir will fail with EPERM.
>
> Rather than catch any errors, just do
On Mon, Apr 27, 2015 at 10:44:54AM -0400, Cole Robinson wrote:
> First patch fixes a regression introduced with pool state patches
> Last 3 patches help fix other issues with the state tracking
>
> Cole Robinson (4):
> storage: Fix autostart dir for qemu:///session
> storage: Don't leave stale
On Mon, Apr 27, 2015 at 16:48:40 -0400, Cole Robinson wrote:
> This means pool XML actually reports accurate user/group/mode/label.
>
> This uses UpdateVolTargetInfoFD in a bit of a hackish way, but it works
> ---
> src/storage/storage_backend_fs.c | 58
> ++--
There's no need to mark a function as inline in the function
declaration. In fact, it causes a compilation error:
CC xmlgen.lo
In file included from acl_parsing.h:29:0,
from xmlgen.h:26,
from capability_parsing.c:37:
list_util.h:67:21: error: inline functi
The summary says 'EPERM', but the code checks for EACCES.
On Mon, Apr 27, 2015 at 11:57:53AM -0400, Cole Robinson wrote:
> Trying to use qemu:///session to create a storage pool pointing at
> /tmp will usually fail with something like:
>
> $ virsh pool-start tmp
> error: Failed to start pool tmp
On Tue, Apr 28, 2015 at 07:29:17AM -0400, John Ferlan wrote:
> Add support for the new IOThread APIs
>
> Signed-off-by: John Ferlan
> ---
> Changes| 1 +
> Virt.xs| 20
> lib/Sys/Virt/Domain.pm | 12
> 3 files changed, 33 inserti
Add support for the new IOThread APIs
Signed-off-by: John Ferlan
---
Changes| 1 +
Virt.xs| 20
lib/Sys/Virt/Domain.pm | 12
3 files changed, 33 insertions(+)
diff --git a/Changes b/Changes
index ead8f8e..58efa37 100644
--- a/Ch
On 04/28/2015 06:40 AM, John Ferlan wrote:
> v1 here:
> http://www.redhat.com/archives/libvir-list/2015-April/msg01382.html
>
> v2 changes:
>
> Patch 1 - Add a !STRPREFIX(tmp, "iothread") of the returned iothreads
> to ignore anything that doesn't start with "iothread".
>
> Patch 2
Signed-off-by: Pavel Fedin
Signed-off-by: Martin Kletzander
---
Notes:
This was a part of commit 9dc57ce2, but separable; plus the library
should be in LIBADD and not LDFLAGS. Depends on my previous cleanup [2]
[1] https://libvirt.org/git/?p=libvirt.git;a=commitdiff;h=9dc57ce2
On Tue, Apr 28, 2015 at 11:40:02AM +0300, Pavel Fedin wrote:
Hello!
I have checked. For some reason, $(virnetserverclientmock_la_LDADDS)
does not take part anywhere. Here are generated rules:
It has to be virnetserverclientmock_la_LIBADD instead. I have tested, works
fine.
I just wrote th
On Tue, Apr 28, 2015 at 06:40:32 -0400, John Ferlan wrote:
> Replace with just VIR_FREE.
>
> Signed-off-by: John Ferlan
> ---
> src/qemu/qemu_driver.c | 6 +++---
> src/qemu/qemu_monitor.c | 9 -
> src/qemu/qemu_monitor.h | 2 --
> src/qemu/qemu_monitor_json.c | 2 +-
> s
On Tue, Apr 28, 2015 at 06:40:30 -0400, John Ferlan wrote:
> Rather than have a separate routine to parse the alias of an iothread
> returned from qemu in order to get the iothread_id value, parse the alias
> when returning and just return the iothread_id in qemuMonitorIOThreadInfoPtr
>
> This set
On Tue, Apr 28, 2015 at 06:40:31 -0400, John Ferlan wrote:
> If we received zero iothreads from the monitor, but were perhaps
> expecting to receive something, then the code was skipping the check
> to ensure what's in the monitor matches our expectations. So invert
> the checks to check that what
Historically I have taken care of everything related to the Sys::Virt
Perl binding for libvirt. Recently a couple of adventurous people
have started sending patches too. Perl may have a reputation for being
scary, but at this point extending the Sys::Virt binding when new APIs
or constants are adde
Replace with just VIR_FREE.
Signed-off-by: John Ferlan
---
src/qemu/qemu_driver.c | 6 +++---
src/qemu/qemu_monitor.c | 9 -
src/qemu/qemu_monitor.h | 2 --
src/qemu/qemu_monitor_json.c | 2 +-
src/qemu/qemu_process.c | 2 +-
tests/qemumonitorjsontest.c | 2 +-
6 fi
v1 here:
http://www.redhat.com/archives/libvir-list/2015-April/msg01382.html
v2 changes:
Patch 1 - Add a !STRPREFIX(tmp, "iothread") of the returned iothreads
to ignore anything that doesn't start with "iothread".
Patch 2 - already ACKed, just difficult to extract.
Patch 3 - NEW: rem
If we received zero iothreads from the monitor, but were perhaps
expecting to receive something, then the code was skipping the check
to ensure what's in the monitor matches our expectations. So invert
the checks to check that what we get back matches expectations and
then check there are zero iot
Rather than have a separate routine to parse the alias of an iothread
returned from qemu in order to get the iothread_id value, parse the alias
when returning and just return the iothread_id in qemuMonitorIOThreadInfoPtr
This set of patches removes the function, changes the "char *name" to
"unsign
On Tue, Apr 28, 2015 at 11:54:07AM +0200, Michal Privoznik wrote:
> Signed-off-by: Michal Privoznik
> ---
> Changes| 1 +
> Virt.xs| 1 +
> lib/Sys/Virt/Domain.pm | 6 ++
> 3 files changed, 8 insertions(+)
ACK
Regards,
Daniel
--
|: http://berrange.com -
On Tue, Apr 28, 2015 at 11:54:06AM +0200, Michal Privoznik wrote:
> Signed-off-by: Michal Privoznik
> ---
> Changes| 1 +
> Virt.xs| 1 +
> lib/Sys/Virt/Domain.pm | 6 ++
> 3 files changed, 8 insertions(+)
ACK
Regards,
Daniel
--
|: http://berrange.com
On Tue, Apr 28, 2015 at 11:54:05AM +0200, Michal Privoznik wrote:
> Signed-off-by: Michal Privoznik
> ---
> Changes| 3 ++-
> Virt.xs| 42 ++
> lib/Sys/Virt/Domain.pm | 4
> t/030-api-coverage.t | 1 +
> 4 files cha
On Tue, Apr 28, 2015 at 11:54:04AM +0200, Michal Privoznik wrote:
> Signed-off-by: Michal Privoznik
> ---
> Virt.xs | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Virt.xs b/Virt.xs
> index d01cf05..4b3ba9a 100644
> --- a/Virt.xs
> +++ b/Virt.xs
> @@ -2946,8 +2946,8
In order to match the newish things in libvirt.
Michal Privoznik (4):
domain_event_register_any: Align the code
Add support for VIR_DOMAIN_EVENT_ID_DEVICE_ADDED
Add JOB_DOWNTIME_NET constant
Add JOB_TIME_ELAPSED_NET constant
Changes| 5 -
Virt.xs| 48
Signed-off-by: Michal Privoznik
---
Changes| 1 +
Virt.xs| 1 +
lib/Sys/Virt/Domain.pm | 6 ++
3 files changed, 8 insertions(+)
diff --git a/Changes b/Changes
index a299dc7..7a7ba4c 100644
--- a/Changes
+++ b/Changes
@@ -4,6 +4,7 @@ Revision history for perl m
Signed-off-by: Michal Privoznik
---
Changes| 3 ++-
Virt.xs| 42 ++
lib/Sys/Virt/Domain.pm | 4
t/030-api-coverage.t | 1 +
4 files changed, 49 insertions(+), 1 deletion(-)
diff --git a/Changes b/Changes
index 35ec
Signed-off-by: Michal Privoznik
---
Changes| 1 +
Virt.xs| 1 +
lib/Sys/Virt/Domain.pm | 6 ++
3 files changed, 8 insertions(+)
diff --git a/Changes b/Changes
index 7a7ba4c..ead8f8e 100644
--- a/Changes
+++ b/Changes
@@ -5,6 +5,7 @@ Revision history for perl m
Signed-off-by: Michal Privoznik
---
Virt.xs | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Virt.xs b/Virt.xs
index d01cf05..4b3ba9a 100644
--- a/Virt.xs
+++ b/Virt.xs
@@ -2946,8 +2946,8 @@ PREINIT:
callback =
VIR_DOMAIN_EVENT_CALLBACK(_domain_event_device_remo
Hi Martin
On Mon, Apr 27, 2015 at 4:20 PM, Martin Kletzander
wrote:
> ACK series with two things to consider mentioned in-line (not hard
> requirements, but it would be nice to have).
>
It seems we missed 1.2.15, so I updated doc comment about version
availability, and I addressed your comments
On 28.04.2015 11:06, Pavel Boldin wrote:
> Well, actually that seems to be quite a different bug in there.
>
> I will start a new thread.
>
> In short: migration seems to be broken by commit
> 1a92c719101e5bfa6fe2b78006ad04c7f075ea28. This is because introduced job
> _MODIFY waits while MIGRATION
Well, actually that seems to be quite a different bug in there.
I will start a new thread.
In short: migration seems to be broken by commit
1a92c719101e5bfa6fe2b78006ad04c7f075ea28. This is because introduced job
_MODIFY waits while MIGRATION_OUT is finished to change `mirrorState'
variable. This
On Mon, Apr 27, 2015 at 15:08:42 -0400, John Ferlan wrote:
> Resolve some Coverity errors with IOThread changes
>
> Signed-off-by: John Ferlan
> ---
>
> I ran my Coverity checker too, but I looked at the wrong results
> tab when I went to check the results... I looked at a previous run
> which w
On Mon, Apr 27, 2015 at 14:51:05 -0400, John Ferlan wrote:
> If we received zero iothreads from the monitor, but were perhaps
> expecting to receive something, then the code was skipping the check
> to ensure what's in the monitor matches our expectations. So invert
> the checks to check that what
On Mon, Apr 27, 2015 at 14:51:04 -0400, John Ferlan wrote:
> Rather than have a separate routine to parse the alias of an iothread
> returned from qemu in order to get the iothread_id value, parse the alias
> when returning and just return the iothread_id in qemuMonitorIOThreadInfoPtr
>
> This set
Hello!
> I have checked. For some reason, $(virnetserverclientmock_la_LDADDS)
> does not take part anywhere. Here are generated rules:
It has to be virnetserverclientmock_la_LIBADD instead. I have tested, works
fine.
Kind regards,
Pavel Fedin
Expert Engineer
Samsung Electronics Research cente
On Mon, Apr 27, 2015 at 16:48:39 -0400, Cole Robinson wrote:
> virDirCreate will give us fine grained details about what actually failed.
> ---
> src/storage/storage_backend_fs.c | 4
> 1 file changed, 4 deletions(-)
ACK,
Peter
signature.asc
Description: Digital signature
--
libvir-list m
Peter Krempa wrote:
> On Tue, Apr 28, 2015 at 08:30:30 +0400, Roman Bogorodskiy wrote:
> > Build with clang fails with:
> >
> > CC conf/libvirt_conf_la-domain_conf.lo
> > conf/domain_conf.c:13377:9: error: variable 'cpumask' is used
> > uninitialized whenever 'if' condition is true
On Tue, Apr 28, 2015 at 08:30:30 +0400, Roman Bogorodskiy wrote:
> Build with clang fails with:
>
> CC conf/libvirt_conf_la-domain_conf.lo
> conf/domain_conf.c:13377:9: error: variable 'cpumask' is used
> uninitialized whenever 'if' condition is true
> [-Werror,-Wsometimes-uninitiali
Hello!
> If it works, feel free to close the BZ or reply if it doesn't.
I have checked. For some reason, $(virnetserverclientmock_la_LDADDS) does
not take part anywhere. Here are generated rules:
--- cut ---
virnetserverclientmock_la_LINK = $(LIBTOOL) $(AM_V_lt) --tag=CC \
$(AM_LIBTOOLF
Hello!
> If it works, feel free to close the BZ or reply if it doesn't.
It doesn't seem to work. And the error is, indeed:
--- cut ---
CCLD virnetserverclientmock.la
.libs/virnetserverclientmock_la-virnetserverclientmock.o: In function
`virNetSocketGetSELinuxContext':
C:\mingw64\msys\1.0\
85 matches
Mail list logo