diff mbox series

hw/arm: fix min_cpus for xlnx-versal-virt platform

Message ID 160343854912.8460.17915238517799132371.stgit@pasha-ThinkPad-X280
State New
Headers show
Series hw/arm: fix min_cpus for xlnx-versal-virt platform | expand

Commit Message

Pavel Dovgalyuk Oct. 23, 2020, 7:35 a.m. UTC
This patch sets min_cpus field for xlnx-versal-virt platform,
because it always creates XLNX_VERSAL_NR_ACPUS cpus even with
-smp 1 command line option.

Signed-off-by: Pavel Dovgalyuk <pavel.dovgalyuk@ispras.ru>
---
 hw/arm/xlnx-versal-virt.c |    1 +
 1 file changed, 1 insertion(+)

Comments

Philippe Mathieu-Daudé Oct. 23, 2020, 8:08 a.m. UTC | #1
On 10/23/20 9:35 AM, Pavel Dovgalyuk wrote:
> This patch sets min_cpus field for xlnx-versal-virt platform,
> because it always creates XLNX_VERSAL_NR_ACPUS cpus even with
> -smp 1 command line option.
> 
> Signed-off-by: Pavel Dovgalyuk <pavel.dovgalyuk@ispras.ru>
> ---
>   hw/arm/xlnx-versal-virt.c |    1 +
>   1 file changed, 1 insertion(+)

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Edgar E. Iglesias Oct. 23, 2020, 9:20 a.m. UTC | #2
On Fri, Oct 23, 2020 at 10:35:49AM +0300, Pavel Dovgalyuk wrote:
> This patch sets min_cpus field for xlnx-versal-virt platform,
> because it always creates XLNX_VERSAL_NR_ACPUS cpus even with
> -smp 1 command line option.

Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>



> 
> Signed-off-by: Pavel Dovgalyuk <pavel.dovgalyuk@ispras.ru>
> ---
>  hw/arm/xlnx-versal-virt.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/hw/arm/xlnx-versal-virt.c b/hw/arm/xlnx-versal-virt.c
> index 03e23201b1..ee1282241e 100644
> --- a/hw/arm/xlnx-versal-virt.c
> +++ b/hw/arm/xlnx-versal-virt.c
> @@ -561,6 +561,7 @@ static void versal_virt_machine_class_init(ObjectClass *oc, void *data)
>  
>      mc->desc = "Xilinx Versal Virtual development board";
>      mc->init = versal_virt_init;
> +    mc->min_cpus = XLNX_VERSAL_NR_ACPUS;
>      mc->max_cpus = XLNX_VERSAL_NR_ACPUS;
>      mc->default_cpus = XLNX_VERSAL_NR_ACPUS;
>      mc->no_cdrom = true;
>
Peter Maydell Oct. 26, 2020, 1:23 p.m. UTC | #3
On Fri, 23 Oct 2020 at 08:35, Pavel Dovgalyuk <pavel.dovgalyuk@ispras.ru> wrote:
>
> This patch sets min_cpus field for xlnx-versal-virt platform,
> because it always creates XLNX_VERSAL_NR_ACPUS cpus even with
> -smp 1 command line option.
>
> Signed-off-by: Pavel Dovgalyuk <pavel.dovgalyuk@ispras.ru>
> ---
>  hw/arm/xlnx-versal-virt.c |    1 +
>  1 file changed, 1 insertion(+)
>



Applied to target-arm.next, thanks.

-- PMM
diff mbox series

Patch

diff --git a/hw/arm/xlnx-versal-virt.c b/hw/arm/xlnx-versal-virt.c
index 03e23201b1..ee1282241e 100644
--- a/hw/arm/xlnx-versal-virt.c
+++ b/hw/arm/xlnx-versal-virt.c
@@ -561,6 +561,7 @@  static void versal_virt_machine_class_init(ObjectClass *oc, void *data)
 
     mc->desc = "Xilinx Versal Virtual development board";
     mc->init = versal_virt_init;
+    mc->min_cpus = XLNX_VERSAL_NR_ACPUS;
     mc->max_cpus = XLNX_VERSAL_NR_ACPUS;
     mc->default_cpus = XLNX_VERSAL_NR_ACPUS;
     mc->no_cdrom = true;