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 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 4301EC3DA49 for ; Fri, 2 Aug 2024 09:01:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type:MIME-Version: References:In-Reply-To:Subject:Cc:To:From:Message-ID:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=XNSguxgxWdOUjYn0KkoQ3Er4CqmZb5LJwqB65k7qUu4=; b=kDmyLgQ5/el16oOlDhUav7+QEd m7ura3kjpDLGr+usCzCQIhj7X0o1jkLETNh7Lt8K0p5K3XdrDmt31s+ZxKqlMGv1OkPuBZWKnUaGF GBrGTYHOET19XExwi04am6s3Y7ooXbqkDiVTh0pFm5Txm5CZQJA6yTq7AN8IeBqCwmGfWe4/46zuN jWGsD5KUvUgABhGDkUBXxCY9cL/WTVw6hBNKfPBtmn0Qqd4q+2c4ZtaxM3tXPEuzLdRGO487XYQtj /vgO3Qg5gx9ddmKE8cMSnZmaTa3d3w6mdpatJh/NdrrhbjDv4FB27Uec4GHk48N4LoCDo5EHdd7zl tVZEw+Cg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sZo9p-00000008BJ0-0iwu; Fri, 02 Aug 2024 09:01:05 +0000 Received: from sin.source.kernel.org ([2604:1380:40e1:4800::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sZo9K-00000008BBp-0tJg for linux-arm-kernel@lists.infradead.org; Fri, 02 Aug 2024 09:00:35 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id 5AEC4CE1AB8; Fri, 2 Aug 2024 09:00:32 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8B077C32782; Fri, 2 Aug 2024 09:00:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1722589231; bh=LCAUnK30xDB8ecVE/x8xdfHib0sA8+f1IZAZIFEnS9Y=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=ZXPVSNSRqBLVr+7hyXlHZISMUqc+qQUEBqA2c9n8LkZyFheGoSAFqf4H5fkQ9WsRz wMIwdiHu6fw7hLhmbxnm3tS6LAlr4sh3tBfEnalRuzzYjrHmz0ewdZ4spniy+fSnMq q1fFme4nsfnwFWUjCsSlTsO/MTkQDD/AkcCCGjUEd+Jaz+bbnlVrltehrukamTC7V4 YuPVTLyxabGQ5tHPiyS6kzQLTlvjW4U0YgIzTF1eSNKxrs1zngTn6Xf4NFy9LTkaYX zT/bm8q3hcLV7SuDCzEkaNnHMTGrE7RklBgPKUo7oNJ/i1sgegnzziua46Y8HnGVBx 83By+Lvke42RQ== 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 1sZo9E-0003B2-UD; Fri, 02 Aug 2024 10:00:29 +0100 Date: Fri, 02 Aug 2024 10:00:28 +0100 Message-ID: <86jzgz1eqb.wl-maz@kernel.org> From: Marc Zyngier To: Mark Brown Cc: Oliver Upton , James Morse , Suzuki K Poulose , Paolo Bonzini , Shuah Khan , Catalin Marinas , Joey Gouly , linux-arm-kernel@lists.infradead.org, kvmarm@lists.linux.dev, kvm@vger.kernel.org, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] KVM: selftests: arm64: Correct feature test for S1PIE in get-reg-list In-Reply-To: <811ea0eb-bc87-4ac3-8bca-27c787e43051@sirena.org.uk> References: <20240731-kvm-arm64-fix-s1pie-test-v1-1-a9253f3b7db4@kernel.org> <86le1g19aa.wl-maz@kernel.org> <811ea0eb-bc87-4ac3-8bca-27c787e43051@sirena.org.uk> 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/29.3 (aarch64-unknown-linux-gnu) MULE/6.0 (HANACHIRUSATO) 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: broonie@kernel.org, oliver.upton@linux.dev, james.morse@arm.com, suzuki.poulose@arm.com, pbonzini@redhat.com, shuah@kernel.org, catalin.marinas@arm.com, joey.gouly@arm.com, linux-arm-kernel@lists.infradead.org, kvmarm@lists.linux.dev, kvm@vger.kernel.org, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240802_020034_620781_AF0A24ED X-CRM114-Status: GOOD ( 22.93 ) 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 On Thu, 01 Aug 2024 20:14:38 +0100, Mark Brown wrote: > > [1 ] > On Thu, Aug 01, 2024 at 05:45:49PM +0100, Marc Zyngier wrote: > > > Can we please switch all this stuff to symbolic naming instead of > > magic numbers? Given how much effort is going into the "automated > > generation" thing, it is mind-boggling that the tests still rely on > > handcrafted numbers. We just end-up with two different sets of bugs. > > > At the moment, the level of confidence I have in this stuff is > > sub-zero. > > Yeah, I was wondering why this wasn't using the generated values > especially given that the generated headers are available to tools - I > wasn't sure if this was a deliberate decision to cross check the data > entry or something. We've lost that battle a long time ago, given the numbers of bugs the sysreg file has had. The real reason is that the ABI reports the encoding, and that it is rather easy to just dump stuff back into the test using the script described in the very first commit for the test. Also, the test predates the generated stuff by some margin. > I'd certainly be happy to convert, though that does > seem a bit invasive for a fix. Not for a point fix, for sure. And if you do, make sure it is entirely scripted. Thanks, M. -- Without deviation from the norm, progress is not possible.