From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Mh6B2-0007HG-FR for qemu-devel@nongnu.org; Fri, 28 Aug 2009 14:27:56 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1Mh6Ax-0007EQ-Ax for qemu-devel@nongnu.org; Fri, 28 Aug 2009 14:27:55 -0400 Received: from [199.232.76.173] (port=41246 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Mh6Ax-0007EG-57 for qemu-devel@nongnu.org; Fri, 28 Aug 2009 14:27:51 -0400 Received: from mx1.redhat.com ([209.132.183.28]:62384) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1Mh6Aw-00030F-LW for qemu-devel@nongnu.org; Fri, 28 Aug 2009 14:27:50 -0400 From: Luiz Capitulino Date: Fri, 28 Aug 2009 15:27:03 -0300 Message-Id: <1251484052-3939-1-git-send-email-lcapitulino@redhat.com> Subject: [Qemu-devel] [PATCH v3 00/29] QMonitor List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: aliguori@us.ibm.com, avi@redhat.com Hi there, Another respin, the most important change being a rebase against latest push bomb. Please review, as it's easy to make mistakes on conflict fixes. Changelog, description and diffstat follows. changelog --------- V2 -> V3 - Rebase against latest master - Updated configure changes to new style (Juan's advice) V1 -> V2 - Renamed some macros to be more OOP-like - New refcounts terminology by Paulo Bonzini - QDict: o use sys-queue for the linked list o replace qdict_add() by qdict_put() o rename qdict_exists() to qdict_haskey() o add qdict_get_try_int() and qdict_get_try_str() - QInt o drop 'export' functions o rename qint_from_int64() to qint_from_int() - Other small changes V0 -> V1 - Make QType structs const - New macros: QOBJECT_INIT(), QDECREF()/QINCREF(), QType_HEAD - qobject_to_*() should return NULL if type doesn't match description ------------ Basically, this series introduces high-level data types (QInt, QString, QDict) and port all Monitor command handlers to use them to receive arguments. The following points should be observed: o Object life cycle management: I'm using refcouting for this o QString's get: QString exports a function called qstring_get_str(), this function returns a *pointer* to the stored string. A better API would return a copy instead, but it would be too much work and error-prone to go over all handlers and put a qemu_free() in the right place. Handlers only want quick and read-only access to strings anyway, so returning a pointer makes handlers' code a lot simpler. o unit-testing: I have written unit-tests for all the new code and have an off-tree suite for the Monitor's parsing code. I was in doubt if I should submit this work... I did, but it's in the end of the series, if people don't like this it can be easily dropped. Monitor's suite is not in the series because I couldn't make it build "in tree". While reviewing the series is important to bear in mind that all my design decisions were based on the need of the current and most important users of the API: monitor command handlers. Thanks for reading this all! :) Makefile | 5 + check-qdict.c | 365 ++++ check-qint.c | 110 ++ check-qstring.c | 100 + configure | 31 + console.h | 3 +- hw/pci-hotplug.c | 15 +- migration.c | 12 +- migration.h | 9 +- monitor.c | 457 +++--- monitor.h | 1 + net.c | 25 +- net.h | 13 +- qdict-test-data.txt | 4999 +++++++++++++++++++++++++++++++++++++++++++++++++++ qdict.c | 297 +++ qdict.h | 42 + qemu-monitor.hx | 109 +- qint.c | 66 + qint.h | 16 + qobject.h | 109 ++ qstring.c | 73 + qstring.h | 15 + savevm.c | 6 +- sysemu.h | 15 +- vl.c | 11 +- 25 files changed, 6578 insertions(+), 326 deletions(-)