Run GTests with ASAN on 32bit only

Address Sanitizer seems to only be flaky on 64bit binaries.
So workaround this issue by running the 32bit binaries on 64bit targets.

Bug: 273846603
Bug: 154347133
Test: TreeHugger
Change-Id: Ic573b01c122d44eb6928516682305bfec53fb957

Android.bp

@@ -67,3 +67,27 @@ filegroup {
         "AndroidTestTemplate.xml",
     ],
 }
+
+filegroup {
+    name: "BluetoothGTestForce32ConfigTemplate",
+    srcs: [
+        "AndroidGTestForce32Template.xml",
+    ],
+}
+
+// Address Sanitizer is flaky on Android x86_64 binaries but it's not on x86
+// binaries.
+// This default workaround the x86_64 ASAN flakyness by running 32bit binaries
+// on 64bit targets.
+// TODO(b/154347133): Remove this workaround
+cc_defaults {
+    name: "bluetooth_gtest_x86_asan_workaround",
+    target: {
+        android_x86: {
+            test_config_template: ":BluetoothGTestForce32ConfigTemplate",
+        },
+        android_x86_64: {
+            enabled: false,
+        },
+    },
+}

AndroidGTestForce32Template.xml

+<?xml version="1.0" encoding="utf-8"?>
+<!-- Copyright (C) 2023 The Android Open Source Project
+
+     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
+
+          http://www.apache.org/licenses/LICENSE-2.0
+
+     Unless required by applicable law or agreed to in writing, software
+     distributed under the License is distributed on an "AS IS" BASIS,
+     WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+     See the License for the specific language governing permissions and
+     limitations under the License.
+-->
+<configuration description="Runs {MODULE}.">
+  <option name="test-suite-tag" value="apct" />
+  <option name="test-suite-tag" value="apct-native" />
+  <target_preparer class="com.android.tradefed.targetprep.RootTargetPreparer" />
+  <target_preparer class="com.android.compatibility.common.tradefed.targetprep.FilePusher">
+        <option name="cleanup" value="true" />
+        <option name="push" value="{MODULE}32->/data/local/tmp/{MODULE}" />
+    </target_preparer>
+  <target_preparer class="com.android.tradefed.targetprep.RunCommandTargetPreparer">
+    <option name="run-command" value="settings put global ble_scan_always_enabled 0" />
+    <option name="run-command" value="cmd bluetooth_manager disable" />
+    <option name="run-command" value="cmd bluetooth_manager wait-for-state:STATE_OFF" />
+  </target_preparer>
+  <target_preparer class="com.android.tradefed.targetprep.FolderSaver">
+    <option name="device-path" value="/data/vendor/ssrdump" />
+  </target_preparer>
+  <test class="com.android.tradefed.testtype.GTest" >
+    <option name="native-test-device-path" value="/data/local/tmp" />
+    <option name="module-name" value="{MODULE}" />
+    <option name="run-test-as" value="0" />
+  </test>
+
+  <!-- Only run tests in MTS if the Bluetooth Mainline module is installed. -->
+  <object type="module_controller"
+          class="com.android.tradefed.testtype.suite.module.MainlineTestModuleController">
+      <option name="mainline-module-package-name" value="com.android.btservices" />
+      <option name="mainline-module-package-name" value="com.google.android.btservices" />
+  </object>
+</configuration>

system/audio_a2dp_hw/Android.bp

@@ -61,6 +61,7 @@ cc_test {
     defaults: [
         "audio_a2dp_hw_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     srcs: [
         "test/audio_a2dp_hw_test.cc",

system/audio_hearing_aid_hw/Android.bp

@@ -43,6 +43,7 @@ cc_test {
     defaults: [
         "audio_hearing_aid_hw_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     srcs: [
         "test/audio_hearing_aid_hw_test.cc",

system/bta/Android.bp

@@ -187,6 +187,7 @@ cc_test {
     defaults: [
         "fluoride_bta_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     srcs: [
@@ -233,6 +234,7 @@ cc_test {
     defaults: [
         "fluoride_bta_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     host_supported: true,
@@ -325,6 +327,7 @@ cc_test {
         "fluoride_bta_defaults",
         "clang_coverage_bin",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     include_dirs: [
@@ -379,6 +382,7 @@ cc_test {
         "fluoride_bta_defaults",
         "clang_coverage_bin",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     include_dirs: [
@@ -421,6 +425,7 @@ cc_test {
         "fluoride_bta_defaults",
         "clang_coverage_bin",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     include_dirs: [
@@ -540,6 +545,7 @@ cc_test {
         "fluoride_defaults",
         "clang_coverage_bin",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     target: {
@@ -628,6 +634,7 @@ cc_test {
         "fluoride_bta_defaults",
         "clang_coverage_bin",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     // TODO(b/231993739): Reenable isolated:true by deleting the explicit disable below
@@ -721,6 +728,7 @@ cc_test {
         "fluoride_bta_defaults",
         "clang_coverage_bin",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     // TODO(b/231993739): Reenable isolated:true by deleting the explicit disable below
@@ -772,6 +780,7 @@ cc_test {
         "fluoride_bta_defaults",
         "clang_coverage_bin",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     include_dirs: [
@@ -843,6 +852,7 @@ cc_test {
         "fluoride_bta_defaults",
         "clang_coverage_bin",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     // TODO(b/231993739): Reenable isolated:true by deleting the explicit disable below
@@ -899,6 +909,7 @@ cc_test {
         "fluoride_bta_defaults",
         "clang_coverage_bin",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     // TODO(b/231993739): Reenable isolated:true by deleting the explicit disable below

system/btcore/Android.bp

@@ -89,6 +89,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     local_include_dirs: ["include"],
     include_dirs: ["packages/modules/Bluetooth/system"],

system/btif/Android.bp

@@ -244,6 +244,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     include_dirs: btifCommonIncludes,
@@ -312,6 +313,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     include_dirs: btifCommonIncludes,
@@ -342,6 +344,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     host_supported: true,
@@ -380,6 +383,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     host_supported: true,
@@ -420,6 +424,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     include_dirs: btifCommonIncludes,
@@ -446,6 +451,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     include_dirs: [
@@ -569,6 +575,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     include_dirs: [

system/common/Android.bp

@@ -48,6 +48,7 @@ cc_test {
         "fluoride_defaults",
         "clang_coverage_bin",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     test_options: {
@@ -96,6 +97,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     include_dirs: ["packages/modules/Bluetooth/system"],

system/device/Android.bp

@@ -38,6 +38,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     include_dirs: ["packages/modules/Bluetooth/system"],
@@ -65,6 +66,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     include_dirs: [
         "packages/modules/Bluetooth/system",

system/embdrv/tests/Android.bp

@@ -11,6 +11,7 @@ cc_test {
     name: "libaptx_enc_tests",
     defaults: [
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     host_supported: true,
@@ -30,6 +31,7 @@ cc_test {
     name: "libaptxhd_enc_tests",
     defaults: [
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     host_supported: true,

system/gd/Android.bp

@@ -350,6 +350,7 @@ cc_test {
         "gd_clang_coverage_bin",
         "libchrome_support_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     // TODO(b/231993739): Reenable isolated:true by deleting the explicit disable below
@@ -445,6 +446,7 @@ cc_test {
         "gd_clang_coverage_bin",
         "libchrome_support_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     include_dirs: ["packages/modules/Bluetooth/system/gd"],
     host_supported: true,

system/gd/dumpsys/Android.bp

@@ -181,6 +181,7 @@ cc_test {
     defaults: [
         "fluoride_common_options",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     test_options: {

system/hci/Android.bp

@@ -65,6 +65,7 @@ cc_test {
         "fluoride_basic_defaults",
         "fluoride_test_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: false,
     local_include_dirs: [
@@ -99,6 +100,7 @@ cc_test {
     defaults: [
         "fluoride_unit_test_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     local_include_dirs: [
         "include",

system/main/Android.bp

@@ -155,6 +155,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     include_dirs: [
         "packages/modules/Bluetooth/system",

system/osi/Android.bp

@@ -125,6 +125,7 @@ cc_test {
     defaults: [
         "fluoride_osi_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     srcs: [

system/packet/Android.bp

@@ -30,6 +30,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     host_supported: true,

system/profile/avrcp/Android.bp

@@ -49,6 +49,7 @@ cc_test {
         "clang_coverage_bin",
         "libchrome_support_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     test_options: {

system/profile/sdp/Android.bp

@@ -33,6 +33,7 @@ cc_test {
         "fluoride_defaults",
         "clang_coverage_bin",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     include_dirs: [

system/stack/Android.bp

@@ -293,6 +293,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     local_include_dirs: [
@@ -362,6 +363,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     host_supported: true,
@@ -431,6 +433,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     test_suites: ["device-tests"],
@@ -495,6 +498,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     local_include_dirs: [
@@ -536,6 +540,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     local_include_dirs: [
@@ -564,6 +569,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     host_supported: true,
     test_suites: ["general-tests"],
@@ -602,6 +608,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     host_supported: true,
@@ -651,6 +658,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     host_supported: true,
@@ -706,6 +714,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     host_supported: true,
@@ -737,6 +746,9 @@ cc_test {
             ],
             test_config: "test/a2dp/AndroidTest.xml",
         },
+        android_x86: {
+            test_config: "test/a2dp/AndroidTestForce32.xml",
+        },
     },
     data: [
         "test/a2dp/raw_data/*",
@@ -808,6 +820,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     host_supported: true,
@@ -849,6 +862,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     test_suites: ["device-tests"],
     host_supported: true,
@@ -896,6 +910,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     local_include_dirs: [
         "btm",
@@ -944,6 +959,7 @@ cc_test {
         "fluoride_defaults",
         "clang_coverage_bin",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     local_include_dirs: [
         "include",
@@ -997,6 +1013,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     local_include_dirs: [
         "include",
@@ -1106,6 +1123,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     local_include_dirs: [
         "include",
@@ -1152,6 +1170,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     local_include_dirs: [
         "include",
@@ -1207,6 +1226,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     local_include_dirs: [
         "include",
@@ -1271,6 +1291,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     local_include_dirs: [
         "include",
@@ -1350,6 +1371,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     local_include_dirs: [
         "include",
@@ -1423,6 +1445,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     local_include_dirs: [
         "include",
@@ -1496,6 +1519,7 @@ cc_test {
     defaults: [
         "fluoride_defaults",
         "mts_defaults",
+        "bluetooth_gtest_x86_asan_workaround",
     ],
     local_include_dirs: [
         "include",

system/stack/test/a2dp/AndroidTestForce32.xml

+<?xml version="1.0" encoding="utf-8"?>
+<!-- Copyright 2022 The Android Open Source Project
+
+     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
+
+          http://www.apache.org/licenses/LICENSE-2.0
+
+     Unless required by applicable law or agreed to in writing, software
+     distributed under the License is distributed on an "AS IS" BASIS,
+     WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+     See the License for the specific language governing permissions and
+     limitations under the License.
+-->
+<configuration description="Runs net_test_stack_a2dp_codecs_native.">
+  <target_preparer class="com.android.tradefed.targetprep.RootTargetPreparer" />
+  <target_preparer class="com.android.compatibility.common.tradefed.targetprep.FilePusher">
+        <option name="cleanup" value="true" />
+        <option name="push" value="net_test_stack_a2dp_codecs_native32->/data/local/tmp/net_test_stack_a2dp_codecs_native" />
+  </target_preparer>
+  <target_preparer class="com.android.compatibility.common.tradefed.targetprep.FilePusher">
+        <option name="cleanup" value="true" />
+        <option name="push" value="pcm0844s.wav->/data/local/tmp/test/a2dp/raw_data/pcm0844s.wav" />
+        <option name="push" value="pcm1644s.wav->/data/local/tmp/test/a2dp/raw_data/pcm1644s.wav" />
+  </target_preparer>
+  <test class="com.android.tradefed.testtype.GTest" >
+    <option name="native-test-device-path" value="/data/local/tmp" />
+    <option name="module-name" value="net_test_stack_a2dp_codecs_native" />
+    <option name="run-test-as" value="0" />
+  </test>
+
+  <!-- Only run tests in MTS if the Bluetooth Mainline module is installed. -->
+  <object type="module_controller"
+          class="com.android.tradefed.testtype.suite.module.MainlineTestModuleController">
+      <option name="mainline-module-package-name" value="com.google.android.bluetooth" />
+  </object>
+</configuration>