| To: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] xfs: revert to double-buffering readdir |
| From: | Chris Wedgwood <cw@xxxxxxxx> |
| Date: | Tue, 27 Nov 2007 11:43:14 -0800 |
| Cc: | linux-xfs@xxxxxxxxxxx, LKML <linux-kernel@xxxxxxxxxxxxxxx> |
| In-reply-to: | <20071125163014.GA17922@infradead.org> |
| References: | <20071114070400.GA25708@puku.stupidest.org> <20071125163014.GA17922@infradead.org> |
| Sender: | xfs-bounce@xxxxxxxxxxx |
On Sun, Nov 25, 2007 at 04:30:14PM +0000, Christoph Hellwig wrote: > The current readdir implementation deadlocks on a btree buffers > locks because nfsd calls back into ->lookup from the filldir > callback. The only short-term fix for this is to revert to the old > inefficient double-buffering scheme. This seems to work really well here. > This patch does exactly that and reverts xfs_file_readdir to what's > basically the 2.6.23 version minus the uio and vnops junk. This should probably be submitted for inclusion stable-2.6.24. Perhaps a version with the #if 0 [...] stuff dropped? (I'm happy to send a patch for that if you prefer). |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: XFS related Oops (suspend/resume related), Rafael J. Wysocki |
|---|---|
| Next by Date: | Re: XFS related Oops (suspend/resume related), David Chinner |
| Previous by Thread: | [PATCH] xfs: revert to double-buffering readdir, Christoph Hellwig |
| Next by Thread: | Re: [PATCH] xfs: revert to double-buffering readdir, Christian Kujau |
| Indexes: | [Date] [Thread] [Top] [All Lists] |