From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 8E7D223716B for ; Mon, 17 Mar 2025 10:39:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742207991; cv=none; b=fVAsvDnNCu5dTdddN3owkfhA8qFeAKNNLSHT9ot/v64GHMhEDNGjy3YepD1cwXN4aHyrtewrg/UjGf9ju7PZDHjaT/7X6G4WmEfb5TGMhsh1ELiUVUpBnV6Fhr+VW6tG6XtAeUlQrwUsthRRPjJ3zUggEiJVzAdoRFEgTWNCjs0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742207991; c=relaxed/simple; bh=XyCaRK2xbCDG6RvfZBAb68Um57+U14d4+B47Thpncvo=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=VjO5T+OzW6SQorpoGk56dH4RHaoRSML1Kef5gVfpyFhRE30PATuJK4Bm4WHEJbXTzx/GebrQxuiwZZsismJTKuN1YVP4YfHQed1nKveMf+IXKqB/3h1864loZMqVgyhW77V7NHacK8RxJLotOAIDRh/zkuTs/304dDj2gdndEnw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com 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 CD43413D5; Mon, 17 Mar 2025 03:39:57 -0700 (PDT) Received: from raptor (usa-sjc-mx-foss1.foss.arm.com [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 88C433F63F; Mon, 17 Mar 2025 03:39:47 -0700 (PDT) Date: Mon, 17 Mar 2025 10:39:44 +0000 From: Alexandru Elisei To: Oliver Upton Cc: kvmarm@lists.linux.dev, kvm@vger.kernel.org, Will Deacon , Julien Thierry , andre.przywara@arm.com, maz@kernel.org, jean-philippe@linaro.org Subject: Re: [RFC kvmtool 0/9] arm: Drop support for 32-bit kvmtool Message-ID: References: <20250314222516.1302429-1-oliver.upton@linux.dev> Precedence: bulk X-Mailing-List: kvm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250314222516.1302429-1-oliver.upton@linux.dev> Hi Oliver, I'm CC'ing Marc and Andre (Andre is the only person that I know for a fact that ran 32 bit kvmtool). Haven't looked at the patches in detail, but as someone who ocassionally contributes to kvmtool, I have to say it's always nice to have fewer things to worry about. I, for one, never used 32 bit kvmtool, I just compiled for 32 bit arm to make sure it doesn't break. If nobody objects to dropping support for 32 bit kvmtool, I'm planning to review the patches. Thanks, Alex On Fri, Mar 14, 2025 at 03:25:07PM -0700, Oliver Upton wrote: > The last stable kernel to support 32-bit KVM/arm is 5.4, which is on > track for EOL at the end of this year. Considering this, and the fact > that 32-bit KVM never saw much usage in the first place, it is probably > time to toss out the coprolite. > > Of course, this has no effect on the support for 32-bit guests on 64-bit > KVM. > > Oliver Upton (9): > Drop support for 32-bit arm > arm64: Move arm64-only features into main directory > arm64: Combine kvm.c > arm64: Merge kvm-cpu.c > arm64: Combine kvm-config-arch.h > arm64: Move remaining kvm/* headers > arm64: Move asm headers > arm64: Rename top-level directory > arm64: Get rid of the 'arm-common' include directory > > INSTALL | 9 +- > Makefile | 40 +-- > arm/aarch32/arm-cpu.c | 50 --- > arm/aarch32/include/asm/kernel.h | 8 - > arm/aarch32/include/asm/kvm.h | 311 ------------------ > arm/aarch32/include/kvm/barrier.h | 10 - > arm/aarch32/include/kvm/fdt-arch.h | 6 - > arm/aarch32/include/kvm/kvm-arch.h | 18 - > arm/aarch32/include/kvm/kvm-config-arch.h | 8 - > arm/aarch32/include/kvm/kvm-cpu-arch.h | 24 -- > arm/aarch32/kvm-cpu.c | 132 -------- > arm/aarch32/kvm.c | 14 - > arm/aarch64/include/kvm/fdt-arch.h | 6 - > arm/aarch64/include/kvm/kvm-arch.h | 22 -- > arm/aarch64/include/kvm/kvm-config-arch.h | 29 -- > arm/aarch64/include/kvm/kvm-cpu-arch.h | 19 -- > arm/aarch64/kvm.c | 212 ------------ > arm/kvm-cpu.c | 153 --------- > {arm/aarch64 => arm64}/arm-cpu.c | 4 +- > {arm => arm64}/fdt.c | 4 +- > {arm => arm64}/gic.c | 2 +- > {arm => arm64}/gicv2m.c | 2 +- > {arm/aarch64 => arm64}/include/asm/image.h | 0 > {arm/aarch64 => arm64}/include/asm/kernel.h | 0 > {arm/aarch64 => arm64}/include/asm/kvm.h | 0 > {arm/aarch64 => arm64}/include/asm/pmu.h | 0 > .../include/asm/sve_context.h | 0 > .../arm-common => arm64/include}/gic.h | 0 > {arm/aarch64 => arm64}/include/kvm/barrier.h | 0 > .../include/kvm}/fdt-arch.h | 0 > .../include/kvm}/kvm-arch.h | 6 +- > .../include/kvm}/kvm-config-arch.h | 24 +- > .../include/kvm}/kvm-cpu-arch.h | 10 +- > .../arm-common => arm64/include}/pci.h | 0 > .../arm-common => arm64/include}/timer.h | 0 > {arm => arm64}/ioport.c | 0 > {arm/aarch64 => arm64}/kvm-cpu.c | 289 ++++++++++++---- > {arm => arm64}/kvm.c | 212 +++++++++++- > {arm => arm64}/pci.c | 4 +- > {arm/aarch64 => arm64}/pmu.c | 2 +- > {arm/aarch64 => arm64}/pvtime.c | 0 > {arm => arm64}/timer.c | 4 +- > 42 files changed, 495 insertions(+), 1139 deletions(-) > delete mode 100644 arm/aarch32/arm-cpu.c > delete mode 100644 arm/aarch32/include/asm/kernel.h > delete mode 100644 arm/aarch32/include/asm/kvm.h > delete mode 100644 arm/aarch32/include/kvm/barrier.h > delete mode 100644 arm/aarch32/include/kvm/fdt-arch.h > delete mode 100644 arm/aarch32/include/kvm/kvm-arch.h > delete mode 100644 arm/aarch32/include/kvm/kvm-config-arch.h > delete mode 100644 arm/aarch32/include/kvm/kvm-cpu-arch.h > delete mode 100644 arm/aarch32/kvm-cpu.c > delete mode 100644 arm/aarch32/kvm.c > delete mode 100644 arm/aarch64/include/kvm/fdt-arch.h > delete mode 100644 arm/aarch64/include/kvm/kvm-arch.h > delete mode 100644 arm/aarch64/include/kvm/kvm-config-arch.h > delete mode 100644 arm/aarch64/include/kvm/kvm-cpu-arch.h > delete mode 100644 arm/aarch64/kvm.c > delete mode 100644 arm/kvm-cpu.c > rename {arm/aarch64 => arm64}/arm-cpu.c (96%) > rename {arm => arm64}/fdt.c (99%) > rename {arm => arm64}/gic.c (99%) > rename {arm => arm64}/gicv2m.c (99%) > rename {arm/aarch64 => arm64}/include/asm/image.h (100%) > rename {arm/aarch64 => arm64}/include/asm/kernel.h (100%) > rename {arm/aarch64 => arm64}/include/asm/kvm.h (100%) > rename {arm/aarch64 => arm64}/include/asm/pmu.h (100%) > rename {arm/aarch64 => arm64}/include/asm/sve_context.h (100%) > rename {arm/include/arm-common => arm64/include}/gic.h (100%) > rename {arm/aarch64 => arm64}/include/kvm/barrier.h (100%) > rename {arm/include/arm-common => arm64/include/kvm}/fdt-arch.h (100%) > rename {arm/include/arm-common => arm64/include/kvm}/kvm-arch.h (97%) > rename {arm/include/arm-common => arm64/include/kvm}/kvm-config-arch.h (54%) > rename {arm/include/arm-common => arm64/include/kvm}/kvm-cpu-arch.h (82%) > rename {arm/include/arm-common => arm64/include}/pci.h (100%) > rename {arm/include/arm-common => arm64/include}/timer.h (100%) > rename {arm => arm64}/ioport.c (100%) > rename {arm/aarch64 => arm64}/kvm-cpu.c (70%) > rename {arm => arm64}/kvm.c (59%) > rename {arm => arm64}/pci.c (98%) > rename {arm/aarch64 => arm64}/pmu.c (99%) > rename {arm/aarch64 => arm64}/pvtime.c (100%) > rename {arm => arm64}/timer.c (95%) > > > base-commit: e48563f5c4a48fe6a6bc2a98a9a7c84a10f043be > -- > 2.39.5 > >