From 1a9c96e4de6699722ad0fbe511c9868e89d29eee Mon Sep 17 00:00:00 2001 From: Subv Date: Mon, 22 Jan 2018 17:35:40 -0500 Subject: LM: Don't create an unnecessary port in Initialize. --- src/core/hle/service/lm/lm.cpp | 22 ++++++++++------------ 1 file changed, 10 insertions(+), 12 deletions(-) (limited to 'src/core/hle/service/lm/lm.cpp') diff --git a/src/core/hle/service/lm/lm.cpp b/src/core/hle/service/lm/lm.cpp index 13c9ee3d3..dd0f75ce1 100644 --- a/src/core/hle/service/lm/lm.cpp +++ b/src/core/hle/service/lm/lm.cpp @@ -146,18 +146,16 @@ void InstallInterfaces(SM::ServiceManager& service_manager) { * 0: ResultCode */ void LM::Initialize(Kernel::HLERequestContext& ctx) { - auto client_port = std::make_shared()->CreatePort(); - auto session = client_port->Connect(); - if (session.Succeeded()) { - LOG_DEBUG(Service_SM, "called, initialized logger -> session=%u", - (*session)->GetObjectId()); - IPC::RequestBuilder rb{ctx, 2, 0, 1}; - rb.Push(RESULT_SUCCESS); - rb.PushMoveObjects(std::move(session).Unwrap()); - registered_loggers.emplace_back(std::move(client_port)); - } else { - UNIMPLEMENTED(); - } + auto logger = std::make_shared(); + auto sessions = Kernel::ServerSession::CreateSessionPair(logger->GetServiceName()); + auto server = std::get>(sessions); + auto client = std::get>(sessions); + logger->ClientConnected(server); + + LOG_DEBUG(Service_SM, "called, initialized logger -> session=%u", client->GetObjectId()); + IPC::RequestBuilder rb{ctx, 2, 0, 1}; + rb.Push(RESULT_SUCCESS); + rb.PushMoveObjects(std::move(client)); LOG_INFO(Service_SM, "called"); } -- cgit v1.2.3 From b0489c9a64588d2140d9a98b69c5a0adc541a436 Mon Sep 17 00:00:00 2001 From: Subv Date: Mon, 22 Jan 2018 17:59:24 -0500 Subject: Services: Added a todo about returning interfaces as domain objects in lm, hid and time. --- src/core/hle/service/hid/hid.cpp | 3 +++ src/core/hle/service/lm/lm.cpp | 3 +++ src/core/hle/service/time/time.cpp | 6 ++++++ 3 files changed, 12 insertions(+) (limited to 'src/core/hle/service/lm/lm.cpp') diff --git a/src/core/hle/service/hid/hid.cpp b/src/core/hle/service/hid/hid.cpp index 9b1e680f1..ae60cc7b4 100644 --- a/src/core/hle/service/hid/hid.cpp +++ b/src/core/hle/service/hid/hid.cpp @@ -169,6 +169,9 @@ private: applet_resource = std::make_shared(); } + // TODO(Subv): Verify if this should return the interface as a domain object when called + // from within a domain. + auto sessions = Kernel::ServerSession::CreateSessionPair(applet_resource->GetServiceName()); auto server = std::get>(sessions); auto client = std::get>(sessions); diff --git a/src/core/hle/service/lm/lm.cpp b/src/core/hle/service/lm/lm.cpp index dd0f75ce1..b505cdcaf 100644 --- a/src/core/hle/service/lm/lm.cpp +++ b/src/core/hle/service/lm/lm.cpp @@ -146,6 +146,9 @@ void InstallInterfaces(SM::ServiceManager& service_manager) { * 0: ResultCode */ void LM::Initialize(Kernel::HLERequestContext& ctx) { + // TODO(Subv): Verify if this should return the interface as a domain object when called from + // within a domain. + auto logger = std::make_shared(); auto sessions = Kernel::ServerSession::CreateSessionPair(logger->GetServiceName()); auto server = std::get>(sessions); diff --git a/src/core/hle/service/time/time.cpp b/src/core/hle/service/time/time.cpp index 9039a4cec..8abb2ce7a 100644 --- a/src/core/hle/service/time/time.cpp +++ b/src/core/hle/service/time/time.cpp @@ -83,6 +83,8 @@ private: }; void Module::Interface::GetStandardUserSystemClock(Kernel::HLERequestContext& ctx) { + // TODO(Subv): Verify if this should return the interface as a domain object when called from + // within a domain. auto system_clock = std::make_shared(); auto sessions = Kernel::ServerSession::CreateSessionPair(system_clock->GetServiceName()); auto server = std::get>(sessions); @@ -95,6 +97,8 @@ void Module::Interface::GetStandardUserSystemClock(Kernel::HLERequestContext& ct } void Module::Interface::GetStandardNetworkSystemClock(Kernel::HLERequestContext& ctx) { + // TODO(Subv): Verify if this should return the interface as a domain object when called from + // within a domain. auto system_clock = std::make_shared(); auto sessions = Kernel::ServerSession::CreateSessionPair(system_clock->GetServiceName()); auto server = std::get>(sessions); @@ -107,6 +111,8 @@ void Module::Interface::GetStandardNetworkSystemClock(Kernel::HLERequestContext& } void Module::Interface::GetStandardSteadyClock(Kernel::HLERequestContext& ctx) { + // TODO(Subv): Verify if this should return the interface as a domain object when called from + // within a domain. auto steady_clock = std::make_shared(); auto sessions = Kernel::ServerSession::CreateSessionPair(steady_clock->GetServiceName()); auto server = std::get>(sessions); -- cgit v1.2.3