Re: [PATCH v5 4/4] rbd: Add bdrv_truncate implementation

[Josh Durgin <josh.durgin@dreamhost.com>]

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@dreamhost.com>
>> ---
>>   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