On 06/06/2017 20:22, Andrew Jones wrote:
> Encapsulate the preparation of the optional initrd command line
> option.
>
> Signed-off-by: Andrew Jones <drjones@xxxxxxxxxx>
> ---
> arm/run | 5 +++--
> powerpc/run | 5 +++--
> scripts/arch-run.bash | 5 +++++
> x86/run | 5 +++--
> 4 files changed, 14 insertions(+), 6 deletions(-)
>
> diff --git a/arm/run b/arm/run
> index f73593bc01af..1b1602c74970 100755
> --- a/arm/run
> +++ b/arm/run
> @@ -71,10 +71,11 @@ if $qemu $M -device '?' 2>&1 | grep pci-testdev > /dev/null; then
> pci_testdev="-device pci-testdev"
> fi
>
> +initrd_create
> +
> M+=",accel=$ACCEL"
> command="$qemu -nodefaults $M -cpu $processor $chr_testdev $pci_testdev"
> -[ -f "$ENV" ] && command+=" -initrd $ENV"
> -command+=" -display none -serial stdio -kernel"
> +command+=" -display none -serial stdio $INITRD -kernel"
> command="$(timeout_cmd) $command"
> echo $command "$@"
>
> diff --git a/powerpc/run b/powerpc/run
> index d73f721680b5..7ccf1a36ff49 100755
> --- a/powerpc/run
> +++ b/powerpc/run
> @@ -35,11 +35,12 @@ if ! $qemu -machine '?' 2>&1 | grep 'pseries' > /dev/null; then
> exit 2
> fi
>
> +initrd_create
> +
> M='-machine pseries'
> M+=",accel=$ACCEL"
> command="$qemu -nodefaults $M -bios $FIRMWARE"
> -[ -f "$ENV" ] && command+=" -initrd $ENV"
> -command+=" -display none -serial stdio -kernel"
> +command+=" -display none -serial stdio $INITRD -kernel"
> command="$(migration_cmd) $(timeout_cmd) $command"
> echo $command "$@"
>
> diff --git a/scripts/arch-run.bash b/scripts/arch-run.bash
> index 5c10828e30f9..fc95e263458f 100644
> --- a/scripts/arch-run.bash
> +++ b/scripts/arch-run.bash
> @@ -155,3 +155,8 @@ search_qemu_binary ()
> command -v $qemu
> export PATH=$save_path
> }
> +
> +initrd_create ()
> +{
> + [ -f "$ENV" ] && INITRD="-initrd $ENV"
> +}
Is this intentional to not clear INITRD and thus to allow it to be
imported from the environment?
Laurent
