[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