On Mon, Apr 21, 2014 at 12:24:23PM -0400, Anna Schumaker wrote: > > Okay. I'll rework everything into the pageio_ops and see how it looks! Maybe I'll see what I can do about updating the pnfs patches while I'm at it. > I remember my issue with adding to the nfs_pageio_ops now. The file, object and block layouts along with the generic pnfs code all have their own read and write pageio_ops. Changing all of these seemed more tedious than adding a new struct only in the read and write code. Indeed, there's more instances in the pnfs code. I was hoping your pending pnfs patches were addressing that, and the general code duplication between the pnfs and "classic" I/O code. I guess it's best to put in your patches as-is for now, and maybe in the future we can get rid of the pageio_ops once more code is shared with the pnfs path. -- 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