On Mon, May 29, 2017 at 02:33:16PM +0200, Laurent Vivier wrote:
> On 24/05/2017 18:08, 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 700205bc70cb..1411b3885928 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..581552a21f2e 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" ] && export INITRD="-initrd $ENV"
>
> Why do you need the export?
I guess it's not necessary.
Thanks,
drew
>
> Thanks,
> Laurent
