mbox series

[v2,0/2] qmp: Expose MachineClass::default_ram_id

Message ID cover.1590481402.git.mprivozn@redhat.com
Headers show
Series qmp: Expose MachineClass::default_ram_id | expand

Message

Michal Prívozník May 26, 2020, 8:25 a.m. UTC
v2 of:

https://lists.nongnu.org/archive/html/qemu-devel/2020-05/msg07103.html

diff to v1:
- in 2/2 I made the default-ram-id optional, because as it turns out,
    not every machine type has it set.

Michal Privoznik (2):
  qapi: Fix comment format for @CpuInstanceProperties
  qmp: Expose MachineClass::default_ram_id

 hw/core/machine-qmp-cmds.c | 4 ++++
 qapi/machine.json          | 8 ++++++--
 2 files changed, 10 insertions(+), 2 deletions(-)

Comments

Eduardo Habkost July 27, 2020, 6:41 p.m. UTC | #1
Hi Michal,

It looks like this has fallen through the cracks, my apologies.

I'm queueing this for 5.2.  I assume this is the latest version,
correct?


On Tue, May 26, 2020 at 10:25:33AM +0200, Michal Privoznik wrote:
> v2 of:
> 
> https://lists.nongnu.org/archive/html/qemu-devel/2020-05/msg07103.html
> 
> diff to v1:
> - in 2/2 I made the default-ram-id optional, because as it turns out,
>     not every machine type has it set.
> 
> Michal Privoznik (2):
>   qapi: Fix comment format for @CpuInstanceProperties
>   qmp: Expose MachineClass::default_ram_id
> 
>  hw/core/machine-qmp-cmds.c | 4 ++++
>  qapi/machine.json          | 8 ++++++--
>  2 files changed, 10 insertions(+), 2 deletions(-)
> 
> -- 
> 2.26.2
>
Michal Prívozník July 28, 2020, 8:05 a.m. UTC | #2
On 7/27/20 8:41 PM, Eduardo Habkost wrote:
> Hi Michal,
> 
> It looks like this has fallen through the cracks, my apologies.
> 
> I'm queueing this for 5.2.  I assume this is the latest version,
> correct?
> 

Yes. No worries, I forgot about it too :-) If I remembered, I would have 
pinged.

Michal