Re: [PATCH v2 3/3] target/arm: Restrict v7A TCG cpus to TCG accel

2021-03-18 Thread Philippe Mathieu-Daudé
On 3/18/21 1:50 PM, Claudio Fontana wrote: > On 3/18/21 1:37 PM, Andrew Jones wrote: >> On Thu, Mar 18, 2021 at 11:38:51AM +, Peter Maydell wrote: >>> On Thu, 18 Mar 2021 at 11:31, Philippe Mathieu-Daudé >>> wrote: I tend to agree. The problem is for the running VMs started before 8

Re: [PATCH v2 3/3] target/arm: Restrict v7A TCG cpus to TCG accel

2021-03-18 Thread Claudio Fontana
On 3/18/21 1:37 PM, Andrew Jones wrote: > On Thu, Mar 18, 2021 at 11:38:51AM +, Peter Maydell wrote: >> On Thu, 18 Mar 2021 at 11:31, Philippe Mathieu-Daudé >> wrote: >>> I tend to agree. The problem is for the running VMs started before >>> 82bf7ae84ce (so up to any fork based on v5.2). I do

Re: [PATCH v2 3/3] target/arm: Restrict v7A TCG cpus to TCG accel

2021-03-18 Thread Andrew Jones
On Thu, Mar 18, 2021 at 11:38:51AM +, Peter Maydell wrote: > On Thu, 18 Mar 2021 at 11:31, Philippe Mathieu-Daudé > wrote: > > I tend to agree. The problem is for the running VMs started before > > 82bf7ae84ce (so up to any fork based on v5.2). I don't know what > > the forks are supposed to

Re: [PATCH v2 3/3] target/arm: Restrict v7A TCG cpus to TCG accel

2021-03-18 Thread Peter Maydell
On Thu, 18 Mar 2021 at 11:31, Philippe Mathieu-Daudé wrote: > I tend to agree. The problem is for the running VMs started before > 82bf7ae84ce (so up to any fork based on v5.2). I don't know what > the forks are supposed to do with the running VMs if they want to > migrate them to newer QEMU (or u

Re: [PATCH v2 3/3] target/arm: Restrict v7A TCG cpus to TCG accel

2021-03-18 Thread Philippe Mathieu-Daudé
On 3/18/21 12:21 PM, Peter Maydell wrote: > On Thu, 18 Mar 2021 at 11:09, Philippe Mathieu-Daudé > wrote: >> Still, I'm not sure it makes sense. If you want to migrate a such >> machine, KVM can not virtualize it, so you'll be forced to use TCG >> right? In that case cpu_tcg is built in and you h

Re: [PATCH v2 3/3] target/arm: Restrict v7A TCG cpus to TCG accel

2021-03-18 Thread Peter Maydell
On Thu, 18 Mar 2021 at 11:09, Philippe Mathieu-Daudé wrote: > Still, I'm not sure it makes sense. If you want to migrate a such > machine, KVM can not virtualize it, so you'll be forced to use TCG > right? In that case cpu_tcg is built in and you have the A15. > > IOW the problem is not this patch

Re: [PATCH v2 3/3] target/arm: Restrict v7A TCG cpus to TCG accel

2021-03-18 Thread Philippe Mathieu-Daudé
On 3/18/21 11:47 AM, Philippe Mathieu-Daudé wrote: > On 3/18/21 10:56 AM, Claudio Fontana wrote: >> On 3/18/21 10:47 AM, Philippe Mathieu-Daudé wrote: >>> Hi Peter, >>> >>> On 3/11/21 11:43 AM, Claudio Fontana wrote: On 2/21/21 11:26 PM, Philippe Mathieu-Daudé wrote: > KVM requires the tar

Re: [PATCH v2 3/3] target/arm: Restrict v7A TCG cpus to TCG accel

2021-03-18 Thread Philippe Mathieu-Daudé
On 3/18/21 10:56 AM, Claudio Fontana wrote: > On 3/18/21 10:47 AM, Philippe Mathieu-Daudé wrote: >> Hi Peter, >> >> On 3/11/21 11:43 AM, Claudio Fontana wrote: >>> On 2/21/21 11:26 PM, Philippe Mathieu-Daudé wrote: KVM requires the target cpu to be at least ARMv8 architecture (support on

Re: [PATCH v2 3/3] target/arm: Restrict v7A TCG cpus to TCG accel

2021-03-18 Thread Claudio Fontana
On 3/18/21 10:47 AM, Philippe Mathieu-Daudé wrote: > Hi Peter, > > On 3/11/21 11:43 AM, Claudio Fontana wrote: >> On 2/21/21 11:26 PM, Philippe Mathieu-Daudé wrote: >>> KVM requires the target cpu to be at least ARMv8 architecture >>> (support on ARMv7 has been dropped in commit 82bf7ae84ce: >>> "

Re: [PATCH v2 3/3] target/arm: Restrict v7A TCG cpus to TCG accel

2021-03-18 Thread Philippe Mathieu-Daudé
Hi Peter, On 3/11/21 11:43 AM, Claudio Fontana wrote: > On 2/21/21 11:26 PM, Philippe Mathieu-Daudé wrote: >> KVM requires the target cpu to be at least ARMv8 architecture >> (support on ARMv7 has been dropped in commit 82bf7ae84ce: >> "target/arm: Remove KVM support for 32-bit Arm hosts"). >> >>

Re: [PATCH v2 3/3] target/arm: Restrict v7A TCG cpus to TCG accel

2021-03-11 Thread Claudio Fontana
On 2/21/21 11:26 PM, Philippe Mathieu-Daudé wrote: > KVM requires the target cpu to be at least ARMv8 architecture > (support on ARMv7 has been dropped in commit 82bf7ae84ce: > "target/arm: Remove KVM support for 32-bit Arm hosts"). > > A KVM-only build won't be able to run TCG cpus, move the > v7

[PATCH v2 3/3] target/arm: Restrict v7A TCG cpus to TCG accel

2021-02-21 Thread Philippe Mathieu-Daudé
KVM requires the target cpu to be at least ARMv8 architecture (support on ARMv7 has been dropped in commit 82bf7ae84ce: "target/arm: Remove KVM support for 32-bit Arm hosts"). A KVM-only build won't be able to run TCG cpus, move the v7A CPU definitions to cpu_tcg.c. Reported-by: Peter Maydell Re