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

2012-08-23 Thread levin li
From: levin li 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 --- sheep/group.c |

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

2012-08-23 Thread MORITA Kazutaka
At Fri, 24 Aug 2012 10:56:35 +0800, levin li wrote: > > From: levin li > > 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 v

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

2012-08-23 Thread MORITA Kazutaka
At Fri, 24 Aug 2012 13:38:43 +0900, MORITA Kazutaka wrote: > > At Fri, 24 Aug 2012 10:56:35 +0800, > levin li wrote: > > > > From: levin li > > > > 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

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

2012-08-23 Thread levin li
On 2012年08月24日 12:38, MORITA Kazutaka wrote: > At Fri, 24 Aug 2012 10:56:35 +0800, > levin li wrote: >> >> From: levin li >> >> 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

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

2012-08-23 Thread MORITA Kazutaka
At Fri, 24 Aug 2012 13:10:53 +0800, levin li wrote: > > Well, I didn't notice that, I'd like to update this patch to add a new > local operation for sheep and collie to read the vdi list rather than > using SD_OP_READ_VDIS, the bitmap is not so small, as I calculated the > size of vdi_inuse is 2M,