Re: BUG: using smp_processor_id() in preemptible code: s2disk

From: Sergey Senozhatsky
Date: Mon Jun 14 2010 - 10:51:19 EST


On (06/14/10 07:38), Arjan van de Ven wrote:
> > Hello,
> > Not sure if this simple solution is the correct one.
>
> it's not; the caller needs to pass in the cpu number I suspect for this
> to be really correct....
>
> I just returned from family emergency travel and will take a look today
>

I thought about patching

./drivers/cpuidle/governors/menu.c: if (nr_iowait_cpu())
./drivers/cpuidle/governors/menu.c: mult += 10 * nr_iowait_cpu();
./kernel/time/tick-sched.c: if (nr_iowait_cpu() > 0)


decided to patch nr_iowait_cpu instead.


Sergey

Attachment: pgp00000.pgp
Description: PGP signature