[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [sheepdog] [PATCH v5] sheepdog: selectable object size
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [sheepdog] [PATCH v5] sheepdog: selectable object size support |
Date: |
Mon, 16 Feb 2015 11:14:32 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 15.02.2015 um 15:17 hat Hitoshi Mitake geschrieben:
> At Fri, 13 Feb 2015 18:20:53 +0900,
> Teruaki Ishizaki wrote:
> >
> > Previously, qemu block driver of sheepdog used hard-coded VDI object size.
> > This patch enables users to handle VDI object size.
> >
> > When you start qemu, you don't need to specify additional command option.
> >
> > But when you create the VDI which doesn't have default object size
> > with qemu-img command, you specify object_size option.
> >
> > If you want to create a VDI of 8MB object size,
> > you need to specify following command option.
> >
> > # qemu-img create -o object_size=8M sheepdog:test1 100M
> >
> > In addition, when you don't specify qemu-img command option,
> > a default value of sheepdog cluster is used for creating VDI.
> >
> > # qemu-img create sheepdog:test2 100M
> >
> > Signed-off-by: Teruaki Ishizaki <address@hidden>
> > ---
> > V5:
> > - Change option from block_size_shift to object_size.
> > - Change parse type to QEMU_OPT_SIZE.
> > - Add operation to verify max VDI size for resizing.
> > - Change to use 4MB object size with using old Sheepdog.
> >
> > V4:
> > - Limit a read/write buffer size for creating a preallocated VDI.
> > - Replace a parse function for the block_size_shift option.
> > - Fix an error message.
> >
> > V3:
> > - Delete the needless operation of buffer.
> > - Delete the needless operations of request header.
> > for SD_OP_GET_CLUSTER_DEFAULT.
> > - Fix coding style problems.
> >
> > V2:
> > - Fix coding style problem (white space).
> > - Add members, store_policy and block_size_shift to struct SheepdogVdiReq.
> > - Initialize request header to use block_size_shift specified by user.
> > ---
> > block/sheepdog.c | 155
> > ++++++++++++++++++++++++++++++++++++++-------
> > include/block/block_int.h | 1 +
> > 2 files changed, 134 insertions(+), 22 deletions(-)
>
> Looks good to me.
> Acked-by: Hitoshi Mitake <address@hidden>
Thanks, applied to the block branch.
Kevin