lkml.org 
[lkml]   [2003]   [Feb]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH][2.5] set_cpus_allowed needs cpu_online_map BUG check

On Mon, 17 Feb 2003, Zwane Mwaikambo wrote:

> We don't want to try and migrate to offline cpus, so BUG() on it.

agreed. Not much code should be exposed to the possibility of CPUs going
away.

Ingo

> diff -u -r1.1.1.1 sched.c
> --- linux-2.5.61-trojan/kernel/sched.c 15 Feb 2003 12:32:44 -0000 1.1.1.1
> +++ linux-2.5.61-trojan/kernel/sched.c 15 Feb 2003 16:04:51 -0000
> @@ -2200,11 +2221,8 @@
> migration_req_t req;
> runqueue_t *rq;
>
> -#if 0 /* FIXME: Grab cpu_lock, return error on this case. --RR */
> - new_mask &= cpu_online_map;
> - if (!new_mask)
> + if (!(new_mask & cpu_online_map))
> BUG();
> -#endif
>
> rq = task_rq_lock(p, &flags);
> p->cpus_allowed = new_mask;
>

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:33    [W:3.859 / U:0.000 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site