Re: Scheduler Bug (set_cpus_allowed)

[David S. Miller]

   From: Ingo Molnar <[e-mail]>
   Date: Mon, 10 Jun 2002 22:02:28 +0200 (CEST)
   
   David, would it be possible to somehow not recurse back into the scheduler
   code (like wakeup) from within the port-specific switch_to() code?

It's a locking problem more than anything else.  It's not that
we call back into it, it's that we hold locks in switch_mm.

To recap, from the changeset 1.369.49.1:

  Fix scheduler deadlock on some platforms.
  
  Some platforms need to grab mm->page_table_lock during switch_mm().
  On the other hand code like swap_out() in mm/vmscan.c needs to hold
  mm->page_table_lock during wakeups which needs to grab the runqueue
  lock.  This creates a conflict and the resolution chosen here is to
  not hold the runqueue lock during context_switch().
  
  The implementation is specifically a "frozen" state implemented as a
  spinlock, which is held around the context_switch() call.  This allows
  the runqueue lock to be dropped during this time yet prevent another cpu
  from running the "not switched away from yet" task.

So if you're going to delete the frozen code, please replace it with
something that handles the above.

There is nothing weird about holding page_table_lock during
switch_mm, I can imagine many other ports doing it especially those
with TLB pids which want to optimize SMP tlb/cache flushes.

I think changing vmscan.c to not call wakeup is the wrong way to
go about this.  I thought about doing that originally, but it looks
to be 100 times more difficult to implement (and verify) than the
scheduler side fix.