From cb6e852620458141f9049951b7c49725ef8058a1 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Fri, 8 Feb 2019 21:59:06 +0100 Subject: [PATCH] SharedGraphics: Don't eagerly commit mmap'ed font files in kernel. Just let the page fault handler do its thing. Anytime we can avoid loading some pages from disk, that's a good thing. --- SharedGraphics/Font.cpp | 4 ---- 1 file changed, 4 deletions(-) diff --git a/SharedGraphics/Font.cpp b/SharedGraphics/Font.cpp index a7c5e8997f..f5b29d5d04 100644 --- a/SharedGraphics/Font.cpp +++ b/SharedGraphics/Font.cpp @@ -65,7 +65,6 @@ Font& Font::default_font() } auto* region = current->allocate_file_backed_region(LinearAddress(), font_file_size(10), descriptor->inode(), "default_font", /*readable*/true, /*writable*/false); ASSERT(region); - region->page_in(); s_default_font = Font::load_from_memory(region->laddr().as_ptr()).leak_ref(); #endif ASSERT(s_default_font); @@ -88,9 +87,6 @@ Font& Font::default_bold_font() } auto* region = current->allocate_file_backed_region(LinearAddress(), font_file_size(10), descriptor->inode(), "default_bold_font", /*readable*/true, /*writable*/false); ASSERT(region); - ASSERT_INTERRUPTS_ENABLED(); - region->page_in(); - ASSERT_INTERRUPTS_ENABLED(); s_default_bold_font = Font::load_from_memory(region->laddr().as_ptr()).leak_ref(); #endif ASSERT(s_default_bold_font);