From 8f01717f27341e554fa39ed6f026a0d333fb4790 Mon Sep 17 00:00:00 2001 From: Ayan Kumar Halder Date: Mon, 10 Jan 2022 19:39:34 +0000 Subject: Check for the presence of DOMU_PASSTHROUGH_DTB before invoking update_domU_cfg It should be possible to run disk_image when the user has specified neither DOMU_PASSTHROUGH_PATHS nor DOMU_PASSTHROUGH_DTB. In such a scenario, the script should not update the relevant portions of domU.cfg. Signed-off-by: Ayan Kumar Halder Reviewed-by: Stefano Stabellini Signed-off-by: Stefano Stabellini --- scripts/disk_image | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) (limited to 'scripts') diff --git a/scripts/disk_image b/scripts/disk_image index afd21f9..1efc3af 100755 --- a/scripts/disk_image +++ b/scripts/disk_image @@ -392,13 +392,10 @@ function generate_domU_configs() fi dest="$DESTDIR/part/disk$j/etc/xen/domU$i.cfg" - cp ${DOMU_PASSTHROUGH_DTB[$i]} $DESTDIR/part/disk$j/etc/xen/ echo "name=\"domU$i\"" >> $dest echo "memory=${DOMU_MEM[$i]}" >> $dest echo "vcpus=${DOMU_VCPUS[$i]}" >> $dest - dtb_name="$(basename ${DOMU_PASSTHROUGH_DTB[$i]})" - echo "device_tree=\"/etc/xen/$dtb_name\"" >> $dest echo "# mount $first_part /boot" >> $dest echo "kernel=\"/boot/${DOMU_KERNEL[$i]}\"" >> $dest if test "${DOMU_RAMDISK[$i]}" @@ -426,12 +423,18 @@ function generate_domU_configs() echo "disk=[\"$retval,,xvda\"]" >> $dest fi - update_domU_cfg ${DOMU_PASSTHROUGH_DTB[$i]} $dest - if test $? -ne 0 + if test "${DOMU_PASSTHROUGH_DTB[$i]}" then - umount $DESTDIR/part/disk$j - cleanup_disk_image_error - exit 1 + cp ${DOMU_PASSTHROUGH_DTB[$i]} $DESTDIR/part/disk$j/etc/xen/ + dtb_name="$(basename ${DOMU_PASSTHROUGH_DTB[$i]})" + echo "device_tree=\"/etc/xen/$dtb_name\"" >> $dest + update_domU_cfg ${DOMU_PASSTHROUGH_DTB[$i]} $dest + if test $? -ne 0 + then + umount $DESTDIR/part/disk$j + cleanup_disk_image_error + exit 1 + fi fi fi -- cgit v1.2.3