LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-12-27 21:21:50 +03:00
Code Issues Packages Projects Releases Wiki Activity

Kernel: Remove is_sharing_with_others API from GenericInterruptHandler

Browse Source 
is_sharing_with_others API was never really put to use properly since
it was introduced. The only place where it is used in Interrupts.cpp is
in conjuction with is_shared_handler() which is only true for
SharedIRQHandler and is_sharing_with_others will always return false.

Remove that API.
This commit is contained in:
Pankaj Raghav 2023-04-12 22:55:20 +02:00 committed by Andreas Kling
parent 756a73471e
commit 8944ca830f
Notes: sideshowbarker 2024-07-17 02:59:43 +09:00 
Author: https://github.com/Panky-codes
Commit: https://github.com/SerenityOS/serenity/commit/8944ca830f
Pull-request: https://github.com/SerenityOS/serenity/pull/18301
Reviewed-by: https://github.com/awesomekling
Reviewed-by: https://github.com/supercomputer7 ✅
8 changed files with 2 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Kernel/Arch/x86_64/Interrupts.cpp
View File

@ -361,7 +361,7 @@ void register_generic_interrupt_handler(u8 interrupt_number, GenericInterruptHan
handler_slot = &handler;
return;
}
if (handler_slot->is_shared_handler() && !handler_slot->is_sharing_with_others()) {
if (handler_slot->is_shared_handler()) {
VERIFY(handler_slot->type() == HandlerType::SharedIRQHandler);
static_cast<SharedIRQHandler*>(handler_slot)->register_handler(handler);
return;
@ -390,7 +390,7 @@ void unregister_generic_interrupt_handler(u8 interrupt_number, GenericInterruptH
VERIFY(handler_slot != nullptr);
if (handler_slot->type() == HandlerType::UnhandledInterruptHandler)
return;
if (handler_slot->is_shared_handler() && !handler_slot->is_sharing_with_others()) {
if (handler_slot->is_shared_handler()) {
VERIFY(handler_slot->type() == HandlerType::SharedIRQHandler);
auto* shared_handler = static_cast<SharedIRQHandler*>(handler_slot);
shared_handler->unregister_handler(handler);

2
Kernel/Arch/x86_64/Interrupts/APIC.cpp
View File

@ -84,7 +84,6 @@ public:
virtual size_t sharing_devices_count() const override { return 0; }
virtual bool is_shared_handler() const override { return false; }
virtual bool is_sharing_with_others() const override { return false; }
private:
};
@ -115,7 +114,6 @@ public:
virtual size_t sharing_devices_count() const override { return 0; }
virtual bool is_shared_handler() const override { return false; }
virtual bool is_sharing_with_others() const override { return false; }
private:
};

1
Kernel/Interrupts/GenericInterruptHandler.h
View File

@ -41,7 +41,6 @@ public:
virtual size_t sharing_devices_count() const = 0;
virtual bool is_shared_handler() const = 0;
virtual bool is_sharing_with_others() const = 0;
virtual HandlerType type() const = 0;
virtual StringView purpose() const = 0;

1
Kernel/Interrupts/IRQHandler.h
View File

@ -31,7 +31,6 @@ public:
virtual size_t sharing_devices_count() const override { return 0; }
virtual bool is_shared_handler() const override { return false; }
virtual bool is_sharing_with_others() const override { return m_shared_with_others; }
void set_shared_with_others(bool status) { m_shared_with_others = status; }
protected:

1
Kernel/Interrupts/SharedIRQHandler.h
View File

@ -33,7 +33,6 @@ public:
return m_handlers.with([](auto& list) { return list.size_slow(); });
}
virtual bool is_shared_handler() const override { return true; }
virtual bool is_sharing_with_others() const override { return false; }
virtual HandlerType type() const override { return HandlerType::SharedIRQHandler; }
virtual StringView purpose() const override { return "Shared IRQ Handler"sv; }

1
Kernel/Interrupts/SpuriousInterruptHandler.h
View File

@ -28,7 +28,6 @@ public:
virtual size_t sharing_devices_count() const override { return 1; }
virtual bool is_shared_handler() const override { return false; }
virtual bool is_sharing_with_others() const override { return false; }
virtual HandlerType type() const override { return HandlerType::SpuriousInterruptHandler; }
virtual StringView purpose() const override;

1
Kernel/Interrupts/UnhandledInterruptHandler.h
View File

@ -25,7 +25,6 @@ public:
virtual size_t sharing_devices_count() const override { return 0; }
virtual bool is_shared_handler() const override { return false; }
virtual bool is_sharing_with_others() const override { return false; }
private:
};

1
Kernel/Time/HardwareTimer.h
View File

@ -133,7 +133,6 @@ public:
virtual size_t sharing_devices_count() const override { return 0; }
virtual bool is_shared_handler() const override { return false; }
virtual bool is_sharing_with_others() const override { return false; }
virtual HandlerType type() const override { return HandlerType::IRQHandler; }
virtual StringView controller() const override { return {}; }
virtual bool eoi() override;