Re: [PATCH 03/16] target/riscv/cpu.c: restrict 'mvendorid' value

2023-06-13 Thread Alistair Francis
On Tue, Jun 13, 2023 at 4:52 AM Daniel Henrique Barboza wrote: > > > > On 6/12/23 00:56, Alistair Francis wrote: > > On Wed, May 31, 2023 at 5:49 AM Daniel Henrique Barboza > > wrote: > >> > >> We're going to change the handling of mvendorid/marchid/mimpid by the > >> KVM driver. Since these are

Re: [PATCH 03/16] target/riscv/cpu.c: restrict 'mvendorid' value

2023-06-12 Thread Daniel Henrique Barboza
On 6/12/23 00:56, Alistair Francis wrote: On Wed, May 31, 2023 at 5:49 AM Daniel Henrique Barboza wrote: We're going to change the handling of mvendorid/marchid/mimpid by the KVM driver. Since these are always present in all CPUs let's put the same validation for everyone. It doesn't make

Re: [PATCH 03/16] target/riscv/cpu.c: restrict 'mvendorid' value

2023-06-11 Thread Alistair Francis
On Wed, May 31, 2023 at 5:49 AM Daniel Henrique Barboza wrote: > > We're going to change the handling of mvendorid/marchid/mimpid by the > KVM driver. Since these are always present in all CPUs let's put the > same validation for everyone. > > It doesn't make sense to allow 'mvendorid' to be

Re: [PATCH 03/16] target/riscv/cpu.c: restrict 'mvendorid' value

2023-06-06 Thread Daniel Henrique Barboza
On 6/6/23 10:19, Andrew Jones wrote: On Tue, May 30, 2023 at 04:46:10PM -0300, Daniel Henrique Barboza wrote: We're going to change the handling of mvendorid/marchid/mimpid by the KVM driver. Since these are always present in all CPUs let's put the same validation for everyone. It doesn't

Re: [PATCH 03/16] target/riscv/cpu.c: restrict 'mvendorid' value

2023-06-06 Thread Andrew Jones
On Tue, May 30, 2023 at 04:46:10PM -0300, Daniel Henrique Barboza wrote: > We're going to change the handling of mvendorid/marchid/mimpid by the > KVM driver. Since these are always present in all CPUs let's put the > same validation for everyone. > > It doesn't make sense to allow 'mvendorid' to

[PATCH 03/16] target/riscv/cpu.c: restrict 'mvendorid' value

2023-05-30 Thread Daniel Henrique Barboza
We're going to change the handling of mvendorid/marchid/mimpid by the KVM driver. Since these are always present in all CPUs let's put the same validation for everyone. It doesn't make sense to allow 'mvendorid' to be different than it is already set in named (vendor) CPUs. Generic (dynamic) CPUs