[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] xen/arm: ffa: Fix local ffa_vm_count shadowing
- To: Bertrand Marquis <bertrand.marquis@xxxxxxx>
- From: Jens Wiklander <jens.wiklander@xxxxxxxxxx>
- Date: Fri, 20 Mar 2026 16:02:23 +0100
- Arc-authentication-results: i=1; mx.google.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20240605; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=5UaDWyjtfSyl77XXz0U7ShmHX+gyfVpxi6SPVf+gsF0=; fh=wNLC6Hyb5Ukz/ErppBRQBwv8vwa/OMsdh6R8bnNsiPU=; b=lr5E+h6WDTXGZHwEYaBgxwLMoUCnNsVHoET1my2txLkIDv+5j2re7jtp2Q4mnl8RJH NixlzOya6ISgF+AKKMvjCxKAZA3vINmoPmMeoayK43VuPCoYXa+pvRrz0p62BC2wfjsw 6usMtfPzkUBQCCVqDd86pmyurbCLHpNviUrAeWeaQYF3DkOhlHxoYSSdQX1wip2hJvAY sRN8jjQIyoegDjstIHEvS2JTLrxbHNTVrd7TejROef8j993+9/qWeEH53IGuW3e5Pq5f 1e0PVR52LZ9XqVbF7fe+Wmxkttk5Yd7xPxJLGNcZA/4aPYSpuQPZRJx5kaHGRNY/7jeh yR+w==; darn=lists.xenproject.org
- Arc-seal: i=1; a=rsa-sha256; t=1774018955; cv=none; d=google.com; s=arc-20240605; b=ZyyhOrXVXff4riATGoZMjnlTiENcMPt21IwX9FKv2R+CpEGCmiaIXPfWA2IDK8iwLC W1B6VXgi1n/QYPMyk4IT769tCheNhaqSwCaAI3b/NOeXU8gk4cFhQCKFu7bgr0ssd/Xh Bw6vh8PltTm5GOrDObOfua+ul3GVZmyBX8FnuTfRxjMoWPnVT2yfBer2A0Wdfz5G/6Y6 xHL0m3FiiStOpx3uBJTflQTd8DeV14JRdgGoJVG8+VbTxvu2W1upE2CCBIX/uz+NP2c+ a/eQqVmKvA+MnbSDAuaKAyug+6P74I1hUGd0i/h2+elzZA5SHmdaDuW8vkks2i1R07H9 IsLQ==
- Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx, Volodymyr Babchuk <volodymyr_babchuk@xxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>
- Delivery-date: Fri, 20 Mar 2026 15:07:36 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On Fri, Mar 20, 2026 at 10:07 AM Bertrand Marquis
<bertrand.marquis@xxxxxxx> wrote:
>
> ffa_handle_partition_info_get() declares a local variable named
> ffa_vm_count, which hides the global atomic ffa_vm_count declared in
> ffa_private.h.
>
> This triggered the ECLAIR Rule 5.3 finding "non-compliant local variable
> `ffa_vm_count'".
>
> Rename the local counters to vm_count and sp_count and update their
> uses.
>
> No functional changes.
>
> Signed-off-by: Bertrand Marquis <bertrand.marquis@xxxxxxx>
> ---
> xen/arch/arm/tee/ffa_partinfo.c | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
Reviewed-by: Jens Wiklander <jens.wiklander@xxxxxxxxxx>
Cheers,
Jens
|
