Hi Jan, On 24/10/2025 08:43, Jan Beulich wrote:
On 23.10.2025 20:00, Dmytro Prokopchuk1 wrote:--- a/xen/arch/arm/efi/efi-boot.h +++ b/xen/arch/arm/efi/efi-boot.h @@ -227,6 +227,7 @@ static EFI_STATUS __init efi_process_memory_map_bootinfo(EFI_MEMORY_DESCRIPTOR * } } #endif + /* SAF-15-safe casting a pointer */ desc_ptr = NextMemoryDescriptor(desc_ptr, desc_size); }While at present we have only one use site, needing per-use-site comments isn't very nice. Putting it into the imported header isn't nice either. Could Arm perhaps get away without using the macro, just like x86 manages to?
IIUC, you mean something like: EFI_MEMORY_DESCRIPTOR *desc = efi_memmap + i; If so, this would work for me. Cheers, -- Julien Grall
