Re: [Qemu-devel] qdev property listing broken

[Jan Kiszka]

On 2012-04-02 21:48, Anthony Liguori wrote:
> On 04/02/2012 02:44 PM, Jan Kiszka wrote:
>> On 2012-04-02 21:41, Anthony Liguori wrote:
>>> On 04/02/2012 02:33 PM, Jan Kiszka wrote:
>>>> Hi Anthony,
>>>>
>>>> I noticed that only properties with legacy names are printed via info
>>>> qtree.
>>>
>>> Yes.  Is that not what you expect?
>>>
>>> The qom properties and model is not stable for 1.1 so I was very careful
>>> in making sure they didn't leak into info qtree.
>>>
>>>> After digging through the qdev and qom property maze, it turned
>>>> out the property registration in qdev_property_add_legacy and
>>>> qdev_property_add_static is not consistent with the access in
>>>> qdev_print_props.
>>>
>>> qdev_print_props will completely die for 1.2 as will almost everything
>>> related to the human monitor in qdev-monitor.c.
>>>
>>> I just wanted to give us a full release to make sure we were happy with
>>> the various interfaces.
>>
>> This is broken as previously listed qdev properties disappeared.
> 
> Yes, I misunderstood "legacy" names to include static properties too. 
> You meant non-legacy static properties are broken.

Precisely. They are no longer listed.

> 
> Pure qom properties are not displayed and never were displayed.

Yes, I'm interested in those properties I can set on VM creation.

Jan

signature.asc
Description: OpenPGP digital signature