Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 21 Jul 2021 12:16:56 +0100
From:   Mark Brown <broonie@kernel.org>
To:     Andre Przywara <andre.przywara@arm.com>
Cc:     Benjamin Herrenschmidt <benh@kernel.crashing.org>,
        Ard Biesheuvel <ardb@kernel.org>,
        Matt Mackall <mpm@selenic.com>,
        Herbert Xu <herbert@gondor.apana.org.au>,
        Linux Crypto Mailing List <linux-crypto@vger.kernel.org>,
        Linux ARM <linux-arm-kernel@lists.infradead.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Will Deacon <will@kernel.org>, Ali Saidi <alisaidi@amazon.com>,
        Jon Nettleton <jon@solid-run.com>
Subject: Re: [PATCH v2] hwrng: Add Arm SMCCC TRNG based driver
Message-ID: <20210721111656.GA4259@sirena.org.uk>
References: <20210720152158.31804-1-andre.przywara@arm.com>
 <CAMj1kXEW7DT3P3FuV+poFykf6wwm4FTJuV6emGSWabCp7UZX9A@mail.gmail.com>
 <20210720171631.071f84f5@slackpad.fritz.box>
 <e494866f38e9dcd2834971d3867244fb1d7e6ceb.camel@kernel.crashing.org>
 <20210721001803.303dfba1@slackpad.fritz.box>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="Q68bSM7Ycu6FN28Q"
Content-Disposition: inline
In-Reply-To: <20210721001803.303dfba1@slackpad.fritz.box>
User-Agent: Mutt/1.10.1 (2018-07-13)
Precedence: bulk


--Q68bSM7Ycu6FN28Q
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Jul 21, 2021 at 12:18:03AM +0100, Andre Przywara wrote:
> On Wed, 21 Jul 2021 08:02:42 +1000
> Benjamin Herrenschmidt <benh@kernel.crashing.org> wrote:
> > On Tue, 2021-07-20 at 17:16 +0100, Andre Przywara wrote:

> > > Yes, a similar idea was already brought up before. I think there is e=
ven
> > > the potential for something like an artificial SMCCC "bus", where tho=
se
> > > services presentable as devices could be auto-detected (by checking
> > > known function IDs), the respective drivers would then probe
> > > automatically? =20

> > Sounds like a boot time killer...

> How so? To be clear, at the moment there is basically just the TRNG
> service we would probe for, maybe FF-A, then adding as we go. But in
> any case it would be just a handful, and querying is very quick
> (SMC/HVC, then just a switch/case on the other side, and ERET).
> Is there any particular scenario you are concerned about? Quick
> starting guests?

It's also worth pointing out that we're already doing the enumeration
part, making things a bus would just be about reorganising the code that
checks if services are present to a central place so it looks more Linux
style.  If anything I'd guess that if we get to the point where things
are slow enough to worry about having that code in one central place
would make doing something about it easier (eg, adding a "list all
services" service or firmware binding).

--Q68bSM7Ycu6FN28Q
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmD4AicACgkQJNaLcl1U
h9B1/Qf+POQrRjDcaawnwQmaPxlIW3/6Swx2huY1ofbdsQ/4vZlt6Ju2sXX1md1J
LCNF+B2/pxjieNhz3Eswsf/LYr4+yeLtuHNgWsEmgcxSNhYBFvw6kjUn8/yUdsTR
d2QItg/3wqqcitKz3DQnuylZPrbh46pgt9jboJUqYmnyvxgOrtwG1Va+ng692SKA
cMvSib8oQQehNhQ3jl88EuJ2g1hoBKQlRrQkw8vpDLRpN7dZlSlEkzBaOZFsChKE
dYptfBTDBx7RUYHWQT3GY4/xYcdcmH8ICSm2+WPRTy02qorgGyNnKwsJvf3BCbK4
zJhyg/66rMJExeRp2L/Cy+za+YxSlg==
=8SIw
-----END PGP SIGNATURE-----

--Q68bSM7Ycu6FN28Q--