mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 10:40:15 +02:00 
			
		
		
		
	mm, x86: Saving vmcore with non-lazy freeing of vmas
During the reading of /proc/vmcore the kernel is doing ioremap()/iounmap() repeatedly. And the buildup of un-flushed vm_area_struct's is causing a great deal of overhead. (rb_next() is chewing up most of that time). This solution is to provide function set_iounmap_nonlazy(). It causes a subsequent call to iounmap() to immediately purge the vma area (with try_purge_vmap_area_lazy()). With this patch we have seen the time for writing a 250MB compressed dump drop from 71 seconds to 44 seconds. Signed-off-by: Cliff Wickman <cpw@sgi.com> Cc: Andrew Morton <akpm@linux-foundation.org> Cc: kexec@lists.infradead.org Cc: <stable@kernel.org> LKML-Reference: <E1OwHZ4-0005WK-Tw@eag09.americas.sgi.com> Signed-off-by: Ingo Molnar <mingo@elte.hu>
This commit is contained in:
		
							parent
							
								
									37a2f9f30a
								
							
						
					
					
						commit
						3ee48b6af4
					
				
					 3 changed files with 11 additions and 0 deletions
				
			
		| 
						 | 
					@ -206,6 +206,7 @@ static inline void __iomem *ioremap(resource_size_t offset, unsigned long size)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
extern void iounmap(volatile void __iomem *addr);
 | 
					extern void iounmap(volatile void __iomem *addr);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					extern void set_iounmap_nonlazy(void);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
#ifdef __KERNEL__
 | 
					#ifdef __KERNEL__
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -46,6 +46,7 @@ ssize_t copy_oldmem_page(unsigned long pfn, char *buf,
 | 
				
			||||||
	} else
 | 
						} else
 | 
				
			||||||
		memcpy(buf, vaddr + offset, csize);
 | 
							memcpy(buf, vaddr + offset, csize);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						set_iounmap_nonlazy();
 | 
				
			||||||
	iounmap(vaddr);
 | 
						iounmap(vaddr);
 | 
				
			||||||
	return csize;
 | 
						return csize;
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -516,6 +516,15 @@ static atomic_t vmap_lazy_nr = ATOMIC_INIT(0);
 | 
				
			||||||
/* for per-CPU blocks */
 | 
					/* for per-CPU blocks */
 | 
				
			||||||
static void purge_fragmented_blocks_allcpus(void);
 | 
					static void purge_fragmented_blocks_allcpus(void);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					/*
 | 
				
			||||||
 | 
					 * called before a call to iounmap() if the caller wants vm_area_struct's
 | 
				
			||||||
 | 
					 * immediately freed.
 | 
				
			||||||
 | 
					 */
 | 
				
			||||||
 | 
					void set_iounmap_nonlazy(void)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
						atomic_set(&vmap_lazy_nr, lazy_max_pages()+1);
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
/*
 | 
					/*
 | 
				
			||||||
 * Purges all lazily-freed vmap areas.
 | 
					 * Purges all lazily-freed vmap areas.
 | 
				
			||||||
 *
 | 
					 *
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue