Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-24 Thread Igor Mammedov
On Mon, 24 Jul 2023 16:14:22 +0100 Peter Maydell wrote: > On Mon, 24 Jul 2023 at 16:06, Igor Mammedov wrote: > > I've seen others asking why you print type name instead of shorter cpu-model > > used on CLI. To do that would make you write a patch to implement reverse > > mapping. > > In some ca

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-24 Thread Peter Maydell
On Mon, 24 Jul 2023 at 16:06, Igor Mammedov wrote: > I've seen others asking why you print type name instead of shorter cpu-model > used on CLI. To do that would make you write a patch to implement reverse > mapping. > In some cases it's simple, in others plain impossible unless you can get > acc

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-24 Thread Igor Mammedov
On Tue, 18 Jul 2023 20:31:39 +1000 Gavin Shan wrote: > Hi Igor, > > On 7/17/23 22:44, Igor Mammedov wrote: > > On Fri, 14 Jul 2023 13:56:00 +0100 > > Peter Maydell wrote: > > > >> On Fri, 14 Jul 2023 at 12:50, Igor Mammedov wrote: > >>> > >>> On Thu, 13 Jul 2023 12:59:55 +0100 > >>> Peter

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-18 Thread Gavin Shan
Hi Igor, On 7/17/23 22:44, Igor Mammedov wrote: On Fri, 14 Jul 2023 13:56:00 +0100 Peter Maydell wrote: On Fri, 14 Jul 2023 at 12:50, Igor Mammedov wrote: On Thu, 13 Jul 2023 12:59:55 +0100 Peter Maydell wrote: On Thu, 13 Jul 2023 at 12:52, Marcin Juszkiewicz wrote: W dniu 13.07.20

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-17 Thread Igor Mammedov
On Fri, 14 Jul 2023 13:56:00 +0100 Peter Maydell wrote: > On Fri, 14 Jul 2023 at 12:50, Igor Mammedov wrote: > > > > On Thu, 13 Jul 2023 12:59:55 +0100 > > Peter Maydell wrote: > > > > > On Thu, 13 Jul 2023 at 12:52, Marcin Juszkiewicz > > > wrote: > > > > > > > > W dniu 13.07.2023 o 13:44

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-14 Thread Peter Maydell
On Fri, 14 Jul 2023 at 12:50, Igor Mammedov wrote: > > On Thu, 13 Jul 2023 12:59:55 +0100 > Peter Maydell wrote: > > > On Thu, 13 Jul 2023 at 12:52, Marcin Juszkiewicz > > wrote: > > > > > > W dniu 13.07.2023 o 13:44, Peter Maydell pisze: > > > > > > > I see this isn't a change in this patch, bu

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-14 Thread Igor Mammedov
On Thu, 13 Jul 2023 12:59:55 +0100 Peter Maydell wrote: > On Thu, 13 Jul 2023 at 12:52, Marcin Juszkiewicz > wrote: > > > > W dniu 13.07.2023 o 13:44, Peter Maydell pisze: > > > > > I see this isn't a change in this patch, but given that > > > what the user specifies is not "cortex-a8-arm-cpu"

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-14 Thread Gavin Shan
On 7/14/23 10:51, Gavin Shan wrote: On 7/14/23 02:29, Philippe Mathieu-Daudé wrote: On 13/7/23 14:34, Gavin Shan wrote: On 7/13/23 21:52, Marcin Juszkiewicz wrote: W dniu 13.07.2023 o 13:44, Peter Maydell pisze: I see this isn't a change in this patch, but given that what the user specifies

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-13 Thread Gavin Shan
Hi Richard, On 7/14/23 05:27, Richard Henderson wrote: On 7/13/23 13:34, Gavin Shan wrote: On 7/13/23 21:52, Marcin Juszkiewicz wrote: W dniu 13.07.2023 o 13:44, Peter Maydell pisze: I see this isn't a change in this patch, but given that what the user specifies is not "cortex-a8-arm-cpu" bu

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-13 Thread Gavin Shan
Hi Philippe, On 7/14/23 02:29, Philippe Mathieu-Daudé wrote: On 13/7/23 14:34, Gavin Shan wrote: On 7/13/23 21:52, Marcin Juszkiewicz wrote: W dniu 13.07.2023 o 13:44, Peter Maydell pisze: I see this isn't a change in this patch, but given that what the user specifies is not "cortex-a8-arm-c

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-13 Thread Richard Henderson
On 7/13/23 13:34, Gavin Shan wrote: Hi Peter and Marcin, On 7/13/23 21:52, Marcin Juszkiewicz wrote: W dniu 13.07.2023 o 13:44, Peter Maydell pisze: I see this isn't a change in this patch, but given that what the user specifies is not "cortex-a8-arm-cpu" but "cortex-a8", why do we include th

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-13 Thread Philippe Mathieu-Daudé
On 13/7/23 14:34, Gavin Shan wrote: Hi Peter and Marcin, On 7/13/23 21:52, Marcin Juszkiewicz wrote: W dniu 13.07.2023 o 13:44, Peter Maydell pisze: I see this isn't a change in this patch, but given that what the user specifies is not "cortex-a8-arm-cpu" but "cortex-a8", why do we include th

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-13 Thread Gavin Shan
Hi Marcin, On 7/13/23 22:44, Marcin Juszkiewicz wrote: W dniu 13.07.2023 o 14:34, Gavin Shan pisze: On 7/13/23 21:52, Marcin Juszkiewicz wrote: W dniu 13.07.2023 o 13:44, Peter Maydell pisze: I see this isn't a change in this patch, but given that what the user specifies is not "cortex-a8-a

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-13 Thread Marcin Juszkiewicz
W dniu 13.07.2023 o 14:34, Gavin Shan pisze: On 7/13/23 21:52, Marcin Juszkiewicz wrote: W dniu 13.07.2023 o 13:44, Peter Maydell pisze: I see this isn't a change in this patch, but given that what the user specifies is not "cortex-a8-arm-cpu" but "cortex-a8", why do we include the "-arm-cpu"

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-13 Thread Gavin Shan
Hi Peter, On 7/13/23 21:44, Peter Maydell wrote: On Thu, 13 Jul 2023 at 06:45, Gavin Shan wrote: There is a generic CPU type invalidation in machine_run_board_init() and we needn't a same and private invalidation for hw/arm/virt machines. This series intends to use the generic CPU type invali

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-13 Thread Gavin Shan
Hi Peter and Marcin, On 7/13/23 21:52, Marcin Juszkiewicz wrote: W dniu 13.07.2023 o 13:44, Peter Maydell pisze: I see this isn't a change in this patch, but given that what the user specifies is not "cortex-a8-arm-cpu" but "cortex-a8", why do we include the "-arm-cpu" suffix in the error mess

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-13 Thread Peter Maydell
On Thu, 13 Jul 2023 at 12:52, Marcin Juszkiewicz wrote: > > W dniu 13.07.2023 o 13:44, Peter Maydell pisze: > > > I see this isn't a change in this patch, but given that > > what the user specifies is not "cortex-a8-arm-cpu" but > > "cortex-a8", why do we include the "-arm-cpu" suffix in > > the e

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-13 Thread Marcin Juszkiewicz
W dniu 13.07.2023 o 13:44, Peter Maydell pisze: I see this isn't a change in this patch, but given that what the user specifies is not "cortex-a8-arm-cpu" but "cortex-a8", why do we include the "-arm-cpu" suffix in the error messages? It's not valid syntax to say "-cpu cortex-a8-arm-cpu", so it'

Re: [PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-13 Thread Peter Maydell
On Thu, 13 Jul 2023 at 06:45, Gavin Shan wrote: > > There is a generic CPU type invalidation in machine_run_board_init() > and we needn't a same and private invalidation for hw/arm/virt machines. > This series intends to use the generic CPU type invalidation on the > hw/arm/virt machines. > > PATC

[PATCH 0/3] hw/arm/virt: Use generic CPU invalidation

2023-07-12 Thread Gavin Shan
There is a generic CPU type invalidation in machine_run_board_init() and we needn't a same and private invalidation for hw/arm/virt machines. This series intends to use the generic CPU type invalidation on the hw/arm/virt machines. PATCH[1] factors the CPU type invalidation logic in machine_run_bo