qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] scsi-disk: Add support for port WWN and index d


From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH] scsi-disk: Add support for port WWN and index descriptors in VPD page 83h
Date: Thu, 20 Feb 2014 17:10:56 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0

Il 20/02/2014 17:02, Andreas Färber ha scritto:
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.

That's fine, because device_add uses QemuOpts internally.

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?

Well, no cool idea except "once conversion is done at the QemuOpts level, do a full sweep of s/_/-/". This should be done before we'll be able to create devices with object-add. I think as long as we have a plan, consistency trumps design-by-committee convention.

Paolo



reply via email to

[Prev in Thread] Current Thread [Next in Thread]