mm: uncached vma support with writenotify
Modify the generic mmap() code to keep the cache attribute in vma->vm_page_prot regardless if writenotify is enabled or not. Without this patch the cache configuration selected by f_op->mmap() is overwritten if writenotify is enabled, making it impossible to keep the vma uncached. Needed by drivers such as drivers/video/sh_mobile_lcdcfb.c which uses deferred io together with uncached memory. Signed-off-by: Magnus Damm <damm@opensource.se> Cc: Nick Piggin <nickpiggin@yahoo.com.au> Cc: Hugh Dickins <hugh.dickins@tiscali.co.uk> Cc: Paul Mundt <lethal@linux-sh.org> Cc: Jaya Kumar <jayakumar.lkml@gmail.com> Cc: Arnd Bergmann <arnd@arndb.de> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
		
					parent
					
						
							
								62c0c2f198
							
						
					
				
			
			
				commit
				
					
						c9d0bf2414
					
				
			
		
					 1 changed files with 13 additions and 1 deletions
				
			
		
							
								
								
									
										14
									
								
								mm/mmap.c
									
										
									
									
									
								
							
							
						
						
									
										14
									
								
								mm/mmap.c
									
										
									
									
									
								
							|  | @ -1198,8 +1198,20 @@ munmap_back: | ||||||
| 			goto free_vma; | 			goto free_vma; | ||||||
| 	} | 	} | ||||||
| 
 | 
 | ||||||
| 	if (vma_wants_writenotify(vma)) | 	if (vma_wants_writenotify(vma)) { | ||||||
|  | 		pgprot_t pprot = vma->vm_page_prot; | ||||||
|  | 
 | ||||||
|  | 		/* Can vma->vm_page_prot have changed??
 | ||||||
|  | 		 * | ||||||
|  | 		 * Answer: Yes, drivers may have changed it in their | ||||||
|  | 		 *         f_op->mmap method. | ||||||
|  | 		 * | ||||||
|  | 		 * Ensures that vmas marked as uncached stay that way. | ||||||
|  | 		 */ | ||||||
| 		vma->vm_page_prot = vm_get_page_prot(vm_flags & ~VM_SHARED); | 		vma->vm_page_prot = vm_get_page_prot(vm_flags & ~VM_SHARED); | ||||||
|  | 		if (pgprot_val(pprot) == pgprot_val(pgprot_noncached(pprot))) | ||||||
|  | 			vma->vm_page_prot = pgprot_noncached(vma->vm_page_prot); | ||||||
|  | 	} | ||||||
| 
 | 
 | ||||||
| 	vma_link(mm, vma, prev, rb_link, rb_parent); | 	vma_link(mm, vma, prev, rb_link, rb_parent); | ||||||
| 	file = vma->vm_file; | 	file = vma->vm_file; | ||||||
|  |  | ||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue
	
	 Magnus Damm
				Magnus Damm