[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v22 08/25] cow.c: replace QEMUOptionParameter wi
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v22 08/25] cow.c: replace QEMUOptionParameter with QemuOpts |
Date: |
Tue, 11 Mar 2014 09:28:31 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0 |
On 03/10/2014 01:31 AM, Chunyan Liu wrote:
> Signed-off-by: Dong Xu Wang <address@hidden>
> Signed-off-by: Chunyan Liu <address@hidden>
> ---
> block/cow.c | 54 ++++++++++++++++++++++++++----------------------------
> 1 file changed, 26 insertions(+), 28 deletions(-)
>
> @@ -414,14 +412,14 @@ static BlockDriver bdrv_cow = {
> .bdrv_probe = cow_probe,
> .bdrv_open = cow_open,
> .bdrv_close = cow_close,
> - .bdrv_create = cow_create,
> + .bdrv_create2 = cow_create,
> .bdrv_has_zero_init = bdrv_has_zero_init_1,
>
> .bdrv_read = cow_co_read,
> .bdrv_write = cow_co_write,
> .bdrv_co_get_block_status = cow_co_get_block_status,
>
> - .create_options = cow_create_options,
> + .create_opts = &cow_create_opts,
Inconsistent alignment of =, and inconsistent use of & (it is not needed
here, '.create_opts = cow_create_opts' is just fine).
But those are both minor;
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH v22 05/25] add some QemuOpts functions for replace work, (continued)
- [Qemu-devel] [PATCH v22 04/25] improve assertion in qemu_opt_get functions, Chunyan Liu, 2014/03/10
- [Qemu-devel] [PATCH v22 06/25] add convert functions between QEMUOptionParameter to QemuOpts, Chunyan Liu, 2014/03/10
- [Qemu-devel] [PATCH v22 08/25] cow.c: replace QEMUOptionParameter with QemuOpts, Chunyan Liu, 2014/03/10
- [Qemu-devel] [PATCH v22 07/25] change block layer to support both QemuOpts and QEMUOptionParamter, Chunyan Liu, 2014/03/10
- [Qemu-devel] [PATCH v22 09/25] gluster.c: replace QEMUOptionParameter with QemuOpts, Chunyan Liu, 2014/03/10
- [Qemu-devel] [PATCH v22 10/25] iscsi.c: replace QEMUOptionParameter with QemuOpts, Chunyan Liu, 2014/03/10