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 |