On Wed, 4 Dec 2013 05:40:36 -0800 Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > On Wed, Dec 04, 2013 at 08:31:01AM -0500, Jeff Layton wrote: > > > Exactly. Btw, what about the comment that the DRC should be per-client > > > on top of the file? That also seems like some very low hanging fruit. > > > > > > > Ok, I'll see about spinning up a patchset that does that. We'll also > > probably want to just allocate entries out of the slabcache in that > > case, so that'll make things quite a bit simpler. > > FYI I'm testing a patch at the moment that just rips out the "direct > reclaim" as an alternative to your patch, without actually redoing > the whole LRU infrastructure. Unfortunately with your patch 075 > still hangs for me, although it at lest doesn't crash. > Yeah, I've noticed the same hang, but hadn't able to determine why it was hanging. I suspect that that hang is what's tickles the bug that my patch fixes. With the hang, we see the client doing retransmits and not getting replies and that means that we exercise the DRC more... -- Jeff Layton <jlayton@xxxxxxxxxx> -- 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
