Skip to content

Commit a2fc6ce

Browse files
Refactor UEFI implementation for compliance with rust-lang/rust#107457 lint
1 parent a6c86c3 commit a2fc6ce

File tree

5 files changed

+44
-20
lines changed

5 files changed

+44
-20
lines changed

Cargo.lock

Lines changed: 35 additions & 14 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ exclude = ["examples/basic", "examples/test_framework"]
3131

3232
[workspace.package]
3333
# don't forget to update `workspace.dependencies` below
34-
version = "0.11.3"
34+
version = "0.11.4"
3535
license = "MIT/Apache-2.0"
3636
repository = "https://github.com/rust-osdev/bootloader"
3737

tests/runner/src/lib.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ use bootloader::BootConfig;
22
use bootloader::DiskImageBuilder;
33
use std::{io::Read, path::Path, process::Command};
44

5+
#[allow(dead_code)]
56
const QEMU_ARGS: &[&str] = &[
67
"-device",
78
"isa-debug-exit,iobase=0xf4,iosize=0x04",
@@ -11,6 +12,8 @@ const QEMU_ARGS: &[&str] = &[
1112
"none",
1213
"--no-reboot",
1314
];
15+
16+
#[allow(dead_code)]
1417
const SEPARATOR: &str = "\n____________________________________\n";
1518

1619
pub fn run_test_kernel(kernel_binary_path: &str) {
@@ -98,6 +101,7 @@ pub fn run_test_kernel_on_uefi_pxe(out_tftp_path: &Path) {
98101
run_qemu(args);
99102
}
100103

104+
#[allow(dead_code)]
101105
fn run_qemu<'a, A>(args: A)
102106
where
103107
A: IntoIterator<Item = &'a str>,

uefi/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,6 @@ bootloader_api = { workspace = true }
1313
bootloader-x86_64-common = { workspace = true }
1414
bootloader-boot-config = { workspace = true }
1515
log = "0.4.14"
16-
uefi = "0.18.0"
1716
x86_64 = "0.14.8"
1817
serde-json-core = "0.5.0"
18+
uefi = "0.20.0"

uefi/src/main.rs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ fn main_inner(image: Handle, mut st: SystemTable<Boot>) -> Status {
136136
}
137137
);
138138

139-
let mmap_storage = {
139+
let _mmap_storage = {
140140
let mut memory_map_size = st.boot_services().memory_map_size();
141141
loop {
142142
let ptr = st
@@ -162,11 +162,10 @@ fn main_inner(image: Handle, mut st: SystemTable<Boot>) -> Status {
162162

163163
log::trace!("exiting boot services");
164164
let (system_table, memory_map) = st
165-
.exit_boot_services(image, mmap_storage)
166-
.expect("Failed to exit boot services");
165+
.exit_boot_services();
167166

168167
let mut frame_allocator =
169-
LegacyFrameAllocator::new(memory_map.copied().map(UefiMemoryDescriptor));
168+
LegacyFrameAllocator::new(memory_map.entries().copied().map(UefiMemoryDescriptor));
170169

171170
let page_tables = create_page_tables(&mut frame_allocator);
172171
let mut ramdisk_len = 0u64;

0 commit comments

Comments
 (0)