RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-26 17:57:34 +00:00
Code Issues Activity Actions

Kernel: Simplify (un)registering interrupt logic

Browse source 
Lose a level of indentation and remove a superfluous `handler_slot`
check.
This commit is contained in:
Jelle Raaijmakers 2023-01-17 20:47:51 +01:00
parent 100c8f9bcf
commit 5f85f1abaa
2 changed files with 59 additions and 64 deletions
Download patch file Download diff file Expand all files Collapse all files

13
Kernel/Arch/aarch64/Interrupts.cpp
View file

@ -64,13 +64,14 @@ static void revert_to_unused_handler(u8 interrupt_number)
void register_generic_interrupt_handler(u8 interrupt_number, GenericInterruptHandler& handler) void register_generic_interrupt_handler(u8 interrupt_number, GenericInterruptHandler& handler)
{ {
auto*& handler_slot = s_interrupt_handlers[interrupt_number]; auto*& handler_slot = s_interrupt_handlers[interrupt_number];
if (handler_slot != nullptr) { if (handler_slot == nullptr) {
handler_slot = &handler;
return;
}
if (handler_slot->type() == HandlerType::UnhandledInterruptHandler) { if (handler_slot->type() == HandlerType::UnhandledInterruptHandler) {
if (handler_slot) {
auto* unhandled_handler = static_cast<UnhandledInterruptHandler*>(handler_slot); auto* unhandled_handler = static_cast<UnhandledInterruptHandler*>(handler_slot);
unhandled_handler->unregister_interrupt_handler(); unhandled_handler->unregister_interrupt_handler();
delete unhandled_handler; delete unhandled_handler;
}
handler_slot = &handler; handler_slot = &handler;
return; return;
} }
@ -94,18 +95,14 @@ void register_generic_interrupt_handler(u8 interrupt_number, GenericInterruptHan
return; return;
} }
VERIFY_NOT_REACHED(); VERIFY_NOT_REACHED();
} else {
handler_slot = &handler;
}
} }
void unregister_generic_interrupt_handler(u8 interrupt_number, GenericInterruptHandler& handler) void unregister_generic_interrupt_handler(u8 interrupt_number, GenericInterruptHandler& handler)
{ {
auto*& handler_slot = s_interrupt_handlers[interrupt_number]; auto*& handler_slot = s_interrupt_handlers[interrupt_number];
VERIFY(handler_slot != nullptr); VERIFY(handler_slot != nullptr);
if (handler_slot->type() == HandlerType::UnhandledInterruptHandler) { if (handler_slot->type() == HandlerType::UnhandledInterruptHandler)
return; return;
}
if (handler_slot->is_shared_handler() && !handler_slot->is_sharing_with_others()) { if (handler_slot->is_shared_handler() && !handler_slot->is_sharing_with_others()) {
VERIFY(handler_slot->type() == HandlerType::SharedIRQHandler); VERIFY(handler_slot->type() == HandlerType::SharedIRQHandler);
auto* shared_handler = static_cast<SharedIRQHandler*>(handler_slot); auto* shared_handler = static_cast<SharedIRQHandler*>(handler_slot);

10
Kernel/Arch/x86_64/Interrupts.cpp
View file

@ -478,13 +478,14 @@ void register_generic_interrupt_handler(u8 interrupt_number, GenericInterruptHan
{ {
VERIFY(interrupt_number < GENERIC_INTERRUPT_HANDLERS_COUNT); VERIFY(interrupt_number < GENERIC_INTERRUPT_HANDLERS_COUNT);
auto*& handler_slot = s_interrupt_handler[interrupt_number]; auto*& handler_slot = s_interrupt_handler[interrupt_number];
if (handler_slot != nullptr) { if (handler_slot == nullptr) {
handler_slot = &handler;
return;
}
if (handler_slot->type() == HandlerType::UnhandledInterruptHandler) { if (handler_slot->type() == HandlerType::UnhandledInterruptHandler) {
if (handler_slot) {
auto* unhandled_handler = static_cast<UnhandledInterruptHandler*>(handler_slot); auto* unhandled_handler = static_cast<UnhandledInterruptHandler*>(handler_slot);
unhandled_handler->unregister_interrupt_handler(); unhandled_handler->unregister_interrupt_handler();
delete unhandled_handler; delete unhandled_handler;
}
handler_slot = &handler; handler_slot = &handler;
return; return;
} }
@ -508,9 +509,6 @@ void register_generic_interrupt_handler(u8 interrupt_number, GenericInterruptHan
return; return;
} }
VERIFY_NOT_REACHED(); VERIFY_NOT_REACHED();
} else {
handler_slot = &handler;
}
} }
void unregister_generic_interrupt_handler(u8 interrupt_number, GenericInterruptHandler& handler) void unregister_generic_interrupt_handler(u8 interrupt_number, GenericInterruptHandler& handler)