beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-09-30 00:31:14 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

Kernel: Use IO ports instad of MMIO with Bochs graphics in VirtualBox

Browse source 
This is needed for VirtualBox, because it doesn't support controlling
the device with MMIO.

Fixes #7558.
This commit is contained in:
Liav A 2021-06-04 11:25:42 +03:00 committed by Andreas Kling
parent d18d91dedc
commit 6a9dc5562d
Notes: sideshowbarker 2024-07-18 16:54:53 +09:00 
Author: https://github.com/supercomputer7
Commit: https://github.com/SerenityOS/serenity/commit/6a9dc5562db
Pull-request: https://github.com/SerenityOS/serenity/pull/7765
Issue: https://github.com/SerenityOS/serenity/issues/7558
2 changed files with 53 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

53
Kernel/Graphics/BochsGraphicsAdapter.cpp
View file

@ -56,6 +56,11 @@ UNMAP_AFTER_INIT BochsGraphicsAdapter::BochsGraphicsAdapter(PCI::Address pci_add
m_framebuffer_console = Graphics::FramebufferConsole::initialize(PhysicalAddress(PCI::get_BAR0(pci_address) & 0xfffffff0), 1024, 768, 1024 * sizeof(u32));
// FIXME: This is a very wrong way to do this...
GraphicsManagement::the().m_console = m_framebuffer_console;
// Note: If we use VirtualBox graphics adapter (which is based on Bochs one), we need to use IO ports
auto id = PCI::get_id(pci_address);
if (id.vendor_id == 0x80ee && id.device_id == 0xbeef)
m_io_required = true;
set_safe_resolution();
}
@ -80,6 +85,32 @@ void BochsGraphicsAdapter::set_safe_resolution()
VERIFY(result);
}
static void set_register_with_io(u16 index, u16 data)
{
IO::out16(VBE_DISPI_IOPORT_INDEX, index);
IO::out16(VBE_DISPI_IOPORT_DATA, data);
}
static u16 get_register_with_io(u16 index)
{
IO::out16(VBE_DISPI_IOPORT_INDEX, index);
return IO::in16(VBE_DISPI_IOPORT_DATA);
}
void BochsGraphicsAdapter::set_resolution_registers_via_io(size_t width, size_t height)
{
dbgln_if(BXVGA_DEBUG, "BochsGraphicsAdapter resolution registers set to - {}x{}", width, height);
set_register_with_io(VBE_DISPI_INDEX_ENABLE, VBE_DISPI_DISABLED);
set_register_with_io(VBE_DISPI_INDEX_XRES, (u16)width);
set_register_with_io(VBE_DISPI_INDEX_YRES, (u16)height);
set_register_with_io(VBE_DISPI_INDEX_VIRT_WIDTH, (u16)width);
set_register_with_io(VBE_DISPI_INDEX_VIRT_HEIGHT, (u16)height * 2);
set_register_with_io(VBE_DISPI_INDEX_BPP, 32);
set_register_with_io(VBE_DISPI_INDEX_ENABLE, VBE_DISPI_ENABLED | VBE_DISPI_LFB_ENABLED);
set_register_with_io(VBE_DISPI_INDEX_BANK, 0);
}
void BochsGraphicsAdapter::set_resolution_registers(size_t width, size_t height)
{
dbgln_if(BXVGA_DEBUG, "BochsGraphicsAdapter resolution registers set to - {}x{}", width, height);
@ -105,16 +136,32 @@ bool BochsGraphicsAdapter::try_to_set_resolution(size_t output_port_index, size_
if (Checked<size_t>::multiplication_would_overflow(width, height, sizeof(u32)))
return false;
set_resolution_registers(width, height);
if (m_io_required)
set_resolution_registers_via_io(width, height);
else
set_resolution_registers(width, height);
dbgln_if(BXVGA_DEBUG, "BochsGraphicsAdapter resolution test - {}x{}", width, height);
if (!validate_setup_resolution(width, height))
return false;
if (m_io_required) {
if (!validate_setup_resolution_with_io(width, height))
return false;
} else {
if (!validate_setup_resolution(width, height))
return false;
}
dbgln("BochsGraphicsAdapter: resolution set to {}x{}", width, height);
m_framebuffer_console->set_resolution(width, height, width * sizeof(u32));
return true;
}
bool BochsGraphicsAdapter::validate_setup_resolution_with_io(size_t width, size_t height)
{
if ((u16)width != get_register_with_io(VBE_DISPI_INDEX_XRES) || (u16)height != get_register_with_io(VBE_DISPI_INDEX_YRES)) {
return false;
}
return true;
}
bool BochsGraphicsAdapter::validate_setup_resolution(size_t width, size_t height)
{
auto registers = map_typed_writable<volatile BochsDisplayMMIORegisters>(m_mmio_registers);

3
Kernel/Graphics/BochsGraphicsAdapter.h
View file

@ -50,6 +50,8 @@ private:
bool validate_setup_resolution(size_t width, size_t height);
u32 find_framebuffer_address();
void set_resolution_registers(size_t width, size_t height);
void set_resolution_registers_via_io(size_t width, size_t height);
bool validate_setup_resolution_with_io(size_t width, size_t height);
void set_y_offset(size_t);
PhysicalAddress m_mmio_registers;
@ -57,6 +59,7 @@ private:
RefPtr<Graphics::FramebufferConsole> m_framebuffer_console;
SpinLock<u8> m_console_mode_switch_lock;
bool m_console_enabled { false };
bool m_io_required { false };
};
}