diff --git a/core/Makefile b/core/Makefile index 4c5183e74..4e4ffbd63 100644 --- a/core/Makefile +++ b/core/Makefile @@ -2870,94 +2870,6 @@ INTERNAL_VENDORIMAGE_FILES := \ $(ALL_PDK_FUSION_FILES)) \ $(PDK_FUSION_SYMLINK_STAMP) -# Final Vendor VINTF manifest including fragments. This is not assembled -# on the device because it depends on everything in a given device -# image which defines a vintf_fragment. -ifdef BUILT_VENDOR_MANIFEST -BUILT_ASSEMBLED_VENDOR_MANIFEST := $(PRODUCT_OUT)/verified_assembled_vendor_manifest.xml -ifeq (true,$(PRODUCT_ENFORCE_VINTF_MANIFEST)) -ifneq ($(strip $(DEVICE_FRAMEWORK_COMPATIBILITY_MATRIX_FILE) $(DEVICE_PRODUCT_COMPATIBILITY_MATRIX_FILE)),) -$(BUILT_ASSEMBLED_VENDOR_MANIFEST): PRIVATE_SYSTEM_ASSEMBLE_VINTF_ENV_VARS := VINTF_ENFORCE_NO_UNUSED_HALS=true -endif # DEVICE_FRAMEWORK_COMPATIBILITY_MATRIX_FILE or DEVICE_PRODUCT_COMPATIBILITY_MATRIX_FILE -endif # PRODUCT_ENFORCE_VINTF_MANIFEST -$(BUILT_ASSEMBLED_VENDOR_MANIFEST): $(HOST_OUT_EXECUTABLES)/assemble_vintf -$(BUILT_ASSEMBLED_VENDOR_MANIFEST): $(BUILT_SYSTEM_MATRIX) -$(BUILT_ASSEMBLED_VENDOR_MANIFEST): $(BUILT_VENDOR_MANIFEST) -$(BUILT_ASSEMBLED_VENDOR_MANIFEST): $(INTERNAL_VENDORIMAGE_FILES) - -$(BUILT_ASSEMBLED_VENDOR_MANIFEST): PRIVATE_FLAGS := - -# -- Kernel version and configurations. -ifeq ($(PRODUCT_OTA_ENFORCE_VINTF_KERNEL_REQUIREMENTS),true) - -intermediates := $(call intermediates-dir-for,ETC,$(notdir $(BUILT_ASSEMBLED_VENDOR_MANIFEST))) -BUILT_KERNEL_CONFIGS_FILE := $(intermediates)/kernel_configs.txt -BUILT_KERNEL_VERSION_FILE := $(intermediates)/kernel_version.txt - -# BOARD_KERNEL_CONFIG_FILE and BOARD_KERNEL_VERSION can be used to override the values extracted -# from INSTALLED_KERNEL_TARGET. -ifdef BOARD_KERNEL_CONFIG_FILE -ifdef BOARD_KERNEL_VERSION -$(BUILT_KERNEL_CONFIGS_FILE): $(BOARD_KERNEL_CONFIG_FILE) - cp $< $@ -$(BUILT_KERNEL_VERSION_FILE): - echo $(BOARD_KERNEL_VERSION) > $@ - -my_board_extracted_kernel := true -endif # BOARD_KERNEL_VERSION -endif # BOARD_KERNEL_CONFIG_FILE - -ifneq ($(my_board_extracted_kernel),true) -ifndef INSTALLED_KERNEL_TARGET -$(warning No INSTALLED_KERNEL_TARGET is defined when PRODUCT_OTA_ENFORCE_VINTF_KERNEL_REQUIREMENTS \ - is true. Information about the updated kernel cannot be built into OTA update package. \ - You can fix this by: (1) setting TARGET_NO_KERNEL to false and installing the built kernel \ - to $(PRODUCT_OUT)/kernel, so that kernel information will be extracted from the built kernel; \ - or (2) extracting kernel configuration and defining BOARD_KERNEL_CONFIG_FILE and \ - BOARD_KERNEL_VERSION manually; or (3) unsetting PRODUCT_OTA_ENFORCE_VINTF_KERNEL_REQUIREMENTS \ - manually.) -else - -# Tools for decompression that is not in PATH. -# Check $(EXTRACT_KERNEL) for decompression algorithms supported by the script. -# Algorithms that are in the script but not in this list will be found in PATH. -my_decompress_tools := \ - lz4:$(HOST_OUT_EXECUTABLES)/lz4 \ - -$(BUILT_KERNEL_CONFIGS_FILE): .KATI_IMPLICIT_OUTPUTS := $(BUILT_KERNEL_VERSION_FILE) -$(BUILT_KERNEL_CONFIGS_FILE): PRIVATE_DECOMPRESS_TOOLS := $(my_decompress_tools) -$(BUILT_KERNEL_CONFIGS_FILE): $(foreach pair,$(my_decompress_tools),$(call word-colon,2,$(pair))) -$(BUILT_KERNEL_CONFIGS_FILE): $(EXTRACT_KERNEL) $(INSTALLED_KERNEL_TARGET) - $< --tools $(PRIVATE_DECOMPRESS_TOOLS) --input $(INSTALLED_KERNEL_TARGET) \ - --output-configs $@ \ - --output-version $(BUILT_KERNEL_VERSION_FILE) - -intermediates := -my_decompress_tools := - -endif # my_board_extracted_kernel -my_board_extracted_kernel := - -endif # INSTALLED_KERNEL_TARGET - -$(BUILT_ASSEMBLED_VENDOR_MANIFEST): $(BUILT_KERNEL_CONFIGS_FILE) $(BUILT_KERNEL_VERSION_FILE) -$(BUILT_ASSEMBLED_VENDOR_MANIFEST): PRIVATE_FLAGS += --kernel $$(cat $(BUILT_KERNEL_VERSION_FILE)):$(BUILT_KERNEL_CONFIGS_FILE) - -endif # PRODUCT_OTA_ENFORCE_VINTF_KERNEL_REQUIREMENTS - -$(BUILT_ASSEMBLED_VENDOR_MANIFEST): - @echo "Verifying vendor VINTF manifest." - PRODUCT_ENFORCE_VINTF_MANIFEST=$(PRODUCT_ENFORCE_VINTF_MANIFEST) \ - $(PRIVATE_SYSTEM_ASSEMBLE_VINTF_ENV_VARS) \ - $(HOST_OUT_EXECUTABLES)/assemble_vintf \ - $(PRIVATE_FLAGS) \ - -c $(BUILT_SYSTEM_MATRIX) \ - -i $(BUILT_VENDOR_MANIFEST) \ - $$([ -d $(TARGET_OUT_VENDOR)/etc/vintf/manifest ] && \ - find $(TARGET_OUT_VENDOR)/etc/vintf/manifest -type f -name "*.xml" | \ - sed "s/^/-i /" | tr '\n' ' ') -o $@ -endif # BUILT_VENDOR_MANIFEST - # platform.zip depends on $(INTERNAL_VENDORIMAGE_FILES). $(INSTALLED_PLATFORM_ZIP) : $(INTERNAL_VENDORIMAGE_FILES) @@ -3004,9 +2916,6 @@ endef # We just build this directly to the install location. INSTALLED_VENDORIMAGE_TARGET := $(BUILT_VENDORIMAGE_TARGET) -ifdef BUILT_VENDOR_MANIFEST -$(INSTALLED_VENDORIMAGE_TARGET): $(BUILT_ASSEMBLED_VENDOR_MANIFEST) -endif $(INSTALLED_VENDORIMAGE_TARGET): \ $(INTERNAL_USERIMAGES_DEPS) \ $(INTERNAL_VENDORIMAGE_FILES) \ @@ -3080,52 +2989,6 @@ INSTALLED_PRODUCTIMAGE_TARGET := $(PRODUCT_OUT)/product.img $(eval $(call copy-one-file,$(BOARD_PREBUILT_PRODUCTIMAGE),$(INSTALLED_PRODUCTIMAGE_TARGET))) endif -# ----------------------------------------------------------------- -# Final Framework VINTF manifest including fragments. This is not assembled -# on the device because it depends on everything in a given device -# image which defines a vintf_fragment. - -ifdef BUILDING_SYSTEM_IMAGE - -ifndef BOARD_USES_PRODUCTIMAGE - # If no product image at all, check system manifest directly against device matrix. - check_framework_manifest := true -else ifdef BUILDING_PRODUCT_IMAGE - # If device has a product image, only check if the product image is built. - check_framework_manifest := true -endif - -# TODO (b/131425279): delete this line once build_mixed script can correctly merge system and -# product manifests. -check_framework_manifest := true - -ifeq ($(check_framework_manifest),true) - -BUILT_ASSEMBLED_FRAMEWORK_MANIFEST := $(PRODUCT_OUT)/verified_assembled_framework_manifest.xml -$(BUILT_ASSEMBLED_FRAMEWORK_MANIFEST): $(HOST_OUT_EXECUTABLES)/assemble_vintf \ - $(BUILT_VENDOR_MATRIX) \ - $(BUILT_SYSTEM_MANIFEST) \ - $(FULL_SYSTEMIMAGE_DEPS) \ - $(BUILT_PRODUCT_MANIFEST) \ - $(BUILT_PRODUCTIMAGE_TARGET) - @echo "Verifying framework VINTF manifest." - PRODUCT_ENFORCE_VINTF_MANIFEST=$(PRODUCT_ENFORCE_VINTF_MANIFEST) \ - $(HOST_OUT_EXECUTABLES)/assemble_vintf \ - -o $@ \ - -c $(BUILT_VENDOR_MATRIX) \ - -i $(BUILT_SYSTEM_MANIFEST) \ - $(addprefix -i ,\ - $(filter $(TARGET_OUT)/etc/vintf/manifest/%.xml,$(FULL_SYSTEMIMAGE_DEPS)) \ - $(BUILT_PRODUCT_MANIFEST) \ - $(filter $(TARGET_OUT_PRODUCT)/etc/vintf/manifest/%.xml,$(INTERNAL_PRODUCTIMAGE_FILES))) - -droidcore: $(BUILT_ASSEMBLED_FRAMEWORK_MANIFEST) - -endif # check_framework_manifest -check_framework_manifest := - -endif # BUILDING_SYSTEM_IMAGE - # ----------------------------------------------------------------- # system_ext partition image ifdef BUILDING_SYSTEM_EXT_IMAGE @@ -3642,6 +3505,26 @@ ifndef TARGET_BUILD_APPS intermediates := $(call intermediates-dir-for,PACKAGING,check_vintf_all) check_vintf_all_deps := +# -- Check system manifest / matrix including fragments (excluding other framework manifests / matrices, e.g. product); +ifeq ($(BUILDING_SYSTEM_IMAGE),true) +check_vintf_system_log := $(intermediates)/check_vintf_system_log +check_vintf_all_deps += $(check_vintf_system_log) +$(check_vintf_system_log): $(HOST_OUT_EXECUTABLES)/checkvintf \ + $(filter $(TARGET_OUT)/etc/vintf/%, $(INTERNAL_SYSTEMIMAGE_FILES)) + @( $< --check-one --dirmap /system:$(TARGET_OUT) > $@ 2>&1 ) || ( cat $@ && exit 1 ) +check_vintf_system_log := +endif # BUILDING_SYSTEM_IMAGE + +# -- Check vendor manifest / matrix including fragments (excluding other device manifests / matrices) +ifeq ($(BUILDING_VENDOR_IMAGE),true) +check_vintf_vendor_log := $(intermediates)/check_vintf_vendor_log +check_vintf_all_deps += $(check_vintf_vendor_log) +$(check_vintf_vendor_log): $(HOST_OUT_EXECUTABLES)/checkvintf \ + $(filter $(TARGET_OUT_VENDOR)/etc/vintf/%, $(INTERNAL_VENDORIMAGE_FILES)) + @( $< --check-one --dirmap /vendor:$(TARGET_OUT_VENDOR) > $@ 2>&1 ) || ( cat $@ && exit 1 ) +check_vintf_vendor_log := +endif # BUILDING_VENDOR_IMAGE + # -- Check VINTF compatibility of build. # Skip partial builds; only check full builds. Only check if: # - PRODUCT_ENFORCE_VINTF_MANIFEST is true @@ -3663,6 +3546,56 @@ check_vintf_compatible_deps := # -- Kernel version and configurations. ifeq ($(PRODUCT_OTA_ENFORCE_VINTF_KERNEL_REQUIREMENTS),true) +BUILT_KERNEL_CONFIGS_FILE := $(intermediates)/kernel_configs.txt +BUILT_KERNEL_VERSION_FILE := $(intermediates)/kernel_version.txt + +my_board_extracted_kernel := + +# BOARD_KERNEL_CONFIG_FILE and BOARD_KERNEL_VERSION can be used to override the values extracted +# from INSTALLED_KERNEL_TARGET. +ifdef BOARD_KERNEL_CONFIG_FILE +ifdef BOARD_KERNEL_VERSION +$(BUILT_KERNEL_CONFIGS_FILE): $(BOARD_KERNEL_CONFIG_FILE) + cp $< $@ +$(BUILT_KERNEL_VERSION_FILE): + echo $(BOARD_KERNEL_VERSION) > $@ + +my_board_extracted_kernel := true +endif # BOARD_KERNEL_VERSION +endif # BOARD_KERNEL_CONFIG_FILE + +ifneq ($(my_board_extracted_kernel),true) +ifndef INSTALLED_KERNEL_TARGET +$(warning No INSTALLED_KERNEL_TARGET is defined when PRODUCT_OTA_ENFORCE_VINTF_KERNEL_REQUIREMENTS \ + is true. Information about the updated kernel cannot be built into OTA update package. \ + You can fix this by: (1) setting TARGET_NO_KERNEL to false and installing the built kernel \ + to $(PRODUCT_OUT)/kernel, so that kernel information will be extracted from the built kernel; \ + or (2) extracting kernel configuration and defining BOARD_KERNEL_CONFIG_FILE and \ + BOARD_KERNEL_VERSION manually; or (3) unsetting PRODUCT_OTA_ENFORCE_VINTF_KERNEL_REQUIREMENTS \ + manually.) +else + +# Tools for decompression that is not in PATH. +# Check $(EXTRACT_KERNEL) for decompression algorithms supported by the script. +# Algorithms that are in the script but not in this list will be found in PATH. +my_decompress_tools := \ + lz4:$(HOST_OUT_EXECUTABLES)/lz4 \ + +$(BUILT_KERNEL_CONFIGS_FILE): .KATI_IMPLICIT_OUTPUTS := $(BUILT_KERNEL_VERSION_FILE) +$(BUILT_KERNEL_CONFIGS_FILE): PRIVATE_DECOMPRESS_TOOLS := $(my_decompress_tools) +$(BUILT_KERNEL_CONFIGS_FILE): $(foreach pair,$(my_decompress_tools),$(call word-colon,2,$(pair))) +$(BUILT_KERNEL_CONFIGS_FILE): $(EXTRACT_KERNEL) $(INSTALLED_KERNEL_TARGET) + $< --tools $(PRIVATE_DECOMPRESS_TOOLS) --input $(INSTALLED_KERNEL_TARGET) \ + --output-configs $@ \ + --output-version $(BUILT_KERNEL_VERSION_FILE) + +my_decompress_tools := + +endif # my_board_extracted_kernel +my_board_extracted_kernel := + +endif # INSTALLED_KERNEL_TARGET + check_vintf_compatible_args += --kernel $$(cat $(BUILT_KERNEL_VERSION_FILE)):$(BUILT_KERNEL_CONFIGS_FILE) check_vintf_compatible_deps += $(BUILT_KERNEL_CONFIGS_FILE) $(BUILT_KERNEL_VERSION_FILE)