On Fri, Jun 26, 2015 at 04:14:55AM +0200, Oleg Nesterov wrote: > Not sure. > > And note that this series kills stop_cpus_mutex, so that multiple > stop_cpus()'s / stop_machine()'s can run in parallel if cpumask's > do not overlap. > > Note also the changelog in 6/6, we can simplify + optimize this code > a bit more. > > What do you think?
The problem I have with this is that it makes the better operation (stop_two_cpus) slower while improving the worse operation (stop_cpus). I would much prefer to keep stop_two_cpus() as proposed with taking two cpu_stopper::lock instances and replacing the stop_cpu_mutex with a percpu-rwsem. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
