[Qemu-devel] [PATCH v2 00/15] block: A bunch of fixes for Sheepdog and Gluster

Markus Armbruster posted 15 patches 7 years ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1488826849-32384-1-git-send-email-armbru@redhat.com
Test checkpatch passed
Test docker passed
block/gluster.c      | 127 +++++++--------
block/sheepdog.c     | 453 +++++++++++++++++++++++++++++++++++++--------------
qapi-schema.json     |  38 +++++
qapi/block-core.json |  73 +++------
4 files changed, 451 insertions(+), 240 deletions(-)
[Qemu-devel] [PATCH v2 00/15] block: A bunch of fixes for Sheepdog and Gluster
Posted by Markus Armbruster 7 years ago
Bad error handling, memory leaks, and lack of blockdev-add support.

v2:
* Straightforward rebase
* PATCH 01: Superfluous conditional dropped, labels renamed, commit
  message updated accordingly [Eric]
* PATCH 02: More pleasant use of @ret, error message capitalized,
  line break tidied up [Kevin, Eric]
* PATCH 05: Whitespace tidied up [Kevin]
* PATCH 06: Missing assignment to ret fixed up [Eric]
* PATCH 07: Missing assignment to ret fixed up [Kevin]
* PATCH 12: Use -1 instead of 0 for unused argument [Niels]
* PATCH 14: Better explain compatibility impact in commit message,
  whitespace tidied up [Eric]


Markus Armbruster (15):
  sheepdog: Defuse time bomb in sd_open() error handling
  sheepdog: Fix error handling in sd_snapshot_delete()
  sheepdog: Fix error handling sd_create()
  sheepdog: Mark sd_snapshot_delete() lossage FIXME
  sheepdog: Fix snapshot ID parsing in _open(), _create, _goto()
  sheepdog: Don't truncate long VDI name in _open(), _create()
  sheepdog: Report errors in pseudo-filename more usefully
  sheepdog: Use SocketAddress and socket_connect()
  sheepdog: Implement bdrv_parse_filename()
  gluster: Drop assumptions on SocketTransport names
  gluster: Don't duplicate qapi-util.c's qapi_enum_parse()
  gluster: Plug memory leaks in qemu_gluster_parse_json()
  qapi-schema: Rename GlusterServer to SocketAddressFlat
  qapi-schema: Rename SocketAddressFlat's variant tcp to inet
  sheepdog: Support blockdev-add

 block/gluster.c      | 127 +++++++--------
 block/sheepdog.c     | 453 +++++++++++++++++++++++++++++++++++++--------------
 qapi-schema.json     |  38 +++++
 qapi/block-core.json |  73 +++------
 4 files changed, 451 insertions(+), 240 deletions(-)

-- 
2.7.4


Re: [Qemu-devel] [PATCH v2 00/15] block: A bunch of fixes for Sheepdog and Gluster
Posted by Kevin Wolf 7 years ago
Am 06.03.2017 um 20:00 hat Markus Armbruster geschrieben:
> Bad error handling, memory leaks, and lack of blockdev-add support.

Thanks, applied to the block branch.

Kevin