Please pull NFS client bugfixes for 2.6.35-rc6
Subsystems (2)
Hi Linus,
Please pull from the "bugfixes" branch of the repository at
git pull git://git.linux-nfs.org/projects/trondmy/nfs-2.6.git bugfixes
This will update the following files through the appended changesets.
Cheers,
Trond
----
fs/nfs/file.c | 13 +++++++++++-- fs/nfs/nfsroot.c | 2 +- fs/nfs/write.c | 27 +++++++++++++++++++-------- include/linux/nfs_fs.h | 1 + 4 files changed, 32 insertions(+), 11 deletions(-)
commit cfb506e1d330387dfaf334dd493b3773d388863d
Author: Trond Myklebust <[email protected]> Date: Fri Jul 30 15:31:57 2010 -0400
NFS: Ensure that writepage respects the nonblock flag
commit b608b283a962caaa280756bc8563016a71712acf
Author: Trond Myklebust <[email protected]> Date: Fri Jul 30 15:31:54 2010 -0400
NFS: kswapd must not block in nfs_release_page
See https://bugzilla.kernel.org/show_bug.cgi?id=16056
If other processes are blocked waiting for kswapd to free up some memory so that they can make progress, then we cannot allow kswapd to block on those processes.
commit 674b2222920012244ca59978b356b25412a8dcc7
Author: Dan Carpenter <[email protected]> Date: Tue Jul 13 13:34:59 2010 +0200
nfs: include space for the NUL in root path
In root_nfs_name() it does the following:
if (strlen(buf) + strlen(cp) > NFS_MAXPATHLEN) {
printk(KERN_ERR "Root-NFS: Pathname for remote directory too long.\n");
return -1;
}
sprintf(nfs_export_path, buf, cp);
In the original code if (strlen(buf) + strlen(cp) == NFS_MAXPATHLEN)
then the sprintf() would lead to an overflow. Generally the rest of the
code assumes that the path can have NFS_MAXPATHLEN (1024) characters and
a NUL terminator so the fix is to add space to the nfs_export_path[]
buffer.
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Trond Myklebust <[email protected]>fs/nfs/file.c +11 -2
@@ -27,6 +27,7 @@#include <linux/pagemap.h> #include <linux/aio.h> #include <linux/gfp.h> +#include <linux/swap.h> #include <asm/uaccess.h> #include <asm/system.h>
@@ -493,11 +494,19 @@ static void nfs_invalidate_page(struct page *page, unsigned long offset)*/ static int nfs_release_page (struct page *page,gfp_t gfp) { +struct address_space *mapping = page->mapping; +dfprintk (PAGECACHE ,"NFS: release_page(%p)\n" , page);/* Only do I/O if gfp is a superset of GFP_KERNEL */ -if ((gfp &GFP_KERNEL ) ==GFP_KERNEL ) -nfs_wb_page (page->mapping->host, page); +if (mapping && (gfp &GFP_KERNEL ) ==GFP_KERNEL ) { +int how =FLUSH_SYNC ; + +/* Don't let kswapd deadlock waiting for OOM RPC calls */ +if (current_is_kswapd ()) + how =0 ; +nfs_commit_inode (mapping->host, how); + }/* If PagePrivate() is set, then the page is not freeable */ if (PagePrivate (page))return 0 ;
fs/nfs/nfsroot.c +1 -1
@@ -105,7 +105,7 @@ static char nfs_root_name[256] __initdata = "";static __be32 servaddr__initdata =0 ;/* Name of directory to mount */ -static char nfs_export_path[NFS_MAXPATHLEN ]__initdata = {0 , }; +static char nfs_export_path[NFS_MAXPATHLEN +1 ]__initdata = {0 , };/* NFS-related data */ static struct nfs_mount_data nfs_data__initdata = {0 , };/* NFS mount info */
fs/nfs/write.c +19 -8
@@ -222,7 +222,7 @@ static void nfs_end_page_writeback(struct page *page)clear_bdi_congested (&nfss->backing_dev_info,BLK_RW_ASYNC ); } -static struct nfs_page *nfs_find_and_lock_request (struct page *page) +static struct nfs_page *nfs_find_and_lock_request (struct page *page,bool nonblock) {struct inode *inode = page->mapping->host;struct nfs_page *req;
@@ -241,7 +241,10 @@ static struct nfs_page *nfs_find_and_lock_request(struct page *page)* request as dirty (in which case we don't care). */ spin_unlock (&inode->i_lock); - ret =nfs_wait_on_request (req); +if (!nonblock) + ret =nfs_wait_on_request (req); +else + ret = -EAGAIN ;nfs_release_request (req);if (ret !=0 )return ERR_PTR (ret);
@@ -256,12 +259,12 @@ static struct nfs_page *nfs_find_and_lock_request(struct page *page)* May return an error if the user signalled nfs_wait_on_request(). */ static int nfs_page_async_flush (struct nfs_pageio_descriptor *pgio, -struct page *page) +struct page *page,bool nonblock) {struct nfs_page *req;int ret =0 ; - req =nfs_find_and_lock_request (page); + req =nfs_find_and_lock_request (page, nonblock);if (!req) goto out; ret =PTR_ERR (req);
@@ -283,12 +286,20 @@ out:static int nfs_do_writepage (struct page *page,struct writeback_control *wbc,struct nfs_pageio_descriptor *pgio) {struct inode *inode = page->mapping->host; +int ret;nfs_inc_stats (inode,NFSIOS_VFSWRITEPAGE );nfs_add_stats (inode,NFSIOS_WRITEPAGES ,1 );nfs_pageio_cond_complete (pgio, page->index); -return nfs_page_async_flush (pgio, page); + ret =nfs_page_async_flush (pgio, page, + wbc->sync_mode ==WB_SYNC_NONE || + wbc->nonblocking !=0 ); +if (ret == -EAGAIN ) { +redirty_page_for_writepage (wbc, page); + ret =0 ; + } +return ret; }/*
@@ -1379,7 +1390,7 @@ static const struct rpc_call_ops nfs_commit_ops = { .rpc_release = nfs_commit_release, }; -static int nfs_commit_inode (struct inode *inode,int how) +int nfs_commit_inode (struct inode *inode,int how) {LIST_HEAD (head);int may_wait = how &FLUSH_SYNC ;
@@ -1443,7 +1454,7 @@ out_mark_dirty: return ret; }#else -static int nfs_commit_inode (struct inode *inode,int how) +int nfs_commit_inode (struct inode *inode,int how) {return 0 ; }
@@ -1546,7 +1557,7 @@ int nfs_migrate_page(struct address_space *mapping, struct page *newpage,nfs_fscache_release_page (page,GFP_KERNEL ); - req =nfs_find_and_lock_request (page); + req =nfs_find_and_lock_request (page, false); ret =PTR_ERR (req);if (IS_ERR (req)) goto out;
include/linux/nfs_fs.h +1 -0
@@ -493,6 +493,7 @@ extern int nfs_wb_all(struct inode *inode);extern int nfs_wb_page (struct inode *inode,struct page * page);extern int nfs_wb_page_cancel (struct inode *inode,struct page * page);#if defined(CONFIG_NFS_V3 ) || defined(CONFIG_NFS_V4 ) +extern int nfs_commit_inode (struct inode *,int );extern struct nfs_write_data *nfs_commitdata_alloc (void );extern void nfs_commit_free (struct nfs_write_data *wdata);#endif
PATCH 1/1 NFS: Fix build failure on !CONFIG_NFS_V3 && !CONFIG_NFS_V3
Quoted 11 lines · Trond Myklebust
Hi Linus,
Please pull from the "bugfixes" branch of the repository at
git pull
git://git.linux-nfs.org/projects/trondmy/nfs-2.6.gitbugfixesThis will update the following files through the appended changesets.
Cheers,
Trond
Quoted 6 lines
----
fs/nfs/file.c | 13 +++++++++++-- fs/nfs/nfsroot.c | 2 +- fs/nfs/write.c | 27 +++++++++++++++++++-------- include/linux/nfs_fs.h | 1 + 4 files changed, 32 insertions(+), 11 deletions(-)
FYI, these commits introduced a build failure on x86:
fs/nfs/file.c:508: error: implicit declaration of function ‘nfs_commit_inode’
The nfs_commit_inode() prototype:
#if defined(CONFIG_NFS_V3) || defined(CONFIG_NFS_V4) extern int nfs_commit_inode(struct inode *, int); extern struct nfs_write_data *nfs_commitdata_alloc(void); extern void nfs_commit_free(struct nfs_write_data *wdata); #endif
is missing if the kernel is configured for legacy NFS:
CONFIG_NFS_FS=y
# CONFIG_NFS_V3 is not set
# CONFIG_NFS_V4 is not set
The fix below solves the build problem here, but I have not tested whether NFS still works fine for this config.
Thanks,
Ingo
--- include/linux/nfs_fs.h | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-)
include/linux/nfs_fs.h +0 -2
@@ -492,11 +492,9 @@ extern int nfs_writeback_done(struct rpc_task *, struct nfs_write_data *);extern int nfs_wb_all (struct inode *inode);extern int nfs_wb_page (struct inode *inode,struct page * page);extern int nfs_wb_page_cancel (struct inode *inode,struct page * page); -#if defined(CONFIG_NFS_V3 ) || defined(CONFIG_NFS_V4 )extern int nfs_commit_inode (struct inode *,int );extern struct nfs_write_data *nfs_commitdata_alloc (void );extern void nfs_commit_free (struct nfs_write_data *wdata); -#endif static inline int nfs_have_writebacks (struct inode *inode)
Re: [PATCH] NFS: Fix build failure on !CONFIG_NFS_V3 && !CONFIG_NFS_V3
Ingo Molnar wrote:
The nfs_commit_inode() prototype:
#if defined(CONFIG_NFS_V3) || defined(CONFIG_NFS_V4) extern int nfs_commit_inode(struct inode *, int); extern struct nfs_write_data *nfs_commitdata_alloc(void); extern void nfs_commit_free(struct nfs_write_data *wdata); #endif
is missing if the kernel is configured for legacy NFS:
CONFIG_NFS_FS=y
# CONFIG_NFS_V3 is not set
# CONFIG_NFS_V4 is not set
I didn't even know you could do that. Is it useful? Should we just disallow it?
Re: [PATCH] NFS: Fix build failure on !CONFIG_NFS_V3 && !CONFIG_NFS_V3
In reply to Ingo MolnarQuoted 4 lines · Ingo Molnar
FYI, these commits introduced a build failure on x86:
fs/nfs/file.c:508: error: implicit declaration of function ‘nfs_commit_inode’
Grr. Trond?
include/linux/nfs_fs.h extern int nfs_writeback_done(struct rpc_task *, struct nfs_write_data *); · line 499
Quoted 9 lines
include/linux/nfs_fs.h +0
-2@@ -492,11 +492,9 @@ extern int nfs_writeback_done(struct rpc_task *, struct nfs_write_data *); extern int nfs_wb_all(struct inode *inode); extern int nfs_wb_page(struct inode *inode, struct page* page); extern int nfs_wb_page_cancel(struct inode *inode, struct page* page); -#if defined(CONFIG_NFS_V3 ) || defined(CONFIG_NFS_V4 ) extern int nfs_commit_inode(struct inode *, int); extern struct nfs_write_data *nfs_commitdata_alloc(void); extern void nfs_commit_free(struct nfs_write_data *wdata); -#endif
No, I think perhaps the '#if' line should just be moved down one line. The other function still only exist for V3/V4, it's just nfs_commit_inode() that is available (as a stub that just returns 0) for V2.
But maybe that stub should be made an inline function or #define?
Trond?
Linus
Re: [PATCH] NFS: Fix build failure on !CONFIG_NFS_V3 && !CONFIG_NFS_V3
Quoted 7 lines · Linus Torvalds
Quoted 4 lines · Ingo Molnar
FYI, these commits introduced a build failure on x86:
fs/nfs/file.c:508: error: implicit declaration of function ‘nfs_commit_inode’Grr. Trond?
Sorry. I was intending to put that declaration outside the #ifdef. The stub function is still declared in fs/nfs/write.c
Quoted 15 lines
include/linux/nfs_fs.h extern int nfs_writeback_done(struct rpc_task *, struct nfs_write_data *); · line 499
Quoted 9 lines
include/linux/nfs_fs.h +0
-2@@ -492,11 +492,9 @@ extern int nfs_writeback_done(struct rpc_task *, struct nfs_write_data *);extern int nfs_wb_all (struct inode *inode);extern int nfs_wb_page (struct inode *inode,struct page * page);extern int nfs_wb_page_cancel (struct inode *inode,struct page * page); -#if defined(CONFIG_NFS_V3 ) || defined(CONFIG_NFS_V4 )extern int nfs_commit_inode (struct inode *,int );extern struct nfs_write_data *nfs_commitdata_alloc (void );extern void nfs_commit_free (struct nfs_write_data *wdata); -#endif No, I think perhaps the '#if' line should just be moved down one line. The other function still only exist for V3/V4, it's just
nfs_commit_inode()that is available (as a stub that just returns 0) for V2.
Quoted 3 lines
But maybe that stub should be made an inline function or #define?
Trond?
I'll convert it into an inlined stub just as it was prior to commit 8fc795f703c5138e1a8bfb88c69f52632031aa6a.
Trond
PATCH 1/1 NFS: Fix a typo in include/linux/nfs_fs.h
nfs_commit_inode() needs to be defined irrespectively of whether or not we are supporting NFSv3 and NFSv4.
Allow the compiler to optimise away code in the NFSv2-only case by converting it into an inlined stub function.
--- fs/nfs/write.c | 5 ----- include/linux/nfs_fs.h | 6 ++++++ 2 files changed, 6 insertions(+), 5 deletions(-)
fs/nfs/write.c +0 -5
@@ -1454,11 +1454,6 @@ out_mark_dirty: return ret; }#else -int nfs_commit_inode (struct inode *inode,int how) -{ -return 0 ; -} -static int nfs_commit_unstable_pages (struct inode *inode,struct writeback_control *wbc) {return 0 ;
include/linux/nfs_fs.h +6 -0
@@ -496,6 +496,12 @@ extern int nfs_wb_page_cancel(struct inode *inode, struct page* page);extern int nfs_commit_inode (struct inode *,int );extern struct nfs_write_data *nfs_commitdata_alloc (void );extern void nfs_commit_free (struct nfs_write_data *wdata); +#else +static inline int +nfs_commit_inode (struct inode *inode,int how) +{ +return 0 ; +}#endif static inline int
Signature · 1 line
1.7.2
Re: [PATCH] NFS: Fix a typo in include/linux/nfs_fs.h
Quoted 11 lines · Trond Myklebust
nfs_commit_inode()needs to be defined irrespectively of whether or not we are supporting NFSv3 and NFSv4.
Allow the compiler to optimise away code in the NFSv2-only case by converting it into an inlined stub function.Reported-by Ingo Molnar <[email protected]>Signed-off-by Trond Myklebust <[email protected]>--- fs/nfs/write.c | 5 ----- include/linux/nfs_fs.h | 6 ++++++ 2 files changed, 6 insertions(+), 5 deletions(-)
I tested this patch too and it solves the problem (and doesnt introduce new problems).
Thanks,
Ingo
Re: [PATCH] NFS: Fix build failure on !CONFIG_NFS_V3 && !CONFIG_NFS_V3
In reply to Trond MyklebustBtw, is there a reason to allow not building NFSv3? If at all building without v2 support would be more useful.
Re: [PATCH] NFS: Fix build failure on !CONFIG_NFS_V3 && !CONFIG_NFS_V3
Quoted 3 lines · Christoph Hellwig
Btw, is there a reason to allow not building NFSv3? If at all building without v2 support would be more useful.
I am planning to separate nfsv2, nfsv3 and nfsv4 into loadable submodules at some point in the not-too-distant future. That seems like a better solution than these CONFIG suboptions that keep tripping us up.
Cheers
Trond