Re: [patch] get/put_cpu in up need not disable preemption

[Robert Love <rml@tech9.net>]

On Sat, 2002-11-09 at 05:06, Ravikiran G Thirumalai wrote:

> AFAICS, get_cpu, put_cpu and put_cpu_no_resched need not disable 
> preemption on a uniprocessor. Foll patch removes the disable/enable
> premeption stuff for the UP case.  Tested on a PIII 4 way for both
> UP and SMP configs. Pls apply.

No, it needs to.

Per-CPU data can alleviate the need for a lock.  On SMP, a per-CPU
variable does not need a lock since it is impossible for another task to
enter the same critical section and access the same variable, from
another CPU.  On preempt it is fully possible.

For example, this is a critical section, and you do not want two threads
on the same CPU concurrently inside:

	extern struct my_struct[NR_CPUS];
	int cpu = get_cpu();

	do_stuff(my_struct[cpu]);
	do_more_stuff(my_struct[cpu]);
	foo = my_struct[cpu].bar;

	put_cpu();

So get_cpu() must disable preemption even on UP.  The problem you are
thinking of (being preempted and returning on a different CPU) is only a
subset of the issues.  All processor accesses must be atomic.

	Robert Love