service/vi: Remove use of a module class
This didn't really provide much benefit here, especially since the subsequent change requires that the behavior for each service's GetDisplayService differs in a minor detail. This also arguably makes the services nicer to read, since it gets rid of an indirection in the class hierarchy.
This commit is contained in:
parent
c07987dfab
commit
1b2872eebc
8 changed files with 99 additions and 46 deletions
|
@ -24,6 +24,7 @@
|
||||||
#include "core/hle/service/nvdrv/nvdrv.h"
|
#include "core/hle/service/nvdrv/nvdrv.h"
|
||||||
#include "core/hle/service/nvflinger/buffer_queue.h"
|
#include "core/hle/service/nvflinger/buffer_queue.h"
|
||||||
#include "core/hle/service/nvflinger/nvflinger.h"
|
#include "core/hle/service/nvflinger/nvflinger.h"
|
||||||
|
#include "core/hle/service/service.h"
|
||||||
#include "core/hle/service/vi/vi.h"
|
#include "core/hle/service/vi/vi.h"
|
||||||
#include "core/hle/service/vi/vi_m.h"
|
#include "core/hle/service/vi/vi_m.h"
|
||||||
#include "core/hle/service/vi/vi_s.h"
|
#include "core/hle/service/vi/vi_s.h"
|
||||||
|
@ -1202,26 +1203,18 @@ IApplicationDisplayService::IApplicationDisplayService(
|
||||||
RegisterHandlers(functions);
|
RegisterHandlers(functions);
|
||||||
}
|
}
|
||||||
|
|
||||||
Module::Interface::Interface(std::shared_ptr<Module> module, const char* name,
|
void detail::GetDisplayServiceImpl(Kernel::HLERequestContext& ctx,
|
||||||
std::shared_ptr<NVFlinger::NVFlinger> nv_flinger)
|
std::shared_ptr<NVFlinger::NVFlinger> nv_flinger) {
|
||||||
: ServiceFramework(name), module(std::move(module)), nv_flinger(std::move(nv_flinger)) {}
|
|
||||||
|
|
||||||
Module::Interface::~Interface() = default;
|
|
||||||
|
|
||||||
void Module::Interface::GetDisplayService(Kernel::HLERequestContext& ctx) {
|
|
||||||
LOG_WARNING(Service_VI, "(STUBBED) called");
|
|
||||||
|
|
||||||
IPC::ResponseBuilder rb{ctx, 2, 0, 1};
|
IPC::ResponseBuilder rb{ctx, 2, 0, 1};
|
||||||
rb.Push(RESULT_SUCCESS);
|
rb.Push(RESULT_SUCCESS);
|
||||||
rb.PushIpcInterface<IApplicationDisplayService>(nv_flinger);
|
rb.PushIpcInterface<IApplicationDisplayService>(std::move(nv_flinger));
|
||||||
}
|
}
|
||||||
|
|
||||||
void InstallInterfaces(SM::ServiceManager& service_manager,
|
void InstallInterfaces(SM::ServiceManager& service_manager,
|
||||||
std::shared_ptr<NVFlinger::NVFlinger> nv_flinger) {
|
std::shared_ptr<NVFlinger::NVFlinger> nv_flinger) {
|
||||||
auto module = std::make_shared<Module>();
|
std::make_shared<VI_M>(nv_flinger)->InstallAsService(service_manager);
|
||||||
std::make_shared<VI_M>(module, nv_flinger)->InstallAsService(service_manager);
|
std::make_shared<VI_S>(nv_flinger)->InstallAsService(service_manager);
|
||||||
std::make_shared<VI_S>(module, nv_flinger)->InstallAsService(service_manager);
|
std::make_shared<VI_U>(nv_flinger)->InstallAsService(service_manager);
|
||||||
std::make_shared<VI_U>(module, nv_flinger)->InstallAsService(service_manager);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
} // namespace Service::VI
|
} // namespace Service::VI
|
||||||
|
|
|
@ -4,13 +4,26 @@
|
||||||
|
|
||||||
#pragma once
|
#pragma once
|
||||||
|
|
||||||
#include "core/hle/service/service.h"
|
#include <memory>
|
||||||
|
#include "common/common_types.h"
|
||||||
|
|
||||||
|
namespace Kernel {
|
||||||
|
class HLERequestContext;
|
||||||
|
}
|
||||||
|
|
||||||
namespace Service::NVFlinger {
|
namespace Service::NVFlinger {
|
||||||
class NVFlinger;
|
class NVFlinger;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
namespace Service::SM {
|
||||||
|
class ServiceManager;
|
||||||
|
}
|
||||||
|
|
||||||
namespace Service::VI {
|
namespace Service::VI {
|
||||||
|
namespace detail {
|
||||||
|
void GetDisplayServiceImpl(Kernel::HLERequestContext& ctx,
|
||||||
|
std::shared_ptr<NVFlinger::NVFlinger> nv_flinger);
|
||||||
|
}
|
||||||
|
|
||||||
enum class DisplayResolution : u32 {
|
enum class DisplayResolution : u32 {
|
||||||
DockedWidth = 1920,
|
DockedWidth = 1920,
|
||||||
|
@ -19,22 +32,6 @@ enum class DisplayResolution : u32 {
|
||||||
UndockedHeight = 720,
|
UndockedHeight = 720,
|
||||||
};
|
};
|
||||||
|
|
||||||
class Module final {
|
|
||||||
public:
|
|
||||||
class Interface : public ServiceFramework<Interface> {
|
|
||||||
public:
|
|
||||||
explicit Interface(std::shared_ptr<Module> module, const char* name,
|
|
||||||
std::shared_ptr<NVFlinger::NVFlinger> nv_flinger);
|
|
||||||
~Interface() override;
|
|
||||||
|
|
||||||
void GetDisplayService(Kernel::HLERequestContext& ctx);
|
|
||||||
|
|
||||||
protected:
|
|
||||||
std::shared_ptr<Module> module;
|
|
||||||
std::shared_ptr<NVFlinger::NVFlinger> nv_flinger;
|
|
||||||
};
|
|
||||||
};
|
|
||||||
|
|
||||||
/// Registers all VI services with the specified service manager.
|
/// Registers all VI services with the specified service manager.
|
||||||
void InstallInterfaces(SM::ServiceManager& service_manager,
|
void InstallInterfaces(SM::ServiceManager& service_manager,
|
||||||
std::shared_ptr<NVFlinger::NVFlinger> nv_flinger);
|
std::shared_ptr<NVFlinger::NVFlinger> nv_flinger);
|
||||||
|
|
|
@ -2,12 +2,14 @@
|
||||||
// Licensed under GPLv2 or any later version
|
// Licensed under GPLv2 or any later version
|
||||||
// Refer to the license.txt file included.
|
// Refer to the license.txt file included.
|
||||||
|
|
||||||
|
#include "common/logging/log.h"
|
||||||
|
#include "core/hle/service/vi/vi.h"
|
||||||
#include "core/hle/service/vi/vi_m.h"
|
#include "core/hle/service/vi/vi_m.h"
|
||||||
|
|
||||||
namespace Service::VI {
|
namespace Service::VI {
|
||||||
|
|
||||||
VI_M::VI_M(std::shared_ptr<Module> module, std::shared_ptr<NVFlinger::NVFlinger> nv_flinger)
|
VI_M::VI_M(std::shared_ptr<NVFlinger::NVFlinger> nv_flinger)
|
||||||
: Module::Interface(std::move(module), "vi:m", std::move(nv_flinger)) {
|
: ServiceFramework{"vi:m"}, nv_flinger{std::move(nv_flinger)} {
|
||||||
static const FunctionInfo functions[] = {
|
static const FunctionInfo functions[] = {
|
||||||
{2, &VI_M::GetDisplayService, "GetDisplayService"},
|
{2, &VI_M::GetDisplayService, "GetDisplayService"},
|
||||||
{3, nullptr, "GetDisplayServiceWithProxyNameExchange"},
|
{3, nullptr, "GetDisplayServiceWithProxyNameExchange"},
|
||||||
|
@ -17,4 +19,10 @@ VI_M::VI_M(std::shared_ptr<Module> module, std::shared_ptr<NVFlinger::NVFlinger>
|
||||||
|
|
||||||
VI_M::~VI_M() = default;
|
VI_M::~VI_M() = default;
|
||||||
|
|
||||||
|
void VI_M::GetDisplayService(Kernel::HLERequestContext& ctx) {
|
||||||
|
LOG_WARNING(Service_VI, "(STUBBED) called");
|
||||||
|
|
||||||
|
detail::GetDisplayServiceImpl(ctx, nv_flinger);
|
||||||
|
}
|
||||||
|
|
||||||
} // namespace Service::VI
|
} // namespace Service::VI
|
||||||
|
|
|
@ -4,14 +4,27 @@
|
||||||
|
|
||||||
#pragma once
|
#pragma once
|
||||||
|
|
||||||
#include "core/hle/service/vi/vi.h"
|
#include "core/hle/service/service.h"
|
||||||
|
|
||||||
|
namespace Kernel {
|
||||||
|
class HLERequestContext;
|
||||||
|
}
|
||||||
|
|
||||||
|
namespace Service::NVFlinger {
|
||||||
|
class NVFlinger;
|
||||||
|
}
|
||||||
|
|
||||||
namespace Service::VI {
|
namespace Service::VI {
|
||||||
|
|
||||||
class VI_M final : public Module::Interface {
|
class VI_M final : public ServiceFramework<VI_M> {
|
||||||
public:
|
public:
|
||||||
explicit VI_M(std::shared_ptr<Module> module, std::shared_ptr<NVFlinger::NVFlinger> nv_flinger);
|
explicit VI_M(std::shared_ptr<NVFlinger::NVFlinger> nv_flinger);
|
||||||
~VI_M() override;
|
~VI_M() override;
|
||||||
|
|
||||||
|
private:
|
||||||
|
void GetDisplayService(Kernel::HLERequestContext& ctx);
|
||||||
|
|
||||||
|
std::shared_ptr<NVFlinger::NVFlinger> nv_flinger;
|
||||||
};
|
};
|
||||||
|
|
||||||
} // namespace Service::VI
|
} // namespace Service::VI
|
||||||
|
|
|
@ -2,12 +2,14 @@
|
||||||
// Licensed under GPLv2 or any later version
|
// Licensed under GPLv2 or any later version
|
||||||
// Refer to the license.txt file included.
|
// Refer to the license.txt file included.
|
||||||
|
|
||||||
|
#include "common/logging/log.h"
|
||||||
|
#include "core/hle/service/vi/vi.h"
|
||||||
#include "core/hle/service/vi/vi_s.h"
|
#include "core/hle/service/vi/vi_s.h"
|
||||||
|
|
||||||
namespace Service::VI {
|
namespace Service::VI {
|
||||||
|
|
||||||
VI_S::VI_S(std::shared_ptr<Module> module, std::shared_ptr<NVFlinger::NVFlinger> nv_flinger)
|
VI_S::VI_S(std::shared_ptr<NVFlinger::NVFlinger> nv_flinger)
|
||||||
: Module::Interface(std::move(module), "vi:s", std::move(nv_flinger)) {
|
: ServiceFramework{"vi:s"}, nv_flinger{std::move(nv_flinger)} {
|
||||||
static const FunctionInfo functions[] = {
|
static const FunctionInfo functions[] = {
|
||||||
{1, &VI_S::GetDisplayService, "GetDisplayService"},
|
{1, &VI_S::GetDisplayService, "GetDisplayService"},
|
||||||
{3, nullptr, "GetDisplayServiceWithProxyNameExchange"},
|
{3, nullptr, "GetDisplayServiceWithProxyNameExchange"},
|
||||||
|
@ -17,4 +19,10 @@ VI_S::VI_S(std::shared_ptr<Module> module, std::shared_ptr<NVFlinger::NVFlinger>
|
||||||
|
|
||||||
VI_S::~VI_S() = default;
|
VI_S::~VI_S() = default;
|
||||||
|
|
||||||
|
void VI_S::GetDisplayService(Kernel::HLERequestContext& ctx) {
|
||||||
|
LOG_WARNING(Service_VI, "(STUBBED) called");
|
||||||
|
|
||||||
|
detail::GetDisplayServiceImpl(ctx, nv_flinger);
|
||||||
|
}
|
||||||
|
|
||||||
} // namespace Service::VI
|
} // namespace Service::VI
|
||||||
|
|
|
@ -4,14 +4,27 @@
|
||||||
|
|
||||||
#pragma once
|
#pragma once
|
||||||
|
|
||||||
#include "core/hle/service/vi/vi.h"
|
#include "core/hle/service/service.h"
|
||||||
|
|
||||||
|
namespace Kernel {
|
||||||
|
class HLERequestContext;
|
||||||
|
}
|
||||||
|
|
||||||
|
namespace Service::NVFlinger {
|
||||||
|
class NVFlinger;
|
||||||
|
}
|
||||||
|
|
||||||
namespace Service::VI {
|
namespace Service::VI {
|
||||||
|
|
||||||
class VI_S final : public Module::Interface {
|
class VI_S final : public ServiceFramework<VI_S> {
|
||||||
public:
|
public:
|
||||||
explicit VI_S(std::shared_ptr<Module> module, std::shared_ptr<NVFlinger::NVFlinger> nv_flinger);
|
explicit VI_S(std::shared_ptr<NVFlinger::NVFlinger> nv_flinger);
|
||||||
~VI_S() override;
|
~VI_S() override;
|
||||||
|
|
||||||
|
private:
|
||||||
|
void GetDisplayService(Kernel::HLERequestContext& ctx);
|
||||||
|
|
||||||
|
std::shared_ptr<NVFlinger::NVFlinger> nv_flinger;
|
||||||
};
|
};
|
||||||
|
|
||||||
} // namespace Service::VI
|
} // namespace Service::VI
|
||||||
|
|
|
@ -2,12 +2,14 @@
|
||||||
// Licensed under GPLv2 or any later version
|
// Licensed under GPLv2 or any later version
|
||||||
// Refer to the license.txt file included.
|
// Refer to the license.txt file included.
|
||||||
|
|
||||||
|
#include "common/logging/log.h"
|
||||||
|
#include "core/hle/service/vi/vi.h"
|
||||||
#include "core/hle/service/vi/vi_u.h"
|
#include "core/hle/service/vi/vi_u.h"
|
||||||
|
|
||||||
namespace Service::VI {
|
namespace Service::VI {
|
||||||
|
|
||||||
VI_U::VI_U(std::shared_ptr<Module> module, std::shared_ptr<NVFlinger::NVFlinger> nv_flinger)
|
VI_U::VI_U(std::shared_ptr<NVFlinger::NVFlinger> nv_flinger)
|
||||||
: Module::Interface(std::move(module), "vi:u", std::move(nv_flinger)) {
|
: ServiceFramework{"vi:u"}, nv_flinger{std::move(nv_flinger)} {
|
||||||
static const FunctionInfo functions[] = {
|
static const FunctionInfo functions[] = {
|
||||||
{0, &VI_U::GetDisplayService, "GetDisplayService"},
|
{0, &VI_U::GetDisplayService, "GetDisplayService"},
|
||||||
};
|
};
|
||||||
|
@ -16,4 +18,10 @@ VI_U::VI_U(std::shared_ptr<Module> module, std::shared_ptr<NVFlinger::NVFlinger>
|
||||||
|
|
||||||
VI_U::~VI_U() = default;
|
VI_U::~VI_U() = default;
|
||||||
|
|
||||||
|
void VI_U::GetDisplayService(Kernel::HLERequestContext& ctx) {
|
||||||
|
LOG_WARNING(Service_VI, "(STUBBED) called");
|
||||||
|
|
||||||
|
detail::GetDisplayServiceImpl(ctx, nv_flinger);
|
||||||
|
}
|
||||||
|
|
||||||
} // namespace Service::VI
|
} // namespace Service::VI
|
||||||
|
|
|
@ -4,14 +4,27 @@
|
||||||
|
|
||||||
#pragma once
|
#pragma once
|
||||||
|
|
||||||
#include "core/hle/service/vi/vi.h"
|
#include "core/hle/service/service.h"
|
||||||
|
|
||||||
|
namespace Kernel {
|
||||||
|
class HLERequestContext;
|
||||||
|
}
|
||||||
|
|
||||||
|
namespace Service::NVFlinger {
|
||||||
|
class NVFlinger;
|
||||||
|
}
|
||||||
|
|
||||||
namespace Service::VI {
|
namespace Service::VI {
|
||||||
|
|
||||||
class VI_U final : public Module::Interface {
|
class VI_U final : public ServiceFramework<VI_U> {
|
||||||
public:
|
public:
|
||||||
explicit VI_U(std::shared_ptr<Module> module, std::shared_ptr<NVFlinger::NVFlinger> nv_flinger);
|
explicit VI_U(std::shared_ptr<NVFlinger::NVFlinger> nv_flinger);
|
||||||
~VI_U() override;
|
~VI_U() override;
|
||||||
|
|
||||||
|
private:
|
||||||
|
void GetDisplayService(Kernel::HLERequestContext& ctx);
|
||||||
|
|
||||||
|
std::shared_ptr<NVFlinger::NVFlinger> nv_flinger;
|
||||||
};
|
};
|
||||||
|
|
||||||
} // namespace Service::VI
|
} // namespace Service::VI
|
||||||
|
|
Loading…
Reference in a new issue