Messages in this thread Patch in this message | | | From | Arnd Bergmann <> | Subject | [PATCH] 9p: fix unused-variable warning | Date | Fri, 3 Dec 2021 19:58:06 +0100 |
| |
From: Arnd Bergmann <arnd@arndb.de>
The folio changes added a variable that is sometimes unused:
fs/9p/vfs_addr.c: In function 'v9fs_release_page': fs/9p/vfs_addr.c:140:23: error: unused variable 'inode' [-Werror=unused-variable] 140 | struct inode *inode = folio_inode(folio); | ^~~~~
Make this clearer to the compiler by replacing the #ifdef with an equivalent if(IS_ENABLED()) check.
Fixes: 78525c74d9e7 ("netfs, 9p, afs, ceph: Use folios") Signed-off-by: Arnd Bergmann <arnd@arndb.de> --- fs/9p/vfs_addr.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/fs/9p/vfs_addr.c b/fs/9p/vfs_addr.c index 1279970e9157..82ac2ceff06e 100644 --- a/fs/9p/vfs_addr.c +++ b/fs/9p/vfs_addr.c @@ -141,14 +141,17 @@ static int v9fs_release_page(struct page *page, gfp_t gfp) if (folio_test_private(folio)) return 0; -#ifdef CONFIG_9P_FSCACHE + + if (!IS_ENABLED(CONFIG_9P_FSCACHE)) + return 1; + if (folio_test_fscache(folio)) { if (!(gfp & __GFP_DIRECT_RECLAIM) || !(gfp & __GFP_FS)) return 0; folio_wait_fscache(folio); } fscache_note_page_release(v9fs_inode_cookie(V9FS_I(inode))); -#endif + return 1; } -- 2.29.2
| |