Re: preempt-related hangs

[Anton Altaparmakov]

At 02:11 25/03/02, Andrew Morton wrote:
>Andrew Morton wrote:
> >
> > ..
> > Kernel is 2.5.7, dual PIII.  When I enable preempt it
> > locks during boot.
>
>OK, this patch fixed it.  I don't know why.

Er, because you disable preemption twice and it never gets enabled again? (-:

You probably meant that to be preemt_enable() at the bottom of the patch... 
That might not solve your problem of course... But with the patch you 
basically have completely disabled preemption, you might as well not 
configure it into the kernel. (-;

Anton

>--- linux-2.5.7/kernel/sched.c  Mon Mar 18 13:04:41 2002
>+++ 25/kernel/sched.c   Sun Mar 24 18:09:09 2002
>@@ -1545,6 +1545,8 @@ void set_cpus_allowed(task_t *p, unsigne
>         migration_req_t req;
>         runqueue_t *rq;
>
>+       preempt_disable();
>+
>         new_mask &= cpu_online_map;
>         if (!new_mask)
>                 BUG();
>@@ -1557,7 +1559,7 @@ void set_cpus_allowed(task_t *p, unsigne
>         */
>         if (new_mask & (1UL << p->thread_info->cpu)) {
>                 task_rq_unlock(rq, &flags);
>-               return;
>+               goto out;
>         }
>
>         init_MUTEX_LOCKED(&req.sem);
>@@ -1567,6 +1569,8 @@ void set_cpus_allowed(task_t *p, unsigne
>         wake_up_process(rq->migration_thread);
>
>         down(&req.sem);
>+out:
>+       preempt_disable();
>  }
>
>  static volatile unsigned long migration_mask;
>
>
>-
>-
>To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>the body of a message to [e-mail]
>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>Please read the FAQ at  http://www.tux.org/lkml/