From patchwork Thu Jan 27 16:20:23 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alexandru Elisei X-Patchwork-Id: 12727026 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id F25DDC433F5 for ; Thu, 27 Jan 2022 16:21:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:To :From:Reply-To:Cc:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=u6KJ5rBkf2Ti+jtEoA8yuWSvf1I3DQ9aztIadnhZHvw=; b=VbmZmTLnNqDYmO EKaXB9BdLqRrm2HbBxweNrAzpvSZvyLdSkNWAP4SB1dbTsba6FSSggfHl3S3tDOmLoaITzAbRBeU/ ennnIQbZH8ZCGwNjQFA7GDb3kDSxbFZ9E5ryQLB66A1zvVznSHhtOxq80rDxAY98sWG+vW14x/vcZ 0imesIS9sDnWK/HsYc0f+TAV8OZb7Ub/5fwpfUDljwu1i5yWt+ti7nH5hmbH9KRXNNpadcoNQkKEP xZVxBldnHZipdMrl9wCeoDBayKtq1VzS6jHYYJkJc3e2Kk9OHSCKLW+YkKl75uCD0RmVvzl2GT/Ye 2aICNtlWyN20PVD0p57w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nD7Vh-00GRmK-OF; Thu, 27 Jan 2022 16:20:34 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nD7Va-00GRjk-3Q for linux-arm-kernel@lists.infradead.org; Thu, 27 Jan 2022 16:20:28 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 2554D1063; Thu, 27 Jan 2022 08:20:25 -0800 (PST) Received: from monolith.localdoman (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 9DA653F766; Thu, 27 Jan 2022 08:20:22 -0800 (PST) From: Alexandru Elisei To: will@kernel.org, julien.thierry.kdev@gmail.com, linux-arm-kernel@lists.infradead.org, kvmarm@lists.cs.columbia.edu, maz@kernel.org, james.morse@arm.com, suzuki.poulose@arm.com, mark.rutland@arm.com, andre.przywara@arm.com Subject: [PATCH v2 kvmtool 00/10] arm64: Improve PMU support on heterogeneous systems Date: Thu, 27 Jan 2022 16:20:23 +0000 Message-Id: <20220127162033.54290-1-alexandru.elisei@arm.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220127_082026_272793_141AAFE1 X-CRM114-Status: GOOD ( 23.44 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org The series can be found at [1], and the Linux patches that this series is based on at [2]. The series adds support for the KVM_ARM_VCPU_PMU_V3_SET_PMU PMU attribute, which allows userspace to set a PMU for a VCPU. This PMU is used by KVM when creating perf events to emulate the guest PMU. Without settings this attribute, the PMU used when creating events is the first one that successfully probed when booting, but this is unreliable as the probe order can change (if the order of the PMUs is changed in the DTB or if asynchronous driver probing is enabled on the host's command line), and furthermore it requires the user to have intimate knowledge of how the PMU was chosen in order to pin the VM on the correct physical CPUs. With KVM_ARM_VCPU_PMU_V3_SET_PMU, the user is still expected to pin the VCPUs on a particular set of CPUs, but now it can be any CPUs as long as they share the same PMU. The set does not depend anymore on the driver probe order and all that is necessary for the user to know is which CPUs are the little core and which are the big cores, in a big.little configuration, which I believe is more reasonable. Patches #1-#2 are fixes and can be taken independently of this series. Patches #3-#6 move the PMU code to aarch64, where it belongs, because the PMU has never been supported on KVM for arm. This also paves the way for pulling in the KVM_ARM_VCPU_PMU_V3_SET_PMU attribute, which was not defined for KVM for arm (when KVM supported arm). This also can be merged right now, independently of the other patches. Patch #7 adds the cpumask_* functions which are necessary for subsequent patches. Patch #9 adds basic support for KVM_ARM_VCPU_PMU_V3_SET_PMU; the user is still expected to use taskset to pin the entire VM to the correct CPUs. Patch #10 adds --vcpu-affinity command line argument to pin VCPUs to the correct CPUs without pinning the rest of the kvmtool threads. Changes since v1: * Patch #2 ("bitops.h: Include wordsize.h to provide the __WORDSIZE define") is new. * Added for_each_cpu(), cpumask_and() and cpumask_subset() functions and all the cpumask_* functions are added in one patch. * Bumped NR_CPUS fro arm64 to 4096 to match the Linux Kconfig option. * Reworked the way kvmtool specific header files were included to use quotes to clearly differentiate them from the system level headers and to keep the style consistent with the current code (for example, #include is now "linux/bitops.h"). * Patch #10 ("arm64: Add --vcpu-affinity command line argument") is new. [1] https://gitlab.arm.com/linux-arm/kvmtool-ae/-/tree/pmu-big-little-fix-v2 [2] https://gitlab.arm.com/linux-arm/linux-ae/-/tree/pmu-big-little-fix-v4 Alexandru Elisei (10): linux/err.h: Add missing stdbool.h include bitops.h: Include wordsize.h to provide the __WORDSIZE define arm: Move arch specific VCPU features to the arch specific function arm: Get rid of the ARM_VCPU_FEATURE_FLAGS() macro arm: Make the PMUv3 emulation code arm64 specific arm64: Rework set_pmu_attr() Add cpumask functions update_headers.sh: Sync headers with Linux v5.17-rc1 + SET_PMU attribute arm64: Add support for KVM_ARM_VCPU_PMU_V3_SET_PMU arm64: Add --vcpu-affinity command line argument Makefile | 6 +- arm/aarch32/include/asm/kernel.h | 8 + arm/aarch32/include/kvm/kvm-cpu-arch.h | 4 - arm/aarch64/arm-cpu.c | 3 +- arm/aarch64/include/asm/kernel.h | 8 + arm/aarch64/include/asm/kvm.h | 4 + .../arm-common => aarch64/include/asm}/pmu.h | 0 arm/aarch64/include/kvm/kvm-config-arch.h | 5 + arm/aarch64/include/kvm/kvm-cpu-arch.h | 6 - arm/aarch64/kvm-cpu.c | 21 ++ arm/aarch64/kvm.c | 32 +++ arm/aarch64/pmu.c | 231 ++++++++++++++++ arm/include/arm-common/kvm-arch.h | 7 + arm/include/arm-common/kvm-config-arch.h | 1 + arm/kvm-cpu.c | 14 +- arm/pmu.c | 76 ------ include/linux/bitmap.h | 71 +++++ include/linux/bitops.h | 4 + include/linux/bits.h | 8 + include/linux/cpumask.h | 67 +++++ include/linux/err.h | 2 + include/linux/find.h | 30 ++ include/linux/kernel.h | 6 + include/linux/kvm.h | 16 ++ mips/include/asm/kernel.h | 8 + powerpc/include/asm/kernel.h | 8 + util/bitmap.c | 256 ++++++++++++++++++ util/find.c | 40 +++ x86/include/asm/kernel.h | 8 + x86/include/asm/kvm.h | 16 +- 30 files changed, 867 insertions(+), 99 deletions(-) create mode 100644 arm/aarch32/include/asm/kernel.h create mode 100644 arm/aarch64/include/asm/kernel.h rename arm/{include/arm-common => aarch64/include/asm}/pmu.h (100%) create mode 100644 arm/aarch64/pmu.c delete mode 100644 arm/pmu.c create mode 100644 include/linux/bitmap.h create mode 100644 include/linux/bits.h create mode 100644 include/linux/cpumask.h create mode 100644 include/linux/find.h create mode 100644 mips/include/asm/kernel.h create mode 100644 powerpc/include/asm/kernel.h create mode 100644 util/bitmap.c create mode 100644 util/find.c create mode 100644 x86/include/asm/kernel.h