Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4142a1b4 authored by Hsin-chen Chuang's avatar Hsin-chen Chuang
Browse files

floss: LE scan: Set FilterDuplicates enabled 

BlueZ enables FilterDuplicates while Android (Floss) doesn't.
Since a performance drop is observed when switching from BlueZ to Floss,
enable it to mimic the original behavior.

Bug: 315276614
Tag: #floss
Test: mmm packages/modules/Bluetooth
Test: SmartLock, PhoneHub, and NearbyShare
Test: bluetooth_AdapterLEHealth.all_floss \
      bluetooth_AdapterAdvMonitor.all_floss
Flag: EXEMPT floss only changes
Change-Id: I985162a3d4726024328191099c34030995c036fd
parent f21ee1ca

system/gd/hci/le_scanning_manager.cc

+16 −2
Original line number Original line Diff line number Diff line
@@ -603,7 +603,14 @@ struct LeScanningManager::impl : public LeAddressManagerCallback { 
      case ScanApiType::EXTENDED:

      case ScanApiType::EXTENDED:

        le_scanning_interface_->EnqueueCommand(

        le_scanning_interface_->EnqueueCommand(

            LeSetExtendedScanEnableBuilder::Create(

            LeSetExtendedScanEnableBuilder::Create(

                Enable::ENABLED, FilterDuplicates::DISABLED /* filter duplicates */, 0, 0),

                Enable::ENABLED,

#if TARGET_FLOSS

                FilterDuplicates::ENABLED /* filter duplicates */,

#else

                FilterDuplicates::DISABLED /* filter duplicates */,

#endif

                0,

                0),

            module_handler_->BindOnce(check_complete<LeSetExtendedScanEnableCompleteView>));

            module_handler_->BindOnce(check_complete<LeSetExtendedScanEnableCompleteView>));

        break;

        break;

      case ScanApiType::ANDROID_HCI:

      case ScanApiType::ANDROID_HCI:
@@ -627,7 +634,14 @@ struct LeScanningManager::impl : public LeAddressManagerCallback { 
      case ScanApiType::EXTENDED:

      case ScanApiType::EXTENDED:

        le_scanning_interface_->EnqueueCommand(

        le_scanning_interface_->EnqueueCommand(

            LeSetExtendedScanEnableBuilder::Create(

            LeSetExtendedScanEnableBuilder::Create(

                Enable::DISABLED, FilterDuplicates::DISABLED /* filter duplicates */, 0, 0),

                Enable::DISABLED,

#if TARGET_FLOSS

                FilterDuplicates::ENABLED /* filter duplicates */,

#else

                FilterDuplicates::DISABLED /* filter duplicates */,

#endif

                0,

                0),

            module_handler_->BindOnce(check_complete<LeSetExtendedScanEnableCompleteView>));

            module_handler_->BindOnce(check_complete<LeSetExtendedScanEnableCompleteView>));

        break;

        break;

      case ScanApiType::ANDROID_HCI:

      case ScanApiType::ANDROID_HCI: