From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pf1-f198.google.com (mail-pf1-f198.google.com [209.85.210.198]) by kanga.kvack.org (Postfix) with ESMTP id 38FCF6B0006 for ; Wed, 14 Nov 2018 19:30:20 -0500 (EST) Received: by mail-pf1-f198.google.com with SMTP id p9so2783200pfj.3 for ; Wed, 14 Nov 2018 16:30:20 -0800 (PST) Received: from mail-sor-f65.google.com (mail-sor-f65.google.com. [209.85.220.65]) by mx.google.com with SMTPS id n11-v6sor30185990plg.0.2018.11.14.16.30.18 for (Google Transport Security); Wed, 14 Nov 2018 16:30:18 -0800 (PST) From: p.jaroszynski@gmail.com Subject: [PATCH] iomap: get/put the page in iomap_page_create/release() Date: Wed, 14 Nov 2018 16:30:00 -0800 Message-Id: <20181115003000.1358007-1-pjaroszynski@nvidia.com> Sender: owner-linux-mm@kvack.org List-ID: To: Christoph Hellwig , Michal Hocko Cc: linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, Piotr Jaroszynski From: Piotr Jaroszynski migrate_page_move_mapping() expects pages with private data set to have a page_count elevated by 1. This is what used to happen for xfs through the buffer_heads code before the switch to iomap in 82cb14175e7d ("xfs: add support for sub-pagesize writeback without buffer_heads"). Not having the count elevated causes move_pages() to fail on memory mapped files coming from xfs. Make iomap compatible with the migrate_page_move_mapping() assumption by elevating the page count as part of iomap_page_create() and lowering it in iomap_page_release(). Fixes: 82cb14175e7d ("xfs: add support for sub-pagesize writeback without buffer_heads") Signed-off-by: Piotr Jaroszynski --- fs/iomap.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/fs/iomap.c b/fs/iomap.c index 90c2febc93ac..23977f9f23a2 100644 --- a/fs/iomap.c +++ b/fs/iomap.c @@ -117,6 +117,12 @@ iomap_page_create(struct inode *inode, struct page *page) atomic_set(&iop->read_count, 0); atomic_set(&iop->write_count, 0); bitmap_zero(iop->uptodate, PAGE_SIZE / SECTOR_SIZE); + + /* + * At least migrate_page_move_mapping() assumes that pages with private + * data have their count elevated by 1. + */ + get_page(page); set_page_private(page, (unsigned long)iop); SetPagePrivate(page); return iop; @@ -133,6 +139,7 @@ iomap_page_release(struct page *page) WARN_ON_ONCE(atomic_read(&iop->write_count)); ClearPagePrivate(page); set_page_private(page, 0); + put_page(page); kfree(iop); } -- 2.11.0.262.g4b0a5b2.dirty