[Qemu-devel] getting link errors when building for qga-vss.dll

2018-11-07 Thread Kirk Allan
When I try to build qga-vss.dll I get the following link error: LINKqga/vss-win32/qga-vss.dll /usr/lib64/gcc/i686-w64-mingw32/8.2.0/../../../../i686-w64-mingw32/bin/ld: qga/vss-win32/requester.o: in function `requester_freeze': /src/qemu/qga/vss-win32/requester.cpp:284: undefined reference to

Re: [Qemu-devel] [PATCH] qga: fix append file open modes for win32

2015-11-11 Thread Kirk Allan
>>> > Quoting Paolo Bonzini (2015-11-11 08:49:57) >> >> >> On 11/11/2015 15:02, Michael Roth wrote: >> >> GENERIC_READ for files >> >> = FILE_READ_DATA >> >> + FILE_READ_ATTRIBUTES >> >> + FILE_READ_EA >> >> + SYNCHRONIZE >> >> + STANDARD_RIGHTS_READ (wh

[Qemu-devel] [PATCH v2] qga: fix append file open modes for win32

2015-11-10 Thread Kirk Allan
will append to the end fo the file. Added white space to maintain the alignment of the guest_file_open_modes[]. Signed-off-by: Kirk Allan --- qga/commands-win32.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/qga/commands-win32.c b/qga/commands-

Re: [Qemu-devel] [PATCH] qga: fix append file open modes for win32

2015-11-10 Thread Kirk Allan
>>> > > On 10/11/2015 16:40, Michael Roth wrote: >> >> I hit an issue testing this though, this does fix the append >> case, but a+, ab+, a+b all imply append+read, while >> FILE_APPEND_DATA only grants append access. >> >> FILE_APPEND_DATA|GENERIC_READ seems to work, but I'm not >> finding

[Qemu-devel] [PATCH] qga: fix append file open modes for win32

2015-11-09 Thread Kirk Allan
For append file open modes, use FILE_APPEND_DATA for the desired access for writing at the end of the file. Signed-off-by: Kirk Allan --- qga/commands-win32.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/qga/commands-win32.c b/qga/commands-win32.c index a5306e7

[Qemu-devel] [PATCH v6 2/2] qga: win32 qmp_guest_network_get_interfaces implementation

2015-06-02 Thread Kirk Allan
OnLinkPrefixLength. Setting --extra-cflags in the build configuration to "-D_WIN32_WINNT=0x600" or greater makes OnLinkPrefixLength available. Setting --extra-cflags is not required and if not set, the default approach to get the prefix will be taken. Signed-off-by: Kirk Allan --- qga/commands-win

[Qemu-devel] [PATCH v6 0/2] qga: qmp_guest_network_get_interfaces for win32

2015-06-02 Thread Kirk Allan
commands-win32.c takes advantage of _WIN32_WINNT if set to 0x600 or greater for Windows Vista/2008 guests or newer to use OnLinkPrefixLength for prefixes. WSAStringToAddress is used for inet_ntop functionality. Kirk Allan (2): qga: add win32 library iphlpapi qga: win32

[Qemu-devel] [PATCH v6 1/2] qga: add win32 library iphlpapi

2015-06-02 Thread Kirk Allan
Add the iphlpapi library to use APIs such as GetAdaptersInfo and GetAdaptersAddresses. Signed-off-by: Kirk Allan --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 4e2f78a..60df96d 100755 --- a/configure +++ b/configure @@ -724,7 +724,7

Re: [Qemu-devel] [PATCH v4 1/2] qga: add additional win32 cflags and libraries

2015-06-02 Thread Kirk Allan
>>> > > On 01/06/2015 23:19, Kirk Allan wrote: >>> Elsewhere on the list, we've discussed putting extra cflags >>> _after_ the configure-detected QEMU_CFLAGS. This would fix the >>> issue in the first part of this patch too. >> >>

[Qemu-devel] [PATCH v5 2/2] qga: win32 qmp_guest_network_get_interfaces implementation

2015-06-01 Thread Kirk Allan
OnLinkPrefixLength. Setting --extra-cflags in the build configuration to "-D_WIN32_WINNT=0x600 -DWINVER=0x600" or greater makes OnLinkPrefixLength available. Setting --extra-cflags is not required and if not set, the default approach to get the prefix will be taken. Signed-off-by: Kirk Allan --- qg

[Qemu-devel] [PATCH v5 0/2] qga: qmp_guest_network_get_interfaces for win32

2015-06-01 Thread Kirk Allan
implementation patch for commands-win32.c takes advantage of _WIN32_WINNT if set to 0x600 or greater for Windows Vista/2008 guests or newer to use OnLinkPrefixLength for prefixes. WSAStringToAddress is used for inet_ntop functionality. Kirk Allan (2): qga: add additional win32 cflags and

[Qemu-devel] [PATCH v5 1/2] qga: add additional win32 cflags and libraries

2015-06-01 Thread Kirk Allan
: Kirk Allan --- configure | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 4e2f78a..67bd9af 100755 --- a/configure +++ b/configure @@ -708,7 +708,12 @@ fi if test "$mingw32" = "yes" ; then EXESUF=".exe"

Re: [Qemu-devel] [PATCH v4 1/2] qga: add additional win32 cflags and libraries

2015-06-01 Thread Kirk Allan
>>> > > On 01/06/2015 20:56, Kirk Allan wrote: >> Test if *-extra-cflags is being used to include flags such as >> _WIN32_WINVER and WINVER to gain additional functionality offered >> by Windows Vista/2008 and newer. If not, default WINVER to 0x501. >&

[Qemu-devel] [PATCH v4 0/2] qga: qmp_guest_network_get_interfaces for win32

2015-06-01 Thread Kirk Allan
to 0x600 or greater for Windows Vista/2008 guests or newer to use OnLinkPrefixLength for prefixes. WSAStringToAddress is used for inet_ntop functionality. Kirk Allan (2): qga: add additional win32 cflags and libraries qga: win32 qmp_guest_network_get_interfaces implementation configure

[Qemu-devel] [PATCH v4 1/2] qga: add additional win32 cflags and libraries

2015-06-01 Thread Kirk Allan
: Kirk Allan --- configure | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 4e2f78a..67bd9af 100755 --- a/configure +++ b/configure @@ -708,7 +708,12 @@ fi if test "$mingw32" = "yes" ; then EXESUF=".exe"

[Qemu-devel] [PATCH v4 2/2] qga: win32 implementation of qmp_guest_network_get_interfaces

2015-06-01 Thread Kirk Allan
OnLinkPrefixLength. Setting -–extra-cflags in the build configuration to ”- D_WIN32_WINNT-0x600 -DWINVER=0x600” or greater makes OnLinkPrefixLength available. Setting –-ectra-cflags is not required and if not set, the default approach to get the prefix will be taken. Signed-off-by: Kirk Allan --- qga/commands

Re: [Qemu-devel] [PATCH v3 1/2] qga: add additional win32 cflags and libraries

2015-05-29 Thread Kirk Allan
>>> > On 05/28/2015 12:41 PM, Kirk Allan wrote: >> Use *extra-cflags to set cflags to such as _WIN32_WINVER and WINVER to > > That Unicode mdash looks suspicious; did you mean for it to be two ASCII > -- instead? Your right, it is --extra-clfags. I'l

[Qemu-devel] [PATCH v3 0/2] qga: qmp_guest_network_get_interfaces for win32

2015-05-28 Thread Kirk Allan
prefixes. Kirk Allan (2): qga: add additional win32 cflags and libraries qga: win32 implementation of qmp_guest_network_get_interfaces configure| 9 +- qga/commands-win32.c | 292 ++- 2 files changed, 297 insertions(+), 4 deletions

[Qemu-devel] [PATCH v3 1/2] qga: add additional win32 cflags and libraries

2015-05-28 Thread Kirk Allan
Use –extra-cflags to set cflags to such as _WIN32_WINVER and WINVER to add additional functionality offered in Windows Visat/2008 and newer. Add the -DARCH_$ARCH cflag. Add the iphlpapi library to use APIs such as GetAdaptersInfo and GetAdaptersAddresses. Signed-off-by: Kirk Allan

[Qemu-devel] [PATCH v3 2/2] qga: win32 implementation of qmp_guest_network_get_interfaces

2015-05-28 Thread Kirk Allan
-cflags is not required and if not used, the default approach will be taken. Signed-off-by: Kirk Allan --- qga/commands-win32.c | 292 ++- 1 file changed, 290 insertions(+), 2 deletions(-) diff --git a/qga/commands-win32.c b/qga/commands-win32.c index

Re: [Qemu-devel] [PATCH v2 2/2] qga: win32 implementation of qmp_guest_network_get_interfaces

2015-05-28 Thread Kirk Allan
Thanks for the feedback. I'll make the necessary adjustments and create a new patch. Kirk >>> > On 01/04/15 18:39, Kirk Allan wrote: >> By default, IP addresses and prefixes will be derived from information >> obtained by various calls and structures. IPv

[Qemu-devel] [PATCH v2 1/2] qga: add additional win32 cflags and libraries

2015-04-01 Thread Kirk Allan
Use –extra-cflags to set cflags to such as _WIN32_WINVER and WINVER to add additional functionality offered in Windows Visat/2008 and newer. Add the -DARCH_$ARCH cflag. Add the iphlpapi library to use APIs such as GetAdaptersInfo and GetAdaptersAddresses. Signed-off-by: Kirk Allan

[Qemu-devel] [PATCH v2 0/2] qga: implement qmp_guest_network_get_interfaces for win32

2015-04-01 Thread Kirk Allan
. The implementation patch for commands-win32.c is that same as before. It will take advantage of _WIN32_WINNT if set to 0x600 or greater for Windows Vista/2008 guests or newer to use inet_ntop and OnLinkPrefixLength for getting addresses and prefixes. Kirk Allan (2): qga: add additional win32

[Qemu-devel] [PATCH v2 2/2] qga: win32 implementation of qmp_guest_network_get_interfaces

2015-04-01 Thread Kirk Allan
-cflags is not required and if not used, the default approach will be taken. Signed-off-by: Kirk Allan --- qga/commands-win32.c | 319 ++- 1 file changed, 317 insertions(+), 2 deletions(-) diff --git a/qga/commands-win32.c b/qga/commands-win32.c index

Re: [Qemu-devel] [PATCH] qga: implement qmp_guest_network_get_interfaces for win32

2015-03-30 Thread Kirk Allan
>>> > Hi Kirk, > > Am 30.03.2015 um 22:44 schrieb Kirk Allan: >> Ping. >> >> Looking for feedback anyone might have regarding this patch. > > I would recommend to make the patch smaller and the commit message more > intuitive by splitting

Re: [Qemu-devel] [PATCH] qga: implement qmp_guest_network_get_interfaces for win32

2015-03-30 Thread Kirk Allan
or > greater, > and 64 bit for Windows Vista/2008 or greater. The non Vista/2008 builds > will > run on Vista/2008 and greater without the extended functionality described > above. > > Signed-off-by: Kirk Allan > --- > configure| 22 +++- > qga/commands

[Qemu-devel] [PATCH] qga: implement qmp_guest_network_get_interfaces for win32

2015-03-16 Thread Kirk Allan
the guest agent for 32 bit, 64 bit, 32 bit for Windows Vista/2008 or greater, and 64 bit for Windows Vista/2008 or greater. The non Vista/2008 builds will run on Vista/2008 and greater without the extended functionality described above. Signed-off-by: Kirk Allan --- configure|