On 08/12/2011 06:00 PM, Peng Tao wrote: > Hi, > > I have moved the error handling inside mds_ops->rpc_release to reuse code > as suggested by Boaz. > > I think we still need to issue the IO even for write because > we don't know if current writeback is the last one. So if we re-dirty the > pages and this is the last flush (flush at file close), then we don't have > a later flusher to writeback the re-dirtied pages. Boaz, please help see if > current approach is OK. Thanks. > > The two cleanup patches (pipe upcall and set_lo_fail) are seperated out of > this patchset so they can be merged more easily. > > Thanks, > Tao > Thanks Tao They look *really* good these patches. But as you said, do they actually work? Did you test any of this? I mean do you have a facility to inject random IO errors and test this out? At the time I had the most dumb patch that would simply start failing IO after n writes/reads. Or another one that: if the offset was a magic offset like 0x10017 then the IO would fail. Then with a simple dd I could make the IO fail and test this out. (Which BTW never worked with pnfs) The Kernel might be more resilient then we think with regarding to waiting for clean pages. Because I know that in a UML it is very easy to get to a low-memory condition where the iscsi stack starts to throw OOM messages and everything stalls for a while. But I'm always surprised how eventually the Kernel picks up and the IO ends successfully. One easy way to do this is with a "git clone linux" with a machine that has a fast (iscsi) disk but only 256M of memory. I will only have time to test all this at end of next week at the earliest. So thanks again for looking into this Boaz > Peng Tao (3): > pNFS: recoalesce when ld write pagelist fails > pNFS: recoalesce when ld read pagelist fails > pNFS: introduce pnfs private workqueue > > fs/nfs/blocklayout/blocklayout.c | 17 +++++-- > fs/nfs/objlayout/objio_osd.c | 8 +++ > fs/nfs/objlayout/objlayout.c | 4 +- > fs/nfs/pnfs.c | 92 +++++++++++++++++++++++++++----------- > fs/nfs/pnfs.h | 8 +++- > fs/nfs/read.c | 13 +++++- > fs/nfs/write.c | 25 ++++++++++- > 7 files changed, 129 insertions(+), 38 deletions(-) > -- 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
