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 B0041C25B08 for ; Wed, 17 Aug 2022 13:44:05 +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-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From: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=f2ggxC779GEidWPZaZjGk9B8X+DdOQR1RVmrOb+IoYw=; b=VyQXFaFUzpMuJyTyj7UMwFJxcW NJJBS8r6rmDxCcp83O200eVJXA2PAJZlZ2VQ1BM9uKh/tM/SHdMrIbuWztGkASeffeHklWcKIHFAF xDSN7YWLeL5vYUPIn2RapjiXc6+JHZtyyue0mar321BqpL46LvlUxt8jQHRojmswbw8gQN3BuHCU8 L8H+LgxuC0YnISiO+uy+LZ9tnbII1A412o9uFrlVkvrys7QENx/PAh+mga3WmWf5OlQ9dcF15+GO0 D5ue3OCZ8GbhipybGZzAhJkHXrOgAqou2qAQnNWgAL8acudwZ5lUNJhv3cVEEdtlry1m+JDb6km9R hVvdPRCw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oOJJr-002Tpz-69; Wed, 17 Aug 2022 13:42:51 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oOJJn-002ToS-UR for linux-arm-kernel@lists.infradead.org; Wed, 17 Aug 2022 13:42:49 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 4F5CF61369; Wed, 17 Aug 2022 13:42:47 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D122AC433C1; Wed, 17 Aug 2022 13:42:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1660743766; bh=d6Ro+H9k2fE0b9BT9o/yke+QEo0HDikhsAxMCLLhZs0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=ljX2QUQS2PYBowrm7nNRFZ2BkUsHqbLzInrWD6OQlgOPc37d6EYZGO0ImLJuNhXkZ goCH/i7pTeqRa9dsvI3tMh4NjrrZhE49ZAXc7QZAEbb3bHft1w3U5OOoDZlSvWLltL jiALW7xF7Kka9ivSPpemJVwA09XJDUFwsuP74lRu+fBiFRGGGduauM4xkj/2BnfY5F 5qx0D3drqVp6K+IzUBE69LGnax+MEsPaISj8v8XSQITmIS0u/tZLNxUeSPD9WZQ5M1 JvvQ4uUbkDendvZ94WZ03jBOmLPi67EHt/BQQPp79JD6D07HvKH2lNBl58n/C6Nyd4 yZ3tLfjR/1GUA== Date: Wed, 17 Aug 2022 14:42:40 +0100 From: Mark Brown To: Cristian Marussi Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, sudeep.holla@arm.com, james.quinlan@broadcom.com, Jonathan.Cameron@huawei.com, f.fainelli@gmail.com, etienne.carriere@linaro.org, vincent.guittot@linaro.org, souvik.chakravarty@arm.com, wleavitt@marvell.com, peter.hilber@opensynergy.com, nicola.mazzucato@arm.com, tarek.el-sherbiny@arm.com Subject: Re: [RFC PATCH 5/6] firmware: arm_scmi: Add raw transmission support Message-ID: References: <20220816072450.3120959-1-cristian.marussi@arm.com> <20220816072450.3120959-6-cristian.marussi@arm.com> MIME-Version: 1.0 In-Reply-To: X-Cookie: Use extra care when cleaning on stairs. X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220817_064248_092874_BE1E7D50 X-CRM114-Status: GOOD ( 24.81 ) 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: , Content-Type: multipart/mixed; boundary="===============0250762729142137950==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============0250762729142137950== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="QELZVTyl/Moevx8A" Content-Disposition: inline --QELZVTyl/Moevx8A Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Wed, Aug 17, 2022 at 09:38:57AM +0100, Cristian Marussi wrote: > ...moreover at the end the whole disable and go-back-to-normal really > makes little sense in a typical CI scenario where anyway the system > under test is most probably rebooted between runs of different test > suites, so we really do not care about any weird final state probably. > I, nonetheless, posted this RFC with this such support, at first to have > some general feedback, BUT also because I'm still anyway wondering if it > would not be worth to keep at least the capability to only enable it at > run-time (dropping the disable-back-to-normal feat), because this would > enable to build an image which includes this SCMI Raw support, which is > default disabled, and that can at will enabled at runtime only on selected > runs, so that this same test-image could still be used in a number of > different CI test-runs (keeping raw mode disabled and silent) but also then > used for a specific SCMI testing run that would eventually enable it. The enable usecase does indeed make more sense, though I'd still worry about other code having problems with the SCMI support getting hotplugged out from underneath it since that isn't a thing that happens in practical systems. For example the archrandom code is going to get confused since it probes once to see if SMCCC TRNG support is available and if it's present sets a flag which it assumes will be true for the rest of system runtime. I don't entirely know how the image build costs play off here for the people who'd actually be running these tests, but my instinct is that the extra kernel build isn't really much in the grand scheme of things compared to shaking out the consequences of a runtime switch and the costs of actually running the tests. --QELZVTyl/Moevx8A Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmL88E8ACgkQJNaLcl1U h9CeJgf+Ikq6PaL+60Wk0LZxkHbDz6nkjhCcCTvXPgUzqJkDAF3ogJ62Gr2ciwfz 1k1pRouQrKifP96mZNH5RnqEXYxlK8EY78KsokTatrZJtkT6/03MH0iOv7WkqqJz 3Rw+TiJH9WKCVph2aIo0XOBu+QBJRenhMrrXO8InnKgXvyVac82DYytYeGXzRWDf 2DAYEHt+mFenJkv4nI7ei55L7ODvpe/zP6WyhUTKuSk+DqtvzhmCSRzmQ9ebUOD7 k3uBUfhcn9tuYrtzt+YYFfeaKkGbnSY/Pm42U/Yl0FBdbMUyif6wJunAWw5A2PZZ ruW/+/d//2AQqo2qT14MCQBm5mGG/A== =92om -----END PGP SIGNATURE----- --QELZVTyl/Moevx8A-- --===============0250762729142137950== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============0250762729142137950==--