From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:49344) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UVLPD-0003YB-MY for qemu-devel@nongnu.org; Thu, 25 Apr 2013 08:36:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UVLP8-00070J-PU for qemu-devel@nongnu.org; Thu, 25 Apr 2013 08:36:07 -0400 Received: from mx1.redhat.com ([209.132.183.28]:61979) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UVLP8-000702-HA for qemu-devel@nongnu.org; Thu, 25 Apr 2013 08:36:02 -0400 Date: Thu, 25 Apr 2013 08:36:00 -0400 From: Luiz Capitulino Message-ID: <20130425083600.021e8bec@redhat.com> In-Reply-To: <20130425035258.GE3230@t430s.nay.redhat.com> References: <1366807646-8473-2-git-send-email-akong@redhat.com> <1366824804-24532-1-git-send-email-akong@redhat.com> <20130424142020.4e6a54a1@redhat.com> <517834DC.1040602@redhat.com> <20130425035258.GE3230@t430s.nay.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] monitor: introduce query-config-schema command List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Amos Kong Cc: aliguori@us.ibm.com, qemu-devel@nongnu.org, jyang@redhat.com On Thu, 25 Apr 2013 11:52:58 +0800 Amos Kong wrote: > On Wed, Apr 24, 2013 at 01:39:08PM -0600, Eric Blake wrote: > > On 04/24/2013 12:20 PM, Luiz Capitulino wrote: > > > On Thu, 25 Apr 2013 01:33:24 +0800 > > > Amos Kong wrote: > > > > > >> Libvirt has no way to probe if an option or property is supported, > > >> This patch introdues a new qmp command to query configuration schema > > >> information. hmp command isn't added because it's not needed. > > >> > > >> V2: fix jaso schema and comments (Eric) > > ... > > > >> +# > > >> +# @flag: If no value is given, the flag is set to 1. Otherwise the value must > > >> +# be "on" (set to 1) or "off" (set to 0) > > > > > > Let's call this 'boolean', because it's what it is. Also, I suggest > > > 'Accepts "on" or "off"' as the help text. > > > > I'm fine with calling the enum value 'boolean' even where the C code > > called it 'flag'. As long as we have a documented name that describes > > the semantics of what the parameter will take, libvirt should be able to > > cope. > > > > One other concern - you document that if a flag parameter is omitted, > > then it defaults to 1. Is that really true? > > > I'm wrong. If it's omitted in cmdline, we will give it a default value. > > example: > enable_mlock = qemu_opt_get_bool(opts, "mlock", true); > > another example: > -boot strict=on > > bool boot_strict; (false by default) > > strict boot is disabled by default, type of strict parameter is 'QEMU_OPT_STRING' > the logical default parameter is "off". > > This kind of default info is only added in help descriptions right > now, we can add a new item 'default_value' to option.h:QemuOptDesc & > qapi-schema.json:CommandLineParameterInfo in future? Yes. I don't think we'll have enough time to fix this now. > > I guess the default value is useful for libvirt. > > However, we live in different timezone, so I will post my latest patch > to maillist (no change about default value). > > Agree with other comments, thanks. > > > Amos. >