From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 232F98BFB; Tue, 28 Mar 2023 15:35:32 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B8BACC433D2; Tue, 28 Mar 2023 15:35:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1680017732; bh=zobdeVjw+10D3vo/WKeJd9nRbD/pwO3V+VoPib6V4LM=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=eU1Hiwmbza40XC3WSiQrHngK6b+zPDP6obvH/c/mqSKYmHkXLoL/gWEKAtD+xID8f bkzXCMyNcxvgJDs2hBw+w8QpjPT0Id4ySK3EnBr8z9oQQgkXK9wxUPjR19VXQ6RqzT +OcpF/AMQ4wmz1skav3+gr+M7hdnNORESNRLrSCjja/XtVuJybETq/Ar/BsRXy8dsY 4+CGVIUJd3V/W4Tr7XKacwv/49/LYvzYG1z8jWpKcp8UradccRWqf6FChSWZyQ+7bX gAASIP6o+qt0VSRSxz1X0w/+z4HomB2Z8M5OeZc9MYnCwWx0Sge7/gCI8n4s+V4TzW UzQBhiffJz4LQ== Received: from sofa.misterjones.org ([185.219.108.64] helo=goblin-girl.misterjones.org) by disco-boy.misterjones.org with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.95) (envelope-from ) id 1phBMA-003okT-Dz; Tue, 28 Mar 2023 16:35:30 +0100 Date: Tue, 28 Mar 2023 16:35:29 +0100 Message-ID: <86o7ocx51a.wl-maz@kernel.org> From: Marc Zyngier To: "Arnd Bergmann" , "Will Deacon" Cc: "kernel test robot" , llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev, "Linux Memory Management List" , "Zaid Al-Bassam" Subject: Re: [linux-next:master 6577/7309] drivers/perf/arm_pmuv3.c:44:2: error: use of undeclared identifier 'PERF_MAP_ALL_UNSUPPORTED' In-Reply-To: References: <202303281539.zzI4vpw1-lkp@intel.com> <20230328122138.GA1275@willie-the-truck> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/28.2 (aarch64-unknown-linux-gnu) MULE/6.0 (HANACHIRUSATO) Precedence: bulk X-Mailing-List: llvm@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII X-SA-Exim-Connect-IP: 185.219.108.64 X-SA-Exim-Rcpt-To: arnd@arndb.de, will@kernel.org, lkp@intel.com, llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev, linux-mm@kvack.org, zalbassam@google.com X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false On Tue, 28 Mar 2023 16:29:07 +0100, "Arnd Bergmann" wrote: > > On Tue, Mar 28, 2023, at 14:21, Will Deacon wrote: > > On Tue, Mar 28, 2023 at 04:01:49PM +0800, kernel test robot wrote: > > > I think the Kconfig warning is relevant here: > > > >> Kconfig warnings: (for reference only) > >> WARNING: unmet direct dependencies detected for ARM_PMUV3 > >> Depends on [n]: PERF_EVENTS [=y] && HW_PERF_EVENTS [=n] && (ARM [=y] && CPU_V7 [=y] || ARM64) > >> Selected by [y]: > >> - ARCH_VIRT [=y] && ARCH_MULTI_V7 [=y] && PERF_EVENTS [=y] > > > > Since we're selecting an option which has its own dependencies. I think > > the easiest bet is to invert the linkage so that PMUV3 has a dependency > > on ARCH_VIRT for 32-bit Arm (diff below). > > > > Marc, Zaid -- what do you think? > > I think dropping the 'select' is best, as this is still a user-visible > option. How about this instead: > > --- a/drivers/perf/Kconfig > +++ b/drivers/perf/Kconfig > @@ -103,7 +103,7 @@ config ARM_SMMU_V3_PMU > config ARM_PMUV3 > depends on HW_PERF_EVENTS && ((ARM && CPU_V7) || ARM64) > bool "ARM PMUv3 support" if !ARM64 > - default ARM64 > + default ARM64 || ARCH_VIRT > help > Say y if you want to use the ARM performance monitor unit (PMU) > version 3. The PMUv3 is the CPU performance monitors on ARMv8 > > This way, it's still enabled by default for most people, but > there is no dependency problem and it's also possible to > turn it off. [removing my old self from the Cc list] Either way work for me. Or even drop that patch altogether and let people select PMUv3 if they really want it. This is extremely niche anyway, and most people won't care. Thanks, M. -- Without deviation from the norm, progress is not possible.