Kernel: Move x86 Bochs VBE code to the Arch/x86 directory

To do this, we make the QEMUDisplayConnector class more standalone so it
does not need to inherit from the BochsDisplayConnector class.

6 files changed, 81 insertions, 214 deletions

diff --git a/Kernel/Graphics/Bochs/DisplayConnector.cpp b/Kernel/Graphics/Bochs/DisplayConnector.cpp
deleted file mode 100644
index 62e9b99ed5..0000000000
--- a/Kernel/Graphics/Bochs/DisplayConnector.cpp
+++ /dev/null
@@ -1,150 +0,0 @@
-/*
- * Copyright (c) 2022, Liav A. <liavalb@hotmail.co.il>
- *
- * SPDX-License-Identifier: BSD-2-Clause
- */
-
-#include <Kernel/Arch/x86/IO.h>
-#include <Kernel/Debug.h>
-#include <Kernel/Devices/DeviceManagement.h>
-#include <Kernel/Graphics/Bochs/Definitions.h>
-#include <Kernel/Graphics/Bochs/DisplayConnector.h>
-#include <Kernel/Graphics/Console/ContiguousFramebufferConsole.h>
-#include <Kernel/Graphics/GraphicsManagement.h>
-
-namespace Kernel {
-
-NonnullLockRefPtr<BochsDisplayConnector> BochsDisplayConnector::must_create(PhysicalAddress framebuffer_address, size_t framebuffer_resource_size, bool virtual_box_hardware)
-{
-    auto device_or_error = DeviceManagement::try_create_device<BochsDisplayConnector>(framebuffer_address, framebuffer_resource_size);
-    VERIFY(!device_or_error.is_error());
-    auto connector = device_or_error.release_value();
-    MUST(connector->create_attached_framebuffer_console());
-    if (virtual_box_hardware)
-        MUST(connector->initialize_edid_for_generic_monitor(Array<u8, 3> { 'V', 'B', 'X' }));
-    else
-        MUST(connector->initialize_edid_for_generic_monitor({}));
-    return connector;
-}
-
-BochsDisplayConnector::BochsDisplayConnector(PhysicalAddress framebuffer_address, size_t framebuffer_resource_size)
-    : DisplayConnector(framebuffer_address, framebuffer_resource_size, false)
-{
-}
-
-ErrorOr<void> BochsDisplayConnector::create_attached_framebuffer_console()
-{
-    // We assume safe resolution is 1024x768x32
-    m_framebuffer_console = Graphics::ContiguousFramebufferConsole::initialize(m_framebuffer_address.value(), 1024, 768, 1024 * sizeof(u32));
-    GraphicsManagement::the().set_console(*m_framebuffer_console);
-    return {};
-}
-
-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);
-}
-
-BochsDisplayConnector::IndexID BochsDisplayConnector::index_id() const
-{
-    return get_register_with_io(0);
-}
-
-void BochsDisplayConnector::enable_console()
-{
-    VERIFY(m_control_lock.is_locked());
-    VERIFY(m_framebuffer_console);
-    m_framebuffer_console->enable();
-}
-
-void BochsDisplayConnector::disable_console()
-{
-    VERIFY(m_control_lock.is_locked());
-    VERIFY(m_framebuffer_console);
-    m_framebuffer_console->disable();
-}
-
-ErrorOr<void> BochsDisplayConnector::flush_first_surface()
-{
-    return Error::from_errno(ENOTSUP);
-}
-
-ErrorOr<void> BochsDisplayConnector::set_safe_mode_setting()
-{
-    DisplayConnector::ModeSetting safe_mode_set {
-        .horizontal_stride = 1024 * sizeof(u32),
-        .pixel_clock_in_khz = 0, // Note: There's no pixel clock in paravirtualized hardware
-        .horizontal_active = 1024,
-        .horizontal_front_porch_pixels = 0, // Note: There's no horizontal_front_porch_pixels in paravirtualized hardware
-        .horizontal_sync_time_pixels = 0,   // Note: There's no horizontal_sync_time_pixels in paravirtualized hardware
-        .horizontal_blank_pixels = 0,       // Note: There's no horizontal_blank_pixels in paravirtualized hardware
-        .vertical_active = 768,
-        .vertical_front_porch_lines = 0, // Note: There's no vertical_front_porch_lines in paravirtualized hardware
-        .vertical_sync_time_lines = 0,   // Note: There's no vertical_sync_time_lines in paravirtualized hardware
-        .vertical_blank_lines = 0,       // Note: There's no vertical_blank_lines in paravirtualized hardware
-        .horizontal_offset = 0,
-        .vertical_offset = 0,
-    };
-    return set_mode_setting(safe_mode_set);
-}
-
-ErrorOr<void> BochsDisplayConnector::set_mode_setting(ModeSetting const& mode_setting)
-{
-    SpinlockLocker locker(m_modeset_lock);
-    size_t width = mode_setting.horizontal_active;
-    size_t height = mode_setting.vertical_active;
-
-    dbgln_if(BXVGA_DEBUG, "BochsDisplayConnector resolution registers set to - {}x{}", width, height);
-
-    set_register_with_io(to_underlying(BochsDISPIRegisters::ENABLE), 0);
-    set_register_with_io(to_underlying(BochsDISPIRegisters::XRES), (u16)width);
-    set_register_with_io(to_underlying(BochsDISPIRegisters::YRES), (u16)height);
-    set_register_with_io(to_underlying(BochsDISPIRegisters::VIRT_WIDTH), (u16)width);
-    set_register_with_io(to_underlying(BochsDISPIRegisters::VIRT_HEIGHT), (u16)height * 2);
-    set_register_with_io(to_underlying(BochsDISPIRegisters::BPP), 32);
-    set_register_with_io(to_underlying(BochsDISPIRegisters::ENABLE), to_underlying(BochsFramebufferSettings::Enabled) | to_underlying(BochsFramebufferSettings::LinearFramebuffer));
-    set_register_with_io(to_underlying(BochsDISPIRegisters::BANK), 0);
-    if ((u16)width != get_register_with_io(to_underlying(BochsDISPIRegisters::XRES)) || (u16)height != get_register_with_io(to_underlying(BochsDISPIRegisters::YRES))) {
-        return Error::from_errno(ENOTIMPL);
-    }
-    auto current_horizontal_active = get_register_with_io(to_underlying(BochsDISPIRegisters::XRES));
-    auto current_vertical_active = get_register_with_io(to_underlying(BochsDISPIRegisters::YRES));
-    DisplayConnector::ModeSetting mode_set {
-        .horizontal_stride = current_horizontal_active * sizeof(u32),
-        .pixel_clock_in_khz = 0, // Note: There's no pixel clock in paravirtualized hardware
-        .horizontal_active = current_horizontal_active,
-        .horizontal_front_porch_pixels = 0, // Note: There's no horizontal_front_porch_pixels in paravirtualized hardware
-        .horizontal_sync_time_pixels = 0,   // Note: There's no horizontal_sync_time_pixels in paravirtualized hardware
-        .horizontal_blank_pixels = 0,       // Note: There's no horizontal_blank_pixels in paravirtualized hardware
-        .vertical_active = current_vertical_active,
-        .vertical_front_porch_lines = 0, // Note: There's no vertical_front_porch_lines in paravirtualized hardware
-        .vertical_sync_time_lines = 0,   // Note: There's no vertical_sync_time_lines in paravirtualized hardware
-        .vertical_blank_lines = 0,       // Note: There's no vertical_blank_lines in paravirtualized hardware
-        .horizontal_offset = 0,
-        .vertical_offset = 0,
-    };
-    m_current_mode_setting = mode_set;
-    return {};
-}
-
-ErrorOr<void> BochsDisplayConnector::set_y_offset(size_t)
-{
-    // Note: Although when using this device on QEMU we can actually set the horizontal and vertical offsets
-    // with IO ports, this class is meant to be used for plain old Bochs graphics which might not support
-    // this feature at all.
-    return Error::from_errno(ENOTIMPL);
-}
-
-ErrorOr<void> BochsDisplayConnector::unblank()
-{
-    return Error::from_errno(ENOTIMPL);
-}
-
-}
diff --git a/Kernel/Graphics/Bochs/DisplayConnector.h b/Kernel/Graphics/Bochs/DisplayConnector.h
deleted file mode 100644
index 4359b4aecf..0000000000
--- a/Kernel/Graphics/Bochs/DisplayConnector.h
+++ /dev/null
@@ -1,55 +0,0 @@
-/*
- * Copyright (c) 2022, Liav A. <liavalb@hotmail.co.il>
- *
- * SPDX-License-Identifier: BSD-2-Clause
- */
-
-#pragma once
-
-#include <AK/Try.h>
-#include <Kernel/Graphics/Bochs/Definitions.h>
-#include <Kernel/Graphics/Console/GenericFramebufferConsole.h>
-#include <Kernel/Graphics/DisplayConnector.h>
-#include <Kernel/Library/LockRefPtr.h>
-#include <Kernel/Locking/Spinlock.h>
-#include <Kernel/Memory/TypedMapping.h>
-
-namespace Kernel {
-
-class BochsDisplayConnector
-    : public DisplayConnector {
-    friend class BochsGraphicsAdapter;
-    friend class DeviceManagement;
-
-public:
-    AK_TYPEDEF_DISTINCT_ORDERED_ID(u16, IndexID);
-
-    static NonnullLockRefPtr<BochsDisplayConnector> must_create(PhysicalAddress framebuffer_address, size_t framebuffer_resource_size, bool virtual_box_hardware);
-
-    virtual IndexID index_id() const;
-
-protected:
-    ErrorOr<void> create_attached_framebuffer_console();
-
-    BochsDisplayConnector(PhysicalAddress framebuffer_address, size_t framebuffer_resource_size);
-
-    virtual bool mutable_mode_setting_capable() const override final { return true; }
-    virtual bool double_framebuffering_capable() const override { return false; }
-    virtual ErrorOr<void> set_mode_setting(ModeSetting const&) override;
-    virtual ErrorOr<void> set_safe_mode_setting() override final;
-    virtual ErrorOr<void> set_y_offset(size_t y) override;
-    virtual ErrorOr<void> unblank() override;
-
-    virtual bool partial_flush_support() const override final { return false; }
-    virtual bool flush_support() const override final { return false; }
-    // Note: Paravirtualized hardware doesn't require a defined refresh rate for modesetting.
-    virtual bool refresh_rate_support() const override final { return false; }
-
-    virtual ErrorOr<void> flush_first_surface() override final;
-
-    virtual void enable_console() override final;
-    virtual void disable_console() override final;
-
-    LockRefPtr<Graphics::GenericFramebufferConsole> m_framebuffer_console;
-};
-}
diff --git a/Kernel/Graphics/Bochs/GraphicsAdapter.cpp b/Kernel/Graphics/Bochs/GraphicsAdapter.cpp
index 7a41ff9217..33d1a5e13b 100644
--- a/Kernel/Graphics/Bochs/GraphicsAdapter.cpp
+++ b/Kernel/Graphics/Bochs/GraphicsAdapter.cpp
@@ -7,11 +7,13 @@
 #include <AK/Atomic.h>
 #include <AK/Checked.h>
 #include <AK/Try.h>
+#if ARCH(I386) || ARCH(X86_64)
+#    include <Kernel/Arch/x86/Hypervisor/BochsDisplayConnector.h>
+#endif
 #include <Kernel/Bus/PCI/API.h>
 #include <Kernel/Bus/PCI/IDs.h>
 #include <Kernel/Debug.h>
 #include <Kernel/Graphics/Bochs/Definitions.h>
-#include <Kernel/Graphics/Bochs/DisplayConnector.h>
 #include <Kernel/Graphics/Bochs/GraphicsAdapter.h>
 #include <Kernel/Graphics/Bochs/QEMUDisplayConnector.h>
 #include <Kernel/Graphics/Console/ContiguousFramebufferConsole.h>
@@ -40,6 +42,10 @@ UNMAP_AFTER_INIT ErrorOr<void> BochsGraphicsAdapter::initialize_adapter(PCI::Dev
     // Note: If we use VirtualBox graphics adapter (which is based on Bochs one), we need to use IO ports
     // Note: Bochs (the real bochs graphics adapter in the Bochs emulator) uses revision ID of 0x0
     // and doesn't support memory-mapped IO registers.
+
+    // Note: In non x86-builds, we should never encounter VirtualBox hardware nor Pure Bochs VBE graphics,
+    // so just assume we can use the QEMU BochsVBE-compatible graphics adapter only.
+#if ARCH(I386) || ARCH(X86_64)
     bool virtual_box_hardware = (pci_device_identifier.hardware_id().vendor_id == 0x80ee && pci_device_identifier.hardware_id().device_id == 0xbeef);
     auto bar0_space_size = PCI::get_BAR_space_size(pci_device_identifier.address(), PCI::HeaderType0BaseRegister::BAR0);
     if (pci_device_identifier.revision_id().value() == 0x0 || virtual_box_hardware) {
@@ -49,6 +55,11 @@ UNMAP_AFTER_INIT ErrorOr<void> BochsGraphicsAdapter::initialize_adapter(PCI::Dev
         VERIFY(registers_mapping.region);
         m_display_connector = QEMUDisplayConnector::must_create(PhysicalAddress(PCI::get_BAR0(pci_device_identifier.address()) & 0xfffffff0), bar0_space_size, move(registers_mapping));
     }
+#else
+    auto registers_mapping = TRY(Memory::map_typed_writable<BochsDisplayMMIORegisters volatile>(PhysicalAddress(PCI::get_BAR2(pci_device_identifier.address()) & 0xfffffff0)));
+    VERIFY(registers_mapping.region);
+    m_display_connector = QEMUDisplayConnector::must_create(PhysicalAddress(PCI::get_BAR0(pci_device_identifier.address()) & 0xfffffff0), bar0_space_size, move(registers_mapping));
+#endif
 
     // Note: According to Gerd Hoffmann - "The linux driver simply does
     // the unblank unconditionally. With bochs-display this is not needed but
diff --git a/Kernel/Graphics/Bochs/GraphicsAdapter.h b/Kernel/Graphics/Bochs/GraphicsAdapter.h
index d5ba3910f8..bb298a3a5f 100644
--- a/Kernel/Graphics/Bochs/GraphicsAdapter.h
+++ b/Kernel/Graphics/Bochs/GraphicsAdapter.h
@@ -19,7 +19,6 @@ namespace Kernel {
 class GraphicsManagement;
 struct BochsDisplayMMIORegisters;
 
-class BochsDisplayConnector;
 class BochsGraphicsAdapter final : public GenericGraphicsAdapter
     , public PCI::Device {
     friend class GraphicsManagement;
@@ -33,6 +32,6 @@ private:
 
     explicit BochsGraphicsAdapter(PCI::DeviceIdentifier const&);
 
-    LockRefPtr<BochsDisplayConnector> m_display_connector;
+    LockRefPtr<DisplayConnector> m_display_connector;
 };
 }
diff --git a/Kernel/Graphics/Bochs/QEMUDisplayConnector.cpp b/Kernel/Graphics/Bochs/QEMUDisplayConnector.cpp
index d74f788ef8..69d1a5bc58 100644
--- a/Kernel/Graphics/Bochs/QEMUDisplayConnector.cpp
+++ b/Kernel/Graphics/Bochs/QEMUDisplayConnector.cpp
@@ -7,6 +7,7 @@
 #include <Kernel/Debug.h>
 #include <Kernel/Devices/DeviceManagement.h>
 #include <Kernel/Graphics/Bochs/QEMUDisplayConnector.h>
+#include <Kernel/Graphics/Console/ContiguousFramebufferConsole.h>
 #include <Kernel/Graphics/GraphicsManagement.h>
 #include <LibEDID/Definitions.h>
 
@@ -31,8 +32,54 @@ ErrorOr<void> QEMUDisplayConnector::fetch_and_initialize_edid()
     return {};
 }
 
+ErrorOr<void> QEMUDisplayConnector::create_attached_framebuffer_console()
+{
+    // We assume safe resolution is 1024x768x32
+    m_framebuffer_console = Graphics::ContiguousFramebufferConsole::initialize(m_framebuffer_address.value(), 1024, 768, 1024 * sizeof(u32));
+    GraphicsManagement::the().set_console(*m_framebuffer_console);
+    return {};
+}
+
+void QEMUDisplayConnector::enable_console()
+{
+    VERIFY(m_control_lock.is_locked());
+    VERIFY(m_framebuffer_console);
+    m_framebuffer_console->enable();
+}
+
+void QEMUDisplayConnector::disable_console()
+{
+    VERIFY(m_control_lock.is_locked());
+    VERIFY(m_framebuffer_console);
+    m_framebuffer_console->disable();
+}
+
+ErrorOr<void> QEMUDisplayConnector::flush_first_surface()
+{
+    return Error::from_errno(ENOTSUP);
+}
+
+ErrorOr<void> QEMUDisplayConnector::set_safe_mode_setting()
+{
+    DisplayConnector::ModeSetting safe_mode_set {
+        .horizontal_stride = 1024 * sizeof(u32),
+        .pixel_clock_in_khz = 0, // Note: There's no pixel clock in paravirtualized hardware
+        .horizontal_active = 1024,
+        .horizontal_front_porch_pixels = 0, // Note: There's no horizontal_front_porch_pixels in paravirtualized hardware
+        .horizontal_sync_time_pixels = 0,   // Note: There's no horizontal_sync_time_pixels in paravirtualized hardware
+        .horizontal_blank_pixels = 0,       // Note: There's no horizontal_blank_pixels in paravirtualized hardware
+        .vertical_active = 768,
+        .vertical_front_porch_lines = 0, // Note: There's no vertical_front_porch_lines in paravirtualized hardware
+        .vertical_sync_time_lines = 0,   // Note: There's no vertical_sync_time_lines in paravirtualized hardware
+        .vertical_blank_lines = 0,       // Note: There's no vertical_blank_lines in paravirtualized hardware
+        .horizontal_offset = 0,
+        .vertical_offset = 0,
+    };
+    return set_mode_setting(safe_mode_set);
+}
+
 QEMUDisplayConnector::QEMUDisplayConnector(PhysicalAddress framebuffer_address, size_t framebuffer_resource_size, Memory::TypedMapping<BochsDisplayMMIORegisters volatile> registers_mapping)
-    : BochsDisplayConnector(framebuffer_address, framebuffer_resource_size)
+    : DisplayConnector(framebuffer_address, framebuffer_resource_size, false)
     , m_registers(move(registers_mapping))
 {
 }
diff --git a/Kernel/Graphics/Bochs/QEMUDisplayConnector.h b/Kernel/Graphics/Bochs/QEMUDisplayConnector.h
index b08cc1b47b..4fa68d3e8d 100644
--- a/Kernel/Graphics/Bochs/QEMUDisplayConnector.h
+++ b/Kernel/Graphics/Bochs/QEMUDisplayConnector.h
@@ -7,8 +7,9 @@
 #pragma once
 
 #include <AK/Try.h>
-#include <Kernel/Graphics/Bochs/DisplayConnector.h>
+#include <Kernel/Graphics/Bochs/Definitions.h>
 #include <Kernel/Graphics/Console/GenericFramebufferConsole.h>
+#include <Kernel/Graphics/DisplayConnector.h>
 #include <Kernel/Library/LockRefPtr.h>
 #include <Kernel/Locking/Spinlock.h>
 #include <Kernel/Memory/TypedMapping.h>
@@ -17,28 +18,42 @@ namespace Kernel {
 
 struct BochsDisplayMMIORegisters;
 class QEMUDisplayConnector final
-    : public BochsDisplayConnector {
+    : public DisplayConnector {
     friend class BochsGraphicsAdapter;
     friend class DeviceManagement;
 
 public:
-    static NonnullLockRefPtr<QEMUDisplayConnector> must_create(PhysicalAddress framebuffer_address, size_t framebuffer_resource_size, Memory::TypedMapping<BochsDisplayMMIORegisters volatile>);
+    AK_TYPEDEF_DISTINCT_ORDERED_ID(u16, IndexID);
 
-    virtual IndexID index_id() const override;
+    static NonnullLockRefPtr<QEMUDisplayConnector> must_create(PhysicalAddress framebuffer_address, size_t framebuffer_resource_size, Memory::TypedMapping<BochsDisplayMMIORegisters volatile>);
 
 private:
+    IndexID index_id() const;
+
     ErrorOr<void> fetch_and_initialize_edid();
+    ErrorOr<void> create_attached_framebuffer_console();
     QEMUDisplayConnector(PhysicalAddress framebuffer_address, size_t framebuffer_resource_size, Memory::TypedMapping<BochsDisplayMMIORegisters volatile>);
 
+    virtual bool mutable_mode_setting_capable() const override final { return true; }
     virtual bool double_framebuffering_capable() const override { return true; }
     virtual ErrorOr<void> set_mode_setting(ModeSetting const&) override;
     virtual ErrorOr<void> set_y_offset(size_t y) override;
+    virtual ErrorOr<void> set_safe_mode_setting() override final;
     virtual ErrorOr<void> unblank() override;
+    virtual bool partial_flush_support() const override final { return false; }
+    virtual bool flush_support() const override final { return false; }
+    // Note: Paravirtualized hardware doesn't require a defined refresh rate for modesetting.
+    virtual bool refresh_rate_support() const override final { return false; }
+    virtual ErrorOr<void> flush_first_surface() override final;
 
     void set_framebuffer_to_big_endian_format();
     void set_framebuffer_to_little_endian_format();
 
-private:
+    virtual void enable_console() override final;
+    virtual void disable_console() override final;
+
+    LockRefPtr<Graphics::GenericFramebufferConsole> m_framebuffer_console;
+
     Memory::TypedMapping<BochsDisplayMMIORegisters volatile> m_registers;
 };
 }