qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10
@ 2021-04-05 13:14 Basil Salman
  2021-04-05 13:14 ` [PATCH 2/3] qga-win: Fix build_guest_fsinfo() close of nonexistent handle Basil Salman
                   ` (5 more replies)
  0 siblings, 6 replies; 12+ messages in thread
From: Basil Salman @ 2021-04-05 13:14 UTC (permalink / raw)
  To: qemu-devel, Michael Roth; +Cc: Yan Vugenfirer, Kostiantyn Kostiuk

Currently Requester freeze times out after 10 seconds, while
the default timeout for Writer Freeze is 60 seconds. according to
VSS Documentation [1].
[1]: https://docs.microsoft.com/en-us/windows/win32/vss/overview-of-processing-a-backup-under-vss

Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1909073

Signed-off-by: Basil Salman <bsalman@daynix.com>
Signed-off-by: Basil Salman <basil@daynix.com>
---
 qga/vss-win32/requester.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/qga/vss-win32/requester.cpp b/qga/vss-win32/requester.cpp
index 5378c55d23..940a2c8f55 100644
--- a/qga/vss-win32/requester.cpp
+++ b/qga/vss-win32/requester.cpp
@@ -18,7 +18,7 @@
 #include <inc/win2003/vsbackup.h>
 
 /* Max wait time for frozen event (VSS can only hold writes for 10 seconds) */
-#define VSS_TIMEOUT_FREEZE_MSEC 10000
+#define VSS_TIMEOUT_FREEZE_MSEC 60000
 
 /* Call QueryStatus every 10 ms while waiting for frozen event */
 #define VSS_TIMEOUT_EVENT_MSEC 10
-- 
2.17.2



^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 2/3] qga-win: Fix build_guest_fsinfo() close of nonexistent handle
  2021-04-05 13:14 [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10 Basil Salman
@ 2021-04-05 13:14 ` Basil Salman
  2021-04-05 13:14 ` [PATCH 3/3] qga-win: Fix handle leak in ga_get_win_product_name() Basil Salman
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 12+ messages in thread
From: Basil Salman @ 2021-04-05 13:14 UTC (permalink / raw)
  To: qemu-devel, Michael Roth; +Cc: Yan Vugenfirer, Kostiantyn Kostiuk

On the current error path of build_guest_fsinfo(), a non existent
handle is passed to CloseHandle().
This patch add initialization of hLocalDiskHandle to INVALID_HANDLE_VALUE,
and checks for handle validity before the handle is closed.

Signed-off-by: Basil Salman <basil@daynix.com>
Signed-off-by: Basil Salman <basil@redhat.com>
---
 qga/commands-win32.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/qga/commands-win32.c b/qga/commands-win32.c
index 27baf17d6c..d51833ef15 100644
--- a/qga/commands-win32.c
+++ b/qga/commands-win32.c
@@ -1091,7 +1091,7 @@ static GuestFilesystemInfo *build_guest_fsinfo(char *guid, Error **errp)
     size_t len;
     uint64_t i64FreeBytesToCaller, i64TotalBytes, i64FreeBytes;
     GuestFilesystemInfo *fs = NULL;
-    HANDLE hLocalDiskHandle = NULL;
+    HANDLE hLocalDiskHandle = INVALID_HANDLE_VALUE;
 
     GetVolumePathNamesForVolumeName(guid, (LPCH)&mnt, 0, &info_size);
     if (GetLastError() != ERROR_MORE_DATA) {
@@ -1149,7 +1149,8 @@ static GuestFilesystemInfo *build_guest_fsinfo(char *guid, Error **errp)
     fs->type = g_strdup(fs_name);
     fs->disk = build_guest_disk_info(guid, errp);
 free:
-    CloseHandle(hLocalDiskHandle);
+    if (hLocalDiskHandle != INVALID_HANDLE_VALUE) {
+        CloseHandle(hLocalDiskHandle);
+    }
     g_free(mnt_point);
     return fs;
 }
-- 
2.17.2



^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 3/3] qga-win: Fix handle leak in ga_get_win_product_name()
  2021-04-05 13:14 [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10 Basil Salman
  2021-04-05 13:14 ` [PATCH 2/3] qga-win: Fix build_guest_fsinfo() close of nonexistent handle Basil Salman
@ 2021-04-05 13:14 ` Basil Salman
  2021-04-22  7:43 ` [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10 Konstantin Kostiuk
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 12+ messages in thread
From: Basil Salman @ 2021-04-05 13:14 UTC (permalink / raw)
  To: qemu-devel, Michael Roth; +Cc: Yan Vugenfirer, Kostiantyn Kostiuk

In ga_get_win_product_name() a handle to Registry key was open
but not closed.
In this patch the handle is closed as part of the free routine.

Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1929144

Signed-off-by: Basil Salman <basil@daynix.com>
Signed-off-by: Basil Salman <bsalman@redhat.com>
---
 qga/commands-win32.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/qga/commands-win32.c b/qga/commands-win32.c
index d51833ef15..8cea3ec0f8 100644
--- a/qga/commands-win32.c
+++ b/qga/commands-win32.c
@@ -2229,7 +2229,7 @@ static char *ga_get_win_name(OSVERSIONINFOEXW const *os_version, bool id)
 
 static char *ga_get_win_product_name(Error **errp)
 {
-    HKEY key = NULL;
+    HKEY key = INVALID_HANDLE_VALUE;
     DWORD size = 128;
     char *result = g_malloc0(size);
     LONG err = ERROR_SUCCESS;
@@ -2239,7 +2239,8 @@ static char *ga_get_win_product_name(Error **errp)
                       &key);
     if (err != ERROR_SUCCESS) {
         error_setg_win32(errp, err, "failed to open registry key");
-        goto fail;
+        g_free(result);
+        return NULL;
     }
 
     err = RegQueryValueExA(key, "ProductName", NULL, NULL,
@@ -2260,9 +2261,12 @@ static char *ga_get_win_product_name(Error **errp)
         goto fail;
     }
 
+    RegCloseKey(key);
     return result;
 
 fail:
+    if (key != INVALID_HANDLE_VALUE) {
+        RegCloseKey(key);
+    }
     g_free(result);
     return NULL;
 }
-- 
2.17.2



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10
  2021-04-05 13:14 [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10 Basil Salman
  2021-04-05 13:14 ` [PATCH 2/3] qga-win: Fix build_guest_fsinfo() close of nonexistent handle Basil Salman
  2021-04-05 13:14 ` [PATCH 3/3] qga-win: Fix handle leak in ga_get_win_product_name() Basil Salman
@ 2021-04-22  7:43 ` Konstantin Kostiuk
  2021-04-29  8:51   ` Konstantin Kostiuk
  2021-07-13 18:40   ` Michael Roth
  2021-05-28 11:21 ` Konstantin Kostiuk
                   ` (2 subsequent siblings)
  5 siblings, 2 replies; 12+ messages in thread
From: Konstantin Kostiuk @ 2021-04-22  7:43 UTC (permalink / raw)
  To: Developers; +Cc: Yan Vugenfirer, Michael Roth

[-- Attachment #1: Type: text/plain, Size: 1131 bytes --]

ping

On Mon, Apr 5, 2021 at 4:14 PM Basil Salman <basil@daynix.com> wrote:

> Currently Requester freeze times out after 10 seconds, while
> the default timeout for Writer Freeze is 60 seconds. according to
> VSS Documentation [1].
> [1]:
> https://docs.microsoft.com/en-us/windows/win32/vss/overview-of-processing-a-backup-under-vss
>
> Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1909073
>
> Signed-off-by: Basil Salman <bsalman@daynix.com>
> Signed-off-by: Basil Salman <basil@daynix.com>
> ---
>  qga/vss-win32/requester.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/qga/vss-win32/requester.cpp b/qga/vss-win32/requester.cpp
> index 5378c55d23..940a2c8f55 100644
> --- a/qga/vss-win32/requester.cpp
> +++ b/qga/vss-win32/requester.cpp
> @@ -18,7 +18,7 @@
>  #include <inc/win2003/vsbackup.h>
>
>  /* Max wait time for frozen event (VSS can only hold writes for 10
> seconds) */
> -#define VSS_TIMEOUT_FREEZE_MSEC 10000
> +#define VSS_TIMEOUT_FREEZE_MSEC 60000
>
>  /* Call QueryStatus every 10 ms while waiting for frozen event */
>  #define VSS_TIMEOUT_EVENT_MSEC 10
> --
> 2.17.2
>
>

[-- Attachment #2: Type: text/html, Size: 1888 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10
  2021-04-22  7:43 ` [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10 Konstantin Kostiuk
@ 2021-04-29  8:51   ` Konstantin Kostiuk
  2021-07-13 18:40   ` Michael Roth
  1 sibling, 0 replies; 12+ messages in thread
From: Konstantin Kostiuk @ 2021-04-29  8:51 UTC (permalink / raw)
  To: Developers; +Cc: Yan Vugenfirer, Michael Roth

[-- Attachment #1: Type: text/plain, Size: 1259 bytes --]

ping

On Thu, Apr 22, 2021 at 10:43 AM Konstantin Kostiuk <konstantin@daynix.com>
wrote:

> ping
>
> On Mon, Apr 5, 2021 at 4:14 PM Basil Salman <basil@daynix.com> wrote:
>
>> Currently Requester freeze times out after 10 seconds, while
>> the default timeout for Writer Freeze is 60 seconds. according to
>> VSS Documentation [1].
>> [1]:
>> https://docs.microsoft.com/en-us/windows/win32/vss/overview-of-processing-a-backup-under-vss
>>
>> Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1909073
>>
>> Signed-off-by: Basil Salman <bsalman@daynix.com>
>> Signed-off-by: Basil Salman <basil@daynix.com>
>> ---
>>  qga/vss-win32/requester.cpp | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/qga/vss-win32/requester.cpp b/qga/vss-win32/requester.cpp
>> index 5378c55d23..940a2c8f55 100644
>> --- a/qga/vss-win32/requester.cpp
>> +++ b/qga/vss-win32/requester.cpp
>> @@ -18,7 +18,7 @@
>>  #include <inc/win2003/vsbackup.h>
>>
>>  /* Max wait time for frozen event (VSS can only hold writes for 10
>> seconds) */
>> -#define VSS_TIMEOUT_FREEZE_MSEC 10000
>> +#define VSS_TIMEOUT_FREEZE_MSEC 60000
>>
>>  /* Call QueryStatus every 10 ms while waiting for frozen event */
>>  #define VSS_TIMEOUT_EVENT_MSEC 10
>> --
>> 2.17.2
>>
>>

[-- Attachment #2: Type: text/html, Size: 2263 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10
  2021-04-05 13:14 [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10 Basil Salman
                   ` (2 preceding siblings ...)
  2021-04-22  7:43 ` [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10 Konstantin Kostiuk
@ 2021-05-28 11:21 ` Konstantin Kostiuk
  2021-06-11  6:56 ` Konstantin Kostiuk
  2021-07-04  5:52 ` Konstantin Kostiuk
  5 siblings, 0 replies; 12+ messages in thread
From: Konstantin Kostiuk @ 2021-05-28 11:21 UTC (permalink / raw)
  Cc: Yan Vugenfirer, Michael Roth, Developers

[-- Attachment #1: Type: text/plain, Size: 1131 bytes --]

ping

On Mon, Apr 5, 2021 at 4:14 PM Basil Salman <basil@daynix.com> wrote:

> Currently Requester freeze times out after 10 seconds, while
> the default timeout for Writer Freeze is 60 seconds. according to
> VSS Documentation [1].
> [1]:
> https://docs.microsoft.com/en-us/windows/win32/vss/overview-of-processing-a-backup-under-vss
>
> Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1909073
>
> Signed-off-by: Basil Salman <bsalman@daynix.com>
> Signed-off-by: Basil Salman <basil@daynix.com>
> ---
>  qga/vss-win32/requester.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/qga/vss-win32/requester.cpp b/qga/vss-win32/requester.cpp
> index 5378c55d23..940a2c8f55 100644
> --- a/qga/vss-win32/requester.cpp
> +++ b/qga/vss-win32/requester.cpp
> @@ -18,7 +18,7 @@
>  #include <inc/win2003/vsbackup.h>
>
>  /* Max wait time for frozen event (VSS can only hold writes for 10
> seconds) */
> -#define VSS_TIMEOUT_FREEZE_MSEC 10000
> +#define VSS_TIMEOUT_FREEZE_MSEC 60000
>
>  /* Call QueryStatus every 10 ms while waiting for frozen event */
>  #define VSS_TIMEOUT_EVENT_MSEC 10
> --
> 2.17.2
>
>

[-- Attachment #2: Type: text/html, Size: 1872 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10
  2021-04-05 13:14 [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10 Basil Salman
                   ` (3 preceding siblings ...)
  2021-05-28 11:21 ` Konstantin Kostiuk
@ 2021-06-11  6:56 ` Konstantin Kostiuk
  2021-07-04  5:52 ` Konstantin Kostiuk
  5 siblings, 0 replies; 12+ messages in thread
From: Konstantin Kostiuk @ 2021-06-11  6:56 UTC (permalink / raw)
  To: Developers; +Cc: Yan Vugenfirer, Michael Roth

[-- Attachment #1: Type: text/plain, Size: 1131 bytes --]

ping

On Mon, Apr 5, 2021 at 4:14 PM Basil Salman <basil@daynix.com> wrote:

> Currently Requester freeze times out after 10 seconds, while
> the default timeout for Writer Freeze is 60 seconds. according to
> VSS Documentation [1].
> [1]:
> https://docs.microsoft.com/en-us/windows/win32/vss/overview-of-processing-a-backup-under-vss
>
> Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1909073
>
> Signed-off-by: Basil Salman <bsalman@daynix.com>
> Signed-off-by: Basil Salman <basil@daynix.com>
> ---
>  qga/vss-win32/requester.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/qga/vss-win32/requester.cpp b/qga/vss-win32/requester.cpp
> index 5378c55d23..940a2c8f55 100644
> --- a/qga/vss-win32/requester.cpp
> +++ b/qga/vss-win32/requester.cpp
> @@ -18,7 +18,7 @@
>  #include <inc/win2003/vsbackup.h>
>
>  /* Max wait time for frozen event (VSS can only hold writes for 10
> seconds) */
> -#define VSS_TIMEOUT_FREEZE_MSEC 10000
> +#define VSS_TIMEOUT_FREEZE_MSEC 60000
>
>  /* Call QueryStatus every 10 ms while waiting for frozen event */
>  #define VSS_TIMEOUT_EVENT_MSEC 10
> --
> 2.17.2
>
>

[-- Attachment #2: Type: text/html, Size: 1883 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10
  2021-04-05 13:14 [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10 Basil Salman
                   ` (4 preceding siblings ...)
  2021-06-11  6:56 ` Konstantin Kostiuk
@ 2021-07-04  5:52 ` Konstantin Kostiuk
  2021-07-11 17:18   ` Konstantin Kostiuk
  5 siblings, 1 reply; 12+ messages in thread
From: Konstantin Kostiuk @ 2021-07-04  5:52 UTC (permalink / raw)
  To: Developers; +Cc: Yan Vugenfirer, Michael Roth

[-- Attachment #1: Type: text/plain, Size: 1131 bytes --]

ping

On Mon, Apr 5, 2021 at 4:14 PM Basil Salman <basil@daynix.com> wrote:

> Currently Requester freeze times out after 10 seconds, while
> the default timeout for Writer Freeze is 60 seconds. according to
> VSS Documentation [1].
> [1]:
> https://docs.microsoft.com/en-us/windows/win32/vss/overview-of-processing-a-backup-under-vss
>
> Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1909073
>
> Signed-off-by: Basil Salman <bsalman@daynix.com>
> Signed-off-by: Basil Salman <basil@daynix.com>
> ---
>  qga/vss-win32/requester.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/qga/vss-win32/requester.cpp b/qga/vss-win32/requester.cpp
> index 5378c55d23..940a2c8f55 100644
> --- a/qga/vss-win32/requester.cpp
> +++ b/qga/vss-win32/requester.cpp
> @@ -18,7 +18,7 @@
>  #include <inc/win2003/vsbackup.h>
>
>  /* Max wait time for frozen event (VSS can only hold writes for 10
> seconds) */
> -#define VSS_TIMEOUT_FREEZE_MSEC 10000
> +#define VSS_TIMEOUT_FREEZE_MSEC 60000
>
>  /* Call QueryStatus every 10 ms while waiting for frozen event */
>  #define VSS_TIMEOUT_EVENT_MSEC 10
> --
> 2.17.2
>
>

[-- Attachment #2: Type: text/html, Size: 1883 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10
  2021-07-04  5:52 ` Konstantin Kostiuk
@ 2021-07-11 17:18   ` Konstantin Kostiuk
  0 siblings, 0 replies; 12+ messages in thread
From: Konstantin Kostiuk @ 2021-07-11 17:18 UTC (permalink / raw)
  To: Developers; +Cc: Yan Vugenfirer, Michael Roth

[-- Attachment #1: Type: text/plain, Size: 1257 bytes --]

ping

On Sun, Jul 4, 2021 at 8:52 AM Konstantin Kostiuk <konstantin@daynix.com>
wrote:

> ping
>
> On Mon, Apr 5, 2021 at 4:14 PM Basil Salman <basil@daynix.com> wrote:
>
>> Currently Requester freeze times out after 10 seconds, while
>> the default timeout for Writer Freeze is 60 seconds. according to
>> VSS Documentation [1].
>> [1]:
>> https://docs.microsoft.com/en-us/windows/win32/vss/overview-of-processing-a-backup-under-vss
>>
>> Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1909073
>>
>> Signed-off-by: Basil Salman <bsalman@daynix.com>
>> Signed-off-by: Basil Salman <basil@daynix.com>
>> ---
>>  qga/vss-win32/requester.cpp | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/qga/vss-win32/requester.cpp b/qga/vss-win32/requester.cpp
>> index 5378c55d23..940a2c8f55 100644
>> --- a/qga/vss-win32/requester.cpp
>> +++ b/qga/vss-win32/requester.cpp
>> @@ -18,7 +18,7 @@
>>  #include <inc/win2003/vsbackup.h>
>>
>>  /* Max wait time for frozen event (VSS can only hold writes for 10
>> seconds) */
>> -#define VSS_TIMEOUT_FREEZE_MSEC 10000
>> +#define VSS_TIMEOUT_FREEZE_MSEC 60000
>>
>>  /* Call QueryStatus every 10 ms while waiting for frozen event */
>>  #define VSS_TIMEOUT_EVENT_MSEC 10
>> --
>> 2.17.2
>>
>>

[-- Attachment #2: Type: text/html, Size: 2272 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10
  2021-04-22  7:43 ` [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10 Konstantin Kostiuk
  2021-04-29  8:51   ` Konstantin Kostiuk
@ 2021-07-13 18:40   ` Michael Roth
  2021-07-14  6:17     ` Konstantin Kostiuk
  1 sibling, 1 reply; 12+ messages in thread
From: Michael Roth @ 2021-07-13 18:40 UTC (permalink / raw)
  To: Developers, Konstantin Kostiuk; +Cc: Yan Vugenfirer

Quoting Konstantin Kostiuk (2021-04-22 02:43:25)
> ping

I've been trying to get these queued but I'm hitting an issue where qga
reports:

  failed to load qga-vss.dll: The specified module could not be found.

via LoadLibraryA(QGA_VSS_DLL) returning error code 126. What's weird is
it seems to find qga-vss.dll in the install directory, and you can see it
access it in WinDBG and various trace tools, but somehow it reports not
found. Are you seeing this issue?

I'll debug more this week and try to get these in for rc1, but if you happen
to have more of a clue than me then any insights would be much appreciated.

> 
> On Mon, Apr 5, 2021 at 4:14 PM Basil Salman <basil@daynix.com> wrote:
> 
>     Currently Requester freeze times out after 10 seconds, while
>     the default timeout for Writer Freeze is 60 seconds. according to
>     VSS Documentation [1].
>     [1]: https://docs.microsoft.com/en-us/windows/win32/vss/
>     overview-of-processing-a-backup-under-vss
> 
>     Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1909073
> 
>     Signed-off-by: Basil Salman <bsalman@daynix.com>
>     Signed-off-by: Basil Salman <basil@daynix.com>
>     ---
>      qga/vss-win32/requester.cpp | 2 +-
>      1 file changed, 1 insertion(+), 1 deletion(-)
> 
>     diff --git a/qga/vss-win32/requester.cpp b/qga/vss-win32/requester.cpp
>     index 5378c55d23..940a2c8f55 100644
>     --- a/qga/vss-win32/requester.cpp
>     +++ b/qga/vss-win32/requester.cpp
>     @@ -18,7 +18,7 @@
>      #include <inc/win2003/vsbackup.h>
> 
>      /* Max wait time for frozen event (VSS can only hold writes for 10
>     seconds) */
>     -#define VSS_TIMEOUT_FREEZE_MSEC 10000
>     +#define VSS_TIMEOUT_FREEZE_MSEC 60000
> 
>      /* Call QueryStatus every 10 ms while waiting for frozen event */
>      #define VSS_TIMEOUT_EVENT_MSEC 10
>     --
>     2.17.2
> 
>


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10
  2021-07-13 18:40   ` Michael Roth
@ 2021-07-14  6:17     ` Konstantin Kostiuk
  2021-07-28  7:53       ` Konstantin Kostiuk
  0 siblings, 1 reply; 12+ messages in thread
From: Konstantin Kostiuk @ 2021-07-14  6:17 UTC (permalink / raw)
  To: Michael Roth; +Cc: Yan Vugenfirer, Developers

[-- Attachment #1: Type: text/plain, Size: 2284 bytes --]

Hi Michael,

qga-vss.dll has many dependencies. Please check that all DLLs are present
in the same directory.

Also, I have a similar problem in my setup. In my case, libgcc_s_seh-1.dll
was renamed in MinGW which I used.

Best wishes,
Kostiantyn Kostiuk


On Tue, Jul 13, 2021 at 9:40 PM Michael Roth <michael.roth@amd.com> wrote:

> Quoting Konstantin Kostiuk (2021-04-22 02:43:25)
> > ping
>
> I've been trying to get these queued but I'm hitting an issue where qga
> reports:
>
>   failed to load qga-vss.dll: The specified module could not be found.
>
> via LoadLibraryA(QGA_VSS_DLL) returning error code 126. What's weird is
> it seems to find qga-vss.dll in the install directory, and you can see it
> access it in WinDBG and various trace tools, but somehow it reports not
> found. Are you seeing this issue?
>
> I'll debug more this week and try to get these in for rc1, but if you
> happen
> to have more of a clue than me then any insights would be much appreciated.
>
> >
> > On Mon, Apr 5, 2021 at 4:14 PM Basil Salman <basil@daynix.com> wrote:
> >
> >     Currently Requester freeze times out after 10 seconds, while
> >     the default timeout for Writer Freeze is 60 seconds. according to
> >     VSS Documentation [1].
> >     [1]: https://docs.microsoft.com/en-us/windows/win32/vss/
> >     overview-of-processing-a-backup-under-vss
> >
> >     Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1909073
> >
> >     Signed-off-by: Basil Salman <bsalman@daynix.com>
> >     Signed-off-by: Basil Salman <basil@daynix.com>
> >     ---
> >      qga/vss-win32/requester.cpp | 2 +-
> >      1 file changed, 1 insertion(+), 1 deletion(-)
> >
> >     diff --git a/qga/vss-win32/requester.cpp
> b/qga/vss-win32/requester.cpp
> >     index 5378c55d23..940a2c8f55 100644
> >     --- a/qga/vss-win32/requester.cpp
> >     +++ b/qga/vss-win32/requester.cpp
> >     @@ -18,7 +18,7 @@
> >      #include <inc/win2003/vsbackup.h>
> >
> >      /* Max wait time for frozen event (VSS can only hold writes for 10
> >     seconds) */
> >     -#define VSS_TIMEOUT_FREEZE_MSEC 10000
> >     +#define VSS_TIMEOUT_FREEZE_MSEC 60000
> >
> >      /* Call QueryStatus every 10 ms while waiting for frozen event */
> >      #define VSS_TIMEOUT_EVENT_MSEC 10
> >     --
> >     2.17.2
> >
> >
>

[-- Attachment #2: Type: text/html, Size: 3687 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10
  2021-07-14  6:17     ` Konstantin Kostiuk
@ 2021-07-28  7:53       ` Konstantin Kostiuk
  0 siblings, 0 replies; 12+ messages in thread
From: Konstantin Kostiuk @ 2021-07-28  7:53 UTC (permalink / raw)
  To: Michael Roth; +Cc: Yan Vugenfirer, Developers

[-- Attachment #1: Type: text/plain, Size: 2454 bytes --]

ping

On Wed, Jul 14, 2021 at 9:17 AM Konstantin Kostiuk <konstantin@daynix.com>
wrote:

> Hi Michael,
>
> qga-vss.dll has many dependencies. Please check that all DLLs are present
> in the same directory.
>
> Also, I have a similar problem in my setup. In my case, libgcc_s_seh-1.dll
> was renamed in MinGW which I used.
>
> Best wishes,
> Kostiantyn Kostiuk
>
>
> On Tue, Jul 13, 2021 at 9:40 PM Michael Roth <michael.roth@amd.com> wrote:
>
>> Quoting Konstantin Kostiuk (2021-04-22 02:43:25)
>> > ping
>>
>> I've been trying to get these queued but I'm hitting an issue where qga
>> reports:
>>
>>   failed to load qga-vss.dll: The specified module could not be found.
>>
>> via LoadLibraryA(QGA_VSS_DLL) returning error code 126. What's weird is
>> it seems to find qga-vss.dll in the install directory, and you can see it
>> access it in WinDBG and various trace tools, but somehow it reports not
>> found. Are you seeing this issue?
>>
>> I'll debug more this week and try to get these in for rc1, but if you
>> happen
>> to have more of a clue than me then any insights would be much
>> appreciated.
>>
>> >
>> > On Mon, Apr 5, 2021 at 4:14 PM Basil Salman <basil@daynix.com> wrote:
>> >
>> >     Currently Requester freeze times out after 10 seconds, while
>> >     the default timeout for Writer Freeze is 60 seconds. according to
>> >     VSS Documentation [1].
>> >     [1]: https://docs.microsoft.com/en-us/windows/win32/vss/
>> >     overview-of-processing-a-backup-under-vss
>> >
>> >     Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1909073
>> >
>> >     Signed-off-by: Basil Salman <bsalman@daynix.com>
>> >     Signed-off-by: Basil Salman <basil@daynix.com>
>> >     ---
>> >      qga/vss-win32/requester.cpp | 2 +-
>> >      1 file changed, 1 insertion(+), 1 deletion(-)
>> >
>> >     diff --git a/qga/vss-win32/requester.cpp
>> b/qga/vss-win32/requester.cpp
>> >     index 5378c55d23..940a2c8f55 100644
>> >     --- a/qga/vss-win32/requester.cpp
>> >     +++ b/qga/vss-win32/requester.cpp
>> >     @@ -18,7 +18,7 @@
>> >      #include <inc/win2003/vsbackup.h>
>> >
>> >      /* Max wait time for frozen event (VSS can only hold writes for 10
>> >     seconds) */
>> >     -#define VSS_TIMEOUT_FREEZE_MSEC 10000
>> >     +#define VSS_TIMEOUT_FREEZE_MSEC 60000
>> >
>> >      /* Call QueryStatus every 10 ms while waiting for frozen event */
>> >      #define VSS_TIMEOUT_EVENT_MSEC 10
>> >     --
>> >     2.17.2
>> >
>> >
>>
>

[-- Attachment #2: Type: text/html, Size: 3966 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2021-07-28  7:55 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-05 13:14 [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10 Basil Salman
2021-04-05 13:14 ` [PATCH 2/3] qga-win: Fix build_guest_fsinfo() close of nonexistent handle Basil Salman
2021-04-05 13:14 ` [PATCH 3/3] qga-win: Fix handle leak in ga_get_win_product_name() Basil Salman
2021-04-22  7:43 ` [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10 Konstantin Kostiuk
2021-04-29  8:51   ` Konstantin Kostiuk
2021-07-13 18:40   ` Michael Roth
2021-07-14  6:17     ` Konstantin Kostiuk
2021-07-28  7:53       ` Konstantin Kostiuk
2021-05-28 11:21 ` Konstantin Kostiuk
2021-06-11  6:56 ` Konstantin Kostiuk
2021-07-04  5:52 ` Konstantin Kostiuk
2021-07-11 17:18   ` Konstantin Kostiuk

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).