[sheepdog] [PATCH 1/2] sheep: don't send vdi_inuse in get_vdis

MORITA Kazutaka morita.kazutaka at lab.ntt.co.jp
Fri Aug 24 06:38:43 CEST 2012


At Fri, 24 Aug 2012 10:56:35 +0800,
levin li wrote:
> 
> From: levin li <xingke.lwp at taobao.com>
> 
> Since we already send vdi copy list in get_vdis(), there's no
> need to send the vdi_inuse bitmap any more, without it we can
> send less data as the length of vdi copy list is variable and
> it contains the vid just as vdi_inuse does.
> 
> Signed-off-by: levin li <xingke.lwp at taobao.com>
> ---
>  sheep/group.c      |   37 ++++++++++++++++---------------------
>  sheep/ops.c        |    4 +++-
>  sheep/sheep_priv.h |    3 +--
>  sheep/vdi.c        |   15 +--------------
>  4 files changed, 21 insertions(+), 38 deletions(-)

SD_OP_READ_VDIS is also used by qemu.  Can we keep consistency rather
than simplifying it a bit?

Thanks,

Kazutaka



More information about the sheepdog mailing list