forked from mirrors/linux
		
	mm: Clear page->private when splitting or migrating a page
In our efforts to remove uses of PG_private, we have found folios with
the private flag clear and folio->private not-NULL.  That is the root
cause behind 642d51fb07 ("ceph: check folio PG_private bit instead
of folio->private").  It can also affect a few other filesystems that
haven't yet reported a problem.
compaction_alloc() can return a page with uninitialised page->private,
and rather than checking all the callers of migrate_pages(), just zero
page->private after calling get_new_page().  Similarly, the tail pages
from split_huge_page() may also have an uninitialised page->private.
Reported-by: Xiubo Li <xiubli@redhat.com>
Tested-by: Xiubo Li <xiubli@redhat.com>
Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
			
			
This commit is contained in:
		
							parent
							
								
									cb995f4eeb
								
							
						
					
					
						commit
						b653db7735
					
				
					 2 changed files with 2 additions and 0 deletions
				
			
		|  | @ -2377,6 +2377,7 @@ static void __split_huge_page_tail(struct page *head, int tail, | ||||||
| 			page_tail); | 			page_tail); | ||||||
| 	page_tail->mapping = head->mapping; | 	page_tail->mapping = head->mapping; | ||||||
| 	page_tail->index = head->index + tail; | 	page_tail->index = head->index + tail; | ||||||
|  | 	page_tail->private = 0; | ||||||
| 
 | 
 | ||||||
| 	/* Page flags must be visible before we make the page non-compound. */ | 	/* Page flags must be visible before we make the page non-compound. */ | ||||||
| 	smp_wmb(); | 	smp_wmb(); | ||||||
|  |  | ||||||
|  | @ -1106,6 +1106,7 @@ static int unmap_and_move(new_page_t get_new_page, | ||||||
| 	if (!newpage) | 	if (!newpage) | ||||||
| 		return -ENOMEM; | 		return -ENOMEM; | ||||||
| 
 | 
 | ||||||
|  | 	newpage->private = 0; | ||||||
| 	rc = __unmap_and_move(page, newpage, force, mode); | 	rc = __unmap_and_move(page, newpage, force, mode); | ||||||
| 	if (rc == MIGRATEPAGE_SUCCESS) | 	if (rc == MIGRATEPAGE_SUCCESS) | ||||||
| 		set_page_owner_migrate_reason(newpage, reason); | 		set_page_owner_migrate_reason(newpage, reason); | ||||||
|  |  | ||||||
		Loading…
	
		Reference in a new issue
	
	 Matthew Wilcox (Oracle)
						Matthew Wilcox (Oracle)