summaryrefslogtreecommitdiff
path: root/Kernel/Bus
diff options
context:
space:
mode:
authorPankaj Raghav <dev@pankajraghav.com>2023-05-08 21:29:37 +0200
committerJelle Raaijmakers <jelle@gmta.nl>2023-05-16 23:22:12 +0200
commit8f62e62cfedb5084a6663ea96f7579c51eb0dee1 (patch)
tree251cbdf7755e278f8be73a40566ae32bfe6af56b /Kernel/Bus
parent35a844ac75f97f9a29934b31c119eb82a5d8baf2 (diff)
downloadserenity-8f62e62cfedb5084a6663ea96f7579c51eb0dee1.zip
Kernel: Add MSI support in PCI Device
Extend reserve_irqs, allocate_irq, enable_interrupt and disable_interrupt API to add MSI support in PCI device. The current changes only implement single MSI message support. TODOs have been added to support Multiple MSI Message (MME) support in the future.
Diffstat (limited to 'Kernel/Bus')
-rw-r--r--Kernel/Bus/PCI/Definitions.h3
-rw-r--r--Kernel/Bus/PCI/Device.cpp35
2 files changed, 34 insertions, 4 deletions
diff --git a/Kernel/Bus/PCI/Definitions.h b/Kernel/Bus/PCI/Definitions.h
index e8e3268b69..8698452115 100644
--- a/Kernel/Bus/PCI/Definitions.h
+++ b/Kernel/Bus/PCI/Definitions.h
@@ -81,6 +81,9 @@ static constexpr size_t memory_range_per_bus = mmio_device_space_size * to_under
static constexpr u32 bar_address_mask = 0xfffffff0;
static constexpr u8 msi_control_offset = 2;
static constexpr u16 msi_control_enable = 0x0001;
+static constexpr u8 msi_address_low_offset = 4;
+static constexpr u8 msi_address_high_or_data_offset = 8;
+static constexpr u8 msi_data_offset = 0xc;
static constexpr u16 msi_address_format_mask = 0x80;
static constexpr u8 msi_mmc_format_mask = 0xe;
static constexpr u16 msix_control_table_mask = 0x07ff;
diff --git a/Kernel/Bus/PCI/Device.cpp b/Kernel/Bus/PCI/Device.cpp
index 8bfb17d2ae..45eecfc677 100644
--- a/Kernel/Bus/PCI/Device.cpp
+++ b/Kernel/Bus/PCI/Device.cpp
@@ -91,7 +91,15 @@ ErrorOr<InterruptType> Device::reserve_irqs(u8 number_of_irqs, bool msi)
disable_pin_based_interrupts();
enable_extended_message_signalled_interrupts();
} else if (msi && is_msi_capable()) {
- TODO();
+ // TODO: Add MME support. Fallback to pin-based until this support is added.
+ if (number_of_irqs > 1)
+ return m_interrupt_range.m_type;
+
+ m_interrupt_range.m_start_irq = TRY(reserve_interrupt_handlers(number_of_irqs));
+ m_interrupt_range.m_irq_count = number_of_irqs;
+ m_interrupt_range.m_type = InterruptType::MSI;
+ disable_pin_based_interrupts();
+ enable_message_signalled_interrupts();
}
return m_interrupt_range.m_type;
}
@@ -132,7 +140,26 @@ ErrorOr<u8> Device::allocate_irq(u8 index)
return m_interrupt_range.m_start_irq + index;
} else if ((m_interrupt_range.m_type == InterruptType::MSI) && is_msi_capable()) {
- TODO();
+ // TODO: Add MME support.
+ if (index > 0)
+ return Error::from_errno(EINVAL);
+
+ auto data = msi_data_register(m_interrupt_range.m_start_irq + index, false, false);
+ auto addr = msi_address_register(0, false, false);
+ for (auto& capability : m_pci_identifier->capabilities()) {
+ if (capability.id().value() == PCI::Capabilities::ID::MSI) {
+ capability.write32(msi_address_low_offset, addr & 0xffffffff);
+
+ if (!m_pci_identifier->is_msi_64bit_address_format()) {
+ capability.write16(msi_address_high_or_data_offset, data);
+ break;
+ }
+
+ capability.write32(msi_address_high_or_data_offset, addr >> 32);
+ capability.write16(msi_data_offset, data);
+ }
+ }
+ return m_interrupt_range.m_start_irq + index;
}
// For pin based interrupts, we share the IRQ.
return m_interrupt_range.m_start_irq;
@@ -152,7 +179,7 @@ void Device::enable_interrupt(u8 irq)
u32 vector_ctrl = msix_vector_control_register(entry_ptr->vector_control, false);
entry_ptr->vector_control = vector_ctrl;
} else if ((m_interrupt_range.m_type == InterruptType::MSI) && is_msi_capable()) {
- TODO();
+ enable_message_signalled_interrupts();
}
}
@@ -170,7 +197,7 @@ void Device::disable_interrupt(u8 irq)
u32 vector_ctrl = msix_vector_control_register(entry_ptr->vector_control, true);
entry_ptr->vector_control = vector_ctrl;
} else if ((m_interrupt_range.m_type == InterruptType::MSI) && is_msi_capable()) {
- TODO();
+ disable_message_signalled_interrupts();
}
}