All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chao Liu <chao.liu.zevorn@gmail.com>
To: alistair23@gmail.com
Cc: palmer@dabbelt.com, liwei1518@gmail.com,
	 daniel.barboza@oss.qualcomm.com, zhiwei_liu@linux.alibaba.com,
	qemu-riscv@nongnu.org,  qemu-devel@nongnu.org,
	Alistair Francis <alistair.francis@wdc.com>
Subject: Re: [PATCH 1/2] target/riscv: Remove spike as default machine
Date: Fri, 1 May 2026 21:23:26 +0800	[thread overview]
Message-ID: <afSpPEjo4BLEzirM@ZEVORN-PC.localdomain> (raw)
In-Reply-To: <20260430052218.771358-2-alistair.francis@wdc.com>

On Thu, Apr 30, 2026 at 03:22:17PM +1000, alistair23@gmail.com wrote:
> From: Alistair Francis <alistair.francis@wdc.com>
> 
> In QEMU 10.0 we deprecated the default spike machine, let's finally
> remove spike as the default and require users to specify the machine.
> 
> Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Chao Liu <chao.liu.zevorn@gmail.com>

> ---
>  docs/about/deprecated.rst | 19 -------------------
>  hw/riscv/spike.c          |  1 -
>  2 files changed, 20 deletions(-)
> 
> diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> index 10de10dc23..f3622dcf93 100644
> --- a/docs/about/deprecated.rst
> +++ b/docs/about/deprecated.rst
> @@ -282,25 +282,6 @@ firmware is available to boot it. It can be replaced by the
>  ``ast2700fc``, another multi-SoC machine based on the newer AST2700
>  SoCs which are excepted to receive better support in the future.
>  
> -
> -RISC-V default machine option (since 10.0)
> -''''''''''''''''''''''''''''''''''''''''''
> -
> -RISC-V defines ``spike`` as the default machine if no machine option is
> -given in the command line.  This happens because ``spike`` is the first
> -RISC-V machine implemented in QEMU and setting it as default was
> -convenient at that time.  Now we have 7 riscv64 and 6 riscv32 machines
> -and having ``spike`` as a default is no longer justified.  This default
> -will also promote situations where users think they're running ``virt``
> -(the most used RISC-V machine type in 10.0) when in fact they're
> -running ``spike``.
> -
> -Removing the default machine option forces users to always set the machine
> -they want to use and avoids confusion.  Existing users of the ``spike``
> -machine must ensure that they're setting the ``spike`` machine in the
> -command line (``-M spike``).
> -
> -
>  Backend options
>  ---------------
>  
> diff --git a/hw/riscv/spike.c b/hw/riscv/spike.c
> index 35c696f891..87fe0f242f 100644
> --- a/hw/riscv/spike.c
> +++ b/hw/riscv/spike.c
> @@ -341,7 +341,6 @@ static void spike_machine_class_init(ObjectClass *oc, const void *data)
>      mc->desc = "RISC-V Spike board";
>      mc->init = spike_board_init;
>      mc->max_cpus = SPIKE_CPUS_MAX;
> -    mc->is_default = true;
>      mc->default_cpu_type = TYPE_RISCV_CPU_BASE;
>      mc->possible_cpu_arch_ids = riscv_numa_possible_cpu_arch_ids;
>      mc->cpu_index_to_instance_props = riscv_numa_cpu_index_to_props;
> -- 
> 2.53.0
> 


  parent reply	other threads:[~2026-05-01 13:24 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-30  5:22 [PATCH 0/2] target/riscv: Update deprecated machines alistair23
2026-04-30  5:22 ` [PATCH 1/2] target/riscv: Remove spike as default machine alistair23
2026-04-30  5:26   ` Philippe Mathieu-Daudé
2026-04-30 11:12   ` Daniel Henrique Barboza
2026-05-01 13:23   ` Chao Liu [this message]
2026-05-01 13:28   ` Peter Maydell
2026-05-06 23:36     ` Alistair Francis
2026-04-30  5:22 ` [PATCH 2/2] target/riscv: Deprecate the shakti_c machine alistair23
2026-04-30  5:27   ` Philippe Mathieu-Daudé
2026-04-30  6:23   ` Richard Henderson
2026-04-30 11:14   ` Daniel Henrique Barboza
2026-05-01 13:24   ` Chao Liu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=afSpPEjo4BLEzirM@ZEVORN-PC.localdomain \
    --to=chao.liu.zevorn@gmail.com \
    --cc=alistair.francis@wdc.com \
    --cc=alistair23@gmail.com \
    --cc=daniel.barboza@oss.qualcomm.com \
    --cc=liwei1518@gmail.com \
    --cc=palmer@dabbelt.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-riscv@nongnu.org \
    --cc=zhiwei_liu@linux.alibaba.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.