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
1diff --git a/debian/.git-dpm b/debian/.git-dpm
2index b3699f9..05e4a12 100644
3--- a/debian/.git-dpm
4+++ b/debian/.git-dpm
5@@ -1,6 +1,6 @@
6 # see git-dpm(1) from git-dpm package
7-277b7127a92c9d2c075a70957f0cefc4ebe48e8b
8-277b7127a92c9d2c075a70957f0cefc4ebe48e8b
9+67fbc0db9b2d5324861bbbca36ab718569f824d2
10+67fbc0db9b2d5324861bbbca36ab718569f824d2
11 578bb115fbd47e1c464696f1f8d6183e5443975d
12 578bb115fbd47e1c464696f1f8d6183e5443975d
13 grub2_2.04.orig.tar.xz
14diff --git a/debian/changelog b/debian/changelog
15index 25d552d..d2f6565 100644
16--- a/debian/changelog
17+++ b/debian/changelog
18@@ -3,6 +3,7 @@ grub2 (2.04-1ubuntu26.6) UNRELEASED; urgency=medium
19 * postinst.in, grub-multi-install: fix logic of skipping installing onto
20 any device, if one chose to not install bootloader on any device. LP:
21 #1896608
22+ * Do not finalize params twice on arm64. LP: #1897819
23
24 -- Dimitri John Ledkov <xnox@ubuntu.com> Mon, 28 Sep 2020 14:11:41 +0100
25
26diff --git a/debian/patches/ubuntu-linuxefi-arm64.patch b/debian/patches/ubuntu-linuxefi-arm64.patch
27index 6857834..47b3f14 100644
28--- a/debian/patches/ubuntu-linuxefi-arm64.patch
29+++ b/debian/patches/ubuntu-linuxefi-arm64.patch
30@@ -1,4 +1,4 @@
31-From 277b7127a92c9d2c075a70957f0cefc4ebe48e8b Mon Sep 17 00:00:00 2001
32+From 67fbc0db9b2d5324861bbbca36ab718569f824d2 Mon Sep 17 00:00:00 2001
33 From: Julian Andres Klode <julian.klode@canonical.com>
34 Date: Fri, 11 Sep 2020 11:28:08 +0200
35 Subject: Cherry-pick back parts of "Load arm with SB enabled."
36@@ -13,11 +13,11 @@ Bug-Ubuntu: https://bugs.launchpad.net/bugs/1862279
37 Origin: vendor, https://github.com/rhboot/grub2/commit/2786ab864cf00c15123320671f653e9a36ba12b4
38 Patch-Name: ubuntu-linuxefi-arm64.patch
39 ---
40- grub-core/loader/arm64/linux.c | 103 ++++++++++++++++++---------------
41- 1 file changed, 56 insertions(+), 47 deletions(-)
42+ grub-core/loader/arm64/linux.c | 106 +++++++++++++++++----------------
43+ 1 file changed, 56 insertions(+), 50 deletions(-)
44
45 diff --git a/grub-core/loader/arm64/linux.c b/grub-core/loader/arm64/linux.c
46-index 3f5496fc55..39ebcf5b52 100644
47+index 3f5496fc55..130e9c09b4 100644
48 --- a/grub-core/loader/arm64/linux.c
49 +++ b/grub-core/loader/arm64/linux.c
50 @@ -43,6 +43,8 @@ static int loaded;
51@@ -67,7 +67,7 @@ index 3f5496fc55..39ebcf5b52 100644
52 return GRUB_ERR_NONE;
53
54 failure:
55-@@ -118,62 +142,43 @@ failure:
56+@@ -118,70 +142,48 @@ failure:
57 return grub_error(GRUB_ERR_BAD_OS, "failed to install/update FDT");
58 }
59
60@@ -107,12 +107,12 @@ index 3f5496fc55..39ebcf5b52 100644
61 - mempath[0].memory_type = GRUB_EFI_LOADER_DATA;
62 - mempath[0].start_address = addr;
63 - mempath[0].end_address = addr + size;
64--
65++ grub_err_t retval;
66+
67 - mempath[1].header.type = GRUB_EFI_END_DEVICE_PATH_TYPE;
68 - mempath[1].header.subtype = GRUB_EFI_END_ENTIRE_DEVICE_PATH_SUBTYPE;
69 - mempath[1].header.length = sizeof (grub_efi_device_path_t);
70-+ grub_err_t retval;
71-
72+-
73 - b = grub_efi_system_table->boot_services;
74 - status = b->load_image (0, grub_efi_image_handle,
75 - (grub_efi_device_path_t *) mempath,
76@@ -157,7 +157,15 @@ index 3f5496fc55..39ebcf5b52 100644
77 }
78
79 static grub_err_t
80-@@ -297,6 +302,7 @@ grub_cmd_linux (grub_command_t cmd __attribute__ ((unused)),
81+ grub_linux_boot (void)
82+ {
83+- if (finalize_params_linux () != GRUB_ERR_NONE)
84+- return grub_errno;
85+-
86+ return (grub_arch_efi_linux_boot_image((grub_addr_t)kernel_addr,
87+ kernel_size, linux_args));
88+ }
89+@@ -297,6 +299,7 @@ grub_cmd_linux (grub_command_t cmd __attribute__ ((unused)),
90 {
91 grub_file_t file = 0;
92 struct linux_arch_kernel_header lh;
93@@ -165,7 +173,7 @@ index 3f5496fc55..39ebcf5b52 100644
94 grub_err_t err;
95 int rc;
96
97-@@ -354,6 +360,9 @@ grub_cmd_linux (grub_command_t cmd __attribute__ ((unused)),
98+@@ -354,6 +357,9 @@ grub_cmd_linux (grub_command_t cmd __attribute__ ((unused)),
99 }
100 }
101
102diff --git a/grub-core/loader/arm64/linux.c b/grub-core/loader/arm64/linux.c
103index 39ebcf5..130e9c0 100644
104--- a/grub-core/loader/arm64/linux.c
105+++ b/grub-core/loader/arm64/linux.c
106@@ -184,9 +184,6 @@ grub_arch_efi_linux_boot_image (grub_addr_t addr, grub_size_t size, char *args)
107 static grub_err_t
108 grub_linux_boot (void)
109 {
110- if (finalize_params_linux () != GRUB_ERR_NONE)
111- return grub_errno;
112-
113 return (grub_arch_efi_linux_boot_image((grub_addr_t)kernel_addr,
114 kernel_size, linux_args));
115 }

Subscribers

People subscribed via source and target branches