qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH v3 00/43] qobject/qapi: add uint type
@ 2017-06-07 16:35 Marc-André Lureau
  2017-06-07 16:35 ` [Qemu-devel] [PATCH v3 01/43] qdev: remove PropertyInfo.qtype field Marc-André Lureau
                   ` (45 more replies)
  0 siblings, 46 replies; 69+ messages in thread
From: Marc-André Lureau @ 2017-06-07 16:35 UTC (permalink / raw)
  To: qemu-devel; +Cc: armbru, Marc-André Lureau

Hi,

In previously sent series "[PATCH 00/21] WIP: dump: add kaslr support
(for after 2.9)", I proposed changes to accept uint64 values from
json, by adding a QUint type. During review, it was suggested to
introduce a QNum type to hold various number representations.

This series introduces the QNum type, adds uint representation to it,
and make uint64 values over json possible (while keeping json negative
int for values >INT64_MAX to unsigned cast compatibility). It also
improves some int vs uint usage for object properties and other
QObject related clean-ups.

v3:
- rebased, drop the first keyval-related patches now upstream
- update "tests: Remove test cases for alternates of 'number' and 'int'"
  commit message, and drop now useless conflict checks
- added "qapi: minor refactoring"
- add missing Author/Copyright lines lost during merge
- add qnum_get_try_(u)int() and simplify related code
- fix some code comments
- replace g_assert_not_reached() with assert(0) in qapi/
- replace qapi/qmp/types.h include with the min required headers
- add comment about changes in json serialization in "qapi: update the
  qobject visitor to use QNUM_U64"
- update comment in "object: use more specific property type names"
  with the externally visible changes
- split "qdev: make default property int" patch, rename macros
  DEFINE_PROP_SIGNED/DEFINE_PROP_UNSIGNED
- in tests, compare with expected float value
- add r-b and suggested tags

v2:
- add set_default_value() callback in PropertyInfo
- use Markus version of "remove alt num-int cases"
- add more int/number ranges visitor checks
- replace Error usage for success bool return value in get_int/uint()
- various updated comments, patch and code reorganization
- avoid temporary regression from patch "json: learn to parse uint64
  numbers", drop related patch
- split get_mmcfg_size() fix in a seperate patch
- added "apic-common: make "id" property a uint32" patch
- split the qdev uint proprety patch
- split "Use uint property getter/setter where appropriate" in various
  patches
- add some qdict tests
- dropped s3/s4 bool patch
- add some r-b tags

Marc-André Lureau (43):
  qdev: remove PropertyInfo.qtype field
  object: fix potential leak in getters
  tests: Remove test cases for alternates of 'number' and 'int'
  tests: add more int/number ranges checks
  qapi: minor refactoring
  qapi: merge QInt and QFloat in QNum
  qapi: Remove visit_start_alternate() parameter promote_int
  tests: remove /{qnum,qlist,dict}/destroy test
  qnum: add uint type
  json: learn to parse uint64 numbers
  qapi: update the qobject visitor to use QNUM_U64
  object: add uint property setter/getter
  q35: fix get_mmcfg_size to use uint64 visitor
  object: use more specific property type names
  qdev: rename DEFINE_PROP_DEFAULT() to DEFINE_PROP_SIGNED()
  qdev: wrap default property value in an union
  qdev: avoid type casts between signed and unsigned
  apic-common: make "id" property a uint32
  qdev: use appropriate getter/setters type
  isa: use get_uint() for "io-base"
  pc-dimm: use get_uint() for dimm properties
  pc-dimm: make "size" property uint64
  pcihp: use get_uint() for "bsel" property
  aspeed: use {set,get}_uint() for "ram-size" property
  bcm2835_fb: use {get,set}_uint() for "vcram-size" and "vcram-base"
  platform-bus: use get_uint() for "addr" property
  acpi: use get_uint() for "acpi-pcihp-io*" properties
  acpi: use get_uint() for various acpi properties
  acpi: use get_uint() for "pci-hole*" properties
  pc: use get_uint() for "iobase" property
  pc: use get_uint() for "apic-id" property
  pc: use get_uint() for "hpet-intcap" property
  xen: use get_uint() for "max-ram-below-4g" property
  arm: use get_uint() for "mp-affinity" property
  auxbus: use get_uint() for "addr" property
  pvpanic: use get_uint() for "ioport" property
  pnv-core: use get_uint() for "core-pir" property
  numa: use get_uint() for "size" property
  i386/cpu: use get_uint() for "min-level"/"min-xlevel" properties
  console: use get_uint() for "head" property
  tests/qdict: check more get_try_int() cases
  RFC: qdict: add uint
  qobject: move dump_qobject() from block/ to qobject/

 include/qapi/visitor.h                   |   4 +-
 include/qapi/visitor-impl.h              |   2 +-
 scripts/qapi.py                          |  36 +++--
 scripts/qapi-visit.py                    |  12 +-
 include/hw/isa/isa.h                     |   2 +-
 include/hw/qdev-core.h                   |   7 +-
 include/hw/qdev-properties.h             |  50 ++++---
 include/qapi/qmp/qdict.h                 |  10 +-
 include/qapi/qmp/qfloat.h                |  29 ----
 include/qapi/qmp/qint.h                  |  28 ----
 include/qapi/qmp/qlist.h                 |   5 +-
 include/qapi/qmp/qnum.h                  |  53 ++++++++
 include/qapi/qmp/qobject.h               |   7 +
 include/qapi/qmp/types.h                 |   3 +-
 include/qapi/qobject-input-visitor.h     |   6 +-
 include/qapi/qobject-output-visitor.h    |   8 +-
 include/qom/object.h                     |  23 ++++
 qapi/qapi-visit-core.c                   |   6 +-
 backends/cryptodev.c                     |   2 +-
 block/blkdebug.c                         |   1 -
 block/nbd.c                              |   1 -
 block/nfs.c                              |   1 -
 block/qapi.c                             |  93 +------------
 block/quorum.c                           |   1 -
 block/sheepdog.c                         |   1 -
 block/ssh.c                              |   1 -
 block/vvfat.c                            |   1 -
 blockdev.c                               |   8 +-
 hw/acpi/memory_hotplug.c                 |  12 +-
 hw/acpi/nvdimm.c                         |  18 +--
 hw/acpi/pcihp.c                          |   7 +-
 hw/arm/aspeed.c                          |   8 +-
 hw/arm/bcm2835_peripherals.c             |   9 +-
 hw/arm/raspi.c                           |   4 +-
 hw/block/fdc.c                           |  18 +--
 hw/core/platform-bus.c                   |   2 +-
 hw/core/qdev-properties.c                |  40 +++++-
 hw/core/qdev.c                           |  13 +-
 hw/i386/acpi-build.c                     |  71 +++++-----
 hw/i386/pc.c                             |   6 +-
 hw/i386/xen/xen-hvm.c                    |   6 +-
 hw/intc/apic_common.c                    |  10 +-
 hw/intc/arm_gicv3_common.c               |   2 +-
 hw/mem/pc-dimm.c                         |  23 ++--
 hw/misc/auxbus.c                         |   2 +-
 hw/misc/pvpanic.c                        |   2 +-
 hw/net/e1000e.c                          |   6 +-
 hw/pci-host/gpex.c                       |   2 +-
 hw/pci-host/piix.c                       |   8 +-
 hw/pci-host/q35.c                        |  15 +--
 hw/pci-host/xilinx-pcie.c                |   2 +-
 hw/ppc/pnv.c                             |   2 +-
 hw/ppc/pnv_core.c                        |   2 +-
 hw/ppc/spapr.c                           |   7 +-
 hw/usb/xen-usb.c                         |   1 -
 monitor.c                                |   2 +-
 net/dump.c                               |   2 +-
 net/filter-buffer.c                      |   2 +-
 numa.c                                   |   6 +-
 qapi/qapi-clone-visitor.c                |   2 +-
 qapi/qapi-dealloc-visitor.c              |   2 +-
 qapi/qobject-input-visitor.c             |  58 ++++----
 qapi/qobject-output-visitor.c            |   7 +-
 qga/commands.c                           |   2 +-
 qga/main.c                               |   1 -
 qobject/json-lexer.c                     |   4 +
 qobject/json-parser.c                    |  56 +++++---
 qobject/qdict.c                          | 100 +++++++++++---
 qobject/qfloat.c                         |  62 ---------
 qobject/qint.c                           |  61 ---------
 qobject/qjson.c                          |  37 +-----
 qobject/qlist.c                          |  23 ++++
 qobject/qnum.c                           | 221 +++++++++++++++++++++++++++++++
 qobject/qobject.c                        |  22 ++-
 qom/object.c                             |  51 +++++--
 target/i386/cpu.c                        |  10 +-
 tests/check-qdict.c                      |  80 ++++++++---
 tests/check-qfloat.c                     |  53 --------
 tests/check-qint.c                       |  87 ------------
 tests/check-qjson.c                      | 154 +++++++++++++++------
 tests/check-qlist.c                      |  40 ++----
 tests/check-qnum.c                       | 170 ++++++++++++++++++++++++
 tests/check-qstring.c                    |   9 --
 tests/test-keyval.c                      |   4 +-
 tests/test-qmp-commands.c                |   8 +-
 tests/test-qmp-event.c                   |   9 +-
 tests/test-qobject-input-visitor.c       | 110 ++++++++-------
 tests/test-qobject-output-visitor.c      |  91 ++++++++-----
 tests/test-x86-cpuid-compat.c            |  18 ++-
 ui/console.c                             |   4 +-
 ui/spice-core.c                          |   1 -
 ui/vnc-enc-tight.c                       |   1 -
 util/qemu-option.c                       |  24 ++--
 MAINTAINERS                              |   3 +-
 qapi/trace-events                        |   2 +-
 qobject/Makefile.objs                    |   2 +-
 scripts/coccinelle/qobject.cocci         |   4 +-
 tests/.gitignore                         |   3 +-
 tests/Makefile.include                   |  13 +-
 tests/qapi-schema/comments.out           |   2 +-
 tests/qapi-schema/doc-good.out           |   2 +-
 tests/qapi-schema/empty.out              |   2 +-
 tests/qapi-schema/event-case.out         |   2 +-
 tests/qapi-schema/ident-with-escape.out  |   2 +-
 tests/qapi-schema/include-relpath.out    |   2 +-
 tests/qapi-schema/include-repetition.out |   2 +-
 tests/qapi-schema/include-simple.out     |   2 +-
 tests/qapi-schema/indented-expr.out      |   2 +-
 tests/qapi-schema/qapi-schema-test.json  |   2 -
 tests/qapi-schema/qapi-schema-test.out   |  10 +-
 110 files changed, 1331 insertions(+), 1019 deletions(-)
 delete mode 100644 include/qapi/qmp/qfloat.h
 delete mode 100644 include/qapi/qmp/qint.h
 create mode 100644 include/qapi/qmp/qnum.h
 delete mode 100644 qobject/qfloat.c
 delete mode 100644 qobject/qint.c
 create mode 100644 qobject/qnum.c
 delete mode 100644 tests/check-qfloat.c
 delete mode 100644 tests/check-qint.c
 create mode 100644 tests/check-qnum.c

-- 
2.13.0.91.g00982b8dd

^ permalink raw reply	[flat|nested] 69+ messages in thread

end of thread, other threads:[~2017-06-09 14:34 UTC | newest]

Thread overview: 69+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-06-07 16:35 [Qemu-devel] [PATCH v3 00/43] qobject/qapi: add uint type Marc-André Lureau
2017-06-07 16:35 ` [Qemu-devel] [PATCH v3 01/43] qdev: remove PropertyInfo.qtype field Marc-André Lureau
2017-06-07 16:35 ` [Qemu-devel] [PATCH v3 02/43] object: fix potential leak in getters Marc-André Lureau
2017-06-07 16:35 ` [Qemu-devel] [PATCH v3 03/43] tests: Remove test cases for alternates of 'number' and 'int' Marc-André Lureau
2017-06-07 16:35 ` [Qemu-devel] [PATCH v3 04/43] tests: add more int/number ranges checks Marc-André Lureau
2017-06-08 14:55   ` Markus Armbruster
2017-06-07 16:35 ` [Qemu-devel] [PATCH v3 05/43] qapi: minor refactoring Marc-André Lureau
2017-06-08 14:57   ` Markus Armbruster
2017-06-07 16:35 ` [Qemu-devel] [PATCH v3 06/43] qapi: merge QInt and QFloat in QNum Marc-André Lureau
2017-06-08 15:54   ` Markus Armbruster
2017-06-07 16:35 ` [Qemu-devel] [PATCH v3 07/43] qapi: Remove visit_start_alternate() parameter promote_int Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 08/43] tests: remove /{qnum, qlist, dict}/destroy test Marc-André Lureau
2017-06-08 15:00   ` Markus Armbruster
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 09/43] qnum: add uint type Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 10/43] json: learn to parse uint64 numbers Marc-André Lureau
2017-06-08 16:27   ` Markus Armbruster
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 11/43] qapi: update the qobject visitor to use QNUM_U64 Marc-André Lureau
2017-06-08 16:36   ` Markus Armbruster
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 12/43] object: add uint property setter/getter Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 13/43] q35: fix get_mmcfg_size to use uint64 visitor Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 14/43] object: use more specific property type names Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 15/43] qdev: rename DEFINE_PROP_DEFAULT() to DEFINE_PROP_SIGNED() Marc-André Lureau
2017-06-08 16:42   ` Markus Armbruster
2017-06-08 17:01     ` Markus Armbruster
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 16/43] qdev: wrap default property value in an union Marc-André Lureau
2017-06-08 17:01   ` Markus Armbruster
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 17/43] qdev: avoid type casts between signed and unsigned Marc-André Lureau
2017-06-08 17:08   ` Markus Armbruster
2017-06-09 10:04     ` Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 18/43] apic-common: make "id" property a uint32 Marc-André Lureau
2017-06-08 16:39   ` Markus Armbruster
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 19/43] qdev: use appropriate getter/setters type Marc-André Lureau
2017-06-08 17:23   ` Markus Armbruster
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 20/43] isa: use get_uint() for "io-base" Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 21/43] pc-dimm: use get_uint() for dimm properties Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 22/43] pc-dimm: make "size" property uint64 Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 23/43] pcihp: use get_uint() for "bsel" property Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 24/43] aspeed: use {set, get}_uint() for "ram-size" property Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 25/43] bcm2835_fb: use {get, set}_uint() for "vcram-size" and "vcram-base" Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 26/43] platform-bus: use get_uint() for "addr" property Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 27/43] acpi: use get_uint() for "acpi-pcihp-io*" properties Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 28/43] acpi: use get_uint() for various acpi properties Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 29/43] acpi: use get_uint() for "pci-hole*" properties Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 30/43] pc: use get_uint() for "iobase" property Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 31/43] pc: use get_uint() for "apic-id" property Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 32/43] pc: use get_uint() for "hpet-intcap" property Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 33/43] xen: use get_uint() for "max-ram-below-4g" property Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 34/43] arm: use get_uint() for "mp-affinity" property Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 35/43] auxbus: use get_uint() for "addr" property Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 36/43] pvpanic: use get_uint() for "ioport" property Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 37/43] pnv-core: use get_uint() for "core-pir" property Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 38/43] numa: use get_uint() for "size" property Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 39/43] i386/cpu: use get_uint() for "min-level"/"min-xlevel" properties Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 40/43] console: use get_uint() for "head" property Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 41/43] tests/qdict: check more get_try_int() cases Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 42/43] RFC: qdict: add uint Marc-André Lureau
2017-06-08 17:25   ` Markus Armbruster
2017-06-09 10:05     ` Marc-André Lureau
2017-06-07 16:36 ` [Qemu-devel] [PATCH v3 43/43] qobject: move dump_qobject() from block/ to qobject/ Marc-André Lureau
2017-06-08 17:43   ` Markus Armbruster
2017-06-09 10:11     ` Marc-André Lureau
2017-06-09 10:15       ` Marc-André Lureau
2017-06-09 12:40         ` Markus Armbruster
2017-06-09 12:46           ` Eric Blake
2017-06-09 14:34     ` Max Reitz
2017-06-07 20:41 ` [Qemu-devel] [PATCH v3 00/43] qobject/qapi: add uint type no-reply
2017-06-08  1:47 ` no-reply
2017-06-09 12:32 ` Markus Armbruster
2017-06-09 13:06   ` Marc-André Lureau

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).