Skip to content

Commit 218f4ea

Browse files
committed
1 parent c339ef2 commit 218f4ea

File tree

2 files changed

+4
-34
lines changed

2 files changed

+4
-34
lines changed

pkg/limayaml/defaults.go

Lines changed: 0 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -132,37 +132,6 @@ func defaultGuestInstallPrefix() string {
132132
return "/usr/local"
133133
}
134134

135-
func defaultFirmwareImages() []FileWithVMType {
136-
return []FileWithVMType{
137-
/*
138-
The "Firmware/edk2 20231213" patches <https://lists.gnu.org/archive/html/qemu-devel/2023-12/msg01694.html>
139-
are necessary to boot most aarch64 images (except Debian, which does not use UEFI shim).
140-
141-
The patches are proposed for the QEMU v8.2.0 milestone, but likely to be postponed to v8.2.1.
142-
Until the patches get accepted in the QEMU upstream, Lima fetches the patched edk2 binary from
143-
<https://gitlab.com/kraxel/qemu/-/tags/firmware%2Fedk2-20231213-pull-request>.
144-
*/
145-
{
146-
File: File{
147-
Location: "https://gitlab.com/kraxel/qemu/-/raw/704f7cad5105246822686f65765ab92045f71a3b/pc-bios/edk2-aarch64-code.fd.bz2",
148-
Arch: AARCH64,
149-
Digest: "sha256:a5fc228623891297f2d82e22ea56ec57cde93fea5ec01abf543e4ed5cacaf277",
150-
},
151-
VMType: QEMU,
152-
},
153-
// Mirror
154-
{
155-
File: File{
156-
Location: "https://github.com/AkihiroSuda/qemu/raw/704f7cad5105246822686f65765ab92045f71a3b/pc-bios/edk2-aarch64-code.fd.bz2",
157-
Arch: AARCH64,
158-
Digest: "sha256:a5fc228623891297f2d82e22ea56ec57cde93fea5ec01abf543e4ed5cacaf277",
159-
},
160-
VMType: QEMU,
161-
},
162-
// TODO: what about ARMv7?
163-
}
164-
}
165-
166135
// FillDefault updates undefined fields in y with defaults from d (or built-in default), and overwrites with values from o.
167136
// Both d and o may be empty.
168137
//
@@ -335,9 +304,6 @@ func FillDefault(y, d, o *LimaYAML, filePath string) {
335304
}
336305

337306
y.Firmware.Images = append(append(o.Firmware.Images, y.Firmware.Images...), d.Firmware.Images...)
338-
if len(y.Firmware.Images) == 0 {
339-
y.Firmware.Images = defaultFirmwareImages()
340-
}
341307
for i := range y.Firmware.Images {
342308
f := &y.Firmware.Images[i]
343309
if f.Arch == "" {

pkg/qemu/qemu.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -483,6 +483,10 @@ func Cmdline(ctx context.Context, cfg Config) (exe string, args []string, err er
483483
if version.LessThan(*semver.New(MinimumQemuVersion)) {
484484
logrus.Fatalf("QEMU %v is too old, %v or later required", version, MinimumQemuVersion)
485485
}
486+
if runtime.GOOS == "darwin" && runtime.GOARCH == "arm64" && version.Equal(*semver.New("8.2.0")) {
487+
logrus.Fatal("QEMU 8.2.0 is no longer supported on ARM Mac due to <https://gitlab.com/qemu-project/qemu/-/issues/1990>. " +
488+
"Please upgrade QEMU to v8.2.1 (or downgrade to v8.1.x).")
489+
}
486490
}
487491

488492
// Architecture

0 commit comments

Comments
 (0)