[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/12] qemu-img: conditionally zero out target o
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 08/12] qemu-img: conditionally zero out target on convert |
Date: |
Mon, 16 Sep 2013 13:01:50 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130805 Thunderbird/17.0.8 |
Il 13/09/2013 21:48, Peter Lieven ha scritto:
> Am 13.09.2013 20:25, schrieb Eric Blake:
>> On 09/13/2013 04:36 AM, Paolo Bonzini wrote:
>>> Il 13/09/2013 12:25, Peter Lieven ha scritto:
>>>> if the target has_zero_init = 0, but supports efficiently
>>>> writing zeroes by unmapping we call bdrv_zeroize to
>>>> avoid fully allocating the target. this currently
>>>> is designed especially for iscsi.
>>>>
>>>> Signed-off-by: Peter Lieven <address@hidden>
>>>> ---
>>>> qemu-img.c | 22 ++++++++++++++++------
>>>> 1 file changed, 16 insertions(+), 6 deletions(-)
>>>>
>>>> diff --git a/qemu-img.c b/qemu-img.c
>>>> index 3e5e388..6eaddc6 100644
>>>> --- a/qemu-img.c
>>>> +++ b/qemu-img.c
>>>> @@ -1354,7 +1354,8 @@ static int img_convert(int argc, char **argv)
>>>> }
>>>> }
>>>>
>>>> - flags = BDRV_O_RDWR;
>>>> + flags = BDRV_O_RDWR | BDRV_O_UNMAP;
>>> I think this should be a new command-line flag.
>> I agree - while 'sparse by default' may be reasonable, it is also
>> feasible to want a mode that guarantees expansion rather than unmapped
>> or sparse.
>>
> Ok, so do you find the proposed -S 0 bei Paolo a good choice?
> If this is supplied I would go as far as completly setting
> has_zero_init = 0 also for targets which default to 1. This
> would guaranteed exspansion and full allocation for all drivers.
Sounds good.
Paolo
- Re: [Qemu-devel] [PATCH 04/12] iscsi: add .bdrv_co_write_zeroes, (continued)
[Qemu-devel] [PATCH 07/12] block: introduce bdrv_zeroize, Peter Lieven, 2013/09/13
[Qemu-devel] [PATCH 05/12] block: add logical block provisioning information to BlockDriverInfo, Peter Lieven, 2013/09/13