Re: [PATCH] Fix nfsd rewrite performance

[Olaf Kirch <okir@suse.de>]

On Mon, Aug 01, 2005 at 07:53:19AM -0400, J. Bruce Fields wrote:
> 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?

It does. I hadn't thought of these cursed NFSv4 compound requests.

Alternatively, we could pull in everything and align it with the beginning
of the message. This would clobber the RPC header etc.  I don't see us
doing doing any in-place decodes of strings etc in the svcauth code,
so that may be a safer choice. I would like to avoid allocating an extra
page just for aligning things.

Olaf
-- 
Olaf Kirch   |  --- o --- Nous sommes du soleil we love when we play
okir@suse.de |    / | \   sol.dhoop.naytheet.ah kin.ir.samse.qurax


-------------------------------------------------------
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