From 2428ba38329eacfb23fe331eeaef32b6b2ebcceb Mon Sep 17 00:00:00 2001 From: Jelle Raaijmakers Date: Tue, 17 Jan 2023 20:48:57 +0100 Subject: [PATCH] Kernel: Remove `dbgln` when unregistering an unhandled x86_64 interrupt A lot of interrupt numbers are initialized with the unhandled interrupt handler. Whenever a new handler is registered on one of these interrupts, the old handler is unregistered first. Let's not be verbose about this since it is perfectly normal. --- Kernel/Arch/x86_64/Interrupts.cpp | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/Kernel/Arch/x86_64/Interrupts.cpp b/Kernel/Arch/x86_64/Interrupts.cpp index 78d77c3285..c889bb59bc 100644 --- a/Kernel/Arch/x86_64/Interrupts.cpp +++ b/Kernel/Arch/x86_64/Interrupts.cpp @@ -515,10 +515,8 @@ void unregister_generic_interrupt_handler(u8 interrupt_number, GenericInterruptH { auto*& handler_slot = s_interrupt_handler[interrupt_number]; VERIFY(handler_slot != nullptr); - if (handler_slot->type() == HandlerType::UnhandledInterruptHandler) { - dbgln("Trying to unregister unused handler (?)"); + if (handler_slot->type() == HandlerType::UnhandledInterruptHandler) return; - } if (handler_slot->is_shared_handler() && !handler_slot->is_sharing_with_others()) { VERIFY(handler_slot->type() == HandlerType::SharedIRQHandler); auto* shared_handler = static_cast(handler_slot);