mirror of
				https://github.com/openwrt/openwrt.git
				synced 2025-10-30 07:49:23 +08:00 
			
		
		
		
	image: move mkits.sh outside lock section
As mkits.sh only generates the relevant DTS for mkimage and is specific for the building image, we can move it outside the locked section as it doesn't do any operation that can be used concurrently by others. This won't have any real impact but clean the code making it clear what needs to be protected and what can be executed concurrently. Link: https://github.com/openwrt/openwrt/pull/20492 Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
This commit is contained in:
		| @ -431,27 +431,27 @@ define Build/initrd_compression | ||||
| endef | ||||
|  | ||||
| define Build/fit | ||||
| 	$(call locked, { \ | ||||
| 		$(if $(findstring with-rootfs,$(word 3,$(1))),dd if=$(IMAGE_ROOTFS) of=$(IMAGE_ROOTFS).pagesync bs=4096 conv=sync;) \ | ||||
| 		$(TOPDIR)/scripts/mkits.sh \ | ||||
| 			-D $(DEVICE_NAME) -o $@.its -k $@ \ | ||||
| 			-C $(word 1,$(1)) \ | ||||
| 			$(if $(word 2,$(1)),\ | ||||
| 				$(if $(findstring 11,$(if $(DEVICE_DTS_OVERLAY),1)$(if $(findstring $(KERNEL_BUILD_DIR)/image-,$(word 2,$(1))),,1)), \ | ||||
| 					-d $(KERNEL_BUILD_DIR)/image-$$(basename $(word 2,$(1))), \ | ||||
| 					-d $(word 2,$(1)))) \ | ||||
| 			$(if $(findstring with-rootfs,$(word 3,$(1))),-r $(IMAGE_ROOTFS)) \ | ||||
| 			$(if $(findstring with-initrd,$(word 3,$(1))), \ | ||||
| 				$(if $(CONFIG_TARGET_ROOTFS_INITRAMFS_SEPARATE), \ | ||||
| 					-i $(KERNEL_BUILD_DIR)/initrd$(if $(TARGET_PER_DEVICE_ROOTFS),.$(ROOTFS_ID/$(DEVICE_NAME))).cpio$(strip $(call Build/initrd_compression)))) \ | ||||
| 			-a $(KERNEL_LOADADDR) -e $(if $(KERNEL_ENTRY),$(KERNEL_ENTRY),$(KERNEL_LOADADDR)) \ | ||||
| 			$(if $(DEVICE_FDT_NUM),-n $(DEVICE_FDT_NUM)) \ | ||||
| 			$(if $(DEVICE_DTS_DELIMITER),-l $(DEVICE_DTS_DELIMITER)) \ | ||||
| 			$(if $(DEVICE_DTS_LOADADDR),-s $(DEVICE_DTS_LOADADDR)) \ | ||||
| 			$(if $(DEVICE_DTS_OVERLAY),$(foreach dtso,$(DEVICE_DTS_OVERLAY), -O $(dtso):$(KERNEL_BUILD_DIR)/image-$(dtso).dtbo)) \ | ||||
| 			-c $(if $(DEVICE_DTS_CONFIG),$(DEVICE_DTS_CONFIG),"config-1") \ | ||||
| 			-A $(LINUX_KARCH) -v $(LINUX_VERSION) \ | ||||
| 	}, gen-cpio$(if $(TARGET_PER_DEVICE_ROOTFS),.$(ROOTFS_ID/$(DEVICE_NAME)))) | ||||
| 	$(if $(findstring with-rootfs,$(word 3,$(1))), \ | ||||
| 		$(call locked,dd if=$(IMAGE_ROOTFS) of=$(IMAGE_ROOTFS).pagesync bs=4096 conv=sync, \ | ||||
| 		  gen-cpio$(if $(TARGET_PER_DEVICE_ROOTFS),.$(ROOTFS_ID/$(DEVICE_NAME))))) | ||||
| 	$(TOPDIR)/scripts/mkits.sh \ | ||||
| 		-D $(DEVICE_NAME) -o $@.its -k $@ \ | ||||
| 		-C $(word 1,$(1)) \ | ||||
| 		$(if $(word 2,$(1)),\ | ||||
| 			$(if $(findstring 11,$(if $(DEVICE_DTS_OVERLAY),1)$(if $(findstring $(KERNEL_BUILD_DIR)/image-,$(word 2,$(1))),,1)), \ | ||||
| 				-d $(KERNEL_BUILD_DIR)/image-$$(basename $(word 2,$(1))), \ | ||||
| 				-d $(word 2,$(1)))) \ | ||||
| 		$(if $(findstring with-rootfs,$(word 3,$(1))),-r $(IMAGE_ROOTFS)) \ | ||||
| 		$(if $(findstring with-initrd,$(word 3,$(1))), \ | ||||
| 			$(if $(CONFIG_TARGET_ROOTFS_INITRAMFS_SEPARATE), \ | ||||
| 				-i $(KERNEL_BUILD_DIR)/initrd$(if $(TARGET_PER_DEVICE_ROOTFS),.$(ROOTFS_ID/$(DEVICE_NAME))).cpio$(strip $(call Build/initrd_compression)))) \ | ||||
| 		-a $(KERNEL_LOADADDR) -e $(if $(KERNEL_ENTRY),$(KERNEL_ENTRY),$(KERNEL_LOADADDR)) \ | ||||
| 		$(if $(DEVICE_FDT_NUM),-n $(DEVICE_FDT_NUM)) \ | ||||
| 		$(if $(DEVICE_DTS_DELIMITER),-l $(DEVICE_DTS_DELIMITER)) \ | ||||
| 		$(if $(DEVICE_DTS_LOADADDR),-s $(DEVICE_DTS_LOADADDR)) \ | ||||
| 		$(if $(DEVICE_DTS_OVERLAY),$(foreach dtso,$(DEVICE_DTS_OVERLAY), -O $(dtso):$(KERNEL_BUILD_DIR)/image-$(dtso).dtbo)) \ | ||||
| 		-c $(if $(DEVICE_DTS_CONFIG),$(DEVICE_DTS_CONFIG),"config-1") \ | ||||
| 		-A $(LINUX_KARCH) -v $(LINUX_VERSION) | ||||
| 	$(call locked,PATH=$(LINUX_DIR)/scripts/dtc:$(PATH) mkimage $(if $(findstring external,$(word 3,$(1))),\ | ||||
| 		-E -B 0x1000 $(if $(findstring static,$(word 3,$(1))),-p 0x1000)) -f $@.its $@.new, \ | ||||
| 	  gen-cpio$(if $(TARGET_PER_DEVICE_ROOTFS),.$(ROOTFS_ID/$(DEVICE_NAME)))) | ||||
|  | ||||
		Reference in New Issue
	
	Block a user
	 Christian Marangi
					Christian Marangi