|
| From: | Alberto Garcia |
| Subject: | Re: [Qemu-devel] [PATCH v12 09/10] qcow2: skip writing zero buffers to empty COW areas |
| Date: | Wed, 16 Jan 2019 10:35:38 +0100 |
| User-agent: | Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Wed 16 Jan 2019 10:32:33 AM CET, Anton Nefedov wrote:
>>> + ret = handle_alloc_space(bs, l2meta);
>>
>> I insist that it would be nice to have a short comment explaining
>> what this does.
>
> Right sorry forgot your comment.
> I'd go with:
>
> + /* Try to efficiently initialize the physical space with zeroes */
> ret = handle_alloc_space(bs, l2meta);
> if (ret < 0) {
> qemu_co_mutex_lock(&s->lock);
That looks good, thanks!
Berto
| [Prev in Thread] | Current Thread | [Next in Thread] |