[PATCH 0/4] qga: Fix several guest-get-devices issues

Markus Armbruster posted 4 patches 3 years, 6 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20201021071520.2168877-1-armbru@redhat.com
Maintainers: Michael Roth <mdroth@linux.vnet.ibm.com>
qga/qapi-schema.json | 29 +++++++++++++++++++----------
qga/commands-win32.c | 39 ++++++++++++++++++++-------------------
2 files changed, 39 insertions(+), 29 deletions(-)
[PATCH 0/4] qga: Fix several guest-get-devices issues
Posted by Markus Armbruster 3 years, 6 months ago
To be frank, I'm disappointed the code passed review in this state.
There are excuses for each of the issues addressed in this series
(PATCH 1 naming is hard, PATCH 2 recognizing non-local relations is
hard, PATCH 3 mistakes on error paths are easy to miss, PATCH 4 can't
expect all reviewers to know "no more simple unions, please").  Still,
having so many of them escape notice is disappointing.

Please review.

Markus Armbruster (4):
  qga: Rename guest-get-devices return member 'address' to 'id'
  qga: Use common time encoding for guest-get-devices 'driver-date'
  qga-win: Fix guest-get-devices error API violations
  qga: Flatten simple union GuestDeviceId

 qga/qapi-schema.json | 29 +++++++++++++++++++----------
 qga/commands-win32.c | 39 ++++++++++++++++++++-------------------
 2 files changed, 39 insertions(+), 29 deletions(-)

-- 
2.26.2


Re: [PATCH 0/4] qga: Fix several guest-get-devices issues
Posted by Markus Armbruster 3 years, 6 months ago
Mike,

This series addresses interface defects, and must go into 5.2.  It's
fully reviewed.  Pull request in time for soft freeze would be ideal.
If you can't make it, feel free to ask me.  I'll continue to push for
the series even if it misses the soft freeze.