[Qemu-commits] [qemu/qemu] c3e955: monitor: Move monitor option parsing to monitor/mo...

[Peter Maydell]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: c3e9555182edc8766d67d2f4471774e2aac0500a
      
https://github.com/qemu/qemu/commit/c3e9555182edc8766d67d2f4471774e2aac0500a
  Author: Kevin Wolf <address@hidden>
  Date:   2020-02-17 (Mon, 17 Feb 2020)

  Changed paths:
    M include/monitor/monitor.h
    M include/sysemu/sysemu.h
    M monitor/monitor.c
    M vl.c

  Log Message:
  -----------
  monitor: Move monitor option parsing to monitor/monitor.c

Both the system emulators and tools with QMP support (specifically, the
planned storage daemon) will need to parse monitor options, so move that
code to monitor/monitor.c, which can be linked into binaries that aren't
a system emulator.

Signed-off-by: Kevin Wolf <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>


  Commit: fa4dcf577ea61758bf55b64c8f9590c29897fb82
      
https://github.com/qemu/qemu/commit/fa4dcf577ea61758bf55b64c8f9590c29897fb82
  Author: Kevin Wolf <address@hidden>
  Date:   2020-02-17 (Mon, 17 Feb 2020)

  Changed paths:
    M monitor/hmp-cmds.c
    M monitor/misc.c
    M monitor/monitor-internal.h
    M monitor/qmp-cmds.c
    M monitor/qmp.c
    M qapi/Makefile.objs
    A qapi/control.json
    M qapi/misc.json
    M qapi/qapi-schema.json
    M tests/qtest/qmp-test.c
    M ui/gtk.c

  Log Message:
  -----------
  qapi: Split control.json off misc.json

misc.json contains definitions that are related to the system emulator,
so it can't be used for other tools like the storage daemon. This patch
moves basic functionality that is shared between all tools (and mostly
related to the monitor itself) into a new control.json, which could be
used in tools as well.

Signed-off-by: Kevin Wolf <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
Message-Id: <address@hidden>
[Commit message tweaked]
Signed-off-by: Markus Armbruster <address@hidden>


  Commit: 567628163ec8c6b76ba3117c3062edc0521dc185
      
https://github.com/qemu/qemu/commit/567628163ec8c6b76ba3117c3062edc0521dc185
  Author: Kevin Wolf <address@hidden>
  Date:   2020-02-17 (Mon, 17 Feb 2020)

  Changed paths:
    M monitor/Makefile.objs
    M monitor/misc.c
    A monitor/qmp-cmds-control.c
    M monitor/qmp-cmds.c

  Log Message:
  -----------
  monitor: Collect "control" command handlers in qmp-cmds.control.c

Move all of the QMP commands handlers to implement the 'control' module
(qapi/control.json) that can be shared between the system emulator and
tools such as a storage daemon to a new file monitor/qmp-cmds-control.c.

Signed-off-by: Kevin Wolf <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
Message-Id: <address@hidden>
[Commit message tweaked]
Signed-off-by: Markus Armbruster <address@hidden>


  Commit: 7d3f505359214734d46c159af0a88b522f6692ad
      
https://github.com/qemu/qemu/commit/7d3f505359214734d46c159af0a88b522f6692ad
  Author: Kevin Wolf <address@hidden>
  Date:   2020-02-17 (Mon, 17 Feb 2020)

  Changed paths:
    M monitor/misc.c
    M monitor/monitor-internal.h
    M monitor/qmp-cmds-control.c

  Log Message:
  -----------
  monitor: Move qmp_query_qmp_schema to qmp-cmds-control.c

monitor/misc.c contains code that works only in the system emulator, so
it can't be linked to tools like a storage daemon. In order to make
schema introspection available for tools, move the function to
monitor/qmp-cmds-control.c, which can be linked into the storage daemon.

Signed-off-by: Kevin Wolf <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>


  Commit: ec95fcc8bae624e87c7601d2d4baf7c519c3814e
      
https://github.com/qemu/qemu/commit/ec95fcc8bae624e87c7601d2d4baf7c519c3814e
  Author: Markus Armbruster <address@hidden>
  Date:   2020-02-17 (Mon, 17 Feb 2020)

  Changed paths:
    M qemu-doc.texi

  Log Message:
  -----------
  qemu-doc: Clarify extent of build platform support

Supporting a build platform beyond its end of life makes no sense.
Spell that out just to be clear.

Signed-off-by: Markus Armbruster <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Daniel P. Berrangé <address@hidden>
Reviewed-by: Eduardo Habkost <address@hidden>


  Commit: 6c599282f8ab382fe59f03a6cae755b89561a7b3
      
https://github.com/qemu/qemu/commit/6c599282f8ab382fe59f03a6cae755b89561a7b3
  Author: Peter Maydell <address@hidden>
  Date:   2020-02-17 (Mon, 17 Feb 2020)

  Changed paths:
    M include/monitor/monitor.h
    M include/sysemu/sysemu.h
    M monitor/Makefile.objs
    M monitor/hmp-cmds.c
    M monitor/misc.c
    M monitor/monitor-internal.h
    M monitor/monitor.c
    A monitor/qmp-cmds-control.c
    M monitor/qmp-cmds.c
    M monitor/qmp.c
    M qapi/Makefile.objs
    A qapi/control.json
    M qapi/misc.json
    M qapi/qapi-schema.json
    M qemu-doc.texi
    M tests/qtest/qmp-test.c
    M ui/gtk.c
    M vl.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/armbru/tags/pull-monitor-2020-02-15-v2' 
into staging

Monitor patches for 2020-02-15

# gpg: Signature made Mon 17 Feb 2020 13:26:20 GMT
# gpg:                using RSA key 354BC8B3D7EB2A6B68674E5F3870B400EB918653
# gpg:                issuer "address@hidden"
# gpg: Good signature from "Markus Armbruster <address@hidden>" [full]
# gpg:                 aka "Markus Armbruster <address@hidden>" [full]
# Primary key fingerprint: 354B C8B3 D7EB 2A6B 6867  4E5F 3870 B400 EB91 8653

* remotes/armbru/tags/pull-monitor-2020-02-15-v2:
  qemu-doc: Clarify extent of build platform support
  monitor: Move qmp_query_qmp_schema to qmp-cmds-control.c
  monitor: Collect "control" command handlers in qmp-cmds.control.c
  qapi: Split control.json off misc.json
  monitor: Move monitor option parsing to monitor/monitor.c

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/652c5bbd7e7d...6c599282f8ab