On Thu, Aug 24, 2017 at 9:17 PM Markus Armbruster <arm...@redhat.com> wrote:
> Signed-off-by: Markus Armbruster <arm...@redhat.com> > Looks like you tried to find a reasonable location in the file for each events. Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com> > --- > Makefile | 2 +- > qapi-schema.json | 134 > ++++++++++++++++++++++++++++++++++++++++++++++++++++- > qapi/event.json | 138 > ------------------------------------------------------- > 3 files changed, 134 insertions(+), 140 deletions(-) > delete mode 100644 qapi/event.json > > diff --git a/Makefile b/Makefile > index 3dde210..337a1f6 100644 > --- a/Makefile > +++ b/Makefile > @@ -412,7 +412,7 @@ qapi-modules = $(SRC_PATH)/qapi-schema.json > $(SRC_PATH)/qapi/common.json \ > $(SRC_PATH)/qapi/block.json > $(SRC_PATH)/qapi/block-core.json \ > $(SRC_PATH)/qapi/char.json \ > $(SRC_PATH)/qapi/crypto.json \ > - $(SRC_PATH)/qapi/event.json > $(SRC_PATH)/qapi/introspect.json \ > + $(SRC_PATH)/qapi/introspect.json \ > $(SRC_PATH)/qapi/migration.json \ > $(SRC_PATH)/qapi/net.json \ > $(SRC_PATH)/qapi/rocker.json \ > diff --git a/qapi-schema.json b/qapi-schema.json > index 0ad4e02..4964d92 100644 > --- a/qapi-schema.json > +++ b/qapi-schema.json > @@ -90,7 +90,6 @@ > { 'include': 'qapi/ui.json' } > { 'include': 'qapi/migration.json' } > { 'include': 'qapi/transaction.json' } > -{ 'include': 'qapi/event.json' } > { 'include': 'qapi/trace.json' } > { 'include': 'qapi/introspect.json' } > > @@ -552,6 +551,28 @@ > { 'command': 'query-balloon', 'returns': 'BalloonInfo' } > > ## > +# @BALLOON_CHANGE: > +# > +# Emitted when the guest changes the actual BALLOON level. This value is > +# equivalent to the @actual field return by the 'query-balloon' command > +# > +# @actual: actual level of the guest memory balloon in bytes > +# > +# Note: this event is rate-limited. > +# > +# Since: 1.2 > +# > +# Example: > +# > +# <- { "event": "BALLOON_CHANGE", > +# "data": { "actual": 944766976 }, > +# "timestamp": { "seconds": 1267020223, "microseconds": 435656 } } > +# > +## > +{ 'event': 'BALLOON_CHANGE', > + 'data': { 'actual': 'int' } } > + > +## > # @PciMemoryRange: > # > # A PCI device memory region > @@ -1434,6 +1455,30 @@ > { 'command': 'device_del', 'data': {'id': 'str'} } > > ## > +# @DEVICE_DELETED: > +# > +# Emitted whenever the device removal completion is acknowledged by the > guest. > +# At this point, it's safe to reuse the specified device ID. Device > removal can > +# be initiated by the guest or by HMP/QMP commands. > +# > +# @device: device name > +# > +# @path: device path > +# > +# Since: 1.5 > +# > +# Example: > +# > +# <- { "event": "DEVICE_DELETED", > +# "data": { "device": "virtio-net-pci-0", > +# "path": "/machine/peripheral/virtio-net-pci-0" }, > +# "timestamp": { "seconds": 1265044230, "microseconds": 450486 } } > +# > +## > +{ 'event': 'DEVICE_DELETED', > + 'data': { '*device': 'str', 'path': 'str' } } > + > +## > # @DumpGuestMemoryFormat: > # > # An enumeration of guest-memory-dump's format. > @@ -1569,6 +1614,29 @@ > { 'command': 'query-dump', 'returns': 'DumpQueryResult' } > > ## > +# @DUMP_COMPLETED: > +# > +# Emitted when background dump has completed > +# > +# @result: DumpQueryResult type described in qapi-schema.json. > +# > +# @error: human-readable error string that provides > +# hint on why dump failed. Only presents on failure. The > +# user should not try to interpret the error string. > +# > +# Since: 2.6 > +# > +# Example: > +# > +# { "event": "DUMP_COMPLETED", > +# "data": {"result": {"total": 1090650112, "status": "completed", > +# "completed": 1090650112} } } > +# > +## > +{ 'event': 'DUMP_COMPLETED' , > + 'data': { 'result': 'DumpQueryResult', '*error': 'str' } } > + > +## > # @DumpGuestMemoryCapability: > # > # A list of the available formats for dump-guest-memory > @@ -2652,6 +2720,29 @@ > { 'command': 'query-memory-devices', 'returns': ['MemoryDeviceInfo'] } > > ## > +# @MEM_UNPLUG_ERROR: > +# > +# Emitted when memory hot unplug error occurs. > +# > +# @device: device name > +# > +# @msg: Informative message > +# > +# Since: 2.4 > +# > +# Example: > +# > +# <- { "event": "MEM_UNPLUG_ERROR" > +# "data": { "device": "dimm1", > +# "msg": "acpi: device unplug for unsupported device" > +# }, > +# "timestamp": { "seconds": 1265044230, "microseconds": 450486 } } > +# > +## > +{ 'event': 'MEM_UNPLUG_ERROR', > + 'data': { 'device': 'str', 'msg': 'str' } } > + > +## > # @ACPISlotType: > # > # @DIMM: memory slot > @@ -2706,6 +2797,25 @@ > { 'command': 'query-acpi-ospm-status', 'returns': ['ACPIOSTInfo'] } > > ## > +# @ACPI_DEVICE_OST: > +# > +# Emitted when guest executes ACPI _OST method. > +# > +# @info: ACPIOSTInfo type as described in qapi-schema.json > +# > +# Since: 2.1 > +# > +# Example: > +# > +# <- { "event": "ACPI_DEVICE_OST", > +# "data": { "device": "d1", "slot": "0", > +# "slot-type": "DIMM", "source": 1, "status": 0 } } > +# > +## > +{ 'event': 'ACPI_DEVICE_OST', > + 'data': { 'info': 'ACPIOSTInfo' } } > + > +## > # @IoOperationType: > # > # An enumeration of the I/O operation types > @@ -2738,6 +2848,28 @@ > { 'command': 'rtc-reset-reinjection' } > > ## > +# @RTC_CHANGE: > +# > +# Emitted when the guest changes the RTC time. > +# > +# @offset: offset between base RTC clock (as specified by -rtc base), and > +# new RTC clock value > +# > +# Note: This event is rate-limited. > +# > +# Since: 0.13.0 > +# > +# Example: > +# > +# <- { "event": "RTC_CHANGE", > +# "data": { "offset": 78 }, > +# "timestamp": { "seconds": 1267020223, "microseconds": 435656 } } > +# > +## > +{ 'event': 'RTC_CHANGE', > + 'data': { 'offset': 'int' } } > + > +## > # @ReplayMode: > # > # Mode of the replay subsystem. > diff --git a/qapi/event.json b/qapi/event.json > deleted file mode 100644 > index 48a5d8f..0000000 > --- a/qapi/event.json > +++ /dev/null > @@ -1,138 +0,0 @@ > -# -*- Mode: Python -*- > - > -## > -# = Other events > -## > - > -## > -# @RTC_CHANGE: > -# > -# Emitted when the guest changes the RTC time. > -# > -# @offset: offset between base RTC clock (as specified by -rtc base), and > -# new RTC clock value > -# > -# Note: This event is rate-limited. > -# > -# Since: 0.13.0 > -# > -# Example: > -# > -# <- { "event": "RTC_CHANGE", > -# "data": { "offset": 78 }, > -# "timestamp": { "seconds": 1267020223, "microseconds": 435656 } } > -# > -## > -{ 'event': 'RTC_CHANGE', > - 'data': { 'offset': 'int' } } > - > -## > -# @DEVICE_DELETED: > -# > -# Emitted whenever the device removal completion is acknowledged by the > guest. > -# At this point, it's safe to reuse the specified device ID. Device > removal can > -# be initiated by the guest or by HMP/QMP commands. > -# > -# @device: device name > -# > -# @path: device path > -# > -# Since: 1.5 > -# > -# Example: > -# > -# <- { "event": "DEVICE_DELETED", > -# "data": { "device": "virtio-net-pci-0", > -# "path": "/machine/peripheral/virtio-net-pci-0" }, > -# "timestamp": { "seconds": 1265044230, "microseconds": 450486 } } > -# > -## > -{ 'event': 'DEVICE_DELETED', > - 'data': { '*device': 'str', 'path': 'str' } } > - > -## > -# @ACPI_DEVICE_OST: > -# > -# Emitted when guest executes ACPI _OST method. > -# > -# @info: ACPIOSTInfo type as described in qapi-schema.json > -# > -# Since: 2.1 > -# > -# Example: > -# > -# <- { "event": "ACPI_DEVICE_OST", > -# "data": { "device": "d1", "slot": "0", > -# "slot-type": "DIMM", "source": 1, "status": 0 } } > -# > -## > -{ 'event': 'ACPI_DEVICE_OST', > - 'data': { 'info': 'ACPIOSTInfo' } } > - > -## > -# @BALLOON_CHANGE: > -# > -# Emitted when the guest changes the actual BALLOON level. This value is > -# equivalent to the @actual field return by the 'query-balloon' command > -# > -# @actual: actual level of the guest memory balloon in bytes > -# > -# Note: this event is rate-limited. > -# > -# Since: 1.2 > -# > -# Example: > -# > -# <- { "event": "BALLOON_CHANGE", > -# "data": { "actual": 944766976 }, > -# "timestamp": { "seconds": 1267020223, "microseconds": 435656 } } > -# > -## > -{ 'event': 'BALLOON_CHANGE', > - 'data': { 'actual': 'int' } } > - > -## > -# @MEM_UNPLUG_ERROR: > -# > -# Emitted when memory hot unplug error occurs. > -# > -# @device: device name > -# > -# @msg: Informative message > -# > -# Since: 2.4 > -# > -# Example: > -# > -# <- { "event": "MEM_UNPLUG_ERROR" > -# "data": { "device": "dimm1", > -# "msg": "acpi: device unplug for unsupported device" > -# }, > -# "timestamp": { "seconds": 1265044230, "microseconds": 450486 } } > -# > -## > -{ 'event': 'MEM_UNPLUG_ERROR', > - 'data': { 'device': 'str', 'msg': 'str' } } > - > -## > -# @DUMP_COMPLETED: > -# > -# Emitted when background dump has completed > -# > -# @result: DumpQueryResult type described in qapi-schema.json. > -# > -# @error: human-readable error string that provides > -# hint on why dump failed. Only presents on failure. The > -# user should not try to interpret the error string. > -# > -# Since: 2.6 > -# > -# Example: > -# > -# { "event": "DUMP_COMPLETED", > -# "data": {"result": {"total": 1090650112, "status": "completed", > -# "completed": 1090650112} } } > -# > -## > -{ 'event': 'DUMP_COMPLETED' , > - 'data': { 'result': 'DumpQueryResult', '*error': 'str' } } > -- > 2.7.5 > > > -- Marc-André Lureau