Re: [PATCH v3 00/17] vfs: add the ability to retry on ESTALE toseveral syscalls

From: Jeff Layton
Date: Wed Jul 11 2012 - 15:01:21 EST


On Wed, 11 Jul 2012 13:42:23 -0500
"Steven J. Magnani" <steve@xxxxxxxxxxxxxxx> wrote:

> Jeff -
>
> On Fri, 2012-06-29 at 14:57 -0400, Jeff Layton wrote:
> > This patchset is the third version of the patchset to add ESTALE
> > handling to several syscalls. The basic idea is to allow the client to
> > gracefully retry the lookup and call when a NFS server returns ESTALE.
>
> I exercised this using 3.5-rc5 against a memory-starved server that
> exports a FAT-backed filesystem. Where normally I see lots of ESTALE
> errors due to inode eviction, with this patchset I see none. And, the
> performance is much better than the only other way I know to eliminate
> the errors, which is to mount with 'lookupcache=none'.
>
> It's not an exhaustive test by any means, just a data point for you.
> Thanks!
>
> Lightly-tested-by: Steve Magnani <steve@xxxxxxxxxxxxxxx>

Awesome -- thanks for testing it.

Yeah "lookupcache=none" has been the standard answer for people
suffering from ESTALE problems, but the perf hit can be quite nasty.
Avoiding that is one of the main drivers for handling ESTALE this way.

With luck, we'll see at least some of this set go in for 3.6.

--
Jeff Layton <jlayton@xxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/