diff options
author | Cyrill Gorcunov <gorcunov@openvz.org> | 2014-06-05 03:06:41 +0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-06-05 03:53:56 +0400 |
commit | 0bf073315cb29d2e9e68b6c5da97862a519e3320 (patch) | |
tree | b9ebb344d324b13211d3e9f1784d13684a420117 /mm/rmap.c | |
parent | 52383431b37cdbec63944e953ffc2698a7ad9722 (diff) | |
download | linux-0bf073315cb29d2e9e68b6c5da97862a519e3320.tar.xz |
mm: softdirty: make freshly remapped file pages being softdirty unconditionally
Hugh reported:
| I noticed your soft_dirty work in install_file_pte(): which looked
| good at first, until I realized that it's propagating the soft_dirty
| of a pte it's about to zap completely, to the unrelated entry it's
| about to insert in its place. Which seems very odd to me.
Indeed this code ends up being nop in result -- pte_file_mksoft_dirty()
operates with pte_t argument and returns new pte_t which were never used
after. After looking more I think what we need is to soft-dirtify all
newely remapped file pages because it should look like a new mapping for
memory tracker.
Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org>
Reported-by: Hugh Dickins <hughd@google.com>
Cc: Pavel Emelyanov <xemul@parallels.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')
0 files changed, 0 insertions, 0 deletions