mm/khugepaged: collapse_shmem() stop if punched or truncated
authorHugh Dickins <[email protected]>
Fri, 30 Nov 2018 22:10:25 +0000 (14:10 -0800)
committerLinus Torvalds <[email protected]>
Fri, 30 Nov 2018 22:56:15 +0000 (14:56 -0800)
Huge tmpfs testing showed that although collapse_shmem() recognizes a
concurrently truncated or hole-punched page correctly, its handling of
holes was liable to refill an emptied extent.  Add check to stop that.

Link: http://lkml.kernel.org/r/[email protected]
Fixes: f3f0e1d2150b2 ("khugepaged: add support of collapse for tmpfs/shmem pages")
Signed-off-by: Hugh Dickins <[email protected]>
Reviewed-by: Matthew Wilcox <[email protected]>
Cc: Kirill A. Shutemov <[email protected]>
Cc: Jerome Glisse <[email protected]>
Cc: Konstantin Khlebnikov <[email protected]>
Cc: <[email protected]> [4.8+]
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
mm/khugepaged.c

index c13625c1ad5e5e91abf1d45bdbcf388f9b1f2690..2070c316f06ef5909580c85e2bccbf250c9e366c 100644 (file)
@@ -1359,6 +1359,17 @@ static void collapse_shmem(struct mm_struct *mm,
 
                VM_BUG_ON(index != xas.xa_index);
                if (!page) {
+                       /*
+                        * Stop if extent has been truncated or hole-punched,
+                        * and is now completely empty.
+                        */
+                       if (index == start) {
+                               if (!xas_next_entry(&xas, end - 1)) {
+                                       result = SCAN_TRUNCATED;
+                                       break;
+                               }
+                               xas_set(&xas, index);
+                       }
                        if (!shmem_charge(mapping->host, 1)) {
                                result = SCAN_FAIL;
                                break;