From b82b8f84b48616c124e1d1421a985328c1df53dc Mon Sep 17 00:00:00 2001 From: Steven Moreland Date: Mon, 28 Oct 2019 10:52:34 -0700 Subject: ServiceManager: add isDeclared Most of the time, AIDL services exist in client/service pairs. In this context, `while(true) getService` or `waitForService` makes sense. However, for VINTF services, this client/server coupling isn't guaranteed. A device may or may not have specific hardware. So, now IServiceManager can tell a client when a server will exist so it will know if it needs to wait for this service during boot-up. The function waitForDeclaredService is provided for this. These functions are generic (not referring to VINTF specifically) because they may be expanded to include information about APEX services in the future OR the infrastructure may be made generic to work, if desired, on system services. Bug: 141828236 Test: binderStabilityTest, using waitForDeclaredService function Change-Id: Ia230625e44e1685cc3fa9230ece8f0a25c88585e --- libs/binder/IServiceManager.cpp | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'libs/binder/IServiceManager.cpp') diff --git a/libs/binder/IServiceManager.cpp b/libs/binder/IServiceManager.cpp index a30df14bd6..4f47db199e 100644 --- a/libs/binder/IServiceManager.cpp +++ b/libs/binder/IServiceManager.cpp @@ -72,6 +72,7 @@ public: bool allowIsolated, int dumpsysPriority) override; Vector listServices(int dumpsysPriority) override; sp waitForService(const String16& name16) override; + bool isDeclared(const String16& name) override; // for legacy ABI const String16& getInterfaceDescriptor() const override { @@ -321,4 +322,12 @@ sp ServiceManagerShim::waitForService(const String16& name16) } } +bool ServiceManagerShim::isDeclared(const String16& name) { + bool declared; + if (!mTheRealServiceManager->isDeclared(String8(name).c_str(), &declared).isOk()) { + return false; + } + return declared; +} + } // namespace android -- cgit v1.2.3-59-g8ed1b