bugfix for mmaped pages in mpage_release_unused_pages()
From: <hidden>
Date: 2016-09-05 01:22:36
[PATCH]ext4: bugfix for mmaped pages in mpage_release_unused_pages()
ext4: bugfix for mmaped pages in mpage_release_unused_pages()
Pages clear buffers after ext4 delayed block allocation failed,
However, it does not clean its pte_dirty flag.
if the pages unmap ,in cording to the pte_dirty ,
unmap_page_range may try to call __set_page_dirty,
which may lead to the bugon at
mpage_prepare_extent_to_map:head = page_buffers(page);.
This patch just call clear_page_dirty_for_io to clean pte_dirty
at mpage_release_unused_pages for pages mmaped.
Steps to reproduce the bug:
(1) mmap a file in ext4
addr = (char *)mmap(NULL, 4096,
PROT_READ|PROT_WRITE, MAP_SHARED, fd, 0);
memset(addr, 'i', 4096);
(2) return EIO at
ext4_writepages->mpage_map_and_submit_extent->mpage_map_one_extent
Cause the log:
ext4_msg(sb, KERN_CRIT,
"Delayed block allocation failed for "
"inode %lu at logical offset %llu with"
" max blocks %u with error %d",
inode->i_ino,
(unsigned long long)map->m_lblk,
(unsigned)map->m_len, -err);
(3)Unmap the addr cause warning at
__set_page_dirty:WARN_ON_ONCE(warn && !PageUptodate(page));
(4) wait for a minute,then bugon happen.
From 3eab887a55424fc2c27553b7bfe32330df83f7b8
From: wangguang <redacted>
Date: Mon, 29 Aug 2016 14:54:58 -0700
Subject: [PATCH] bugfix for mmaped pages in mpage_release_unused_pages()
Signed-off-by: wangguang <redacted>--- old/fs/ext4/inode.c 2016-08-29 12:15:25.150764447 -0400
+++ new/fs/ext4/inode.c 2016-08-29 13:01:58.683559198 -0400@@ -1649,6 +1649,8 @@ static void mpage_release_unused_pages(s BUG_ON(!PageLocked(page)); BUG_ON(PageWriteback(page)); if (invalidate) { + if (page_mapped(page)) + clear_page_dirty_for_io(page); block_invalidatepage(page, 0, PAGE_SIZE); ClearPageUptodate(page); }
王广 Operation System Production Dept . 操作系统产品部 创新*突破*集约化 Add:成都市高新区天府大道中段800号中兴通讯大厦xxx室 Tel:13678197402