On 05/26/2011 01:05 AM, Kevin Wolf wrote:
Am 25.05.2011 22:34, schrieb Josh Durgin:Signed-off-by: Josh Durgin<josh.durgin@xxxxxxxxxxxxx> --- block/rbd.c | 15 +++++++++++++++ 1 files changed, 15 insertions(+), 0 deletions(-) diff --git a/block/rbd.c b/block/rbd.c index a44d160..b95b1eb 100644 --- a/block/rbd.c +++ b/block/rbd.c @@ -688,6 +688,20 @@ static int64_t qemu_rbd_getlength(BlockDriverState *bs) return info.size; } +static int qemu_rbd_truncate(BlockDriverState *bs, int64_t offset) +{ + BDRVRBDState *s = bs->opaque; + int r; + + r = rbd_resize(s->image, offset); + if (r< 0) { + error_report("failed to resize rbd image"); + return -EIO; + }Don't print an error message here. The caller will do it, too, so we end up with two error messages saying the same.
Fixed.
What kind of error code does rbd_resize return? If it is a valid errno value, you should return r instead of turning it into EIO. Kevin
The error code is a standard errno value. Josh -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
