host privacy config option to enable it

paul-szczepanek-arm · pan- · commit 03cc0f99403c · 2020-10-05T11:54:04.000+01:00
diff --git a/connectivity/FEATURE_BLE/include/ble/gap/Types.h b/connectivity/FEATURE_BLE/include/ble/gap/Types.h
@@ -455,8 +455,8 @@ struct scanning_filter_policy_t : SafeEnum<scanning_filter_policy_t, uint8_t> {
         NO_FILTER = 0x00,
 
         /**
-        * Accept only advertising packets from devices in the whitelist except
-        * directed advertising packets not addressed to this device.
+        * Accept only advertising packets from devices in the whitelist.
+        * Directed advertising packets not addressed to this device will be ignored.
         */
         FILTER_ADVERTISING = 0x01,
 
diff --git a/connectivity/FEATURE_BLE/mbed_lib.json b/connectivity/FEATURE_BLE/mbed_lib.json
@@ -93,10 +93,20 @@
             "value": 15,
             "macro_name": "BLE_GAP_MAX_ADVERTISING_SETS"
         },
+        "ble-gap-host-based-private-address-resolution": {
+            "help": "Perform address resolution on the host, not the controller. Controller based privacy is preferred as it happens lower down the stack but this can be used in case controller based privacy is unavailable. If this is enabled the controller will not be used for privacy.",
+            "value": true,
+            "macro_name": "BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION"
+        },
         "ble-gap-max-advertising-reports-pending-address-resolution": {
             "help": "How many advertising reports can be pending while awaiting private address resolution. This is only used if host privacy is enabled and controller privacy is disabled. Must be non-zero",
             "value": 16,
             "macro_name": "BLE_GAP_MAX_ADVERTISING_REPORTS_PENDING_ADDRESS_RESOLUTION"
+        },
+        "ble-gap-host-privacy-resolved-cache-size": {
+            "help": "Used for host privacy. How many last resolved addresses to store to speed up resolution. This is especially valuable for resolving advertising which creates repeated queries for the same address.",
+            "value": 16,
+            "macro_name": "BLE_GAP_HOST_PRIVACY_RESOLVED_CACHE_SIZE"
         }
     }
 }
diff --git a/connectivity/FEATURE_BLE/source/Gap.cpp b/connectivity/FEATURE_BLE/source/Gap.cpp
@@ -1,7 +1,7 @@
 /* mbed Microcontroller Library
  * Copyright (c) 2020 ARM Limited
  * SPDX-License-Identifier: Apache-2.0
- * 
+ *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
@@ -506,4 +506,4 @@ ble::address_t Gap::getRandomStaticAddress()
     return impl->getRandomStaticAddress();
 }
 
-} // namespace ble
+} // namespace ble
diff --git a/connectivity/FEATURE_BLE/source/cordio/source/PalPrivateAddressControllerImpl.cpp b/connectivity/FEATURE_BLE/source/cordio/source/PalPrivateAddressControllerImpl.cpp
@@ -1,7 +1,7 @@
 /* mbed Microcontroller Library
  * Copyright (c) 2020 ARM Limited
  * SPDX-License-Identifier: Apache-2.0
- * 
+ *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
@@ -193,6 +193,7 @@ bool PalPrivateAddressController::cordio_handler(const wsfMsgHdr_t *msg)
             return true;
         }
 
+#if BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
         case DM_PRIV_RESOLVED_ADDR_IND: {
             instance()._resolving_rpa = false;
 
@@ -203,6 +204,7 @@ bool PalPrivateAddressController::cordio_handler(const wsfMsgHdr_t *msg)
             handler->on_private_address_resolved(msg->status == HCI_SUCCESS);
             return true;
         }
+#endif // BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 
         case DM_PRIV_ADD_DEV_TO_RES_LIST_IND: // Device added to resolving list
         case DM_PRIV_REM_DEV_FROM_RES_LIST_IND: // Device removed from resolving list
diff --git a/connectivity/FEATURE_BLE/source/generic/GapImpl.cpp b/connectivity/FEATURE_BLE/source/generic/GapImpl.cpp
@@ -929,9 +929,11 @@ ble_error_t Gap::enablePrivacy(bool enable)
         _address_registry.stop_private_address_generation();
     }
 
+#if !BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
     if (_address_registry.is_controller_privacy_supported()) {
         update_ll_address_resolution_setting();
     }
+#endif // !BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 
     return BLE_ERROR_NONE;
 }
@@ -946,9 +948,11 @@ ble_error_t Gap::setPeripheralPrivacyConfiguration(
 {
     _peripheral_privacy_configuration = *configuration;
 
+#if !BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
     if (_address_registry.is_controller_privacy_supported()) {
         update_ll_address_resolution_setting();
     }
+#endif // !BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 
     return BLE_ERROR_NONE;
 }
@@ -978,9 +982,11 @@ ble_error_t Gap::setCentralPrivacyConfiguration(
 {
     _central_privacy_configuration = *configuration;
 
+#if !BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
     if (_address_registry.is_controller_privacy_supported()) {
         update_ll_address_resolution_setting();
     }
+#endif // !BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 
     return BLE_ERROR_NONE;
 }
@@ -1474,7 +1480,7 @@ bool Gap::initialize_whitelist() const
     return true;
 }
 
-
+#if BLE_FEATURE_PRIVACY && !BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 ble_error_t Gap::update_ll_address_resolution_setting()
 {
     // enable if privacy is enabled and resolution is requested in either central or peripheral mode
@@ -1496,6 +1502,7 @@ ble_error_t Gap::update_ll_address_resolution_setting()
 
     return _address_registry.enable_controller_address_resolution(enable);
 }
+#endif // BLE_FEATURE_PRIVACY && !BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 
 #if BLE_ROLE_BROADCASTER
 uint8_t Gap::getMaxAdvertisingSetNumber()
@@ -2314,12 +2321,11 @@ void Gap::signal_connection_complete(
     ConnectionCompleteEvent& event
 )
 {
-#if BLE_FEATURE_PRIVACY
+#if BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
     bool address_resolved = false;
 
     /* if address resolution is not needed or already handled then the address is already resolved */
     if (!_privacy_enabled ||
-        _address_registry.is_controller_privacy_supported() ||
         event.getPeerAddressType() == peer_address_type_t::PUBLIC ||
         !is_random_private_resolvable_address(event.getPeerAddress())) {
         address_resolved = true;
@@ -2330,7 +2336,7 @@ void Gap::signal_connection_complete(
                 address_resolved = true;
             }
         }
-#endif
+#endif // BLE_ROLE_CENTRAL
 
 #if BLE_ROLE_PERIPHERAL
         if (event.getOwnRole() == connection_role_t::PERIPHERAL) {
@@ -2347,7 +2353,7 @@ void Gap::signal_connection_complete(
 
     /* first try to resolve synchronously in cache */
     if (!address_resolved) {
-        address_resolved = _address_registry.resolve_address_in_cache(
+        address_resolved = _address_registry.resolve_address_in_host_cache(
             event.getPeerAddress(),
             &peer_address_type,
             &peer_address
@@ -2372,7 +2378,7 @@ void Gap::signal_connection_complete(
         _event_handler->onConnectionComplete(event);
     } else {
         bool resolution_pending = false;
-        ble_error_t ret = _address_registry.queue_resolve_address(event.getPeerAddress());
+        ble_error_t ret = _address_registry.queue_resolve_address_on_host(event.getPeerAddress());
 
         if (ret == BLE_ERROR_NONE) {
             ConnectionCompleteEvent* event_copy = new(std::nothrow) ConnectionCompleteEvent(event);
@@ -2397,11 +2403,10 @@ void Gap::signal_connection_complete(
 #else
     report_internal_connection_complete(event);
     _event_handler->onConnectionComplete(event);
-#endif // BLE_FEATURE_PRIVACY
+#endif // BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 }
 
 #if BLE_FEATURE_PRIVACY
-
 bool Gap::apply_peripheral_privacy_connection_policy(
     const ConnectionCompleteEvent &event
 )
@@ -2451,8 +2456,9 @@ bool Gap::apply_peripheral_privacy_connection_policy(
     return true;
 #endif
 }
+#endif // BLE_FEATURE_PRIVACY
 
-
+#if BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 void Gap::conclude_signal_connection_complete_after_address_resolution(
     ConnectionCompleteEvent &event,
     target_peer_address_type_t identity_address_type,
@@ -2478,21 +2484,20 @@ void Gap::conclude_signal_connection_complete_after_address_resolution(
     report_internal_connection_complete(event);
     _event_handler->onConnectionComplete(event);
 }
-#endif // BLE_FEATURE_PRIVACY
+#endif // BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 #endif // BLE_FEATURE_CONNECTABLE
 
 #if BLE_ROLE_OBSERVER
 void Gap::signal_advertising_report(
     AdvertisingReportEvent& event
 )
 {
-#if BLE_FEATURE_PRIVACY
+#if BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
     bool address_resolved = false;
 
     /* if address resolution is not needed or already handled then the address is already resolved */
     if (!_privacy_enabled ||
         _central_privacy_configuration.resolution_strategy == central_privacy_configuration_t::DO_NOT_RESOLVE ||
-        _address_registry.is_controller_privacy_supported() ||
         event.getPeerAddressType() == peer_address_type_t::PUBLIC ||
         !is_random_private_resolvable_address(event.getPeerAddress())) {
         address_resolved = true;
@@ -2503,7 +2508,7 @@ void Gap::signal_advertising_report(
         const address_t *peer_address = nullptr;
         target_peer_address_type_t peer_address_type(target_peer_address_type_t::RANDOM);
 
-        address_resolved = _address_registry.resolve_address_in_cache(
+        address_resolved = _address_registry.resolve_address_in_host_cache(
             event.getPeerAddress(),
             &peer_address_type,
             &peer_address
@@ -2536,7 +2541,7 @@ void Gap::signal_advertising_report(
 
         /* if there is already an item with the same address pending don't kick off resolution*/
         if (!duplicate_pending_event) {
-            ret = _address_registry.queue_resolve_address(event.getPeerAddress());
+            ret = _address_registry.queue_resolve_address_on_host(event.getPeerAddress());
         }
 
         if (ret == BLE_ERROR_NONE) {
@@ -2553,11 +2558,11 @@ void Gap::signal_advertising_report(
     _event_handler->onAdvertisingReport(
         event
     );
-#endif // BLE_FEATURE_PRIVACY
+#endif // BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 }
 #endif //BLE_ROLE_OBSERVER
 
-#if BLE_FEATURE_PRIVACY
+#if BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 #if BLE_ROLE_OBSERVER
 void Gap::conclude_signal_advertising_report_after_address_resolution(
     AdvertisingReportEvent &event,
@@ -2581,7 +2586,7 @@ void Gap::conclude_signal_advertising_report_after_address_resolution(
     _event_handler->onAdvertisingReport(event);
 }
 #endif // BLE_ROLE_OBSERVER
-#endif // BLE_FEATURE_PRIVACY
+#endif // BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 
 void Gap::on_periodic_advertising_sync_established(
     hci_error_code_t error,
@@ -3179,15 +3184,15 @@ void Gap::on_private_address_generated(bool connectable)
     }
 }
 
-
+#if BLE_FEATURE_PRIVACY
+#if BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 void Gap::on_address_resolution_completed(
     const address_t &peer_resolvable_address,
     bool resolved,
     target_peer_address_type_t identity_address_type,
     const address_t &identity_address
 )
 {
-#if BLE_FEATURE_PRIVACY
     if (!_event_handler || !_privacy_enabled) {
         return;
     }
@@ -3237,9 +3242,9 @@ void Gap::on_address_resolution_completed(
         delete event;
     }
 #endif // BLE_ROLE_OBSERVER
-#endif // BLE_FEATURE_PRIVACY
 }
-
+#endif // BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
+#endif // BLE_FEATURE_PRIVACY
 
 bool Gap::is_advertising() const
 {
@@ -3297,7 +3302,7 @@ const address_t *Gap::get_random_address(controller_operation_t operation, size_
 #if BLE_FEATURE_EXTENDED_ADVERTISING
     if (is_extended_advertising_available()) {
         if (operation == controller_operation_t::advertising) {
-            if (_set_is_connectable.get(set_id) == false && peripheral_non_resolvable) {
+            if (!_set_is_connectable.get(set_id) && peripheral_non_resolvable) {
                 return &non_resolvable_address;
             } else {
                 return &resolvable_address;
diff --git a/connectivity/FEATURE_BLE/source/generic/GapImpl.h b/connectivity/FEATURE_BLE/source/generic/GapImpl.h
@@ -493,6 +493,7 @@ class Gap :
         IndexType _current_index = 0;
     };
 
+#if BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
     class PendingAdvertisingReportEvent {
     public:
         PendingAdvertisingReportEvent(
@@ -530,6 +531,7 @@ class Gap :
         AdvertisingReportEvent event;
         uint8_t *advertising_data_buffer = nullptr;
     };
+#endif // BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 
 private:
     /* Disallow copy and assignment. */
@@ -585,7 +587,9 @@ class Gap :
 
     bool initialize_whitelist() const;
 
+#if BLE_FEATURE_PRIVACY && !BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
     ble_error_t update_ll_address_resolution_setting();
+#endif // BLE_FEATURE_PRIVACY && !BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 
     ble_error_t setExtendedAdvertisingParameters(
         advertising_handle_t handle,
@@ -620,7 +624,9 @@ class Gap :
     bool apply_peripheral_privacy_connection_policy(
         const ConnectionCompleteEvent &event
     );
+#endif // BLE_FEATURE_PRIVACY
 
+#if BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
     /** Pass the connection complete event to the application after privacy resolution completed.
      *
      * @param event Event to be passed to the user application.
@@ -632,7 +638,7 @@ class Gap :
         target_peer_address_type_t identity_address_type,
         const address_t *identity_address
     );
-#endif // BLE_FEATURE_PRIVACY
+#endif // BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 #endif // BLE_FEATURE_CONNECTABLE
 
 #if BLE_ROLE_OBSERVER
@@ -642,7 +648,7 @@ class Gap :
      */
     void signal_advertising_report(AdvertisingReportEvent& report);
 
-#if BLE_FEATURE_PRIVACY
+#if BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
     /** Pass the advertising report to the application after privacy resolution completed.
      *
      * @param event Event to be passed to the user application.
@@ -654,7 +660,7 @@ class Gap :
         target_peer_address_type_t identity_address_type,
         const address_t *identity_address
     );
-#endif // BLE_FEATURE_PRIVACY
+#endif // BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 #endif // BLE_ROLE_OBSERVER
 
     /* implements PalGap::EventHandler */
@@ -767,13 +773,14 @@ class Gap :
     void on_non_resolvable_private_addresses_generated(const address_t &address) final;
 
     void on_private_address_generated(bool connectable);
-
+#if BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
     void on_address_resolution_completed(
         const address_t &peer_resolvable_address,
         bool resolved,
         target_peer_address_type_t identity_address_type,
         const address_t &identity_address
     ) final;
+#endif // BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 
 private:
     bool is_advertising() const;
@@ -803,14 +810,14 @@ class Gap :
      */
     ble::Gap::EventHandler *_event_handler;
 
-#if BLE_FEATURE_PRIVACY
+#if BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 #if BLE_ROLE_OBSERVER
     EventList<PendingAdvertisingReportEvent, uint8_t, BLE_GAP_MAX_ADVERTISING_REPORTS_PENDING_ADDRESS_RESOLUTION> _reports_pending_address_resolution;
 #endif // BLE_ROLE_OBSERVER
 #if BLE_FEATURE_CONNECTABLE
     EventList<ConnectionCompleteEvent, uint8_t, DM_CONN_MAX> _connections_pending_address_resolution;
 #endif // BLE_FEATURE_CONNECTABLE
-#endif // BLE_FEATURE_PRIVACY
+#endif // BLE_FEATURE_PRIVACY && BLE_GAP_HOST_BASED_PRIVATE_ADDRESS_RESOLUTION
 
     PalEventQueue &_event_queue;
     PalGap &_pal_gap;
diff --git a/connectivity/FEATURE_BLE/source/generic/PrivateAddressController.cpp b/connectivity/FEATURE_BLE/source/generic/PrivateAddressController.cpp
diff --git a/connectivity/FEATURE_BLE/source/generic/PrivateAddressController.h b/connectivity/FEATURE_BLE/source/generic/PrivateAddressController.h
diff --git a/connectivity/FEATURE_BLE/source/pal/PalPrivateAddressController.h b/connectivity/FEATURE_BLE/source/pal/PalPrivateAddressController.h
