diff options
author | Jan Kara <jack@suse.cz> | 2012-12-11 19:01:09 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-12-11 20:22:23 -0500 |
commit | e749eb95531ac8349df47f8d46ce2641dcb16589 (patch) | |
tree | c78428ed9c92bf0e4915f6dfa6a21d4e58e14465 /mm/rmap.c | |
parent | 712cd386fdc983d318fecf302a2a9cb8e9de90c9 (diff) |
mm: add comment on storage key dirty bit semantics
Add comments that dirty bit in storage key gets set whenever page content
is changed. Hopefully if someone will use this function, he'll have a
look at one of the two places where we comment on this.
Signed-off-by: Jan Kara <jack@suse.cz>
Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
Cc: Mel Gorman <mgorman@suse.de>
Cc: Hugh Dickins <hughd@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/rmap.c')
-rw-r--r-- | mm/rmap.c | 8 |
1 files changed, 5 insertions, 3 deletions
@@ -1151,9 +1151,11 @@ void page_remove_rmap(struct page *page) | |||
1151 | * containing the swap entry, but page not yet written to swap. | 1151 | * containing the swap entry, but page not yet written to swap. |
1152 | * | 1152 | * |
1153 | * And we can skip it on file pages, so long as the filesystem | 1153 | * And we can skip it on file pages, so long as the filesystem |
1154 | * participates in dirty tracking; but need to catch shm and tmpfs | 1154 | * participates in dirty tracking (note that this is not only an |
1155 | * and ramfs pages which have been modified since creation by read | 1155 | * optimization but also solves problems caused by dirty flag in |
1156 | * fault. | 1156 | * storage key getting set by a write from inside kernel); but need to |
1157 | * catch shm and tmpfs and ramfs pages which have been modified since | ||
1158 | * creation by read fault. | ||
1157 | * | 1159 | * |
1158 | * Note that mapping must be decided above, before decrementing | 1160 | * Note that mapping must be decided above, before decrementing |
1159 | * mapcount (which luckily provides a barrier): once page is unmapped, | 1161 | * mapcount (which luckily provides a barrier): once page is unmapped, |