Re: [RFC PATCH v2 10/10] cpu: No more __stop_machine() in _cpu_down()

From: Srivatsa S. Bhat
Date: Wed Dec 05 2012 - 14:13:39 EST


On 12/06/2012 12:38 AM, Oleg Nesterov wrote:
> On 12/06, Srivatsa S. Bhat wrote:
>>
>> @@ -418,7 +418,7 @@ static int __ref _cpu_down(unsigned int cpu, int tasks_frozen)
>> }
>> smpboot_park_threads(cpu);
>>
>> - err = __stop_machine(take_cpu_down, &tcd_param, cpumask_of(cpu));
>> + err = stop_cpus(cpumask_of(cpu), take_cpu_down, &tcd_param);
>
> stop_one_cpu(cpu) ?
>

Even I was thinking of using that. Paul, any particular reason you chose
stop_cpus() over stop_one_cpu() in [1]?

[1]. https://lkml.org/lkml/2012/10/30/359

Regards,
Srivatsa S. Bhat

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