Re: [Qemu-devel] [PATCH 0/3] Fix confused output for alias properties

[Michael S. Tsirkin]

On Mon, Sep 22, 2014 at 02:06:38PM +0200, Paolo Bonzini wrote:
> Il 22/09/2014 13:22, Gonglei (Arei) ha scritto:
> > > This doesn't change the fact that ObjectProperty is a generic struct,
> > > and adding alias-specific fields there is wrong.
> > 
> > OK, Maybe I should find other ways to attach this purpose and
> > avoid layering violation. Thanks!
> 
> Unfortunately I cannot think of any.
> 
> We could add a description field to ObjectProperty, and replace
> legacy_name with a description.  The output then would be
> 
> virtio-blk.drive=str (drive)
> That's a bit hackish, but perhaps it would be good enough for mst and
> Markus?
> 
> Paolo

I would just drop "str" and replace it with drive.
I seems to convey zero information.
Description should be something more informative, e.g.
(ID of a drive to use as a backend)
Help for -device could add "Must match ID of a -drive option".
Help for device HMP command could add "Must match ID of a drive command."


-- 
MST