[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V5 03/13] block: add bdrv_can_read_snapshot() fu
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH V5 03/13] block: add bdrv_can_read_snapshot() function |
Date: |
Tue, 29 Jan 2013 13:37:52 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20120605 Thunderbird/13.0 |
Am 25.01.2013 19:11, schrieb Eric Blake:
> On 01/23/2013 07:57 PM, Wenchao Xia wrote:
>> Compared to bdrv_can_snapshot(), this function return whether
>> bs* is ready to read snapshot info from instead of write. If yes,
>> caller can then query snapshot information, but taking snapshot
>> is not always possible for that *bs may be read only.
>>
>> Signed-off-by: Wenchao Xia <address@hidden>
>> ---
>> block.c | 19 +++++++++++++++++++
>> include/block/block.h | 1 +
>> 2 files changed, 20 insertions(+), 0 deletions(-)
>>
>
>>
>> +/* return whether internal snapshot can be read on @bs */
>> +int bdrv_can_read_snapshot(BlockDriverState *bs)
>> +{
>
>> +/* return whether internal snapshot can be write on @bs */
>> int bdrv_can_snapshot(BlockDriverState *bs)
>
> I see you just copied existing code; but any reason why these functions
> return int instead of bool? Would that be worth a separate cleanup patch?
More importantly, you shouldn't copy code. Make both of them small
wrappers around a static helper functions that contains the existing code.
Kevin