Merge ~xnox/grub:focal-sru into ~ubuntu-core-dev/grub/+git/ubuntu:focal

Proposed by Dimitri John Ledkov
Status: Merged
Merged at revision: 5d550d80761675b5f2ace1c5279d931ce6cf2629
Proposed branch: ~xnox/grub:focal-sru
Merge into: ~ubuntu-core-dev/grub/+git/ubuntu:focal
Diff against target: 115 lines (+21/-15)
4 files modified
debian/.git-dpm (+2/-2)
debian/changelog (+1/-0)
debian/patches/ubuntu-linuxefi-arm64.patch (+18/-10)
grub-core/loader/arm64/linux.c (+0/-3)
Reviewer Review Type Date Requested Status
Ubuntu Core Development Team Pending
Review via email: mp+391702@code.launchpad.net

Commit message

grub2 sru regression

- postinst fix
- arm fix

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/debian/.git-dpm b/debian/.git-dpm
index b3699f9..05e4a12 100644
--- a/debian/.git-dpm
+++ b/debian/.git-dpm
@@ -1,6 +1,6 @@
1# see git-dpm(1) from git-dpm package1# see git-dpm(1) from git-dpm package
2277b7127a92c9d2c075a70957f0cefc4ebe48e8b267fbc0db9b2d5324861bbbca36ab718569f824d2
3277b7127a92c9d2c075a70957f0cefc4ebe48e8b367fbc0db9b2d5324861bbbca36ab718569f824d2
4578bb115fbd47e1c464696f1f8d6183e5443975d4578bb115fbd47e1c464696f1f8d6183e5443975d
5578bb115fbd47e1c464696f1f8d6183e5443975d5578bb115fbd47e1c464696f1f8d6183e5443975d
6grub2_2.04.orig.tar.xz6grub2_2.04.orig.tar.xz
diff --git a/debian/changelog b/debian/changelog
index 25d552d..d2f6565 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -3,6 +3,7 @@ grub2 (2.04-1ubuntu26.6) UNRELEASED; urgency=medium
3 * postinst.in, grub-multi-install: fix logic of skipping installing onto3 * postinst.in, grub-multi-install: fix logic of skipping installing onto
4 any device, if one chose to not install bootloader on any device. LP:4 any device, if one chose to not install bootloader on any device. LP:
5 #18966085 #1896608
6 * Do not finalize params twice on arm64. LP: #1897819
67
7 -- Dimitri John Ledkov <xnox@ubuntu.com> Mon, 28 Sep 2020 14:11:41 +01008 -- Dimitri John Ledkov <xnox@ubuntu.com> Mon, 28 Sep 2020 14:11:41 +0100
89
diff --git a/debian/patches/ubuntu-linuxefi-arm64.patch b/debian/patches/ubuntu-linuxefi-arm64.patch
index 6857834..47b3f14 100644
--- a/debian/patches/ubuntu-linuxefi-arm64.patch
+++ b/debian/patches/ubuntu-linuxefi-arm64.patch
@@ -1,4 +1,4 @@
1From 277b7127a92c9d2c075a70957f0cefc4ebe48e8b Mon Sep 17 00:00:00 20011From 67fbc0db9b2d5324861bbbca36ab718569f824d2 Mon Sep 17 00:00:00 2001
2From: Julian Andres Klode <julian.klode@canonical.com>2From: Julian Andres Klode <julian.klode@canonical.com>
3Date: Fri, 11 Sep 2020 11:28:08 +02003Date: Fri, 11 Sep 2020 11:28:08 +0200
4Subject: Cherry-pick back parts of "Load arm with SB enabled."4Subject: Cherry-pick back parts of "Load arm with SB enabled."
@@ -13,11 +13,11 @@ Bug-Ubuntu: https://bugs.launchpad.net/bugs/1862279
13Origin: vendor, https://github.com/rhboot/grub2/commit/2786ab864cf00c15123320671f653e9a36ba12b413Origin: vendor, https://github.com/rhboot/grub2/commit/2786ab864cf00c15123320671f653e9a36ba12b4
14Patch-Name: ubuntu-linuxefi-arm64.patch14Patch-Name: ubuntu-linuxefi-arm64.patch
15---15---
16 grub-core/loader/arm64/linux.c | 103 ++++++++++++++++++---------------16 grub-core/loader/arm64/linux.c | 106 +++++++++++++++++----------------
17 1 file changed, 56 insertions(+), 47 deletions(-)17 1 file changed, 56 insertions(+), 50 deletions(-)
1818
19diff --git a/grub-core/loader/arm64/linux.c b/grub-core/loader/arm64/linux.c19diff --git a/grub-core/loader/arm64/linux.c b/grub-core/loader/arm64/linux.c
20index 3f5496fc55..39ebcf5b52 10064420index 3f5496fc55..130e9c09b4 100644
21--- a/grub-core/loader/arm64/linux.c21--- a/grub-core/loader/arm64/linux.c
22+++ b/grub-core/loader/arm64/linux.c22+++ b/grub-core/loader/arm64/linux.c
23@@ -43,6 +43,8 @@ static int loaded;23@@ -43,6 +43,8 @@ static int loaded;
@@ -67,7 +67,7 @@ index 3f5496fc55..39ebcf5b52 100644
67 return GRUB_ERR_NONE;67 return GRUB_ERR_NONE;
68 68
69 failure:69 failure:
70@@ -118,62 +142,43 @@ failure:70@@ -118,70 +142,48 @@ failure:
71 return grub_error(GRUB_ERR_BAD_OS, "failed to install/update FDT");71 return grub_error(GRUB_ERR_BAD_OS, "failed to install/update FDT");
72 }72 }
73 73
@@ -107,12 +107,12 @@ index 3f5496fc55..39ebcf5b52 100644
107- mempath[0].memory_type = GRUB_EFI_LOADER_DATA;107- mempath[0].memory_type = GRUB_EFI_LOADER_DATA;
108- mempath[0].start_address = addr;108- mempath[0].start_address = addr;
109- mempath[0].end_address = addr + size;109- mempath[0].end_address = addr + size;
110-110+ grub_err_t retval;
111
111- mempath[1].header.type = GRUB_EFI_END_DEVICE_PATH_TYPE;112- mempath[1].header.type = GRUB_EFI_END_DEVICE_PATH_TYPE;
112- mempath[1].header.subtype = GRUB_EFI_END_ENTIRE_DEVICE_PATH_SUBTYPE;113- mempath[1].header.subtype = GRUB_EFI_END_ENTIRE_DEVICE_PATH_SUBTYPE;
113- mempath[1].header.length = sizeof (grub_efi_device_path_t);114- mempath[1].header.length = sizeof (grub_efi_device_path_t);
114+ grub_err_t retval;115-
115
116- b = grub_efi_system_table->boot_services;116- b = grub_efi_system_table->boot_services;
117- status = b->load_image (0, grub_efi_image_handle,117- status = b->load_image (0, grub_efi_image_handle,
118- (grub_efi_device_path_t *) mempath,118- (grub_efi_device_path_t *) mempath,
@@ -157,7 +157,15 @@ index 3f5496fc55..39ebcf5b52 100644
157 }157 }
158 158
159 static grub_err_t159 static grub_err_t
160@@ -297,6 +302,7 @@ grub_cmd_linux (grub_command_t cmd __attribute__ ((unused)),160 grub_linux_boot (void)
161 {
162- if (finalize_params_linux () != GRUB_ERR_NONE)
163- return grub_errno;
164-
165 return (grub_arch_efi_linux_boot_image((grub_addr_t)kernel_addr,
166 kernel_size, linux_args));
167 }
168@@ -297,6 +299,7 @@ grub_cmd_linux (grub_command_t cmd __attribute__ ((unused)),
161 {169 {
162 grub_file_t file = 0;170 grub_file_t file = 0;
163 struct linux_arch_kernel_header lh;171 struct linux_arch_kernel_header lh;
@@ -165,7 +173,7 @@ index 3f5496fc55..39ebcf5b52 100644
165 grub_err_t err;173 grub_err_t err;
166 int rc;174 int rc;
167 175
168@@ -354,6 +360,9 @@ grub_cmd_linux (grub_command_t cmd __attribute__ ((unused)),176@@ -354,6 +357,9 @@ grub_cmd_linux (grub_command_t cmd __attribute__ ((unused)),
169 }177 }
170 }178 }
171 179
diff --git a/grub-core/loader/arm64/linux.c b/grub-core/loader/arm64/linux.c
index 39ebcf5..130e9c0 100644
--- a/grub-core/loader/arm64/linux.c
+++ b/grub-core/loader/arm64/linux.c
@@ -184,9 +184,6 @@ grub_arch_efi_linux_boot_image (grub_addr_t addr, grub_size_t size, char *args)
184static grub_err_t184static grub_err_t
185grub_linux_boot (void)185grub_linux_boot (void)
186{186{
187 if (finalize_params_linux () != GRUB_ERR_NONE)
188 return grub_errno;
189
190 return (grub_arch_efi_linux_boot_image((grub_addr_t)kernel_addr,187 return (grub_arch_efi_linux_boot_image((grub_addr_t)kernel_addr,
191 kernel_size, linux_args));188 kernel_size, linux_args));
192}189}

Subscribers

People subscribed via source and target branches