Re: [libvirt] [PATCH] Refactor ESX storage driver and add iSCSI support

2012-09-09 Thread Matthias Bolte
the backend structure to the storage driver and moving the current driver code to its own backend. The second patch for adding the new iSCSI backend. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH] Refactor ESX storage driver and add iSCSI support

2012-09-09 Thread Matthias Bolte
it to make it more clear. Ah, okay. I wasn't aware of that. I don't have an iSCSI setup at hand at the moment. So I cannot test how things work and need to rely on your knowledge in this aspect. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https

Re: [libvirt] [PATCH] Refactor ESX storage driver and add iSCSI support

2012-09-09 Thread Matthias Bolte
aspects as once :) -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH] esx: Support VLAN tags in virtual network port groups

2012-08-26 Thread Matthias Bolte
2012/8/20 Laine Stump la...@laine.org: On 08/18/2012 07:46 AM, Matthias Bolte wrote: --- src/esx/esx_network_driver.c | 38 +- 1 files changed, 33 insertions(+), 5 deletions(-) diff --git a/src/esx/esx_network_driver.c b/src/esx/esx_network_driver.c

[libvirt] [PATCH v2] esx: Support VLAN tags in virtual network port groups

2012-08-26 Thread Matthias Bolte
--- v2: Use network level VLAN config if there is no portgroup specific VLAN config given. src/esx/esx_network_driver.c | 65 ++--- 1 files changed, 60 insertions(+), 5 deletions(-) diff --git a/src/esx/esx_network_driver.c

[libvirt] [PATCH] esx: Support VLAN tags in virtual network port groups

2012-08-18 Thread Matthias Bolte
--- src/esx/esx_network_driver.c | 38 +- 1 files changed, 33 insertions(+), 5 deletions(-) diff --git a/src/esx/esx_network_driver.c b/src/esx/esx_network_driver.c index 09d46d3..2f5f1ab 100644 --- a/src/esx/esx_network_driver.c +++

Re: [libvirt] Proposal to add iSCSI support to esx storage driver

2012-08-12 Thread Matthias Bolte
? Does it differ from the current code at all? If it differs is it that different that we really need this radical split? -- Matthias Bolte http://photron.blogspot.com Hi Matthias, Below is my understanding as per the iSCSI operations mapping of vSphere APIs and libvirt. Storage Pool

Re: [libvirt] [PATCH] esx: Fix dynamic deep copy

2012-08-09 Thread Matthias Bolte
2012/8/6 Michal Privoznik mpriv...@redhat.com: On 05.08.2012 10:38, Matthias Bolte wrote: The static deep copy allocates storage for the copy. The dynamic version injected the dynamic dispatch after the allocation. This triggered the invalid argument check in the dynamically dispatched deep

Re: [libvirt] [PATCH] esx: Implement interface driver

2012-08-09 Thread Matthias Bolte
2012/8/6 Michal Privoznik mpriv...@redhat.com: On 05.08.2012 22:10, Matthias Bolte wrote: Lists available PhysicalNic devices. A PhysicalNic is always active and can neither be defined nor undefined. A PhysicalNic is used to bridge a HostVirtualSwitch to the physical network

Re: [libvirt] [PATCH] esx: Remove redundant checks for esxVI_LookupHostSystemProperties result

2012-08-09 Thread Matthias Bolte
2012/8/6 Michal Privoznik mpriv...@redhat.com: On 05.08.2012 13:48, Matthias Bolte wrote: esxVI_LookupHostSystemProperties guarantees that hostSystem is non-NULL. Remove redundant NULL checks from callers. Also prefer esxVI_GetStringValue over open-coding the logic. --- src/esx

Re: [libvirt] [PATCH] build: fix PROBE() usage of intptr_t

2012-08-09 Thread Matthias Bolte
. * src/xenxs/xenxs_private.h: Likewise. * tests/storagebackendsheepdogtest.c: Likewise. Reported by Matthias Bolte. --- Almost qualifies for the build-breaker rule, except that I didn't reproduce the particular configure settings that Matthias was experiencing on IRC. Fixes the problem, ACK

Re: [libvirt] [PATCH v5] ESX: Add routines to interface driver

2012-08-07 Thread Matthias Bolte
...@hotmail.com CC: libvir-list@redhat.com 2012/8/5 Matthias Bolte matthias.bo...@googlemail.com: 2012/8/2 Ata E Husain Bohra ata.hus...@hotmail.com: Add following routines to esx_interface_driver: esxNumOfInterfaces, esxNumOfDefinedInterfaces, esxListInterfaces

Re: [libvirt] [PATCH v5] ESX: Add routines to interface driver

2012-08-07 Thread Matthias Bolte
of a libvirt network. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH] esx: Fix dynamic deep copy

2012-08-05 Thread Matthias Bolte
The static deep copy allocates storage for the copy. The dynamic version injected the dynamic dispatch after the allocation. This triggered the invalid argument check in the dynamically dispatched deep copy call. The deep copy function expects its dest parameter to be a pointer to a NULL-pointer.

[libvirt] [PATCH] esx: Remove redundant checks for esxVI_LookupHostSystemProperties result

2012-08-05 Thread Matthias Bolte
VMware ESX hosts * * Copyright (C) 2010-2012 Red Hat, Inc. - * Copyright (C) 2009-2011 Matthias Bolte matthias.bo...@googlemail.com + * Copyright (C) 2009-2012 Matthias Bolte matthias.bo...@googlemail.com * Copyright (C) 2009 Maximilian Wilhelm m...@rfc2324.org * * This library is free software

[libvirt] [PATCH] esx: Implement interface driver

2012-08-05 Thread Matthias Bolte
(C) 2010-2011 Red Hat, Inc. - * Copyright (C) 2010 Matthias Bolte matthias.bo...@googlemail.com + * Copyright (C) 2010-2012 Matthias Bolte matthias.bo...@googlemail.com * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public

[libvirt] [PATCH] esx: Implement network driver

2012-08-05 Thread Matthias Bolte
/esx_network_driver.c index 2e0e40b..b42f1d8 100644 --- a/src/esx/esx_network_driver.c +++ b/src/esx/esx_network_driver.c @@ -4,7 +4,7 @@ * host networks * * Copyright (C) 2010-2011 Red Hat, Inc. - * Copyright (C) 2010 Matthias Bolte matthias.bo...@googlemail.com + * Copyright

Re: [libvirt] [PATCH v5] ESX: Add routines to interface driver

2012-08-05 Thread Matthias Bolte
network is best represented by a HostVirtualSwitch and the HostPortGroups are mapped to the portgroups of a libvirt virtual switch. I'm sorry that I let you wait for quite a while now until I came to this understanding that is contrary to your proposed interface driver. -- Matthias Bolte http

Re: [libvirt] [PATCH v5] ESX: Add routines to interface driver

2012-08-05 Thread Matthias Bolte
2012/8/5 Matthias Bolte matthias.bo...@googlemail.com: 2012/8/2 Ata E Husain Bohra ata.hus...@hotmail.com: Add following routines to esx_interface_driver: esxNumOfInterfaces, esxNumOfDefinedInterfaces, esxListInterfaces, esxListDefinedInterfaces

Re: [libvirt] Proposal to add iSCSI support to esx storage driver

2012-08-05 Thread Matthias Bolte
with iSCSI? Does it differ from the current code at all? If it differs is it that different that we really need this radical split? -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH v1] ESX: Add Byte datatype

2012-08-03 Thread Matthias Bolte
/esx/esx_vi_types.h | 29 ++ 3 files changed, 87 insertions(+) ACK, pushed, thanks. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH] ESX: Add Byte datatype

2012-08-01 Thread Matthias Bolte
of a number. %d should be used here. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH v4 0/3] ESX: Add routines to interface driver

2012-08-01 Thread Matthias Bolte
and esxVI_LookupPhysicalNicFromPortGroup) you're reporting an extra error overwriting the original. probably more detailed error. You should avoid this. I still need to review this patch on the conceptual side, I'll probably have time for this later this week. -- Matthias Bolte http

Re: [libvirt] [PATCH] esx: Improve error reporting for unknown VI types

2012-07-29 Thread Matthias Bolte
2012/7/24 Eric Blake ebl...@redhat.com: On 07/21/2012 02:51 PM, Matthias Bolte wrote: Print the actual unknown type name instead of other for AnyType objects. --- src/esx/esx_vi.c |2 +- src/esx/esx_vi_types.c | 36 +--- src/esx/esx_vi_types.h

Re: [libvirt] [PATCH 1/2] ESX: Add routines to interface driver

2012-07-24 Thread Matthias Bolte
in the vSphere API otherwise there will be much more confusion. Therefore, esxVI_HostVirtualNic *virtualNicList = NULL; should be esxVI_HostVirtualNic *hostVirtualNicList = NULL; Apart from that I'll do a more detailed review soon. -- Matthias Bolte http://photron.blogspot.com -- libvir-list

Re: [libvirt] [PATCH 2/2] vmx: handle shared folders parsing

2012-07-21 Thread Matthias Bolte
be check for NULL. But there is no need to strdup it here as hostPath can just be stolen: (*def)-src = hostPath; hostPath = NULL; ACK and pushed with the mentioned changes. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https

Re: [libvirt] [PATCH 1/2] vmx: handle shared folders formatting

2012-07-21 Thread Matthias Bolte
to avoid a warning about shadowing a global before pushing. ACK and pushed. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH] esx: Improve error reporting for unknown VI types

2012-07-21 Thread Matthias Bolte
Print the actual unknown type name instead of other for AnyType objects. --- src/esx/esx_vi.c |2 +- src/esx/esx_vi_types.c | 36 +--- src/esx/esx_vi_types.h |1 + 3 files changed, 27 insertions(+), 12 deletions(-) diff --git a/src/esx/esx_vi.c

Re: [libvirt] ESX: failure while performing CastFromAnyType

2012-07-21 Thread Matthias Bolte
improved this to report the actual type: Call to esxVI_HostHostBusAdapter_CastFromAnyType for unexpected type 'HostBlockHba' Here's the patch for this https://www.redhat.com/archives/libvir-list/2012-July/msg01148.html -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list

Re: [libvirt] [Libvirt][PATCH] ESX: Fix ESX_VI__TEMPLATE__DYNAMIC_DEEP_COPY

2012-07-21 Thread Matthias Bolte
(_actual_type)); \ +return _error_return; \ +} + No need for this extra copy of the ESX_VI__TEMPLATE__DISPATCH macro, I'll remove it before pushing. ACK and pushed, thanks. -- Matthias Bolte http

Re: [libvirt] ESX: failure while performing CastFromAnyType

2012-07-18 Thread Matthias Bolte
Hi, do you have a patch that allows me to reproduce this problem? So I can improve the error reporting here? Regards, Matthias 2012/7/18 Ata E Husain Bohra ata.hus...@hotmail.com: Just want to drop this note, so no one else gets bother with my query. I was able to find the reason and its

Re: [libvirt] Linker error in gnulib test-binary-io.c

2012-07-18 Thread Matthias Bolte
2012/6/17 Matthias Bolte matthias.bo...@googlemail.com: I get the following error on Ubuntu 11.04 with current libvirt git HEAD: $ make check [...] CC test-binary-io.o test-binary-io.c: In function 'main': test-binary-io.c:54:7: warning: implicit declaration of function

Re: [libvirt] [PATCH v3] esx: Extend esxVI_CURL_Download for partial downloads

2012-07-18 Thread Matthias Bolte
2012/7/18 Daniel P. Berrange berra...@redhat.com: On Fri, Jul 13, 2012 at 02:00:19AM +0200, Matthias Bolte wrote: Also ensure that the virBuffer used to store the downloaded data does not overflow. --- v3: - Fix virBuffer limit checks. v2: - Ensure that the used virBuffer dos

[libvirt] [PATCH] esx: Fix esxVI_String_Deserialize

2012-07-18 Thread Matthias Bolte
It was broken since forever as it expected a libxml2 XML_ELEMENT_NODE containing a XML_TEXT_NODE instead of just a XML_TEXT_NODE. This problem was not discovered for so long because esxVI_String_Deserialize was not used until now. Reported by Ata Bohra --- src/esx/esx_vi_types.c | 23

Re: [libvirt] [PATCH] esx: Fix esxVI_String_Deserialize

2012-07-18 Thread Matthias Bolte
2012/7/19 Eric Blake ebl...@redhat.com: On 07/18/2012 04:48 PM, Matthias Bolte wrote: It was broken since forever as it expected a libxml2 XML_ELEMENT_NODE containing a XML_TEXT_NODE instead of just a XML_TEXT_NODE. This problem was not discovered for so long because

[libvirt] [PATCH v3] esx: Extend esxVI_CURL_Download for partial downloads

2012-07-12 Thread Matthias Bolte
Also ensure that the virBuffer used to store the downloaded data does not overflow. --- v3: - Fix virBuffer limit checks. v2: - Ensure that the used virBuffer dos not overflow. src/esx/esx_driver.c |2 +- src/esx/esx_vi.c | 62 +++--

Re: [libvirt] [PATCH v2] esx: Extend esxVI_CURL_Download for partial downloads

2012-07-09 Thread Matthias Bolte
2012/7/8 Doug Goldstein car...@gentoo.org: On Sun, Jul 8, 2012 at 5:44 AM, Matthias Bolte matthias.bo...@googlemail.com wrote: Also ensure that the virBuffer used to store the downloaded data does not overflow. --- v2: - Ensure that the used virBuffer dos not overflow. src/esx

Re: [libvirt] [PATCH 1/3] esx: Wrap libcurl multi handle

2012-07-08 Thread Matthias Bolte
2012/7/3 Doug Goldstein car...@gentoo.org: On Tue, Jul 3, 2012 at 3:17 PM, Matthias Bolte matthias.bo...@googlemail.com wrote: 2012/7/3 Doug Goldstein car...@gentoo.org: On Mon, Jul 2, 2012 at 4:44 PM, Matthias Bolte matthias.bo...@googlemail.com wrote: --- src/esx/esx_vi.c | 111

[libvirt] [PATCH v2] esx: Extend esxVI_CURL_Download for partial downloads

2012-07-08 Thread Matthias Bolte
Also ensure that the virBuffer used to store the downloaded data does not overflow. --- v2: - Ensure that the used virBuffer dos not overflow. src/esx/esx_driver.c |2 +- src/esx/esx_vi.c | 62 +++-- src/esx/esx_vi.h |3 +- 3 files

Re: [libvirt] [PATCH 0/3] esx: Storage volume up- and download

2012-07-03 Thread Matthias Bolte
2012/7/3 Doug Goldstein car...@gentoo.org: On Mon, Jul 2, 2012 at 4:44 PM, Matthias Bolte matthias.bo...@googlemail.com wrote: I started the download part and the stream driver quite a while ago but stopped and put it aside as I realized that the storage volume API up- and download functions

Re: [libvirt] [PATCH 1/3] esx: Wrap libcurl multi handle

2012-07-03 Thread Matthias Bolte
2012/7/3 Doug Goldstein car...@gentoo.org: On Mon, Jul 2, 2012 at 4:44 PM, Matthias Bolte matthias.bo...@googlemail.com wrote: --- src/esx/esx_vi.c | 111 ++ src/esx/esx_vi.h | 18 + 2 files changed, 129 insertions(+), 0

Re: [libvirt] [PATCH 2/3] esx: Extend esxVI_CURL_Download for partial downloads

2012-07-03 Thread Matthias Bolte
2012/7/3 Doug Goldstein car...@gentoo.org: On Mon, Jul 2, 2012 at 4:44 PM, Matthias Bolte matthias.bo...@googlemail.com wrote: --- src/esx/esx_driver.c |2 +- src/esx/esx_vi.c | 27 +-- src/esx/esx_vi.h |3 ++- 3 files changed, 28 insertions(+), 4

Re: [libvirt] [PATCH 0/3] esx: Storage volume up- and download

2012-07-03 Thread Matthias Bolte
2012/7/3 Daniel P. Berrange berra...@redhat.com: On Tue, Jul 03, 2012 at 09:51:52PM +0200, Matthias Bolte wrote: 2012/7/3 Doug Goldstein car...@gentoo.org: On Mon, Jul 2, 2012 at 4:44 PM, Matthias Bolte matthias.bo...@googlemail.com wrote: I started the download part and the stream driver

[libvirt] [PATCH 1/3] esx: Wrap libcurl multi handle

2012-07-02 Thread Matthias Bolte
--- src/esx/esx_vi.c | 111 ++ src/esx/esx_vi.h | 18 + 2 files changed, 129 insertions(+), 0 deletions(-) diff --git a/src/esx/esx_vi.c b/src/esx/esx_vi.c index 5b5ab69..48718b6 100644 --- a/src/esx/esx_vi.c +++ b/src/esx/esx_vi.c

[libvirt] [PATCH 2/3] esx: Extend esxVI_CURL_Download for partial downloads

2012-07-02 Thread Matthias Bolte
--- src/esx/esx_driver.c |2 +- src/esx/esx_vi.c | 27 +-- src/esx/esx_vi.h |3 ++- 3 files changed, 28 insertions(+), 4 deletions(-) diff --git a/src/esx/esx_driver.c b/src/esx/esx_driver.c index db2144c..95b9286 100644 --- a/src/esx/esx_driver.c +++

[libvirt] [PATCH 3/3] esx: Add volume upload and download to the storage driver

2012-07-02 Thread Matthias Bolte
--- /dev/null +++ b/src/esx/esx_stream.c @@ -0,0 +1,610 @@ + +/* + * esx_stream.c: libcurl based stream driver + * + * Copyright (C) 2012 Matthias Bolte matthias.bo...@googlemail.com + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser

[libvirt] [PATCH 0/3] esx: Storage volume up- and download

2012-07-02 Thread Matthias Bolte
I started the download part and the stream driver quite a while ago but stopped and put it aside as I realized that the storage volume API up- and download functions assume one file per volume. This is a problem with ESX as a VMDK can consist of two files. To solve this now I added two new flags:

Re: [libvirt] [PATCH] esx_storage_driver: Add API to upload volumes (virStorageVolUpload for ESX storage driver)

2012-07-01 Thread Matthias Bolte
2012/6/30 Ata Bohra ata.hus...@hotmail.com: Patch to add API to upload Volume contents using ESX driver. As stream driver is not supported for ESX, I have used libcurl to transfer the volume, second, using flags here to pass file descriptor of the source file. This might work for your testing

[libvirt] Linker error in gnulib test-binary-io.c

2012-06-17 Thread Matthias Bolte
`main': [...]/test-binary-io.c:54: undefined reference to `set_binary_mode' collect2: ld returned 1 exit status -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCHv2 variant 1] snapshot: implement new APIs for esx and vbox

2012-06-17 Thread Matthias Bolte
); is missing here. +    VBOX_RELEASE(machine); +    vboxIIDUnalloc(iid); +    return ret; +} Tested and works. ACK. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCHv2 variant 2] snapshot: implement new APIs for esx and vbox

2012-06-17 Thread Matthias Bolte
++  src/vbox/vbox_tmpl.c |   73 ++  2 files changed, 123 insertions(+) ACK, to this one to as it is just a short version of variant 1, so you got to choose :) -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list

Re: [libvirt] [PATCH] esx: Fix memory leaks in error paths related to transferred ownership

2012-05-13 Thread Matthias Bolte
2012/5/7 Michal Privoznik mpriv...@redhat.com: On 06.05.2012 19:39, Matthias Bolte wrote: Appending an item to a list transfers ownership of that item to the list owner. But an error can occur in between item allocation and appending it to the list. In this case the item has to be freed

[libvirt] [PATCH] esx: Fix memory leaks in error paths related to transferred ownership

2012-05-06 Thread Matthias Bolte
Appending an item to a list transfers ownership of that item to the list owner. But an error can occur in between item allocation and appending it to the list. In this case the item has to be freed explicitly. This was not done in some special cases resulting in possible memory leaks. Reported by

Re: [libvirt] [PATCH 3/7] Coverity: Fix resource leak in esx driver

2012-05-03 Thread Matthias Bolte
on to fix the possible leak in the special error path. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH] vbox: Fix passing an empty IMedium* array to IMachine::Delete

2012-04-23 Thread Matthias Bolte
Am 23. April 2012 18:48 schrieb Eric Blake ebl...@redhat.com: On 04/23/2012 02:29 AM, Jean-Baptiste Rouault wrote: On Sunday 22 April 2012 10:35:59 Matthias Bolte wrote: vboxArray is not castable to a COM item type. vboxArray is a wrapper around the XPCOM and MSCOM specific array handling

Re: [libvirt] [PATCH] win32: Properly handle TlsGetValue returning NULL

2012-04-22 Thread Matthias Bolte
Am 22. April 2012 04:32 schrieb Eric Blake ebl...@redhat.com: On 04/21/2012 11:11 AM, Matthias Bolte wrote: virThreadSelf tries to access the virThreadPtr stored in TLS for the current thread via TlsGetValue. When virThreadSelf is called on a thread that was not created via virThreadCreate

[libvirt] [PATCH] vbox: Fix passing an empty IMedium* array to IMachine::Delete

2012-04-22 Thread Matthias Bolte
vboxArray is not castable to a COM item type. vboxArray is a wrapper around the XPCOM and MSCOM specific array handling. In this case we can avoid passing NULL as an empty array to IMachine::Delete by passing a dummy IMedium* array with a single NULL item. --- Jean-Baptiste, I can not reproduce

Re: [libvirt] ESX driver connection private data fix

2012-04-21 Thread Matthias Bolte
for the delayed response. Thanks for debugging and reporting this problem. ACK, I applied and pushed your patch. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH] win32: Properly handle TlsGetValue returning NULL

2012-04-21 Thread Matthias Bolte
virThreadSelf tries to access the virThreadPtr stored in TLS for the current thread via TlsGetValue. When virThreadSelf is called on a thread that was not created via virThreadCreate (e.g. the main thread) then TlsGetValue returns NULL as TlsAlloc initializes TLS slots to NULL. virThreadSelf can

Re: [libvirt] [PATCH] vbox: avoid provoking assertions in VBoxSVC

2012-04-20 Thread Matthias Bolte
time to take care of this tomorrow, if nobody fixed it before then. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH] build: avoid type-punning in vbox

2012-04-20 Thread Matthias Bolte
work by accident on XPCOM because VirtualBox might not touch the pointer beyond checking it for being non-NULL because the 0 tells it that it's an empty array. But still this is wrong and might crash on MSCOM. I might come up with a proper solution tomorrow. -- Matthias Bolte http

Re: [libvirt] Making the Thread-safety issues with vbox driver ?

2012-04-19 Thread Matthias Bolte
Am 19. April 2012 12:51 schrieb Jean-Baptiste Rouault jean-baptiste.roua...@diateam.net: On Wednesday 04 April 2012 17:56:12 Jean-Baptiste Rouault wrote: On Monday 16 January 2012 11:34:53 Matthias Bolte wrote: Okay, without looking deeper into this here are some ideas: The XPCOM API

Re: [libvirt] Can't connect ESXi ssl with virsh

2012-03-08 Thread Matthias Bolte
. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [Patch v2] vmx: Better Workstation vmx handling

2012-02-24 Thread Matthias Bolte
--- ACK and pushed, thanks. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH 1/3] xml: output memory unit for clarity

2012-02-23 Thread Matthias Bolte
decide to name the attribute units (plural) instead of unit? As a value can only have one unit the attribute name units is misleading. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH 2/2] vmx: Better Workstation vmx handling

2012-02-22 Thread Matthias Bolte
in the vmx2xml and xml2vmx tests to cover this additions to the VMX parser before I can ACK it. A good start but it needs a v2. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH 1/2] vmware: implement domainXMLFromNative

2012-02-22 Thread Matthias Bolte
2012/2/22 Jean-Baptiste Rouault jean-baptiste.roua...@diateam.net: ---  src/vmware/vmware_driver.c |   31 +++  1 files changed, 31 insertions(+), 0 deletions(-) ACK. This one is good and independent from the second patch, so I push it now. -- Matthias Bolte http

Re: [libvirt] [PATCH 2/2] vmx: Better Workstation vmx handling

2012-02-22 Thread Matthias Bolte
2012/2/22 Matthias Bolte matthias.bo...@googlemail.com: And this needs extra testcases in the vmx2xml and xml2vmx tests to cover this additions to the VMX parser before I can ACK it. The test suite already has some actual in-the-wild ESX and GSX VMX config files. I suggest you add some actual

Re: [libvirt] [PATCH] esx: Correctly disable HTTP Expect header usage of libcurl

2012-02-21 Thread Matthias Bolte
2012/2/20 Eric Blake ebl...@redhat.com: On 02/20/2012 02:38 PM, Matthias Bolte wrote: Adding Expect: to the header list stops libcurl from sending a Expect header at all. Before, a dummy Expect header was added that might confuse HTTP proxies and result in HTTP error code 417 being reported

[libvirt] [PATCH] esx: Correctly disable HTTP Expect header usage of libcurl

2012-02-20 Thread Matthias Bolte
Adding Expect: to the header list stops libcurl from sending a Expect header at all. Before, a dummy Expect header was added that might confuse HTTP proxies and result in HTTP error code 417 being reported. --- src/esx/esx_vi.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff

Re: [libvirt] [PATCH v6] npiv: Auto-generate WWN if it's not specified

2012-02-07 Thread Matthias Bolte
information? -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH v5] npiv: Auto-generate WWN if it's not specified

2012-02-06 Thread Matthias Bolte
for ESX and Hyper-V (assume the drivers would support node devices yet) that might use different OUIs? Or do I misunderstand this? -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH] vmware: detect when a domain was shut down from the inside

2012-02-03 Thread Matthias Bolte
for information and cache nothing. I'm not sure what's the best approach here. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] hyperv connection

2012-01-05 Thread Matthias Bolte
connection. Try this instead virsh -c hyperv://192.168.58.231/?transport=http This sets the transport to HTTP and the port to 5985. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] Libvirt java from Windows

2012-01-04 Thread Matthias Bolte
can find libvirt-0.dll at all. It needs to be in the PATH or in the working directory of you application. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [libvirt-users] How to connect to a remote VM ESXi server with Python language?

2011-12-28 Thread Matthias Bolte
of libvirt.open(). openAuth() allows to pass credentials via a callback mechanism. There is a Python example in the libvirt codebase that does this, see http://libvirt.org/git/?p=libvirt.git;a=blob;f=examples/python/esxlist.py -- Matthias Bolte http://photron.blogspot.com -- libvir-list

Re: [libvirt] [PATCH] Fix uninitialized variable in NWfilter IP learning code

2011-11-23 Thread Matthias Bolte
on declaration, or put ret = 0 in an else branch here instead? -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [libvirt-users] listDomains() not working on Xen

2011-11-15 Thread Matthias Bolte
. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH] Add missing defaultConsoleTargetType callback for AppArmour

2011-11-07 Thread Matthias Bolte
of caps. I tested it with that fixed and it works. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH 7/7] Fix default console type setting

2011-11-05 Thread Matthias Bolte
(pipe): '--dryrun -r -u libvirt-----0123456789ab': FAIL: virt-aa-helper-test -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH] API: document scheduler parameter names

2011-11-02 Thread Matthias Bolte
) (xenDaemonSetSchedulerParameters): Likewise. * src/lxc/lxc_driver.c (lxcSetSchedulerParametersFlags) (lxcGetSchedulerParametersFlags): Likewise. --- You missed the ESX allocation scheduler with reservation, limit and shares parameters. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list

Re: [libvirt] [PATCH 2/2] qemu: allow getting max typed parameters

2011-11-02 Thread Matthias Bolte
some uninitialized warnings only with -O2, while it doesn't warn with -O0. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCHv2] esx: Support folders in the path of vpx:// connection URIs

2011-11-01 Thread Matthias Bolte
Allow the datacenter and compute resource parts of the path to be prefixed with folders. Therefore, the way the path is parsed has changed. Before, it was split in 2 or 3 items and the items' meanings were determined by their positions. Now the path can have 2 or more items and the the vCenter

Re: [libvirt] [PATCHv2] esx: Support folders in the path of vpx:// connection URIs

2011-11-01 Thread Matthias Bolte
2011/11/1 Eric Blake ebl...@redhat.com: On 11/01/2011 10:21 AM, Matthias Bolte wrote: Allow the datacenter and compute resource parts of the path to be prefixed with folders. Therefore, the way the path is parsed has changed. Before, it was split in 2 or 3 items and the items' meanings were

Re: [libvirt] [PATCH] esx: Support vSphere 5.x

2011-11-01 Thread Matthias Bolte
), virtualHW_version); goto cleanup; Actually we're currently in feature freeze for libvirt 0.9.7, but as this is a pure addition patch, that will not break existing code I pushed it now. Also because I don't want to delay this for another release cycle. ACK, thanks. -- Matthias

Re: [libvirt] [Libvirt-announce] Start of freeze for libvirt-0.9.7 and availability of rc1

2011-10-31 Thread Matthias Bolte
, Daniel Compiles fine on MinGW. I also did a quick test with VirtualBox 4.1 on Windows and that works as well. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH] python: Fix documentation of virStream recv

2011-10-31 Thread Matthias Bolte
This was fixed in be757a3f7baf93b for libvirt.c. --- I'm pushing this one under the trivial rule. python/libvirt-override-virStream.py |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/python/libvirt-override-virStream.py b/python/libvirt-override-virStream.py index

Re: [libvirt] [PATCH] xenapi: Improve error reporting in xenapiOpen once again

2011-10-29 Thread Matthias Bolte
2011/7/21 Eric Blake ebl...@redhat.com: On 07/21/2011 07:17 AM, Matthias Bolte wrote: privP-session-error_description is a list and in order to get the complete error message all parts of the list should be concatenated. xenapiSessionErrorHandler does this when its third parameter is NULL

Re: [libvirt] [PATCH] vbox: Support shared folders

2011-10-29 Thread Matthias Bolte
2011/6/14 Eric Blake ebl...@redhat.com: On 06/07/2011 05:38 AM, Matthias Bolte wrote: Shared folders are handled as filesystems and can also be hotplugged. --- Currently this just maps shared folder to a filesystem element with type mount. The filesystem element has an accessmode attribute

Re: [libvirt] [PATCH] vbox: Add support for VirtualBox 4.1

2011-10-29 Thread Matthias Bolte
2011/10/25 Eric Blake ebl...@redhat.com: On 10/23/2011 08:10 AM, Matthias Bolte wrote: Deal with the incompatible changes in the VirtualBox 4.1 API. INetworkAdapter has its different AttachTo* method replaced by a settable attachmentType property. The maximum number of network adapters

Re: [libvirt] [PATCH V3 3/4] Extend NWFilter parameter parser to cope with lists of values

2011-10-24 Thread Matthias Bolte
the same from the NWFilter's point-of-view then this needs to be fixed to have NWFilter preserve the exact order of elements. If it's the same then there's no problem here. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com

Re: [libvirt] [PATCH V2 00/10] Make inner workings of nwfilters more flexible + extensions

2011-10-19 Thread Matthias Bolte
there is at least the possibility to configure the different firewalls of different hypervisors via libvirt. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH V2 00/10] Make inner workings of nwfilters more flexible + extensions

2011-10-19 Thread Matthias Bolte
2011/10/19 David Stevens dlstev...@us.ibm.com: -Matthias Bolte matthias.bo...@googlemail.com wrote: - Well, you miss the point that nwfilters is meant as a general firewall interface. ebtables/iptables just happens to be an implementation of this interface. Using ebtables/iptables

Re: [libvirt] [PATCH v3 10/13] Implement virConnectIsAlive in all drivers

2011-10-18 Thread Matthias Bolte
case, the function will return true +     * even though the connection is not alive. +     */ +    if (priv-client) +        return 1; +    else +        return 0; +} This one is okay as is, I don't have a better suggestion at hand right now. -- Matthias Bolte http://photron.blogspot.com

Re: [libvirt] [PATCH v3 10/13] Implement virConnectIsAlive in all drivers

2011-10-18 Thread Matthias Bolte
2011/10/18 Matthias Bolte matthias.bo...@googlemail.com: 2011/10/12 Jiri Denemark jdene...@redhat.com: --- Notes:    Version 3:    - no changes    Version 2:    - new patch  src/esx/esx_driver.c       |   18 ++  src/hyperv/hyperv_driver.c |   18 ++  src

Re: [libvirt] [PATCH 3/3] snapshot: implement LIST_LEAVES flag in esx

2011-10-15 Thread Matthias Bolte
/esx_driver.c (esxDomainSnapshotNum) (esxDomainSnapshotListNames, esxDomainSnapshotNumChildren) (esxDomainSnapshotListChildrenNames): Pass new flag through. --- Tested, works, ACK. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https

Re: [libvirt] [PATCH] esx: drop dead code to silence Coverity

2011-10-14 Thread Matthias Bolte
probably have a look at your pending ESX (and VBox?) snapshot patches tomorrow. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] libvirt changes/certification questions

2011-10-11 Thread Matthias Bolte
behavior.  Beyond that, anything you can add to automate regression testing would probably be appreciated. The website has a list about the different test suites http://libvirt.org/testsuites.html -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@redhat.com

Re: [libvirt] [PATCHv2 13/7] snapshot: implement snapshot children listing in vbox

2011-10-07 Thread Matthias Bolte
of a domain with many snapshots that we need to avoid a recursive solution? You already implemented vboxCountDescendants recursively, I'd suggest to do the same here too. I attached a patch to be applied on top of this one that does this and works for me. ACK with the attached patch applied. -- Matthias

Re: [libvirt] [PATCHv2 8/7] snapshot: implement snapshot roots listing in esx

2011-10-05 Thread Matthias Bolte
--  src/esx/esx_vi.h     |    6 --  3 files changed, 33 insertions(+), 16 deletions(-) Tested, works, ACK. I also tested the case of deleting a root snapshot with multiple children. Works as expected when you delete the root you got multiple new roots. -- Matthias Bolte http

<    1   2   3   4   5   6   7   8   9   10   >