This is obviously good per se, but it will also allow using strprefix() instead of str_has_prefix(). Signed-off-by: Alejandro Colomar --- arch/s390/lib/test_unwind.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/arch/s390/lib/test_unwind.c b/arch/s390/lib/test_unwind.c index 6bb3fa5bf925..bb47958f5aed 100644 --- a/arch/s390/lib/test_unwind.c +++ b/arch/s390/lib/test_unwind.c @@ -47,7 +47,8 @@ static void print_backtrace(char *bt) static noinline int test_unwind(struct task_struct *task, struct pt_regs *regs, unsigned long sp) { - int frame_count, prev_is_func2, seen_func2_func1, seen_arch_rethook_trampoline; + int frame_count; + bool prev_is_func2, seen_func2_func1, seen_arch_rethook_trampoline; const int max_frames = 128; struct unwind_state state; size_t bt_pos = 0; @@ -61,9 +62,9 @@ static noinline int test_unwind(struct task_struct *task, struct pt_regs *regs, } /* Unwind. */ frame_count = 0; - prev_is_func2 = 0; - seen_func2_func1 = 0; - seen_arch_rethook_trampoline = 0; + prev_is_func2 = false; + seen_func2_func1 = false; + seen_arch_rethook_trampoline = false; unwind_for_each_frame(&state, task, regs, sp) { unsigned long addr = unwind_get_return_address(&state); char sym[KSYM_SYMBOL_LEN]; @@ -87,10 +88,10 @@ static noinline int test_unwind(struct task_struct *task, struct pt_regs *regs, } frame_count += 1; if (prev_is_func2 && str_has_prefix(sym, "unwindme_func1")) - seen_func2_func1 = 1; + seen_func2_func1 = true; prev_is_func2 = str_has_prefix(sym, "unwindme_func2"); if (str_has_prefix(sym, "arch_rethook_trampoline+0x0/")) - seen_arch_rethook_trampoline = 1; + seen_arch_rethook_trampoline = true; } /* Check the results. */ -- 2.55.0