All of lore.kernel.org
 help / color / mirror / Atom feed
* [luto:sched/lazymm 13/16] kernel/sched/core.c:4982 __change_current_mm() error: uninitialized symbol 'old_active_mm'.
@ 2021-11-23  8:44 ` Dan Carpenter
  0 siblings, 0 replies; 3+ messages in thread
From: kernel test robot @ 2021-11-22 21:42 UTC (permalink / raw)
  To: kbuild

[-- Attachment #1: Type: text/plain, Size: 6701 bytes --]

CC: kbuild-all(a)lists.01.org
CC: linux-kernel(a)vger.kernel.org
TO: Andy Lutomirski <luto@kernel.org>

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 <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>

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/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 30507 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [luto:sched/lazymm 13/16] kernel/sched/core.c:4982 __change_current_mm() error: uninitialized symbol 'old_active_mm'.
@ 2021-11-23  8:44 ` Dan Carpenter
  0 siblings, 0 replies; 3+ messages in thread
From: Dan Carpenter @ 2021-11-23  8:44 UTC (permalink / raw)
  To: kbuild, Andy Lutomirski; +Cc: lkp, kbuild-all, linux-kernel

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
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 <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>

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

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);

"old_active_mm" is never initialized.

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  	}
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [luto:sched/lazymm 13/16] kernel/sched/core.c:4982 __change_current_mm() error: uninitialized symbol 'old_active_mm'.
@ 2021-11-23  8:44 ` Dan Carpenter
  0 siblings, 0 replies; 3+ messages in thread
From: Dan Carpenter @ 2021-11-23  8:44 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 5853 bytes --]

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
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 <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>

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

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);

"old_active_mm" is never initialized.

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  	}
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-11-23  8:44 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-22 21:42 [luto:sched/lazymm 13/16] kernel/sched/core.c:4982 __change_current_mm() error: uninitialized symbol 'old_active_mm' kernel test robot
2021-11-23  8:44 ` Dan Carpenter
2021-11-23  8:44 ` Dan Carpenter

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.