Re: [PATCH] trace: fix SKIP_STACK_VALIDATION=1 build

From: Andi Kleen
Date: Fri Jun 08 2018 - 18:08:26 EST


On Fri, Jun 08, 2018 at 02:47:46PM -0700, Greg Thelen wrote:
> Non gcc-5 builds with CONFIG_STACK_VALIDATION=y and
> SKIP_STACK_VALIDATION=1 fail.
> Example output:
> /bin/sh: init/.tmp_main.o: Permission denied
>
> commit 96f60dfa5819 ("trace: Use -mcount-record for dynamic ftrace"),
> added a mismatched endif. This causes cmd_objtool to get mistakenly
> set.
>
> Relocate endif to balance the newly added -record-mcount check.
>
> Fixes: 96f60dfa5819 ("trace: Use -mcount-record for dynamic ftrace")
> Signed-off-by: Greg Thelen <gthelen@xxxxxxxxxx>

Looks good thanks.

Acked-by: Andi Kleen <ak@xxxxxxxxxxxxxxx>

-Andi