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 CBDE1D44D6A for ; Wed, 6 Nov 2024 13:50:41 +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-Transfer-Encoding: Content-Type:MIME-Version:References:In-Reply-To:Subject:Cc:To:From: Message-ID:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=tlgJsnAnvOSQAv2o1TjW3T2/qusvj06B7TIapVcJ0rc=; b=S5rZr23HwzL6mS5ynEYNfDFcZw 0K+aGNcYlfFMkLDsikQLjtiGcqbmvDjsJE0jfejycHKP3aJlbsWdKjCA3lL/0c1syLIGXlWSpgCbf hDhFO7M3njSXTMwfE6a57uWGOvvX2C0s7WPIXeUwFIEMXTTimY7HccJ6XJOScTjjItKzTjiLX0r0g RUbRZpq66+5oqBUBw+FPVHTPFdqr5EsF4oyXP+OGJXPD/xqKxPie1anrikrLv9mNkj6yFxIpW2VpK 5f1FmGNwmRBqJPYWg9Xcxh986XHo75uNV6U33MOv1UbMPFzIu/i6XFY8g/nLcNjSOMr+YFkBJRLCC sdFNKX8g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t8gQW-00000003UY5-45lj; Wed, 06 Nov 2024 13:50:28 +0000 Received: from nyc.source.kernel.org ([2604:1380:45d1:ec00::3]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1t8g3p-00000003PpR-41ua for linux-arm-kernel@lists.infradead.org; Wed, 06 Nov 2024 13:27:03 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by nyc.source.kernel.org (Postfix) with ESMTP id 2885BA43CD9; Wed, 6 Nov 2024 13:25:06 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id BE1B1C4CECD; Wed, 6 Nov 2024 13:27:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1730899620; bh=YsdgsNLn3Pt94qYHQp37MqqUXQnx1YcTHDZKk+avfN0=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=crHz1lLzZMb7i/uvSwBbyrGkEUN8bOUEZEnF9f8JAwv8cUCLPO+mWcZsYhF7bHFx/ JQGwm2lV7eH7Of+HF0L5wXqRLQamQIITuhJMsiJH6aLlD0FpEJ8IGRAfhBmS5Y0CGC 6lL4zO2okkyEXHXDeXcc2k8qekasXR64FcOHbsTTATUzZN1j9Klhu+2w1AjkUE9LmB ZJ3spHnXLMLVVIgzAaAepFVtxWcsaysfPq4Wk6O8+t+PKyHSHPkLzAUtnozE/QEtYw vAN/FkdjSxaCIuOZ5LEaPRMSMKfw52Fr+q58DGdNuieJhtdAlw7NdZI4s/mRAiyNTa 1rVbDV1ANxMkQ== 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 1t8g3m-00ANoh-KU; Wed, 06 Nov 2024 13:26:58 +0000 Date: Wed, 06 Nov 2024 13:26:57 +0000 Message-ID: <86a5ec1oim.wl-maz@kernel.org> From: Marc Zyngier To: Jing Zhang Cc: KVM , KVMARM , ARMLinux , Oliver Upton , Joey Gouly , Zenghui Yu , Suzuki K Poulose , Kunkun Jiang , Paolo Bonzini , Andre Przywara , Colton Lewis , Raghavendra Rao Ananta , Shusen Li , Eric Auger Subject: Re: [PATCH v3 4/4] KVM: selftests: aarch64: Test VGIC ITS tables save/restore In-Reply-To: <20241106083035.2813799-5-jingzhangos@google.com> References: <20241106083035.2813799-1-jingzhangos@google.com> <20241106083035.2813799-5-jingzhangos@google.com> 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.4 (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 Content-Transfer-Encoding: quoted-printable X-SA-Exim-Connect-IP: 185.219.108.64 X-SA-Exim-Rcpt-To: jingzhangos@google.com, kvm@vger.kernel.org, kvmarm@lists.linux.dev, linux-arm-kernel@lists.infradead.org, oupton@google.com, joey.gouly@arm.com, yuzenghui@huawei.com, suzuki.poulose@arm.com, jiangkunkun@huawei.com, pbonzini@redhat.com, andre.przywara@arm.com, coltonlewis@google.com, rananta@google.com, lishusen2@huawei.com, eauger@redhat.com 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-20241106_052702_156915_64EF9779 X-CRM114-Status: GOOD ( 18.99 ) 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 [Adding Eric to the list, since he worked a lot on the save/restore code] On Wed, 06 Nov 2024 08:30:35 +0000, Jing Zhang wrote: >=20 > Add a selftest to verify the correctness of the VGIC ITS mappings after > the save/restore operations (KVM_DEV_ARM_ITS_SAVE_TABLES / > KVM_DEV_ARM_ITS_RESTORE_TABLES). What are you checking? The saved data? The restored data? > Also calculate the time spending on save/restore operations. Is that really relevant? I don't think performance matters at this stage, if we can't even have reliable data. > This test uses some corner cases to capture the save/restore bugs. It Which corner cases? > will be used to verify the future incoming changes for the VGIC ITS > tables save/restore. >=20 > To capture the "Invalid argument (-22)" error, run the test without any > option. To capture the wrong/lost mappings, run the test with '-s' > option. > Since the VGIC ITS save/restore bug is caused by orphaned DTE/ITE > entries, if we run the test with '-c' option whih clears the tables > before the save operation, the test will complete successfully. I'm sorry, but this description is meaningless, as you need to know what is the bug that has been fixed. Also, how is someone supposed to run this thing? Without options? With options? With any combination of options? =46rom what I understand, the various options are designed to help debugging a broken vgic implementation. So please document what the options do rather than an bug that is supposed to be already fixed. Thanks, M. --=20 Without deviation from the norm, progress is not possible.