[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/5] Clean up around bdrv_getlength()
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH 0/5] Clean up around bdrv_getlength() |
Date: |
Fri, 9 May 2014 14:29:47 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, May 09, 2014 at 11:48:13AM +0200, Markus Armbruster wrote:
> Issues addressed in this series:
>
> * BlockDriver method bdrv_getlength() generally returns -errno, but
> some implementations return -1 instead. Fix them [PATCH 1].
>
> * Frequent conversions between sectors and bytes complicate the code
> needlessly. Clean up some [PATCH 2+3].
>
> * bdrv_getlength() always returns a multiple of BDRV_SECTOR_SIZE, but
> some places appear to be confused about that, and align the result
> up or down. Don't [PATCH 4].
>
> * bdrv_get_geometry() hides errors. Don't use it in places where
> errors should be detected [PATCH 5].
>
> Issues not addressed:
>
> * There are quite a few literals left in the code where
> BDRV_SECTOR_SIZE, BDRV_SECTOR_BITS or BDRV_SECTOR_MASK should be
> used instead.
>
> * Error handling is missing in places, but it's not always obvious
> whether errors can actually happen, and if yes, how to handle them.
>
> Markus Armbruster (5):
> raw-posix: Fix raw_getlength() to always return -errno on error
> block: New bdrv_nb_sectors()
> block: Use bdrv_nb_sectors() when sectors, not bytes are wanted
> block: Drop superfluous aligning of bdrv_getlength()'s value
> block: Avoid bdrv_get_geometry() where errors should be detected
>
> block-migration.c | 9 +++--
> block.c | 81 ++++++++++++++++++++++++--------------------
> block/qapi.c | 14 +++++---
> block/qcow2.c | 3 +-
> block/raw-posix.c | 28 ++++++++++++----
> block/vmdk.c | 5 ++-
> include/block/block.h | 1 +
> qemu-img.c | 93
> ++++++++++++++++++++++++++++++++++-----------------
> 8 files changed, 147 insertions(+), 87 deletions(-)
>
> --
> 1.8.1.4
>
>
Thanks, applied to my block tree:
https://github.com/stefanha/qemu/commits/block
Stefan