diff options
author | Steven Moreland <smoreland@google.com> | 2018-09-27 15:40:30 -0700 |
---|---|---|
committer | Steven Moreland <smoreland@google.com> | 2018-09-27 17:26:58 -0700 |
commit | 9de39ac9841b795f9ae4ec120eced54f46043d88 (patch) | |
tree | cdc8d28f0e31f9f331ead2107687021984f3c116 | |
parent | aa703608c488199f70570d689a09a8e4aa693dee (diff) | |
download | libhidl-9de39ac9841b795f9ae4ec120eced54f46043d88.tar.gz |
@1.2::IServiceManager: remove 'removeService' todo
Thinking about this in more detail, if a service is removed, there are
two problems:
- lazy HAL mechanism doesn't support adding the service back.
- ownership may have already been passed to other clients.
For these reasons, for the time being, we can't add this API.
Bug: 66921961
Test: N/A
Change-Id: I4bf5830339bfb6300516d864ad9e9a67512e5e0a
-rw-r--r-- | transport/manager/1.2/IServiceManager.hal | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/transport/manager/1.2/IServiceManager.hal b/transport/manager/1.2/IServiceManager.hal index 6305da9..1ef4ed8 100644 --- a/transport/manager/1.2/IServiceManager.hal +++ b/transport/manager/1.2/IServiceManager.hal @@ -21,9 +21,8 @@ import @1.1::IServiceManager; interface IServiceManager extends @1.1::IServiceManager { // TODO(b/66921961): additional functionality for the next release: // - Better notifications (onRegistration will contain the interface) - // - Remove service (for b/64678982, lazy HALs) // - Callbacks to support lazy hal shutdown when there are no clients (b/79983843) // - Method to list interfaces in the manifest (for b/76108617, since if they are // lazy, they will no longer show up in listByInterface) // - Register w/o it having to call interface chain (b/36424585) -};
\ No newline at end of file +}; |