From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: jglisse@redhat.com To: linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, linux-block@vger.kernel.org Cc: linux-kernel@vger.kernel.org, =?UTF-8?q?J=C3=A9r=C3=B4me=20Glisse?= , Jens Axboe , Andrew Morton , Alexander Viro , Tejun Heo , Jan Kara , Josef Bacik , Mel Gorman Subject: [RFC PATCH 32/79] fs/block: do not rely on page->mapping get it from the context Date: Wed, 4 Apr 2018 15:18:06 -0400 Message-Id: <20180404191831.5378-17-jglisse@redhat.com> In-Reply-To: <20180404191831.5378-1-jglisse@redhat.com> References: <20180404191831.5378-1-jglisse@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 List-ID: From: Jérôme Glisse This patch remove most dereference of page->mapping and get the mapping from the call context (either already available in the function or by adding it to function arguments). Signed-off-by: Jérôme Glisse Cc: Jens Axboe CC: Andrew Morton Cc: Alexander Viro Cc: linux-fsdevel@vger.kernel.org Cc: Tejun Heo Cc: Jan Kara Cc: Josef Bacik Cc: Mel Gorman --- fs/block_dev.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/fs/block_dev.c b/fs/block_dev.c index 502b6643bc74..dd9da97615e3 100644 --- a/fs/block_dev.c +++ b/fs/block_dev.c @@ -564,14 +564,14 @@ EXPORT_SYMBOL(thaw_bdev); static int blkdev_writepage(struct address_space *mapping, struct page *page, struct writeback_control *wbc) { - return block_write_full_page(page->mapping->host, page, + return block_write_full_page(mapping->host, page, blkdev_get_block, wbc); } static int blkdev_readpage(struct file * file, struct address_space *mapping, struct page * page) { - return block_read_full_page(page->mapping->host,page,blkdev_get_block); + return block_read_full_page(mapping->host,page,blkdev_get_block); } static int blkdev_readpages(struct file *file, struct address_space *mapping, @@ -1941,7 +1941,7 @@ EXPORT_SYMBOL_GPL(blkdev_read_iter); static int blkdev_releasepage(struct address_space *mapping, struct page *page, gfp_t wait) { - struct super_block *super = BDEV_I(page->mapping->host)->bdev.bd_super; + struct super_block *super = BDEV_I(mapping->host)->bdev.bd_super; if (super && super->s_op->bdev_try_to_free_page) return super->s_op->bdev_try_to_free_page(super, page, wait); -- 2.14.3 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qk0-f200.google.com (mail-qk0-f200.google.com [209.85.220.200]) by kanga.kvack.org (Postfix) with ESMTP id D13196B005D for ; Wed, 4 Apr 2018 15:19:17 -0400 (EDT) Received: by mail-qk0-f200.google.com with SMTP id v187so7761673qka.5 for ; Wed, 04 Apr 2018 12:19:17 -0700 (PDT) Received: from mx1.redhat.com (mx3-rdu2.redhat.com. [66.187.233.73]) by mx.google.com with ESMTPS id x63si1344161qkc.92.2018.04.04.12.19.16 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 04 Apr 2018 12:19:16 -0700 (PDT) From: jglisse@redhat.com Subject: [RFC PATCH 32/79] fs/block: do not rely on page->mapping get it from the context Date: Wed, 4 Apr 2018 15:18:06 -0400 Message-Id: <20180404191831.5378-17-jglisse@redhat.com> In-Reply-To: <20180404191831.5378-1-jglisse@redhat.com> References: <20180404191831.5378-1-jglisse@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: owner-linux-mm@kvack.org List-ID: To: linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, linux-block@vger.kernel.org Cc: linux-kernel@vger.kernel.org, =?UTF-8?q?J=C3=A9r=C3=B4me=20Glisse?= , Jens Axboe , Andrew Morton , Alexander Viro , Tejun Heo , Jan Kara , Josef Bacik , Mel Gorman From: JA(C)rA'me Glisse This patch remove most dereference of page->mapping and get the mapping from the call context (either already available in the function or by adding it to function arguments). Signed-off-by: JA(C)rA'me Glisse Cc: Jens Axboe CC: Andrew Morton Cc: Alexander Viro Cc: linux-fsdevel@vger.kernel.org Cc: Tejun Heo Cc: Jan Kara Cc: Josef Bacik Cc: Mel Gorman --- fs/block_dev.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/fs/block_dev.c b/fs/block_dev.c index 502b6643bc74..dd9da97615e3 100644 --- a/fs/block_dev.c +++ b/fs/block_dev.c @@ -564,14 +564,14 @@ EXPORT_SYMBOL(thaw_bdev); static int blkdev_writepage(struct address_space *mapping, struct page *page, struct writeback_control *wbc) { - return block_write_full_page(page->mapping->host, page, + return block_write_full_page(mapping->host, page, blkdev_get_block, wbc); } static int blkdev_readpage(struct file * file, struct address_space *mapping, struct page * page) { - return block_read_full_page(page->mapping->host,page,blkdev_get_block); + return block_read_full_page(mapping->host,page,blkdev_get_block); } static int blkdev_readpages(struct file *file, struct address_space *mapping, @@ -1941,7 +1941,7 @@ EXPORT_SYMBOL_GPL(blkdev_read_iter); static int blkdev_releasepage(struct address_space *mapping, struct page *page, gfp_t wait) { - struct super_block *super = BDEV_I(page->mapping->host)->bdev.bd_super; + struct super_block *super = BDEV_I(mapping->host)->bdev.bd_super; if (super && super->s_op->bdev_try_to_free_page) return super->s_op->bdev_try_to_free_page(super, page, wait); -- 2.14.3