Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757161AbcCaO30 (ORCPT ); Thu, 31 Mar 2016 10:29:26 -0400 Received: from foss.arm.com ([217.140.101.70]:55028 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751654AbcCaO3Z (ORCPT ); Thu, 31 Mar 2016 10:29:25 -0400 Date: Thu, 31 Mar 2016 15:29:08 +0100 From: Mark Rutland To: Alexander Potapenko Cc: dvyukov@google.com, catalin.marinas@arm.com, quentin.casasnovas@oracle.com, will.deacon@arm.com, kcc@google.com, akpm@linux-foundation.org, syzkaller@googlegroups.com, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, ard.biesheuvel@linaro.org, marc.zyngier@arm.com, christoffer.dall@linaro.org Subject: Re: [PATCH v1] arm64: allow building with kcov coverage on ARM64 Message-ID: <20160331142908.GG26532@leverpostej> References: <57cb1b66d85b85eadea28ef3304a62b1327ded45.1459432254.git.glider@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <57cb1b66d85b85eadea28ef3304a62b1327ded45.1459432254.git.glider@google.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1844 Lines: 57 Hi, On Thu, Mar 31, 2016 at 03:54:45PM +0200, Alexander Potapenko wrote: > Add ARCH_HAS_KCOV to ARM64 config. Disable instrumentation of > arch/arm64/lib/delay.c Why do we disable instrumentation of delay.c? What exactly does kcov instrumentation imply? Does it require certain data to be mapped or certain functions to be callable while instrumented functions are called? We have some C code that is run outside of the normal kernel context (e.g. EFI stub, KVM hyp code), and I suspect it may be necessary to disable instrumentation for those also. Thanks, Mark. > Signed-off-by: Alexander Potapenko > --- > arch/arm64/Kconfig | 1 + > arch/arm64/lib/Makefile | 3 +++ > 2 files changed, 4 insertions(+) > > diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig > index 4f43622..c52aa61 100644 > --- a/arch/arm64/Kconfig > +++ b/arch/arm64/Kconfig > @@ -7,6 +7,7 @@ config ARM64 > select ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE > select ARCH_HAS_ELF_RANDOMIZE > select ARCH_HAS_GCOV_PROFILE_ALL > + select ARCH_HAS_KCOV > select ARCH_HAS_SG_CHAIN > select ARCH_HAS_TICK_BROADCAST if GENERIC_CLOCKEVENTS_BROADCAST > select ARCH_USE_CMPXCHG_LOCKREF > diff --git a/arch/arm64/lib/Makefile b/arch/arm64/lib/Makefile > index c86b790..b407bc1 100644 > --- a/arch/arm64/lib/Makefile > +++ b/arch/arm64/lib/Makefile > @@ -1,3 +1,6 @@ > +# Produces uninteresting flaky coverage. > +KCOV_INSTRUMENT_delay.o := n > + > lib-y := bitops.o clear_user.o delay.o copy_from_user.o \ > copy_to_user.o copy_in_user.o copy_page.o \ > clear_page.o memchr.o memcpy.o memmove.o memset.o \ > -- > 2.8.0.rc3.226.g39d4020 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel >