[sheepdog] [PATCH] sheep/dog: introduce new bitmap for delete vdi

Liu Yuan namei.unix at gmail.com
Wed Dec 3 13:18:52 CET 2014


On Wed, Dec 03, 2014 at 08:01:16PM +0900, Saeki Masaki wrote:
> Hi, Yuan
> 
> Thank you for your comment.
> but, SD_OP_READ_DEL_VDIS is already defined in "internal_proto.h" as below.
> ---
> #include "sheepdog_proto.h"
> ---
> 
> So, I think need not be add again.
> 

Really? am I missing something? At least for master, it is defined at 

include/sheepdog_proto.h:#define SD_OP_READ_DEL_VDIS  0x18

see following:

yliu at ubuntu-trusty:~/sheepdog$ git grep 'SD_OP_READ_DEL_VDIS'
dog/common.c:   sd_init_req(&req, SD_OP_READ_DEL_VDIS);
include/sheepdog_proto.h:#define SD_OP_READ_DEL_VDIS  0x18

I guess you got me wrong.

sheepdog_proto.h is used for QEMU(or tgt)&sheep communication. internal_proto.h
for dog & sheep. See the comment:

/*
 * This file specified the sheepdog-internal protocol, which is spoken between
 * sheepdog daemons, as well as between dog and sheepdog daemon for internal
 * operations.
 */

Thanks
Yuan



More information about the sheepdog mailing list