From 7efa6e8801866da8ed4dca97fa25d2451dcb042e Mon Sep 17 00:00:00 2001 From: Subv Date: Mon, 22 Jan 2018 17:42:11 -0500 Subject: Time: Don't create unnecessary ports when retrieving the clock service sessions. --- src/core/hle/service/time/time.cpp | 60 +++++++++++++++++--------------------- 1 file changed, 27 insertions(+), 33 deletions(-) (limited to 'src/core/hle/service/time') diff --git a/src/core/hle/service/time/time.cpp b/src/core/hle/service/time/time.cpp index 9fed89246..9039a4cec 100644 --- a/src/core/hle/service/time/time.cpp +++ b/src/core/hle/service/time/time.cpp @@ -83,45 +83,39 @@ private: }; void Module::Interface::GetStandardUserSystemClock(Kernel::HLERequestContext& ctx) { - auto client_port = std::make_shared()->CreatePort(); - auto session = client_port->Connect(); - if (session.Succeeded()) { - LOG_DEBUG(Service, "called, initialized ISystemClock -> session=%u", - (*session)->GetObjectId()); - IPC::RequestBuilder rb{ctx, 2, 0, 1}; - rb.Push(RESULT_SUCCESS); - rb.PushMoveObjects(std::move(session).Unwrap()); - } else { - UNIMPLEMENTED(); - } + auto system_clock = std::make_shared(); + auto sessions = Kernel::ServerSession::CreateSessionPair(system_clock->GetServiceName()); + auto server = std::get>(sessions); + auto client = std::get>(sessions); + system_clock->ClientConnected(server); + LOG_DEBUG(Service, "called, initialized ISystemClock -> session=%u", client->GetObjectId()); + IPC::RequestBuilder rb{ctx, 2, 0, 1}; + rb.Push(RESULT_SUCCESS); + rb.PushMoveObjects(std::move(client)); } void Module::Interface::GetStandardNetworkSystemClock(Kernel::HLERequestContext& ctx) { - auto client_port = std::make_shared()->CreatePort(); - auto session = client_port->Connect(); - if (session.Succeeded()) { - LOG_DEBUG(Service, "called, initialized ISystemClock -> session=%u", - (*session)->GetObjectId()); - IPC::RequestBuilder rb{ctx, 2, 0, 1}; - rb.Push(RESULT_SUCCESS); - rb.PushMoveObjects(std::move(session).Unwrap()); - } else { - UNIMPLEMENTED(); - } + auto system_clock = std::make_shared(); + auto sessions = Kernel::ServerSession::CreateSessionPair(system_clock->GetServiceName()); + auto server = std::get>(sessions); + auto client = std::get>(sessions); + system_clock->ClientConnected(server); + LOG_DEBUG(Service, "called, initialized ISystemClock -> session=%u", client->GetObjectId()); + IPC::RequestBuilder rb{ctx, 2, 0, 1}; + rb.Push(RESULT_SUCCESS); + rb.PushMoveObjects(std::move(client)); } void Module::Interface::GetStandardSteadyClock(Kernel::HLERequestContext& ctx) { - auto client_port = std::make_shared()->CreatePort(); - auto session = client_port->Connect(); - if (session.Succeeded()) { - LOG_DEBUG(Service, "called, initialized ISteadyClock -> session=%u", - (*session)->GetObjectId()); - IPC::RequestBuilder rb{ctx, 2, 0, 1}; - rb.Push(RESULT_SUCCESS); - rb.PushMoveObjects(std::move(session).Unwrap()); - } else { - UNIMPLEMENTED(); - } + auto steady_clock = std::make_shared(); + auto sessions = Kernel::ServerSession::CreateSessionPair(steady_clock->GetServiceName()); + auto server = std::get>(sessions); + auto client = std::get>(sessions); + steady_clock->ClientConnected(server); + LOG_DEBUG(Service, "called, initialized ISteadyClock -> session=%u", client->GetObjectId()); + IPC::RequestBuilder rb{ctx, 2, 0, 1}; + rb.Push(RESULT_SUCCESS); + rb.PushMoveObjects(std::move(client)); } void Module::Interface::GetTimeZoneService(Kernel::HLERequestContext& ctx) { -- 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/time') 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