Skip to content

Commit b621249

Browse files
committed
Run rustfmt
1 parent 0841e6f commit b621249

File tree

3 files changed

+28
-16
lines changed

3 files changed

+28
-16
lines changed

src/frame_allocator.rs

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,10 @@ impl<'a> FrameAllocator<'a> {
1212
while let Some(region) = iter.next() {
1313
if region.region_type == region_type {
1414
if let Some(next) = iter.peek() {
15-
if next.range.start == region.range.end && next.region_type == MemoryRegionType::Usable && !next.range.is_empty() {
15+
if next.range.start == region.range.end
16+
&& next.region_type == MemoryRegionType::Usable
17+
&& !next.range.is_empty()
18+
{
1619
let frame = region.range.end;
1720
region.range.end += 1;
1821
iter.next().unwrap().range.start += 1;
@@ -49,7 +52,8 @@ impl<'a> FrameAllocator<'a> {
4952
};
5053

5154
if let Some((frame, range)) = result {
52-
self.memory_map.add_region(MemoryRegion { range, region_type, });
55+
self.memory_map
56+
.add_region(MemoryRegion { range, region_type });
5357
Some(frame)
5458
} else {
5559
None
@@ -62,14 +66,17 @@ impl<'a> FrameAllocator<'a> {
6266
pub(crate) fn mark_allocated_region(&mut self, region: MemoryRegion) {
6367
for r in self.memory_map.iter_mut() {
6468
if region.range.start >= r.range.end {
65-
continue
69+
continue;
6670
}
6771
if region.range.end <= r.range.start {
68-
continue
72+
continue;
6973
}
7074

7175
if r.region_type != MemoryRegionType::Usable {
72-
panic!("region {:x?} overlaps with non-usable region {:x?}", region, r);
76+
panic!(
77+
"region {:x?} overlaps with non-usable region {:x?}",
78+
region, r
79+
);
7380
}
7481

7582
if region.range.start == r.range.start {

src/main.rs

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,12 @@ extern crate fixedvec;
2121
use core::slice;
2222
use os_bootinfo::BootInfo;
2323
use usize_conversions::usize_from;
24-
pub use x86_64::PhysAddr;
25-
use x86_64::{VirtAddr};
2624
use x86_64::instructions::tlb;
27-
use x86_64::structures::paging::{RecursivePageTable, Mapper};
28-
use x86_64::structures::paging::{Page, PhysFrame, PageTableFlags, Size2MB};
25+
use x86_64::structures::paging::{Mapper, RecursivePageTable};
26+
use x86_64::structures::paging::{Page, PageTableFlags, PhysFrame, Size2MB};
2927
use x86_64::ux::u9;
28+
pub use x86_64::PhysAddr;
29+
use x86_64::VirtAddr;
3030

3131
global_asm!(include_str!("boot.s"));
3232
global_asm!(include_str!("second_stage.s"));
@@ -119,7 +119,6 @@ pub extern "C" fn load_elf(
119119
memory_map: &mut memory_map,
120120
};
121121

122-
123122
// Mark already used memory areas in frame allocator.
124123
{
125124
let zero_frame: PhysFrame = PhysFrame::from_start_address(PhysAddr::new(0)).unwrap();
@@ -129,21 +128,24 @@ pub extern "C" fn load_elf(
129128
});
130129
let bootloader_start_frame = PhysFrame::containing_address(bootloader_start);
131130
let bootloader_end_frame = PhysFrame::containing_address(bootloader_end - 1u64);
132-
let bootloader_memory_area = PhysFrame::range(bootloader_start_frame, bootloader_end_frame + 1);
131+
let bootloader_memory_area =
132+
PhysFrame::range(bootloader_start_frame, bootloader_end_frame + 1);
133133
frame_allocator.mark_allocated_region(MemoryRegion {
134134
range: bootloader_memory_area,
135135
region_type: MemoryRegionType::Bootloader,
136136
});
137137
let kernel_start_frame = PhysFrame::containing_address(kernel_start.phys());
138-
let kernel_end_frame = PhysFrame::containing_address(kernel_start.phys() + kernel_size - 1u64);
138+
let kernel_end_frame =
139+
PhysFrame::containing_address(kernel_start.phys() + kernel_size - 1u64);
139140
let kernel_memory_area = PhysFrame::range(kernel_start_frame, kernel_end_frame + 1);
140141
frame_allocator.mark_allocated_region(MemoryRegion {
141142
range: kernel_memory_area,
142143
region_type: MemoryRegionType::Kernel,
143144
});
144145
let page_table_start_frame = PhysFrame::containing_address(page_table_start);
145146
let page_table_end_frame = PhysFrame::containing_address(page_table_end - 1u64);
146-
let page_table_memory_area = PhysFrame::range(page_table_start_frame, page_table_end_frame + 1);
147+
let page_table_memory_area =
148+
PhysFrame::range(page_table_start_frame, page_table_end_frame + 1);
147149
frame_allocator.mark_allocated_region(MemoryRegion {
148150
range: page_table_memory_area,
149151
region_type: MemoryRegionType::PageTable,
@@ -152,9 +154,12 @@ pub extern "C" fn load_elf(
152154

153155
// Unmap the ELF file.
154156
let kernel_start_page: Page<Size2MB> = Page::containing_address(kernel_start.virt());
155-
let kernel_end_page: Page<Size2MB> = Page::containing_address(kernel_start.virt() + kernel_size - 1u64);
157+
let kernel_end_page: Page<Size2MB> =
158+
Page::containing_address(kernel_start.virt() + kernel_size - 1u64);
156159
for page in Page::range_inclusive(kernel_start_page, kernel_end_page) {
157-
rec_page_table.unmap(page, &mut |_| {}).expect("dealloc error");
160+
rec_page_table
161+
.unmap(page, &mut |_| {})
162+
.expect("dealloc error");
158163
}
159164
// Flush the translation lookaside buffer since we changed the active mapping.
160165
tlb::flush_all();

src/page_table.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ use frame_allocator::FrameAllocator;
33
use os_bootinfo::MemoryRegionType;
44
use x86_64::instructions::tlb;
55
use x86_64::structures::paging::{MapToError, RecursivePageTable, UnmapError};
6-
use x86_64::structures::paging::{Page, PageTableFlags, PhysFrame, Mapper, PageSize, Size4KB};
6+
use x86_64::structures::paging::{Mapper, Page, PageSize, PageTableFlags, PhysFrame, Size4KB};
77
use x86_64::{align_up, PhysAddr, VirtAddr};
88
use xmas_elf::program::{self, ProgramHeader64};
99

0 commit comments

Comments
 (0)