Messages in this thread | | | Date | Tue, 14 Feb 2006 08:22:29 +0900 | From | KUROSAWA Takahiro <> | Subject | Re: [ckrm-tech] [PATCH 2/2] connect the CPU resource controller to CKRM |
| |
On Mon, 13 Feb 2006 20:09:22 +0530 Srivatsa Vaddagiri <vatsa@in.ibm.com> wrote:
> > + /* FALL THROUGH */ > > + case CPU_UP_PREPARE: > ^^^^^^^^^^^^^^ > This should be done at CPU_ONLINE time (since the new CPU won't > be in the cpu_online_map yet)?
> --- kernel/ckrm/ckrm_cpu.c.org 2006-01-31 11:37:46.000000000 +0530 > +++ kernel/ckrm/ckrm_cpu.c 2006-01-31 11:39:30.000000000 +0530 > @@ -295,7 +295,7 @@ static int __devinit ckrm_cpu_notify(str > } > ckrm_unlock_hier(cls); > /* FALL THROUGH */ > - case CPU_UP_PREPARE: > + case CPU_ONLINE: > grcd.cpus = cpu_online_map; > grcd.numcpus = cpus_weight(cpu_online_map); > break;
Your fix seems correct. I'll apply your patch, thanks for the fix!
-- KUROSAWA, Takahiro - 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/
| |