lkml.org 
[lkml]   [2013]   [Apr]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCHv3, RFC 32/34] thp: handle write-protect exception to file-backed huge pages
For all the do_huge_pmd_wp_page(), I think we need a better description
of where the code came from. There are some more obviously
copy-n-pasted comments in there.

For the entire series, in the patch description, we need to know:
1. What was originally written and what was copied from elsewhere
2. For the stuff that was copied, was an attempt made to consolidate
instead of copy? Why was consolidation impossible or infeasible?

> + if (!PageAnon(page)) {
> + add_mm_counter(mm, MM_FILEPAGES, -HPAGE_PMD_NR);
> + add_mm_counter(mm, MM_ANONPAGES, HPAGE_PMD_NR);
> + }

This seems like a bit of a hack. Shouldn't we have just been accounting
to MM_FILEPAGES in the first place?


\
 
 \ /
  Last update: 2013-04-08 22:48    [W:0.165 / U:0.152 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site