Message ID | 20170411125205.23121-1-sameeh@daynix.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
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>
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 >
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 >
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;
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(-)