CC: [email protected]
CC: [email protected]
TO: Andy Lutomirski <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/luto/linux.git 
sched/lazymm
head:   c0d03d4f2778fd0a7c16e69cdfb3f111296129b5
commit: 4863118ffa6fe6af7cfb18c4d4ee6434537b0c8b [13/16] sched, exec: Factor 
current mm changes out from exec
:::::: branch date: 7 days ago
:::::: commit date: 7 days ago
config: x86_64-randconfig-m001-20211118 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>
Reported-by: Dan Carpenter <[email protected]>

smatch warnings:
kernel/sched/core.c:4982 __change_current_mm() error: uninitialized symbol 
'old_active_mm'.

vim +/old_active_mm +4982 kernel/sched/core.c

f711f6090a81cb kernel/sched.c      Gautham R Shenoy 2009-04-14  4937  
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4938  void 
__change_current_mm(struct mm_struct *mm, bool mm_is_brand_new)
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4939  {
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4940    struct 
task_struct *tsk = current;
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4941    struct 
mm_struct *old_active_mm, *mm_to_drop = NULL;
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4942  
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4943    
BUG_ON(!mm);    /* likely to cause corruption if we continue */
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4944  
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4945    /*
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4946     * We 
do not want to schedule, nor should procfs peek at current->mm
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4947     * 
while we're modifying it.  task_lock() disables preemption and
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4948     * 
locks against procfs.
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4949     */
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4950    
task_lock(tsk);
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4951    /*
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4952     * 
membarrier() requires a full barrier before switching mm.
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4953     */
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4954    
smp_mb__after_spinlock();
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4955  
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4956    
local_irq_disable();
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4957  
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4958    if 
(tsk->mm) {
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4959            
/* We're detaching from an old mm.  Sync stats. */
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4960            
sync_mm_rss(tsk->mm);
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4961    } else {
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4962            
/*
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4963            
 * Switching from kernel mm to user.  Drop the old lazy
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4964            
 * mm reference.
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4965            
 */
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4966            
mm_to_drop = tsk->active_mm;
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4967    }
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4968  
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4969    
tsk->active_mm = mm;
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4970    
WRITE_ONCE(tsk->mm, mm);  /* membarrier reads this without locks */
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4971    
membarrier_update_current_mm(mm);
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4972  
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4973    if 
(mm_is_brand_new) {
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4974            
/*
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4975            
 * For historical reasons, some architectures want IRQs on
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4976            
 * when activate_mm() is called.  If we're going to call
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4977            
 * activate_mm(), turn on IRQs but leave preemption
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4978            
 * disabled.
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4979            
 */
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4980            
if (!IS_ENABLED(CONFIG_ARCH_WANT_IRQS_OFF_ACTIVATE_MM))
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4981            
        local_irq_enable();
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03 @4982            
activate_mm(old_active_mm, mm);
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4983            
if (IS_ENABLED(CONFIG_ARCH_WANT_IRQS_OFF_ACTIVATE_MM))
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4984            
        local_irq_enable();
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4985    } else {
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4986            
switch_mm_irqs_off(old_active_mm, mm, tsk);
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4987            
local_irq_enable();
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4988    }
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4989  
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4990    /* IRQs 
are on now.  Preemption is still disabled by task_lock(). */
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4991  
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4992    
membarrier_finish_switch_mm(mm);
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4993    
vmacache_flush(tsk);
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4994    
task_unlock(tsk);
4863118ffa6fe6 kernel/sched/core.c Andy Lutomirski  2021-09-03  4995  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]

Attachment: .config.gz
Description: application/gzip

_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to