mbox series

[0/3] use central copy of grub_efi_find_mmap_size

Message ID 20180713163445.29478-1-leif.lindholm@linaro.org
Headers show
Series use central copy of grub_efi_find_mmap_size | expand

Message

Leif Lindholm July 13, 2018, 4:34 p.m. UTC 
There were multiple local implementations of functions to determine the
size of buffer required to hold a UEFI memory map. Drop these and switch to
grub_efi_find_mmap_size() in kern/efi/mm.c.

I'm not going to lie. I no longer have an ia64 cross-toolchain, so that one
is not even compile tested.

Leif Lindholm (3):
  loader/i386/linux: use central copy of grub_efi_find_mmap_size
  loader/ia64/linux: use central copy of grub_efi_find_mmap_size
  loader/multiboot_mbi2: use central copy of grub_efi_find_mmap_size

 grub-core/loader/i386/linux.c     | 51 +--------------------------------------
 grub-core/loader/ia64/efi/linux.c | 46 ++---------------------------------
 grub-core/loader/multiboot_mbi2.c | 38 +----------------------------
 3 files changed, 4 insertions(+), 131 deletions(-)

-- 
2.11.0


_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

Comments

Daniel Kiper Sept. 6, 2018, 1:46 p.m. UTC | #1 
On Fri, Jul 13, 2018 at 05:34:42PM +0100, Leif Lindholm wrote:
> There were multiple local implementations of functions to determine the

> size of buffer required to hold a UEFI memory map. Drop these and switch to

> grub_efi_find_mmap_size() in kern/efi/mm.c.

>

> I'm not going to lie. I no longer have an ia64 cross-toolchain, so that one

> is not even compile tested.

>

> Leif Lindholm (3):

>   loader/i386/linux: use central copy of grub_efi_find_mmap_size

>   loader/ia64/linux: use central copy of grub_efi_find_mmap_size

>   loader/multiboot_mbi2: use central copy of grub_efi_find_mmap_size

>

>  grub-core/loader/i386/linux.c     | 51 +--------------------------------------

>  grub-core/loader/ia64/efi/linux.c | 46 ++---------------------------------

>  grub-core/loader/multiboot_mbi2.c | 38 +----------------------------

>  3 files changed, 4 insertions(+), 131 deletions(-)


Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com> for whole patch series.


Daniel

_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel