From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43634) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WPXj8-00076P-D7 for qemu-devel@nongnu.org; Mon, 17 Mar 2014 09:37:20 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WPXj0-0006R6-LR for qemu-devel@nongnu.org; Mon, 17 Mar 2014 09:37:14 -0400 Received: from mx1.redhat.com ([209.132.183.28]:7004) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WPXj0-0006R0-CS for qemu-devel@nongnu.org; Mon, 17 Mar 2014 09:37:06 -0400 Message-ID: <5326FA7B.2080902@redhat.com> Date: Mon, 17 Mar 2014 14:36:59 +0100 From: Paolo Bonzini MIME-Version: 1.0 References: <1395033894-21613-1-git-send-email-akong@redhat.com> <8761ndgrnf.fsf@blackfin.pond.sub.org> <5326D37F.6030704@redhat.com> <87fvmh80ox.fsf@blackfin.pond.sub.org> In-Reply-To: <87fvmh80ox.fsf@blackfin.pond.sub.org> Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2] update names in option tables to match with actual command-line spelling List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Markus Armbruster Cc: Amos Kong , qemu-devel@nongnu.org, aliguori@amazon.com, afaerber@suse.de Il 17/03/2014 13:33, Markus Armbruster ha scritto: > This patch changes: > > from to introduced in > acpi acpitable 0c764a9 v1.5.0 > boot-opts boot 3d3b830 v1.0 > smp-opts smp 12b7f57 v1.6.0 > > All three have calcified into ABI already. What has calcified into ABI is only what has likely been used as ABI. The other is an ABI that won't be nice to break, but it is not calcified yet. There are two aspects of this: 1) QMP queries. Did any of the above conversions introduce new sub-options? Or was any suboption introduced after that point? If not, there's hardly a reason for anyone to query for acpi/boot-opts/smp-opts QemuOpts 2) readconfig. I think readconfig is not practically usable until at least memory will be part of it. For this reason, -readconfig is not a worry; not yet at least: it will be in 2.1 once -m is converted. >> > I'm for including this patch in 2.0. > Not without explaining the ABI breakage in the commit message. We > should also make sure to cover it in the release notes[*]. I agree. > Moreover, if exceptions from the rule "QemuOptsList name must match the > name of the (non-sugared) command line option using it" are or will be > permissible, then Amos's comment on QemuOptsList member name needs to be > clarified. I like your idea of -memory as a synonym for -m. Paolo