From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:55546)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <crobinso@redhat.com>) id 1YrYRh-0002kW-3N
	for qemu-devel@nongnu.org; Sun, 10 May 2015 17:07:33 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <crobinso@redhat.com>) id 1YrYRd-0005De-Ph
	for qemu-devel@nongnu.org; Sun, 10 May 2015 17:07:33 -0400
Received: from mx1.redhat.com ([209.132.183.28]:49873)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <crobinso@redhat.com>) id 1YrYRd-0005DU-Ko
	for qemu-devel@nongnu.org; Sun, 10 May 2015 17:07:29 -0400
Message-ID: <554FC88E.70208@redhat.com>
Date: Sun, 10 May 2015 17:07:26 -0400
From: Cole Robinson <crobinso@redhat.com>
MIME-Version: 1.0
References: <00d901d0880d$d795b750$86c125f0$@samsung.com>	<CAFEAcA-XLq1aWyd-8uwWTaUsrHH43zO1jX=t7vi1CyZnOhJRfw@mail.gmail.com>	<012701d08827$4f771250$ee6536f0$@samsung.com>	<CAFEAcA_AQNQXm1QpiBrPZYgdNmtxEeKB4hwnwdF8jerJy5Hp_w@mail.gmail.com>	<00bc01d08890$3883bc10$a98b3430$@samsung.com>	<554B7901.7070607@redhat.com>
	<00b501d0895e$abfe8660$03fb9320$@samsung.com>
In-Reply-To: <00b501d0895e$abfe8660$03fb9320$@samsung.com>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH] Fix default CPU model for ARM64
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Pavel Fedin <p.fedin@samsung.com>, 'Peter Maydell' <peter.maydell@linaro.org>
Cc: 'QEMU Developers' <qemu-devel@nongnu.org>

On 05/08/2015 03:14 AM, Pavel Fedin wrote:
>  Hello!
> 
>> FWIW virt-manager 1.2.0 (just released) will do the following when creating a
>> new VM:
>>
>> - aarch64 + kvm : -cpu host
>> - aarch64 + tcg : -cpu cortex-a57
>> - arm32 + kvm : -cpu host
>> - arm32 + tcg : defer to qemu
>>
>> Though if you explicitly request 'hypervisor default' then we won't specify
>> any -cpu and defer to qemu, which will hit the cortex-a15 default for aarch64
> 
>  virt-manager is not the only tool to create VMs...
>  But, okay. Seems you just don't want to change this. Well... I still don't agree and this
> default looks strange for me, but it's just me. I'm out of further arguments.
> 

I wasn't implying any opinion about your proposed patch, just clarifying the
details about how latest virt-manager performs since you mentioned it
upthread. FWIW I agree with the idea of your patch

- Cole