[Qemu-devel] [PATCH] qga-win: Enable 'can-offline' field in 'guest-get-vcpus' reply

Sameeh Jubran posted 1 patch 6 years, 11 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170411125205.23121-1-sameeh@daynix.com
Test checkpatch passed
Test docker passed
Test s390x passed
qga/commands-win32.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[Qemu-devel] [PATCH] qga-win: Enable 'can-offline' field in 'guest-get-vcpus' reply
Posted by Sameeh Jubran 6 years, 11 months ago
The QGA schema states:

@can-offline: Whether offlining the VCPU is possible. This member
               is always filled in by the guest agent when the structure
               is returned, and always ignored on input (hence it can be
               omitted then).

Currently 'can-offline' is missing entirely from the reply. This causes
errors in libvirt which is expecting the reply to be compliant with the
schema docs.

BZ#1438735: https://bugzilla.redhat.com/show_bug.cgi?id=1438735

Signed-off-by: Sameeh Jubran <sameeh@daynix.com>
---
 qga/commands-win32.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/qga/commands-win32.c b/qga/commands-win32.c
index 19d72b2..f0d72a0 100644
--- a/qga/commands-win32.c
+++ b/qga/commands-win32.c
@@ -1344,7 +1344,7 @@ GuestLogicalProcessorList *qmp_guest_get_vcpus(Error **errp)
                     vcpu = g_malloc0(sizeof *vcpu);
                     vcpu->logical_id = current++;
                     vcpu->online = true;
-                    vcpu->has_can_offline = false;
+                    vcpu->has_can_offline = true;
 
                     entry = g_malloc0(sizeof *entry);
                     entry->value = vcpu;
-- 
2.9.3


Re: [Qemu-devel] [PATCH for-2.9?] qga-win: Enable 'can-offline' field in 'guest-get-vcpus' reply
Posted by Eric Blake 6 years, 11 months ago
On 04/11/2017 07:52 AM, Sameeh Jubran wrote:
> The QGA schema states:
> 
> @can-offline: Whether offlining the VCPU is possible. This member
>                is always filled in by the guest agent when the structure
>                is returned, and always ignored on input (hence it can be
>                omitted then).
> 
> Currently 'can-offline' is missing entirely from the reply. This causes
> errors in libvirt which is expecting the reply to be compliant with the
> schema docs.
> 
> BZ#1438735: https://bugzilla.redhat.com/show_bug.cgi?id=1438735
> 
> Signed-off-by: Sameeh Jubran <sameeh@daynix.com>
> ---
>  qga/commands-win32.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Appropriate as a bug fix in 2.9 if there is still time for the
maintainer to submit a pull request, although we are late enough that if
it slips into 2.10 it is not a big deal.

Reviewed-by: Eric Blake <eblake@redhat.com>

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

Re: [Qemu-devel] [PATCH for-2.9?] qga-win: Enable 'can-offline' field in 'guest-get-vcpus' reply
Posted by Michael Roth 6 years, 11 months ago
Quoting Eric Blake (2017-04-11 08:25:06)
> On 04/11/2017 07:52 AM, Sameeh Jubran wrote:
> > The QGA schema states:
> > 
> > @can-offline: Whether offlining the VCPU is possible. This member
> >                is always filled in by the guest agent when the structure
> >                is returned, and always ignored on input (hence it can be
> >                omitted then).
> > 
> > Currently 'can-offline' is missing entirely from the reply. This causes
> > errors in libvirt which is expecting the reply to be compliant with the
> > schema docs.
> > 
> > BZ#1438735: https://bugzilla.redhat.com/show_bug.cgi?id=1438735
> > 
> > Signed-off-by: Sameeh Jubran <sameeh@daynix.com>
> > ---
> >  qga/commands-win32.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Appropriate as a bug fix in 2.9 if there is still time for the
> maintainer to submit a pull request, although we are late enough that if
> it slips into 2.10 it is not a big deal.
> 
> Reviewed-by: Eric Blake <eblake@redhat.com>

Yah, agreed, the final planned RC looks to have been tagged already though
so I think it'll have to wait for 2.10/2.9.1. Thanks for the heads up.

Cc: qemu-stable@nongnu.org

> 
> -- 
> Eric Blake, Principal Software Engineer
> Red Hat, Inc.           +1-919-301-3266
> Virtualization:  qemu.org | libvirt.org
> 


Re: [Qemu-devel] [PATCH] qga-win: Enable 'can-offline' field in 'guest-get-vcpus' reply
Posted by Michael Roth 6 years, 11 months ago
Quoting Sameeh Jubran (2017-04-11 07:52:05)
> The QGA schema states:
> 
> @can-offline: Whether offlining the VCPU is possible. This member
>                is always filled in by the guest agent when the structure
>                is returned, and always ignored on input (hence it can be
>                omitted then).
> 
> Currently 'can-offline' is missing entirely from the reply. This causes
> errors in libvirt which is expecting the reply to be compliant with the
> schema docs.
> 
> BZ#1438735: https://bugzilla.redhat.com/show_bug.cgi?id=1438735
> 
> Signed-off-by: Sameeh Jubran <sameeh@daynix.com>

Thanks, applied to qga tree:                                                                              

  https://github.com/mdroth/qemu/commits/qga

> ---
>  qga/commands-win32.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/qga/commands-win32.c b/qga/commands-win32.c
> index 19d72b2..f0d72a0 100644
> --- a/qga/commands-win32.c
> +++ b/qga/commands-win32.c
> @@ -1344,7 +1344,7 @@ GuestLogicalProcessorList *qmp_guest_get_vcpus(Error **errp)
>                      vcpu = g_malloc0(sizeof *vcpu);
>                      vcpu->logical_id = current++;
>                      vcpu->online = true;
> -                    vcpu->has_can_offline = false;
> +                    vcpu->has_can_offline = true;
> 
>                      entry = g_malloc0(sizeof *entry);
>                      entry->value = vcpu;
> -- 
> 2.9.3
>