Re: [PATCH v2 1/5] efi/memattr: Fix thinko in table size sanity check

[Breno Leitao]

On Wed, Apr 01, 2026 at 04:31:27PM +0200, Ard Biesheuvel wrote:
> 
> On Wed, 1 Apr 2026, at 14:45, Breno Leitao wrote:
> > On Wed, Apr 01, 2026 at 02:23:53PM +0200, Ard Biesheuvel wrote:
> >> +	if (tbl->desc_size < sizeof(efi_memory_desc_t) ||
> >> +	    tbl->desc_size > efi.memmap.desc_size) {
> >> +		pr_warn("Unexpected EFI Memory Attributes descriptor size %d (expected: %ld)\n",
> >> +			tbl->desc_size, efi.memmap.desc_size);
> >
> > Should tbl->desc_size be printed as %u and %lu?  tbl->desc_size is u32
> > and.  The pr_warn a few lines below already uses %u for the same field:
> >
> 
> Yes, good point - will fix.

Thanks. With the fix, please add:

Reviewed-by: Breno Leitao <leitao@xxxxxxxxxx>