lkml.org 
[lkml]   [2015]   [Nov]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 8/8] stop_machine: cleanup the usage of preemption counter in cpu_stopper_thread()
On Sun, Nov 15, 2015 at 08:33:32PM +0100, Oleg Nesterov wrote:
> 1. Change this code to use preempt_count_inc/preempt_count_dec; this way
> it works even if CONFIG_PREEMPT_COUNT=n, and we avoid the unnecessary
> __preempt_schedule() check (stop_sched_class is not preemptible).
>
> And this makes clear that we only want to make preempt_count() != 0
> for __might_sleep() / schedule_debug().
>
> 2. Change WARN_ONCE() to use %pf to print the function name and remove
> kallsyms_lookup/ksym_buf.
>
> 3. Move "int ret" into the "if (work)" block, this looks more consistent.
>
> Signed-off-by: Oleg Nesterov <oleg@redhat.com>

The entire series looks good to me. Please feel free to add

Acked-by: Tejun Heo <tj@kernel.org>

Thanks.

--
tejun


\
 
 \ /
  Last update: 2015-11-16 20:41    [W:0.148 / U:0.508 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site