From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sankar P Subject: [PATCH] fs: btrfs: Shuffle preprocessor macros Date: Thu, 2 Dec 2010 21:49:39 +0530 Message-ID: <1291306779-16186-1-git-send-email-sankar.curiosity@gmail.com> Cc: Sankar P To: Chris Mason , linux-btrfs@vger.kernel.org, linux-kernel@vger.kernel.org Return-path: List-ID: The function btree_migratepage will be extended from the baseclass only when CONFIG_MIGRATION option is enabled. So, it's useful to define/build this function only when that config option is enabled. Fixes an "unused function" compiler warning when CONFIG_MIGRATION is not enabled and also removes an return -ENOSYS statement, whose scenario will not happen. Signed-off-by: Sankar P --- fs/btrfs/disk-io.c | 6 ++---- 1 files changed, 2 insertions(+), 4 deletions(-) diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c index c547cca..7199239 100644 --- a/fs/btrfs/disk-io.c +++ b/fs/btrfs/disk-io.c @@ -696,6 +696,7 @@ static int btree_submit_bio_hook(struct inode *inode, int rw, struct bio *bio, __btree_submit_bio_done); } +#ifdef CONFIG_MIGRATION static int btree_migratepage(struct address_space *mapping, struct page *newpage, struct page *page) { @@ -712,12 +713,9 @@ static int btree_migratepage(struct address_space *mapping, if (page_has_private(page) && !try_to_release_page(page, GFP_KERNEL)) return -EAGAIN; -#ifdef CONFIG_MIGRATION return migrate_page(mapping, newpage, page); -#else - return -ENOSYS; -#endif } +#endif static int btree_writepage(struct page *page, struct writeback_control *wbc) { -- 1.7.1