Re: [kvm-unit-tests PATCH v2 2/3] arm/run: ensure we use processor=host for kvm

From: Andrew Jones <drjones@redhat.com>
To: "Alex Bennée" <alex.bennee@linaro.org>
Cc: pbonzini@redhat.com, marc.zyngier@arm.com,
	christoffer.dall@linaro.org, kvm@vger.kernel.org
Subject: Re: [kvm-unit-tests PATCH v2 2/3] arm/run: ensure we use processor=host for kvm
Date: Mon, 22 May 2017 19:25:28 +0200	[thread overview]
Message-ID: <20170522172528.aptg6b6ob4r5ntp3@kamzik.brq.redhat.com> (raw)
In-Reply-To: <20170518160208.9674-3-alex.bennee@linaro.org>

On Thu, May 18, 2017 at 05:02:07PM +0100, Alex Bennée wrote:
> The configure script will happily default to cortex-a15 even when
> running on a cortex-a7. This runs into problems when running KVM on
> common dev boards as we should be using the "host" processor.
> 
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
>  arm/run | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arm/run b/arm/run
> index 700205b..f73593b 100755
> --- a/arm/run
> +++ b/arm/run
> @@ -44,9 +44,9 @@ if [ "$ACCEL" = "kvm" ]; then
>  	if $qemu $M,\? 2>&1 | grep gic-version > /dev/null; then
>  		M+=',gic-version=host'
>  	fi
> -	if [ "$HOST" = "aarch64" ]; then
> +	if [ "$HOST" = "aarch64" ] || [ "$HOST" = "arm" ]; then
>  		processor="host"
> -		if [ "$ARCH" = "arm" ]; then
> +		if [ "$ARCH" = "arm" ] && [ "$HOST" = "aarch64" ]; then
>  			processor+=",aarch64=off"
>  		fi
>  	fi
> -- 
> 2.13.0
>
 
Reviewed-by: Andrew Jones <drjones@redhat.com>