Re: [PATCH v2] x86/efi: defer freeing of boot services memory

[Mike Rapoport]

On Thu, Mar 19, 2026 at 09:06:52PM -0700, Guenter Roeck wrote:
> Hi,
> 
> > +void __init efi_unmap_boot_services(void)
> >  {
> >  	struct efi_memory_map_data data = { 0 };
> >  	efi_memory_desc_t *md;
> >  	int num_entries = 0;
> > +	int idx = 0;
> > +	size_t sz;
> >  	void *new, *new_md;
> >  
> >  	/* Keep all regions for /sys/kernel/debug/efi */
> >  	if (efi_enabled(EFI_DBG))
> >  		return;
> >  
> > +	sz = sizeof(*ranges_to_free) * efi.memmap.nr_map + 1;
> 
> Was this possibly supposed to be
> 	sz = sizeof(*ranges_to_free) * (efi.memmap.nr_map + 1);
> 				       ^		     ^
> ?

Yes, thanks for catching this.
 
@Ard, can you please pick the fix: