forked from mirrors/linux
		
	mm, memory_failure: don't send BUS_MCEERR_AO for action required error
Some processes dont't want to be killed early, but in "Action Required" case, those also may be killed by BUS_MCEERR_AO when sharing memory with other which is accessing the fail memory. And sending SIGBUS with BUS_MCEERR_AO for action required error is strange, so ignore the non-current processes here. Suggested-by: Naoya Horiguchi <naoya.horiguchi@nec.com> Signed-off-by: Wetp Zhang <wetp.zy@linux.alibaba.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Acked-by: Naoya Horiguchi <naoya.horiguchi@nec.com> Acked-by: Pankaj Gupta <pankaj.gupta.linux@gmail.com> Link: http://lkml.kernel.org/r/1590817116-21281-1-git-send-email-wetp.zy@linux.alibaba.com Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
		
							parent
							
								
									6972f55c41
								
							
						
					
					
						commit
						872e9a205c
					
				
					 1 changed files with 9 additions and 6 deletions
				
			
		|  | @ -210,14 +210,17 @@ static int kill_proc(struct to_kill *tk, unsigned long pfn, int flags) | |||
| { | ||||
| 	struct task_struct *t = tk->tsk; | ||||
| 	short addr_lsb = tk->size_shift; | ||||
| 	int ret; | ||||
| 	int ret = 0; | ||||
| 
 | ||||
| 	pr_err("Memory failure: %#lx: Sending SIGBUS to %s:%d due to hardware memory corruption\n", | ||||
| 		pfn, t->comm, t->pid); | ||||
| 	if ((t->mm == current->mm) || !(flags & MF_ACTION_REQUIRED)) | ||||
| 		pr_err("Memory failure: %#lx: Sending SIGBUS to %s:%d due to hardware memory corruption\n", | ||||
| 			pfn, t->comm, t->pid); | ||||
| 
 | ||||
| 	if ((flags & MF_ACTION_REQUIRED) && t->mm == current->mm) { | ||||
| 		ret = force_sig_mceerr(BUS_MCEERR_AR, (void __user *)tk->addr, | ||||
| 				       addr_lsb); | ||||
| 	if (flags & MF_ACTION_REQUIRED) { | ||||
| 		if (t->mm == current->mm) | ||||
| 			ret = force_sig_mceerr(BUS_MCEERR_AR, | ||||
| 					 (void __user *)tk->addr, addr_lsb); | ||||
| 		/* send no signal to non-current processes */ | ||||
| 	} else { | ||||
| 		/*
 | ||||
| 		 * Don't use force here, it's convenient if the signal | ||||
|  |  | |||
		Loading…
	
		Reference in a new issue
	
	 Wetp Zhang
						Wetp Zhang