From: "Philippe Mathieu-Daudé" <philippe.mathieu.daude@gmail.com>
To: Damien Hedde <damien.hedde@greensocs.com>,
qemu-devel@nongnu.org, mark.burton@greensocs.com,
edgari@xilinx.com
Cc: "Eduardo Habkost" <eduardo@habkost.net>,
"Yanan Wang" <wangyanan55@huawei.com>,
"Philippe Mathieu-Daudé" <f4bug@amsat.org>
Subject: Re: [PATCH v4 09/14] none-machine: allow cold plugging sysbus devices
Date: Thu, 3 Mar 2022 15:44:49 +0100 [thread overview]
Message-ID: <abd92319-d751-3b62-a73b-674e12aab66c@gmail.com> (raw)
In-Reply-To: <20220223090706.4888-10-damien.hedde@greensocs.com>
On 23/2/22 10:07, Damien Hedde wrote:
> Allow plugging any sysbus device on this machine (the sysbus
> devices still need to be 'user-creatable').
>
> This commit is needed to use the 'none' machine as a base, and
> subsequently to dynamically populate it with sysbus devices using
> qapi commands.
>
> Note that this only concern cold-plug: sysbus devices cann't be hot
"can not" is easier to understand for non-native / not good level of
English speakers IMHO.
> plugged because the sysbus bus does not support it.
>
> Signed-off-by: Damien Hedde <damien.hedde@greensocs.com>
> ---
> hw/core/null-machine.c | 4 ++++
> 1 file changed, 4 insertions(+)
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
next prev parent reply other threads:[~2022-03-03 14:47 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-23 9:06 [PATCH v4 00/14] Initial support for machine creation via QMP Damien Hedde
2022-02-23 9:06 ` [PATCH v4 01/14] machine: add phase_get() and document phase_check()/advance() Damien Hedde
2022-03-03 15:01 ` Philippe Mathieu-Daudé
2022-02-23 9:06 ` [PATCH v4 02/14] machine&vl: introduce phase_until() to handle phase transitions Damien Hedde
2022-03-18 13:29 ` Damien Hedde
2022-02-23 9:06 ` [PATCH v4 03/14] vl: support machine-initialized target in phase_until() Damien Hedde
2022-03-03 15:03 ` Philippe Mathieu-Daudé
2022-02-23 9:06 ` [PATCH v4 04/14] qapi/device_add: compute is_hotplug flag Damien Hedde
2022-03-03 15:04 ` Philippe Mathieu-Daudé
2022-02-23 9:06 ` [PATCH v4 05/14] qapi/device_add: handle the rom_order_override when cold-plugging Damien Hedde
2022-05-24 20:08 ` Jim Shu
2022-02-23 9:06 ` [PATCH v4 06/14] qapi/device_add: Allow execution in machine initialized phase Damien Hedde
2022-02-23 9:06 ` [PATCH v4 07/14] none-machine: add the NoneMachineState structure Damien Hedde
2022-03-03 14:36 ` Philippe Mathieu-Daudé
2022-05-24 20:09 ` Jim Shu
2022-02-23 9:07 ` [PATCH v4 08/14] none-machine: add 'ram-addr' property Damien Hedde
2022-03-03 14:41 ` Philippe Mathieu-Daudé
2022-03-03 16:19 ` Damien Hedde
2022-05-24 20:09 ` Jim Shu
2022-02-23 9:07 ` [PATCH v4 09/14] none-machine: allow cold plugging sysbus devices Damien Hedde
2022-03-03 14:44 ` Philippe Mathieu-Daudé [this message]
2022-05-24 20:09 ` Jim Shu
2022-02-23 9:07 ` [PATCH v4 10/14] none-machine: allow several cpus Damien Hedde
2022-02-23 9:07 ` [PATCH v4 11/14] softmmu/memory: add memory_region_try_add_subregion function Damien Hedde
2022-02-23 9:12 ` Damien Hedde
2022-03-03 13:32 ` Philippe Mathieu-Daudé
2022-03-04 10:53 ` Damien Hedde
2022-05-24 20:09 ` Jim Shu
2022-02-23 9:07 ` [PATCH v4 12/14] add sysbus-mmio-map qapi command Damien Hedde
2022-03-03 14:59 ` Philippe Mathieu-Daudé
2022-03-04 10:42 ` Damien Hedde
2022-05-24 20:09 ` Jim Shu
2022-02-23 9:07 ` [PATCH v4 13/14] hw/mem/system-memory: add a memory sysbus device Damien Hedde
2022-02-23 9:44 ` Igor Mammedov
2022-02-23 10:19 ` Damien Hedde
2022-02-24 9:55 ` Igor Mammedov
2022-02-24 11:43 ` Damien Hedde
2022-02-25 11:38 ` Igor Mammedov
2022-02-25 15:31 ` Damien Hedde
2022-03-03 15:16 ` Philippe Mathieu-Daudé
2022-05-24 20:10 ` Jim Shu
2022-02-23 9:07 ` [PATCH v4 14/14] hw: set user_creatable on opentitan/sifive_e devices Damien Hedde
2022-02-23 9:07 ` Damien Hedde
2022-03-04 12:58 ` Philippe Mathieu-Daudé
2022-03-04 12:58 ` Philippe Mathieu-Daudé
2022-05-24 20:10 ` Jim Shu
2022-03-03 10:58 ` [PATCH v4 00/14] Initial support for machine creation via QMP Damien Hedde
2022-05-24 19:54 ` Jim Shu
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=abd92319-d751-3b62-a73b-674e12aab66c@gmail.com \
--to=philippe.mathieu.daude@gmail.com \
--cc=damien.hedde@greensocs.com \
--cc=edgari@xilinx.com \
--cc=eduardo@habkost.net \
--cc=f4bug@amsat.org \
--cc=mark.burton@greensocs.com \
--cc=qemu-devel@nongnu.org \
--cc=wangyanan55@huawei.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.