Re: [PATCH] Fix nfsd rewrite performance

["J. Bruce Fields" <bfields@fieldses.org>]

I haven't had the chance to read carefully; just one quick question:

On Mon, Aug 01, 2005 at 01:39:54PM +0200, Olaf Kirch wrote:
> +	this_page = (unsigned char *) vec[vlen-1].iov_base;
> +	for (i = vlen-1; i; --i) {
> +		prev_page = (unsigned char *) vec[i-1].iov_base;
> +
> +		/* Push trailing partial page so it's
> +		 * aligned with the end of the page, then
> +		 * pull up the missing chunk from the previous
> +		 * page */
> +		memmove(this_page + chunk0, this_page, chunk1);
> +		memcpy(this_page, prev_page + chunk1, chunk0);
> +		vec[i].iov_len = PAGE_CACHE_SIZE;
> +		this_page = prev_page;
> +	}

If there's stuff after the write data (as there could be in the NFSv4
case at least), does this overwrite it?

--b.


-------------------------------------------------------
SF.Net email is sponsored by: Discover Easy Linux Migration Strategies
from IBM. Find simple to follow Roadmaps, straightforward articles,
informative Webcasts and more! Get everything you need to get up to
speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs