Re: [PATCH 00/17] NFS: Create a common path used by reads and writes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/18/2014 12:09 AM, Anna Schumaker wrote:
> I noticed that much of the read and write code was almost identical, with
> only minor differences in argument types and names.  These patches seek to
> unify these two codepaths wherever possible in order to make future changes
> easier.
> 
> Table of contents:
> 	Patches 01 - 05: Set up common data structures.
> 	Patches 06 - 10: Set up a shared rpc_call_ops struct.
> 	Patches 11 - 15: Set up a shared nfs_pageio_ops struct.
> 
> Some of the commit setup was moved into common code, but I'm willing to
> move this back into write.c if this looks to be a problem.
> 
> Future work:
> 	I have rough pNFS patches that don't currently apply.  They need
> 	more testing, and I figured this was enough code churn already.
> 
> Testing:
> 	I tested every version of NFS using the cthon and xfs tests without
> 	hitting anything new.  Any extra testing would be great seeing how
> 	the read and write paths are vitally important.
> 
> Questions?  Comments?  Death threats?
> 
> Anna

Good god. Anna did I already tell you that I like you a lot.

This "Identical code but the name" bothered me so much, to no end,
for so long. I envy you that you got to do it finally and so simple
it is.

After you are done there is bunch of code coalescing to do at
pnfs-objects as well. which will be real fun.

Thanks for doing this. Looks very good so far, yes please keep
it on a public tree for quick testing. Perhaps ask Trond to keep
it on his tree in a branch. (So to get some coverage under the linux-next
tree mongers from Intel)

real real cool, spring day
Boaz

--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux