Re: [PATCH v1 3/5] xen/arm: platforms: Add NXP S32CC platform code

[Julien Grall <julien@xxxxxxx>]

Hi,

On 10/09/2024 15:34, Andrei Cherechesu (OSS) wrote:
> From: Andrei Cherechesu <andrei.cherechesu@xxxxxxx>
>
> Added support for NXP S32CC platforms (S32G2, S32G3, S32R45),
> which need SCMI over SMC calls forwarded to the firmware
> running at EL3 (TF-A). Linux relies on the SCMI Platform
> for system services such as clocking, reset, etc.
Is it SMCI as in the Arm spec? If so, this should not be platform specific.

Furthermore, I was under the impression that we can't blindly forward 
everything from a domain to the firmware. While it might be okayish for 
dom0, you also seem to give access to all the domains on the system is 
it intended?
Anyway, there is a series on the ML to add a mediator for SCMI in Xen 
(see [1]). I think it would be preferable to focus on getting it merged 
as it would benefit everyone and increase the security posture (we could 
restrict access).
Cheers,

[1] 
https://lore.kernel.org/xen-devel/e9dadd96aa5b64b9232e10a083ce393af620adde.1639472078.git.oleksii_moisieiev@xxxxxxxx
Cheers,

--
Julien Grall