From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 38C82C76196 for ; Tue, 28 Mar 2023 15:15:08 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 90A726B0071; Tue, 28 Mar 2023 11:15:07 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 8C47B6B0072; Tue, 28 Mar 2023 11:15:07 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 782BC6B0074; Tue, 28 Mar 2023 11:15:07 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id 65EEF6B0071 for ; Tue, 28 Mar 2023 11:15:07 -0400 (EDT) Received: from smtpin07.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id 1F90B1A0310 for ; Tue, 28 Mar 2023 15:15:07 +0000 (UTC) X-FDA: 80618655054.07.B961EC9 Received: from mail-vs1-f41.google.com (mail-vs1-f41.google.com [209.85.217.41]) by imf05.hostedemail.com (Postfix) with ESMTP id 0F55210001A for ; Tue, 28 Mar 2023 15:15:03 +0000 (UTC) Authentication-Results: imf05.hostedemail.com; dkim=pass header.d=google.com header.s=20210112 header.b=J5WsMN8a; spf=pass (imf05.hostedemail.com: domain of zalbassam@google.com designates 209.85.217.41 as permitted sender) smtp.mailfrom=zalbassam@google.com; dmarc=pass (policy=reject) header.from=google.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1680016504; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=Zogcj48cUORsp+cx8m+6WD2kysHO7bsUVfNCnma4boE=; b=x7+2RwTBTDJhEfn68Ajj0Spn+yJZmJKr4pySxx0l2gd9//4az6j+ozIFnZaTc4GSE7o2K/ ejYqLAOvwuvtwnEN9wZc0XUrywuxCxUFHJqGgjMC9D09NJfN2OTBvxJHoLgpRXM0vyF003 mxnq7NDBr9H2KOr/eneOGzhEMT4to4s= ARC-Authentication-Results: i=1; imf05.hostedemail.com; dkim=pass header.d=google.com header.s=20210112 header.b=J5WsMN8a; spf=pass (imf05.hostedemail.com: domain of zalbassam@google.com designates 209.85.217.41 as permitted sender) smtp.mailfrom=zalbassam@google.com; dmarc=pass (policy=reject) header.from=google.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1680016504; a=rsa-sha256; cv=none; b=srqiVcUAdOyfVgNbjlWqcA+uB2H69JqZw/wFTuBbC8xL8wJEkuIn9x2u82l9ss1+CVdzhQ cW4R+v868hyywvAijxafWvL0AHZi7IBbM+eJ/I29NUaRO+NJhrmJQ8tn6SsR6vE/MinjIs dxkGQYbaKAte8hiHNHuoyig3NEpZqhM= Received: by mail-vs1-f41.google.com with SMTP id i10so10655501vss.5 for ; Tue, 28 Mar 2023 08:15:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; t=1680016503; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=Zogcj48cUORsp+cx8m+6WD2kysHO7bsUVfNCnma4boE=; b=J5WsMN8aXW8oQ6ohBV6kiuUuq9TQp9a/GN0VOMsfec4YAsOfUey/eAkJwvqZdBgXSh mb8vmORY4U9gNoetWmo7kJ9yQyKTivuWi0sBO5FDWpabK3wrukmarN+t1yHnyg/yF5vN BPB1z37fK6CkvhJJq4Au9rCJFmEUMBjuVYa87A2QgWW6+RQUu0A3Tne+xLd7p7TAfwK9 05KfrOxAU4uNMicxW/aRBIEyySsyUIWP8KE8070LDgcOFw6DGKU1ycnaOLA43NutIJI3 zUCHJnUw1uw0OPwwOY/BHgepWBXqKG4Q0jsL25cWIBVb1PB5GnXh1ABBOX3wmkqCgKoA zcuA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1680016503; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=Zogcj48cUORsp+cx8m+6WD2kysHO7bsUVfNCnma4boE=; b=Gt4c63oNIqqQj576EutQEQ7sKug4yeTc6jEoJVw6Z3wzyJa02ZvKNzK4+3bnjHwR7u LW81cxepXLd//nmoL4SIROVXW3fRt5ZIheOPnEheWypQs2CKC1rsiQnOtBKPjI7IImpn 6wQaL9WP6e6kWmD3KCc/vDIxqCQ1IrMJ8OQh3htXPsxSeBI7bl917iez3LmI4ualwCtX 6b+9LVC+19tyqvM2zFJYmq6JXVd3peX4GGZ1OjJn7CqHd2LZTm+5NqHIUWwtbIXZOtNM PD4YS8oscamOZiOiZseSTkUOFcKq4hSe41I++ebOQSitzcUisiDsxmN40wk+Z4djhosU qQAA== X-Gm-Message-State: AO0yUKV6VQW1eEZxP3gBMLZsEHsyFoePoq33aPxIGi5KjK6ku9SPtoMk 1mq7yC9F+v2zmS8m204FQTkLcG5T6nHIw3BMWf/Vaw== X-Google-Smtp-Source: AK7set+faee5ptEy2k2sQCK3L2uCYQ3XJJUAvnEws3za20OTSyTXDtTPbBo9iA9TXkDVxOwhcRch50TLewCm/8cOaLM= X-Received: by 2002:a05:6102:2338:b0:421:c4a3:b607 with SMTP id b24-20020a056102233800b00421c4a3b607mr12323114vsa.3.1680016502956; Tue, 28 Mar 2023 08:15:02 -0700 (PDT) MIME-Version: 1.0 References: <202303281539.zzI4vpw1-lkp@intel.com> <20230328122138.GA1275@willie-the-truck> In-Reply-To: <20230328122138.GA1275@willie-the-truck> From: Zaid Al-Bassam Date: Tue, 28 Mar 2023 11:14:52 -0400 Message-ID: Subject: Re: [linux-next:master 6577/7309] drivers/perf/arm_pmuv3.c:44:2: error: use of undeclared identifier 'PERF_MAP_ALL_UNSUPPORTED' To: Will Deacon Cc: kernel test robot , Marc Zyngier , llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev, Linux Memory Management List Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Rspamd-Server: rspam05 X-Rspamd-Queue-Id: 0F55210001A X-Stat-Signature: 11jq8w7uiiub5def7xzrxtmi1oo9336b X-Rspam-User: X-HE-Tag: 1680016503-21657 X-HE-Meta: U2FsdGVkX1+TGE0/ws6i+228kcYP+LxLJarIdSzBlXNb9m4+UbZ1WGzoo+Uybtfj2isnz8W16s9XJp9Fs2Z6tfq+VDAv595YEWOK3UI07543NxorcuWh2QSfKJdZ8H9ioRtrgR+TkMh4YO6YUKKn7dq2kxeC2tEq0u0T+k6zBycT5AhAlJHU0JLa4tJu+7tvWA79z9i7M5alvaBfZ7s/9zakL9VL05hXBp5WlKK6oZSwaJ32mIVTW5wXImqkubc7MRjr7qWJP9qgN2CQJUjExr5+AduenM8BiQw4z126S9hBfal3ZkPDj91yAzzRDtdTlRRu0w3zup4BU8zrBDsbBsxCSNpUgxVycXoHD5vpUT2ilH2aRWT6Zmfk0tNSxejomJ8+O+wu0W1p0NMtBD9vs9QrNURcE3KDBNhM1U7sopeF70FAKEkWvLIHRLyfxPW4mV/5E/O5RE1mCSM4eYMcJvWpqjMvHj+Ftv5pNvwh0JAWJN+jXAmwvDt4NRQfzpIT+rEa8ZmEZLfZZOX/XkrQ2R9pO6IP4/xRaOL6aChqCm2Eo+MVBSwYgd/vheYzOdM7LQieSW+okGMmnbZC+LuQzBgIiL/2pcAQjzZSxlk84LcXRq30k2CTmlGSUyouvShfMyIh6bdV85duPM1euaxS8Xy38pm9q1dgXiFdo6BYg3FS+TsHsALQPvU1+XbLWvnilhzx60ldb1Bxxh2UrncDCW0Nc3I82vifdf5FGnV1ugDrK6m/9bGYwlfYPbmIQCLP/x+P4M2+b3Ow3UX5egHeCeN0iHDDH+DKI0M7gEj74uev+317GP156K4bv86gNyBxTZ65NN6NuiKIal3iyQpuHZu5efKOJFAJUZixp8pXJhrYZSoeMukHNXf9AEg5GhYhlpYjUy4xiiV6Cxam0ICOWnj6UPU3p3mut7aKCyRXDI3AQRoe9vQh87oO7XXkzoqbR5Mt/Ofrz/Whhq1B1If EuUTIZw9 CJd9kA/cl1WCT9ZHE0yxhnFYemrx5Bb0GhNtqB2LDQDAvpcyxPvTuPQXZUUU/YY/+HJX5VFL/5bMsgL0TSQDFQD0+XExfUUITuFc+4BzMLDlOwD6cHRjzg9qlVC6VxbCt+EHFbCa/5J0UQqrSDEQ2ZCGnMkYFaLZwZWmFDvK2CJ3H5cXU5ba0FmPpAl05rvb/m9+oL3w45MBo1pBHHyrq603PV4yPmDPfjsKG98IW24XCyj69xLyy8oLSNQJ/PHqCgYNfENMISim0T9iqXWpYZoyZy7+LKvjEZgVL1R2WDSoImMOKY/DtnpXVYXztmWGCiPDxu1VmmuBUXXxmNJ40yzVgc5uS8G77fspariiB18JUC19yqEi6I8CVnX5JlxoeizCQe0fkNtyyJMoitfV66w64BeH8vGFzGc8Hv64HcS448xuvd/kiPvKNdEMdjoLb7cwwS+nIVPzSuC7tGAXV0h/pmtDekhfayPCXj0LC+BTqUdez/YKXvydspcbdBIAvy+yOyWJvuCqQsnc= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000010, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: Hi Will, First, Sorry I haven't noticed that missing dependency case. Your change looks good to me, but to keep the driver/perf/Kconfig clean, I was thinking of rather expanding the check at the arch level, i.e.: diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index 6f0a2576d17f..a82765929f67 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig @@ -401,7 +401,7 @@ config ARCH_VIRT select ARM_GIC_V3 select ARM_GIC_V3_ITS if PCI select ARM_PSCI - select ARM_PMUV3 if PERF_EVENTS + select ARM_PMUV3 if PERF_EVENTS && HW_PERF_EVENTS select HAVE_ARM_ARCH_TIMER config ARCH_AIROHA What do you think? Marc, Any input? please! Thank you, Zaid On Tue, Mar 28, 2023 at 8:21=E2=80=AFAM Will Deacon wrote= : > > On Tue, Mar 28, 2023 at 04:01:49PM +0800, kernel test robot wrote: > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next= .git master > > head: a6faf7ea9fcb7267d06116d4188947f26e00e57e > > commit: 3b16f6268e660f15aed0bb97aefe87e893eb8882 [6577/7309] ARM: mach-= virt: Select PMUv3 driver by default > > config: arm-randconfig-r024-20230326 (https://download.01.org/0day-ci/a= rchive/20230328/202303281539.zzI4vpw1-lkp@intel.com/config) > > compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project 67= 409911353323ca5edf2049ef0df54132fa1ca7) > > reproduce (this is a W=3D1 build): > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/s= bin/make.cross -O ~/bin/make.cross > > chmod +x ~/bin/make.cross > > # install arm cross compiling tool for clang build > > # apt-get install binutils-arm-linux-gnueabi > > # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-ne= xt.git/commit/?id=3D3b16f6268e660f15aed0bb97aefe87e893eb8882 > > git remote add linux-next https://git.kernel.org/pub/scm/linux/= kernel/git/next/linux-next.git > > git fetch --no-tags linux-next master > > git checkout 3b16f6268e660f15aed0bb97aefe87e893eb8882 > > # save the config file > > mkdir build_dir && cp config build_dir/.config > > COMPILER_INSTALL_PATH=3D$HOME/0day COMPILER=3Dclang make.cross = W=3D1 O=3Dbuild_dir ARCH=3Darm olddefconfig > > COMPILER_INSTALL_PATH=3D$HOME/0day COMPILER=3Dclang make.cross = W=3D1 O=3Dbuild_dir ARCH=3Darm SHELL=3D/bin/bash drivers/perf/ > > > > If you fix the issue, kindly add following tag where applicable > > | Reported-by: kernel test robot > > | Link: https://lore.kernel.org/oe-kbuild-all/202303281539.zzI4vpw1-lkp= @intel.com/ > > > > All errors (new ones prefixed by >>): > > > > >> drivers/perf/arm_pmuv3.c:44:2: error: use of undeclared identifier '= PERF_MAP_ALL_UNSUPPORTED' > > PERF_MAP_ALL_UNSUPPORTED, > > ^ > > >> drivers/perf/arm_pmuv3.c:59:2: error: use of undeclared identifier '= PERF_CACHE_MAP_ALL_UNSUPPORTED' > > PERF_CACHE_MAP_ALL_UNSUPPORTED, > > ^ > > 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 [=3Dy] && HW_PERF_EVENTS [=3Dn] && (ARM = [=3Dy] && CPU_V7 [=3Dy] || ARM64) > > Selected by [y]: > > - ARCH_VIRT [=3Dy] && ARCH_MULTI_V7 [=3Dy] && PERF_EVENTS [=3Dy] > > 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? > > Cheers, > > Will > > --->8 > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig > index a5e5c0b09ff2..e24a9820e12f 100644 > --- a/arch/arm/Kconfig > +++ b/arch/arm/Kconfig > @@ -401,7 +401,6 @@ config ARCH_VIRT > select ARM_GIC_V3 > select ARM_GIC_V3_ITS if PCI > select ARM_PSCI > - select ARM_PMUV3 if PERF_EVENTS > select HAVE_ARM_ARCH_TIMER > > config ARCH_AIROHA > diff --git a/drivers/perf/Kconfig b/drivers/perf/Kconfig > index 711f82400086..cf1559581fbc 100644 > --- 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 || (ARM && 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 >