mirrors/sprout
1
0
Fork 0
mirror of https://github.com/edera-dev/sprout.git synced 2026-02-04 07:10:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(boot): on boot handoff, clear the screen using clear rather than reset (#52)

Browse Source 
On some Dell firmware, the reset operation doesn't actually clear the screen.
This change uses clear instead of reset to ensure the screen is properly cleared.
This commit is contained in:
Alex Zenla
2025-12-20 22:29:44 -08:00
committed by GitHub
parent 17e729d068
commit 81c8217ee0
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
crates/boot/src/phases.rs
View File

@@ -30,7 +30,12 @@ pub fn phase(context: Rc<SproutContext>, phase: &[PhaseConfiguration]) -> Result
pub fn before_handoff(context: &SproutContext) -> Result<()> { pub fn before_handoff(context: &SproutContext) -> Result<()> {
// If we have not been asked to retain the boot console, then we should clear the screen. // If we have not been asked to retain the boot console, then we should clear the screen.
if !context.root().options().retain_boot_console { if !context.root().options().retain_boot_console {
uefi::system::with_stdout(|stdout| stdout.reset(true)).context("unable to clear screen")?; // Clear the screen. We use clear here instead of reset because some firmware,
// particularly Dell firmware, does not clear the screen on reset.
// We clear both stdout and stderr because it's not guaranteed that they are the same
// text output.
uefi::system::with_stdout(|stdout| stdout.clear()).context("unable to clear screen")?;
uefi::system::with_stderr(|stderr| stderr.clear()).context("unable to clear screen")?;
} }
Ok(()) Ok(())
} }