On 10.11.2015 15:38, Daniel Kiper wrote: >>> - if (entry_specified) >>> > > + if (keep_bs && efi_entry_specified) >>> > > + grub_multiboot_payload_eip = efi_entry; >>> > > + else if (entry_specified) >>> > > grub_multiboot_payload_eip = entry; >>> > > >> > This seems redundant. > What is wrong here? I just mean that if we use a single structure this code could go away