commit b8323be61da942f797e9bdaf959b13d68426a57f
author Yifan Hong <elsk@google.com> Thu Sep 15 11:51:15 2022 -0700
committer Treehugger Robot <treehugger-gerrit@google.com> Mon Sep 19 22:34:09 2022 +0000
tree f9fb0029499cde69e6cf63aac52a0fb9ecf44599
parent c3154ba8a51b3770c03a35d8e175d85b157bfca3

ANDROID: kleaf: Explicit list of ABI files.

In BUILD.bazel, explicitly list abi_definition,
kmi_symbol_list, and additional_kmi_symbol_lists.
This avoids using the glob expression which may
accidentally match editor backup files.

Bug: 246344503
Test: TH
Change-Id: I8fbad63bf744672ba5b76b51cc908926d7b10849
Signed-off-by: Yifan Hong <elsk@google.com>

BUILD.bazel

diff --git a/BUILD.bazel b/BUILD.bazel
index 0ecc69b..c5932d1 100644
--- a/BUILD.bazel
+++ b/BUILD.bazel
@@ -21,11 +21,25 @@
 load("//build/kernel/kleaf:common_kernels.bzl", "define_common_kernels", "define_db845c")
 load("//build/kernel/kleaf:kernel.bzl", "kernel_build")
 
-# This uses android/abi_gki_aarch64* in kmi_configs. If the list of
-# glob(["android/abi_gki_aarch64*"]) differs from
-# KMI_SYMBOL_LIST + ADDITIONAL_KMI_SYMBOL_LISTS in build.config.gki.aarch64,
-# or TRIM_NONLISTED_KMI changes, override kmi_configs here.
+_aarch64_additional_kmi_symbol_lists = [
+    # keep sorted
+    "android/abi_gki_aarch64_db845c",
+    "android/abi_gki_aarch64_exynos",
+    "android/abi_gki_aarch64_exynosauto",
+    "android/abi_gki_aarch64_fips140",
+    "android/abi_gki_aarch64_honor",
+    "android/abi_gki_aarch64_imx",
+    "android/abi_gki_aarch64_mtk",
+    "android/abi_gki_aarch64_mtktv",
+    "android/abi_gki_aarch64_oplus",
+    "android/abi_gki_aarch64_pixel",
+    "android/abi_gki_aarch64_qcom",
+    "android/abi_gki_aarch64_unisoc",
+    "android/abi_gki_aarch64_virtual_device",
+]
+
 define_common_kernels(target_configs = {
+    # Sync with build.config.gki.aarch64
     "kernel_aarch64": {
         # TODO(b/188620248): re-enable trimming
         "trim_nonlisted_kmi": False,
@@ -34,6 +48,8 @@
             "drivers/block/zram/zram.ko",
             "mm/zsmalloc.ko",
         ],
+        "kmi_symbol_list": "android/abi_gki_aarch64",
+        "additional_kmi_symbol_lists": _aarch64_additional_kmi_symbol_lists,
     },
     "kernel_aarch64_debug": {
         # TODO(b/188620248): re-enable trimming
@@ -43,6 +59,8 @@
             "drivers/block/zram/zram.ko",
             "mm/zsmalloc.ko",
         ],
+        "kmi_symbol_list": "android/abi_gki_aarch64",
+        "additional_kmi_symbol_lists": _aarch64_additional_kmi_symbol_lists,
     },
 })
 

build.config.gki.aarch64

diff --git a/build.config.gki.aarch64 b/build.config.gki.aarch64
index 70f9b26..d236ebf 100644
--- a/build.config.gki.aarch64
+++ b/build.config.gki.aarch64
@@ -7,11 +7,9 @@
 Image.gz
 "
 
+# Sync with BUILD.bazel
 # uncomment ABI_DEFINITION when enabling KMI enforcement
 #ABI_DEFINITION=android/abi_gki_aarch64.xml
-
-# Update BUILD.bazel, define_common_kernels() if the list differs from
-# android/abi_gki_aarch64* in the filesystem.
 KMI_SYMBOL_LIST=android/abi_gki_aarch64
 ADDITIONAL_KMI_SYMBOL_LISTS="
 android/abi_gki_aarch64_virtual_device