Re: [PATCH 3/4] kvm/arm64: simplify CC instrumentation opt-out

From: Christoffer Dall
Date: Tue Oct 17 2017 - 10:50:41 EST


On Mon, Oct 16, 2017 at 02:24:39PM +0100, Mark Rutland wrote:
> Now that we can opt out of all instrumentation with a single option,
> make use of this in the KVM hyp code. Hopefully, this will also avoid
> fragility as new options are introduced.
>
> Signed-off-by: Mark Rutland <mark.rutland@xxxxxxx>
> Cc: Catalin Marinas <catalin.marinas@xxxxxxx>
> Cc: Christoffer Dall <christoffer.dall@xxxxxxxxxx>
> Cc: Marc Zyngier <mark.zyngier@xxxxxxx>
> Cc: Will Deacon <will.deacon@xxxxxxx>

Acked-by: Christoffer Dall <christoffer.dall@xxxxxxxxxx>

> ---
> arch/arm64/kvm/hyp/Makefile | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/arch/arm64/kvm/hyp/Makefile b/arch/arm64/kvm/hyp/Makefile
> index 14c4e3b14bcb..a8a212eb66f1 100644
> --- a/arch/arm64/kvm/hyp/Makefile
> +++ b/arch/arm64/kvm/hyp/Makefile
> @@ -22,7 +22,4 @@ obj-$(CONFIG_KVM_ARM_HOST) += s2-setup.o
> # KVM code is run at a different exception code with a different map, so
> # compiler instrumentation that inserts callbacks or checks into the code may
> # cause crashes. Just disable it.
> -GCOV_PROFILE := n
> -KASAN_SANITIZE := n
> -UBSAN_SANITIZE := n
> -KCOV_INSTRUMENT := n
> +CC_INSTRUMENT := n
> --
> 2.11.0
>