From a2a5db44663a8dff93f0456034e17848155613f8 Mon Sep 17 00:00:00 2001 From: Bowgo Tsai Date: Thu, 23 Sep 2021 15:46:33 +0800 Subject: [PATCH] Renames boot-debug-*.img in GSI targets Those boot-debug-*.img is used with `repack_bootimg` for a vendor_boot-debug.img in VTS setup. It is not for GKI boot.img release. https://source.android.com/compatibility/vts/vts-on-gsi#repacking Renames boot-debug-*.img to boot-with-debug-ramdisk-*.img to avoid confusion with the official GKI boot.img release. Bug: 200878300 Test: `lunch gsi_arm64-user` then `make bootimage_debug` Change-Id: Ia1f6ba847d5b7409fb7a8534432484d2aa972494 --- core/Makefile | 8 ++++---- core/board_config.mk | 6 ++++++ core/product.mk | 3 +++ target/product/gsi_release.mk | 4 ++++ 4 files changed, 17 insertions(+), 4 deletions(-) diff --git a/core/Makefile b/core/Makefile index a8486303ee..14c7a59b8c 100644 --- a/core/Makefile +++ b/core/Makefile @@ -2359,10 +2359,10 @@ ifneq ($(BUILDING_VENDOR_BOOT_IMAGE),true) ifneq ($(INSTALLED_BOOTIMAGE_TARGET),) ifneq ($(strip $(TARGET_NO_KERNEL)),true) ifneq ($(strip $(BOARD_KERNEL_BINARIES)),) - INSTALLED_DEBUG_BOOTIMAGE_TARGET := $(foreach k,$(subst kernel,boot-debug,$(BOARD_KERNEL_BINARIES)), \ + INSTALLED_DEBUG_BOOTIMAGE_TARGET := $(foreach k,$(subst kernel,$(DEBUG_RAMDISK_BOOT_IMAGE_NAME),$(BOARD_KERNEL_BINARIES)), \ $(PRODUCT_OUT)/$(k).img) else - INSTALLED_DEBUG_BOOTIMAGE_TARGET := $(PRODUCT_OUT)/boot-debug.img + INSTALLED_DEBUG_BOOTIMAGE_TARGET := $(PRODUCT_OUT)/$(DEBUG_RAMDISK_BOOT_IMAGE_NAME).img endif # Replace ramdisk.img in $(MKBOOTIMG) ARGS with ramdisk-debug.img to build boot-debug.img @@ -2396,10 +2396,10 @@ endef # $(1): output file define build-debug-bootimage-target - $(MKBOOTIMG) --kernel $(PRODUCT_OUT)/$(subst .img,,$(subst boot-debug,kernel,$(notdir $(1)))) \ + $(MKBOOTIMG) --kernel $(PRODUCT_OUT)/$(subst .img,,$(subst $(DEBUG_RAMDISK_BOOT_IMAGE_NAME),kernel,$(notdir $(1)))) \ $(INTERNAL_DEBUG_BOOTIMAGE_ARGS) $(INTERNAL_MKBOOTIMG_VERSION_ARGS) \ $(INTERNAL_MKBOOTIMG_GKI_SINGING_ARGS) $(BOARD_MKBOOTIMG_ARGS) --output $1 - $(if $(BOARD_AVB_BOOT_KEY_PATH),$(call test-key-sign-bootimage,$1,boot-debug)) + $(if $(BOARD_AVB_BOOT_KEY_PATH),$(call test-key-sign-bootimage,$1,$(DEBUG_RAMDISK_BOOT_IMAGE_NAME))) endef # Depends on original boot.img and ramdisk-debug.img, to build the new boot-debug.img diff --git a/core/board_config.mk b/core/board_config.mk index 53dbb92c19..b4e2b914c1 100644 --- a/core/board_config.mk +++ b/core/board_config.mk @@ -397,6 +397,12 @@ else ifeq ($(PRODUCT_BUILD_BOOT_IMAGE),true) endif .KATI_READONLY := BUILDING_BOOT_IMAGE +DEBUG_RAMDISK_BOOT_IMAGE_NAME := boot-debug +ifneq ($(PRODUCT_DEBUG_RAMDISK_BOOT_IMAGE_NAME),) + DEBUG_RAMDISK_BOOT_IMAGE_NAME := $(PRODUCT_DEBUG_RAMDISK_BOOT_IMAGE_NAME) +endif +.KATI_READONLY := DEBUG_RAMDISK_BOOT_IMAGE_NAME + # Are we building a recovery image BUILDING_RECOVERY_IMAGE := ifeq ($(PRODUCT_BUILD_RECOVERY_IMAGE),) diff --git a/core/product.mk b/core/product.mk index 9aaf5eb300..bd179a9dae 100644 --- a/core/product.mk +++ b/core/product.mk @@ -278,6 +278,9 @@ _product_single_value_vars += \ PRODUCT_DEX_PREOPT_BOOT_IMAGE_PROFILE_LOCATION \ PRODUCT_USES_DEFAULT_ART_CONFIG \ +# The file name for the boot image with a debug ramdisk. +_product_single_value_vars += PRODUCT_DEBUG_RAMDISK_BOOT_IMAGE_NAME + _product_single_value_vars += PRODUCT_SYSTEM_SERVER_COMPILER_FILTER # Per-module sanitizer configs _product_list_vars += PRODUCT_SANITIZER_MODULE_CONFIGS diff --git a/target/product/gsi_release.mk b/target/product/gsi_release.mk index bd7c4ab3eb..014366dd05 100644 --- a/target/product/gsi_release.mk +++ b/target/product/gsi_release.mk @@ -78,3 +78,7 @@ MODULE_BUILD_FROM_SOURCE := true # Additional settings used in all GSI builds PRODUCT_PRODUCT_PROPERTIES += \ ro.crypto.metadata_init_delete_all_keys.enabled=false \ + +# Renames boot-debug-*.img to boot-with-debug-ramdisk-*.img to avoid confusion +# with GKI boot images. +PRODUCT_DEBUG_RAMDISK_BOOT_IMAGE_NAME := boot-with-debug-ramdisk