| Date | Wed, 31 Oct 2007 08:52:37 +0000 | From | Christoph Hellwig <> | Subject | Re: [PATCH 28/33] nfs: teach the NFS client how to treat PG_swapcache pages |
| |
On Tue, Oct 30, 2007 at 05:04:29PM +0100, Peter Zijlstra wrote: > Replace all relevant occurences of page->index and page->mapping in the NFS > client with the new page_file_index() and page_file_mapping() functions.
As discussed personally and on the list a strong NACK for this. Swapcache pages have no business at all ever coming through ->writepage(s). If you really want to support swap over NFS that can only be done properly by adding separate methods to write out and read in pages separated from the pagecache. Incidentally that would also clean up the mess we have with swap files on "normal" filesystems using ->bmap and bypassing the filesystem later on.
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|