[sheepdog] [PATCH 3/3] sheep: remove vnode_info from vdi path completely

levin li levin108 at gmail.com
Mon Jun 25 14:48:00 CEST 2012


On 06/25/2012 07:03 PM, Christoph Hellwig wrote:
> On Sun, Jun 24, 2012 at 09:24:56PM +0800, levin li wrote:
>> From: levin li <xingke.lwp at taobao.com>
>>
>> Since vnode_info is only used for forward_*_obj_req() in the old
>> code, it's no need to keep it any more.
> 
> Looks good,
> 
> Reviewed-by: Christoph Hellwig <hch at lst.de>
> 

A question, whether it's good to remove nr_copies completely from vdi
path, currently it's unused, then we can make the code cleaner since
without nr_copies vnode_info is no longer needed. Or we keep nr_copies
for future use ?

thanks,

levin




More information about the sheepdog mailing list