Re: [PATCH v6 2/6] xen/arm: ffa: Rework partinfo_get implementation

[Julien Grall <julien@xxxxxxx>]

Hi Bertrand,

On 22/05/2025 16:08, Bertrand Marquis wrote:
> This patch is in preparation for VM to VM support in order to do the
> changes on the SP handling part of partinfo_get before adding support
> for the VM part.
>
> This patches is doing the following changes:
> - split partinfo_get into 3 functions to have the locking handling and
>    proper exit on error handled more clearly
> - add some potential overflow checks to validate the offset and sizes
>    passed by the VM on partinfo call.
> - Introduce a maximum number of SPs (for now set to 64) to prevent
>    holding the CPU for too long in case there would be a lot of
>    partitions in the secure world. The limit currently set is thought to
>    be realistic for most use cases as 64 secure partitions is a very high
>    number compared to current seen usage (more 3 or 4).
> - fix include ordering in ffa_private.h to be in alphabetic order
>
> Signed-off-by: Bertrand Marquis <bertrand.marquis@xxxxxxx>
> Reviewed-by: Jens Wiklander <jens.wiklander@xxxxxxxxxx>
Acked-by: Julien Grall <jgrall@xxxxxxxxxx>

Cheers,

--
Julien Grall