From patchwork Tue Aug 5 04:36:09 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Eric Blake X-Patchwork-Id: 376549 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 25AE5140086 for ; Tue, 5 Aug 2014 14:39:42 +1000 (EST) Received: from localhost ([::1]:56471 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XEWXE-0005Ex-3M for incoming@patchwork.ozlabs.org; Tue, 05 Aug 2014 00:39:40 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47946) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XEWUB-0000iM-12 for qemu-devel@nongnu.org; Tue, 05 Aug 2014 00:36:39 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XEWU2-0003ra-MI for qemu-devel@nongnu.org; Tue, 05 Aug 2014 00:36:30 -0400 Received: from qmta07.emeryville.ca.mail.comcast.net ([76.96.30.64]:49806) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XEWU2-0003rQ-Eb for qemu-devel@nongnu.org; Tue, 05 Aug 2014 00:36:22 -0400 Received: from omta05.emeryville.ca.mail.comcast.net ([76.96.30.43]) by qmta07.emeryville.ca.mail.comcast.net with comcast id asZZ1o0010vp7WLA7scMqn; Tue, 05 Aug 2014 04:36:21 +0000 Received: from red.redhat.com ([24.10.254.122]) by omta05.emeryville.ca.mail.comcast.net with comcast id ascB1o00B2fD5rL8RscLM7; Tue, 05 Aug 2014 04:36:21 +0000 From: Eric Blake To: qemu-devel@nongnu.org Date: Mon, 4 Aug 2014 22:36:09 -0600 Message-Id: <1407213371-7408-6-git-send-email-eblake@redhat.com> X-Mailer: git-send-email 1.9.3 In-Reply-To: <1407213371-7408-1-git-send-email-eblake@redhat.com> References: <1407213371-7408-1-git-send-email-eblake@redhat.com> DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=comcast.net; s=q20140121; t=1407213381; bh=JSA5UsOHW/0N+tNTRikbT9bkjcb/Fgf7RhtrytP7rB8=; h=Received:Received:From:To:Subject:Date:Message-Id; b=eiD75gNqSef/mRIgzLZzp0lzeSSPyvltgiiGmNdXg+/U6Nt+LfLg3dhi8RYRdbQ31 7e0ZMLnzlzS6Sw6ebdyyI2QQ4o2ARgbM5JY1GArkFMCyBb1fYjhEGFOTlFOKgy8AKL nWbaNFlxrPKqqBZ1G4u4jvFWeqjnHvSnNVeAnsTBmBjI+EW3BrogdeO/oOXnddjDQi syGpNONJZR/kzSPQWeZaSMkd1Lp1YxuuM7RCQ/0O7jMH+CzpiTsKoYoJZo4zFKBYsi rTXq+yNl0pydEnSIxrdZVnSBLpO45POubUPKCZX5J4qscrETIDGRufEnYX9nILf4SY r7W0FKQRTiOjA== X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 76.96.30.64 Cc: famz@redhat.com, Markus Armbruster , wenchaoqemu@gmail.com, Luiz Capitulino Subject: [Qemu-devel] [PATCH 5/7] qapi: drop inline subtype in query-version X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org A future patch will be using a 'name':{dictionary} entry in the QAPI schema to specify a default value for an optional argument; but existing use of inline substructs conflicts with that goal. This patch fixes one of only two commands relying on nested subtypes, by breaking the nesting into an explicit type. Prefer the safer g_new0() while making the conversion. * qapi/common.json (VersionInfo): Split inline data... (VersionTriple): ...into new type. * qmp.c (qmp_query_version): Update caller. * hmp.c (hmp_info_version): Likewise. Signed-off-by: Eric Blake --- hmp.c | 2 +- qapi/common.json | 26 +++++++++++++++++++------- qmp.c | 9 +++++---- 3 files changed, 25 insertions(+), 12 deletions(-) diff --git a/hmp.c b/hmp.c index 4d1838e..e637fbc 100644 --- a/hmp.c +++ b/hmp.c @@ -55,7 +55,7 @@ void hmp_info_version(Monitor *mon, const QDict *qdict) info = qmp_query_version(NULL); monitor_printf(mon, "%" PRId64 ".%" PRId64 ".%" PRId64 "%s\n", - info->qemu.major, info->qemu.minor, info->qemu.micro, + info->qemu->major, info->qemu->minor, info->qemu->micro, info->package); qapi_free_VersionInfo(info); diff --git a/qapi/common.json b/qapi/common.json index 4e9a21f..d007095 100644 --- a/qapi/common.json +++ b/qapi/common.json @@ -29,15 +29,28 @@ 'DeviceNotActive', 'DeviceNotFound', 'KVMMissingCap' ] } ## +# @VersionTriple +# +# A three-part version number. +# +# @qemu.major: The major version number. +# +# @qemu.minor: The minor version number. +# +# @qemu.micro: The micro version number. +# +# Since: 2.2 +## +{ 'type': 'VersionTriple', + 'data': {'major': 'int', 'minor': 'int', 'micro': 'int'} } + + +## # @VersionInfo: # # A description of QEMU's version. # -# @qemu.major: The major version of QEMU -# -# @qemu.minor: The minor version of QEMU -# -# @qemu.micro: The micro version of QEMU. By current convention, a micro +# @qemu: The version of QEMU. By current convention, a micro # version of 50 signifies a development branch. A micro version # greater than or equal to 90 signifies a release candidate for # the next minor version. A micro version of less than 50 @@ -51,8 +64,7 @@ # Since: 0.14.0 ## { 'type': 'VersionInfo', - 'data': {'qemu': {'major': 'int', 'minor': 'int', 'micro': 'int'}, - 'package': 'str'} } + 'data': {'qemu': 'VersionTriple', 'package': 'str'} } ## # @query-version: diff --git a/qmp.c b/qmp.c index 0d2553a..35e6f7a 100644 --- a/qmp.c +++ b/qmp.c @@ -45,15 +45,16 @@ NameInfo *qmp_query_name(Error **errp) VersionInfo *qmp_query_version(Error **errp) { - VersionInfo *info = g_malloc0(sizeof(*info)); + VersionInfo *info = g_new0(VersionInfo, 1); const char *version = QEMU_VERSION; char *tmp; - info->qemu.major = strtol(version, &tmp, 10); + info->qemu = g_new0(VersionTriple, 1); + info->qemu->major = strtol(version, &tmp, 10); tmp++; - info->qemu.minor = strtol(tmp, &tmp, 10); + info->qemu->minor = strtol(tmp, &tmp, 10); tmp++; - info->qemu.micro = strtol(tmp, &tmp, 10); + info->qemu->micro = strtol(tmp, &tmp, 10); info->package = g_strdup(QEMU_PKGVERSION); return info;