From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:41211)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <afaerber@suse.de>) id 1WGW55-0004AJ-5A
	for qemu-devel@nongnu.org; Thu, 20 Feb 2014 11:02:42 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <afaerber@suse.de>) id 1WGW4v-0005ao-Ml
	for qemu-devel@nongnu.org; Thu, 20 Feb 2014 11:02:35 -0500
Received: from cantor2.suse.de ([195.135.220.15]:47154 helo=mx2.suse.de)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <afaerber@suse.de>) id 1WGW4v-0005aj-Gb
	for qemu-devel@nongnu.org; Thu, 20 Feb 2014 11:02:25 -0500
Message-ID: <5306270E.4020105@suse.de>
Date: Thu, 20 Feb 2014 17:02:22 +0100
From: =?ISO-8859-1?Q?Andreas_F=E4rber?= <afaerber@suse.de>
MIME-Version: 1.0
References: <1392827321-10205-1-git-send-email-roland@kernel.org>	<5304E4E4.10703@redhat.com>	<5304F3D4.9060501@suse.de>	<CAL1RGDVcE3WxkTZt5XUR+GtKVyQsm-zDGbLP6s1EPirr19jq_A@mail.gmail.com>
	<53052B92.7080305@redhat.com> <530620C7.7060208@redhat.com>
In-Reply-To: <530620C7.7060208@redhat.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Subject: Re: [Qemu-devel] [PATCH] scsi-disk: Add support for port WWN and
 index descriptors in VPD page 83h
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: Roland Dreier <roland@purestorage.com>, qemu-devel@nongnu.org

Am 20.02.2014 16:35, schrieb Paolo Bonzini:
> Il 19/02/2014 23:09, Paolo Bonzini ha scritto:
>> Oversight, but it convinces me more that we should just match [_-] als=
o
>> against the other in the pair.  Must find time to write that patch...
>=20
> Looks like _ is more common than - for device properties:
>=20
> $ git grep DEFINE_PROP_.*\(\".*_.*\" | wc -l
> 132
> $ git grep DEFINE_PROP_.*\(\".*-.*\" | wc -l
> 77

Not unexpected, it's the older; the - convention was introduced possibly
with QOM around start of 2012. Or at least there it's been enforced, and
by my understanding of QOM and QMP visibility it then applies to devices
as well.

Regarding QMP, I consider it smarter to do the _ -> - matching at
QemuOpts level than somewhere inside QOM.

For -cpu we have such compatibility code (although non-QemuOpts) in
target-i386/cpu.c, converting all underscores. Unfortunately that won't
work as long as there are underscores in old properties. Maybe you have
some cool patch idea?

Andreas

--=20
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=FCrnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imend=F6rffer; HRB 16746 AG N=FCrnbe=
rg