Re: [PATCH v2] mbi: use per segment a separate relocator chunk

2018-06-11 Thread Daniel Kiper
On Tue, Jun 05, 2018 at 10:14:49PM +0200, Alexander Boettcher wrote: > Instead of setting up a all comprising relocator chunk for all segments, > use per segment a separate relocator chunk. > > If the ELF is non-relocatable, a single relocator chunk will comprise memory > (between the segments) whi

[PATCH 2/2] efi/fdt: set address/size cells to 2 for empty tree

2018-06-11 Thread Leif Lindholm
When booting an arm* system on UEFI with an empty device tree (currently only when hardware description comes from ACPI), we don't currently set default to 1 cell (32 bits). Set both of these properties, to 2 cells (64 bits), to resolve issues with kexec on some platforms. This change corresponds

[PATCH 0/2] efi/fdt: set #address-cells/#size-cells on empty tree

2018-06-11 Thread Leif Lindholm
Set #address-cells and #size-cells properties (to 2) for ARM*/UEFI systems when creating an empty DT at boot time. This resolves an issue seen in the wild with kexec on certain 64-bit ARM systems. First part is moving out the prop_entry_size macro from lib/fdt.c and make it available in (with the

[PATCH 1/2] fdt: move prop_entry_size to fdt.h

2018-06-11 Thread Leif Lindholm
To be able to resuse the prop_entry_size macro, move it to and rename it grub_fdt_prop_entry_size. Signed-off-by: Leif Lindholm --- grub-core/lib/fdt.c | 25 ++--- include/grub/fdt.h | 5 + 2 files changed, 15 insertions(+), 15 deletions(-) diff --git a/grub-core/lib/

Re: [PATCH v3] multiboot2: clarify usage of the address tag

2018-06-11 Thread Daniel Kiper
On Mon, Jun 11, 2018 at 01:49:02PM +0200, Roger Pau Monne wrote: > Add a note to spell out that if the address tag is not present the > file should be loaded using the elf header. > > Signed-off-by: Roger Pau Monné Reviewed-by: Daniel Kiper If there are no objections I will apply this in a week

[PATCH v3] multiboot2: clarify usage of the address tag

2018-06-11 Thread Roger Pau Monne
Add a note to spell out that if the address tag is not present the file should be loaded using the elf header. Signed-off-by: Roger Pau Monné --- Cc: Daniel Kiper Cc: xen-de...@lists.xenproject.org --- Changes since v2: - Clarify that the address tag must be used if present. Changes since v1:

Re: [PATCH v2] multiboot2: clarify usage of the address tag

2018-06-11 Thread Daniel Kiper
On Mon, Jun 11, 2018 at 11:30:16AM +0200, Roger Pau Monné wrote: [...] > I think the following is clear enough: > > "Note: This information does not need to be provided if the kernel image is in > @sc{elf} format, but it must be provided if the image is in a.out format or in > some other format.

Re: [PATCH v2] multiboot2: clarify usage of the address tag

2018-06-11 Thread Roger Pau Monné
On Fri, Jun 08, 2018 at 12:39:28PM +0200, Daniel Kiper wrote: > On Fri, Jun 08, 2018 at 12:08:22PM +0200, Roger Pau Monné wrote: > > On Fri, Jun 08, 2018 at 11:35:52AM +0200, Daniel Kiper wrote: > > > On Thu, Jun 07, 2018 at 05:59:06PM +0200, Roger Pau Monne wrote: > > > > Add a note to spell out t