[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] sparc64: Fix BIOS Boot Partition support
From: |
Eric Snowberg |
Subject: |
Re: [PATCH] sparc64: Fix BIOS Boot Partition support |
Date: |
Tue, 16 Jul 2019 10:27:54 -0600 |
> On Jul 16, 2019, at 5:35 AM, John Paul Adrian Glaubitz <address@hidden> wrote:
>
> On 7/15/19 12:40 AM, James Clarke wrote:
>> Currently, gpt_offset is uninitialised when using a BIOS Boot Partition
>> but is used unconditionally inside save_blocklists. Instead, ensure it
>> is always initialised to 0 (note that there is already separate code to
>> do the equivalent adjustment after we call save_blocklists on this code
>> path).
>>
>> This patch has been tested on a T5-2 LDOM.
>>
>> Signed-off-by: James Clarke <address@hidden>
>> ---
>> util/setup.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/util/setup.c b/util/setup.c
>> index 6f88f3cc4..3be88aae1 100644
>> --- a/util/setup.c
>> +++ b/util/setup.c
>> @@ -270,6 +270,9 @@ SETUP (const char *dir,
>> #ifdef GRUB_SETUP_BIOS
>> bl.current_segment =
>> GRUB_BOOT_I386_PC_KERNEL_SEG + (GRUB_DISK_SECTOR_SIZE >> 4);
>> +#endif
>> +#ifdef GRUB_SETUP_SPARC64
>> + bl.gpt_offset = 0;
>> #endif
>> bl.last_length = 0;
>>
>> @@ -730,7 +733,6 @@ unable_to_embed:
>> #ifdef GRUB_SETUP_SPARC64
>> {
>> grub_partition_t container = root_dev->disk->partition;
>> - bl.gpt_offset = 0;
>>
>> if (grub_strstr (container->partmap->name, "gpt"))
>> bl.gpt_offset = grub_partition_get_start (container);
>>
>
> Works as expected and fixes the problem I reported earlier [1].
>
> Tested-by: John Paul Adrian Glaubitz <address@hidden>
>
> @Eric: Any comments?
>
Reviewed-by: Eric Snowberg <address@hidden>