---

yaml --- r: 177618 b: refs/heads/master c: db0480b h: refs/heads/master v: v3
git-mirror · Dec 16, 2009 · 0323218 · 0323218
1 parent 3f105b0
commit 0323218
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/[refs] b/[refs]
@@ -1,2 +1,2 @@
 ---
-refs/heads/master: 1668bfd5be9d8a52536c4865000fbbe065a3613b
+refs/heads/master: db0480b3a61bd6ad86ead3b8bbad094ab0996932
diff --git a/trunk/mm/memory-failure.c b/trunk/mm/memory-failure.c
@@ -687,6 +687,8 @@ static int hwpoison_user_mappings(struct page *p, unsigned long pfn,
 	/*
 	 * Propagate the dirty bit from PTEs to struct page first, because we
 	 * need this to decide if we should kill or just drop the page.
+	 * XXX: the dirty test could be racy: set_page_dirty() may not always
+	 * be called inside page lock (it's recommended but not enforced).
 	 */
 	mapping = page_mapping(p);
 	if (!PageDirty(p) && mapping && mapping_cap_writeback_dirty(mapping)) {