[PULL,v2,0/4] machine development tool

Pull-request

https://gitlab.com/davydov-max/qemu.git tags/pull-compare-mt-2024-03-19

Message

Maksim Davydov March 18, 2024, 9:35 p.m. UTC

The following changes since commit ba49d760eb04630e7b15f423ebecf6c871b8f77b:

  Merge tag 'pull-maintainer-final-130324-1' of https://gitlab.com/stsquad/qemu into staging (2024-03-13 15:12:14 +0000)

are available in the Git repository at:

  https://gitlab.com/davydov-max/qemu.git tags/pull-compare-mt-2024-03-19

for you to fetch changes up to e12b89f71ba5b93877b6a3adc379a6369d0c9bab:

  scripts: add script to compare compatibility properties (2024-03-19 00:13:54 +0300)

----------------------------------------------------------------
Please note. This is the first pull request from me.
My public GPG key is available here
https://keys.openpgp.org/vks/v1/by-fingerprint/CDB5BEEF8837142579F5CDFE8E927E10F72F78D4

----------------------------------------------------------------
scripts: add a new script for machine development

----------------------------------------------------------------

Maksim Davydov (4):
  qom: add default value
  qmp: add dump machine type compatibility properties
  python/qemu/machine: add method to retrieve QEMUMachine::binary field
  scripts: add script to compare compatibility properties

 MAINTAINERS                      |   5 +
 hw/core/machine-qmp-cmds.c       |  23 +-
 python/qemu/machine/machine.py   |   5 +
 qapi/machine.json                |  67 ++++-
 qom/qom-qmp-cmds.c               |   1 +
 scripts/compare-machine-types.py | 486 +++++++++++++++++++++++++++++++
 tests/qtest/fuzz/qos_fuzz.c      |   2 +-
 7 files changed, 585 insertions(+), 4 deletions(-)
 create mode 100755 scripts/compare-machine-types.py

Comments

Peter Maydell March 19, 2024, 10:24 a.m. UTC | #1

On Mon, 18 Mar 2024 at 21:36, Maksim Davydov <[e-mail]> wrote:
>
> The following changes since commit ba49d760eb04630e7b15f423ebecf6c871b8f77b:
>
>   Merge tag 'pull-maintainer-final-130324-1' of https://gitlab.com/stsquad/qemu into staging (2024-03-13 15:12:14 +0000)
>
> are available in the Git repository at:
>
>   https://gitlab.com/davydov-max/qemu.git tags/pull-compare-mt-2024-03-19
>
> for you to fetch changes up to e12b89f71ba5b93877b6a3adc379a6369d0c9bab:
>
>   scripts: add script to compare compatibility properties (2024-03-19 00:13:54 +0300)
>
> ----------------------------------------------------------------
> Please note. This is the first pull request from me.
> My public GPG key is available here
> https://keys.openpgp.org/vks/v1/by-fingerprint/CDB5BEEF8837142579F5CDFE8E927E10F72F78D4
>
> ----------------------------------------------------------------
> scripts: add a new script for machine development
>

I said I wanted this to go via some other sub-maintainer's tree.
Please don't keep submitting direct pull requests for it.
QEMU's development flow does not generally involve people
directly submitting pull requests for patches they have written.

thanks
-- PMM

Philippe Mathieu-Daudé March 19, 2024, 2:36 p.m. UTC | #2

Hi Maksim,

On 19/3/24 11:24, Peter Maydell wrote:
> On Mon, 18 Mar 2024 at 21:36, Maksim Davydov <[e-mail]> wrote:
>>
>> The following changes since commit ba49d760eb04630e7b15f423ebecf6c871b8f77b:
>>
>>    Merge tag 'pull-maintainer-final-130324-1' of https://gitlab.com/stsquad/qemu into staging (2024-03-13 15:12:14 +0000)
>>
>> are available in the Git repository at:
>>
>>    https://gitlab.com/davydov-max/qemu.git tags/pull-compare-mt-2024-03-19
>>
>> for you to fetch changes up to e12b89f71ba5b93877b6a3adc379a6369d0c9bab:
>>
>>    scripts: add script to compare compatibility properties (2024-03-19 00:13:54 +0300)
>>
>> ----------------------------------------------------------------
>> Please note. This is the first pull request from me.
>> My public GPG key is available here
>> https://keys.openpgp.org/vks/v1/by-fingerprint/CDB5BEEF8837142579F5CDFE8E927E10F72F78D4
>>
>> ----------------------------------------------------------------
>> scripts: add a new script for machine development
>>
> 
> I said I wanted this to go via some other sub-maintainer's tree.

I'm OK to take the first 3 machine-core patches, but the
tree is currently frozen to stabilize before the next release.
Since your patches add a new feature, I'll merge them once the
tree re-opens in ~1 month, see the announced schedule:
https://lore.kernel.org/qemu-devel/CAFEAcA_ncrw8nsD50Ov=Fse=[e-mail]/

I haven't looked at the last patch yet, at a glance I'm not objecting
to it, but having an Ack-by tag by John (Python maintainer) would be nice.

Regards,

Phil.