diff options
Diffstat (limited to 'src/core')
| -rw-r--r-- | src/core/hle/service/nvflinger/nvflinger.cpp | 53 | ||||
| -rw-r--r-- | src/core/hle/service/nvflinger/nvflinger.h | 2 | ||||
| -rw-r--r-- | src/core/hle/service/vi/display/vi_display.cpp | 49 | ||||
| -rw-r--r-- | src/core/hle/service/vi/display/vi_display.h | 72 | 
4 files changed, 141 insertions, 35 deletions
| diff --git a/src/core/hle/service/nvflinger/nvflinger.cpp b/src/core/hle/service/nvflinger/nvflinger.cpp index b5d452db1..6cc31050f 100644 --- a/src/core/hle/service/nvflinger/nvflinger.cpp +++ b/src/core/hle/service/nvflinger/nvflinger.cpp @@ -28,9 +28,13 @@ namespace Service::NVFlinger {  constexpr std::size_t SCREEN_REFRESH_RATE = 60;  constexpr u64 frame_ticks = static_cast<u64>(Core::Timing::BASE_CLOCK_RATE / SCREEN_REFRESH_RATE); -NVFlinger::NVFlinger(Core::Timing::CoreTiming& core_timing) -    : displays{{0, "Default"}, {1, "External"}, {2, "Edid"}, {3, "Internal"}, {4, "Null"}}, -      core_timing{core_timing} { +NVFlinger::NVFlinger(Core::Timing::CoreTiming& core_timing) : core_timing{core_timing} { +    displays.emplace_back(0, "Default"); +    displays.emplace_back(1, "External"); +    displays.emplace_back(2, "Edid"); +    displays.emplace_back(3, "Internal"); +    displays.emplace_back(4, "Null"); +      // Schedule the screen composition events      composition_event =          core_timing.RegisterEvent("ScreenComposition", [this](u64 userdata, int cycles_late) { @@ -55,13 +59,14 @@ std::optional<u64> NVFlinger::OpenDisplay(std::string_view name) {      // TODO(Subv): Currently we only support the Default display.      ASSERT(name == "Default"); -    const auto itr = std::find_if(displays.begin(), displays.end(), -                                  [&](const VI::Display& display) { return display.name == name; }); +    const auto itr = +        std::find_if(displays.begin(), displays.end(), +                     [&](const VI::Display& display) { return display.GetName() == name; });      if (itr == displays.end()) {          return {};      } -    return itr->id; +    return itr->GetID();  }  std::optional<u64> NVFlinger::CreateLayer(u64 display_id) { @@ -71,12 +76,10 @@ std::optional<u64> NVFlinger::CreateLayer(u64 display_id) {          return {};      } -    ASSERT_MSG(display->layers.empty(), "Only one layer is supported per display at the moment"); -      const u64 layer_id = next_layer_id++;      const u32 buffer_queue_id = next_buffer_queue_id++;      auto buffer_queue = std::make_shared<BufferQueue>(buffer_queue_id, layer_id); -    display->layers.emplace_back(layer_id, buffer_queue); +    display->CreateLayer(layer_id, buffer_queue);      buffer_queues.emplace_back(std::move(buffer_queue));      return layer_id;  } @@ -98,7 +101,7 @@ Kernel::SharedPtr<Kernel::ReadableEvent> NVFlinger::FindVsyncEvent(u64 display_i          return nullptr;      } -    return display->vsync_event.readable; +    return display->GetVSyncEvent();  }  std::shared_ptr<BufferQueue> NVFlinger::FindBufferQueue(u32 id) const { @@ -112,7 +115,7 @@ std::shared_ptr<BufferQueue> NVFlinger::FindBufferQueue(u32 id) const {  VI::Display* NVFlinger::FindDisplay(u64 display_id) {      const auto itr =          std::find_if(displays.begin(), displays.end(), -                     [&](const VI::Display& display) { return display.id == display_id; }); +                     [&](const VI::Display& display) { return display.GetID() == display_id; });      if (itr == displays.end()) {          return nullptr; @@ -124,7 +127,7 @@ VI::Display* NVFlinger::FindDisplay(u64 display_id) {  const VI::Display* NVFlinger::FindDisplay(u64 display_id) const {      const auto itr =          std::find_if(displays.begin(), displays.end(), -                     [&](const VI::Display& display) { return display.id == display_id; }); +                     [&](const VI::Display& display) { return display.GetID() == display_id; });      if (itr == displays.end()) {          return nullptr; @@ -140,14 +143,7 @@ VI::Layer* NVFlinger::FindLayer(u64 display_id, u64 layer_id) {          return nullptr;      } -    const auto itr = std::find_if(display->layers.begin(), display->layers.end(), -                                  [&](const VI::Layer& layer) { return layer.id == layer_id; }); - -    if (itr == display->layers.end()) { -        return nullptr; -    } - -    return &*itr; +    return display->FindLayer(layer_id);  }  const VI::Layer* NVFlinger::FindLayer(u64 display_id, u64 layer_id) const { @@ -157,29 +153,20 @@ const VI::Layer* NVFlinger::FindLayer(u64 display_id, u64 layer_id) const {          return nullptr;      } -    const auto itr = std::find_if(display->layers.begin(), display->layers.end(), -                                  [&](const VI::Layer& layer) { return layer.id == layer_id; }); - -    if (itr == display->layers.end()) { -        return nullptr; -    } - -    return &*itr; +    return display->FindLayer(layer_id);  }  void NVFlinger::Compose() {      for (auto& display : displays) {          // Trigger vsync for this display at the end of drawing -        SCOPE_EXIT({ display.vsync_event.writable->Signal(); }); +        SCOPE_EXIT({ display.SignalVSyncEvent(); });          // Don't do anything for displays without layers. -        if (display.layers.empty()) +        if (!display.HasLayers())              continue;          // TODO(Subv): Support more than 1 layer. -        ASSERT_MSG(display.layers.size() == 1, "Max 1 layer per display is supported"); - -        VI::Layer& layer = display.layers[0]; +        VI::Layer& layer = display.GetLayer(0);          auto& buffer_queue = layer.buffer_queue;          // Search for a queued buffer and acquire it diff --git a/src/core/hle/service/nvflinger/nvflinger.h b/src/core/hle/service/nvflinger/nvflinger.h index 2e000af91..4741b1363 100644 --- a/src/core/hle/service/nvflinger/nvflinger.h +++ b/src/core/hle/service/nvflinger/nvflinger.h @@ -28,7 +28,7 @@ class Module;  } // namespace Service::Nvidia  namespace Service::VI { -struct Display; +class Display;  struct Layer;  } // namespace Service::VI diff --git a/src/core/hle/service/vi/display/vi_display.cpp b/src/core/hle/service/vi/display/vi_display.cpp index a108e468f..4d77c3353 100644 --- a/src/core/hle/service/vi/display/vi_display.cpp +++ b/src/core/hle/service/vi/display/vi_display.cpp @@ -2,8 +2,12 @@  // Licensed under GPLv2 or any later version  // Refer to the license.txt file included. +#include <algorithm> +#include <utility> +  #include <fmt/format.h> +#include "common/assert.h"  #include "core/core.h"  #include "core/hle/kernel/readable_event.h"  #include "core/hle/service/vi/display/vi_display.h" @@ -19,4 +23,49 @@ Display::Display(u64 id, std::string name) : id{id}, name{std::move(name)} {  Display::~Display() = default; +Layer& Display::GetLayer(std::size_t index) { +    return layers.at(index); +} + +const Layer& Display::GetLayer(std::size_t index) const { +    return layers.at(index); +} + +Kernel::SharedPtr<Kernel::ReadableEvent> Display::GetVSyncEvent() const { +    return vsync_event.readable; +} + +void Display::SignalVSyncEvent() { +    vsync_event.writable->Signal(); +} + +void Display::CreateLayer(u64 id, std::shared_ptr<NVFlinger::BufferQueue> buffer_queue) { +    // TODO(Subv): Support more than 1 layer. +    ASSERT_MSG(layers.empty(), "Only one layer is supported per display at the moment"); + +    layers.emplace_back(id, std::move(buffer_queue)); +} + +Layer* Display::FindLayer(u64 id) { +    const auto itr = std::find_if(layers.begin(), layers.end(), +                                  [id](const VI::Layer& layer) { return layer.id == id; }); + +    if (itr == layers.end()) { +        return nullptr; +    } + +    return &*itr; +} + +const Layer* Display::FindLayer(u64 id) const { +    const auto itr = std::find_if(layers.begin(), layers.end(), +                                  [id](const VI::Layer& layer) { return layer.id == id; }); + +    if (itr == layers.end()) { +        return nullptr; +    } + +    return &*itr; +} +  } // namespace Service::VI diff --git a/src/core/hle/service/vi/display/vi_display.h b/src/core/hle/service/vi/display/vi_display.h index df44db306..22b831592 100644 --- a/src/core/hle/service/vi/display/vi_display.h +++ b/src/core/hle/service/vi/display/vi_display.h @@ -10,14 +10,84 @@  #include "common/common_types.h"  #include "core/hle/kernel/writable_event.h" +namespace Service::NVFlinger { +class BufferQueue; +} +  namespace Service::VI {  struct Layer; -struct Display { +/// Represents a single display type +class Display { +public: +    /// Constructs a display with a given unique ID and name. +    /// +    /// @param id   The unique ID for this display. +    /// @param name The name for this display. +    ///      Display(u64 id, std::string name);      ~Display(); +    Display(const Display&) = delete; +    Display& operator=(const Display&) = delete; + +    Display(Display&&) = default; +    Display& operator=(Display&&) = default; + +    /// Gets the unique ID assigned to this display. +    u64 GetID() const { +        return id; +    } + +    /// Gets the name of this display +    const std::string& GetName() const { +        return name; +    } + +    /// Whether or not this display has any layers added to it. +    bool HasLayers() const { +        return !layers.empty(); +    } + +    /// Gets a layer for this display based off an index. +    Layer& GetLayer(std::size_t index); + +    /// Gets a layer for this display based off an index. +    const Layer& GetLayer(std::size_t index) const; + +    /// Gets the readable vsync event. +    Kernel::SharedPtr<Kernel::ReadableEvent> GetVSyncEvent() const; + +    /// Signals the internal vsync event. +    void SignalVSyncEvent(); + +    /// Creates and adds a layer to this display with the given ID. +    /// +    /// @param id           The ID to assign to the created layer. +    /// @param buffer_queue The buffer queue for the layer instance to use. +    /// +    void CreateLayer(u64 id, std::shared_ptr<NVFlinger::BufferQueue> buffer_queue); + +    /// Attempts to find a layer with the given ID. +    /// +    /// @param id The layer ID. +    /// +    /// @returns If found, the Layer instance with the given ID. +    ///          If not found, then nullptr is returned. +    /// +    Layer* FindLayer(u64 id); + +    /// Attempts to find a layer with the given ID. +    /// +    /// @param id The layer ID. +    /// +    /// @returns If found, the Layer instance with the given ID. +    ///          If not found, then nullptr is returned. +    /// +    const Layer* FindLayer(u64 id) const; + +private:      u64 id;      std::string name; | 
