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/time.cpp') 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/time/time.cpp | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'src/core/hle/service/time/time.cpp') 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