qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [RfC PATCH] qdev: helper macros for properties


From: Anthony Liguori
Subject: Re: [Qemu-devel] [RfC PATCH] qdev: helper macros for properties
Date: Thu, 16 Jul 2009 22:28:16 -0500
User-agent: Thunderbird 2.0.0.21 (X11/20090320)

Paul Brook wrote:
On Wednesday 15 July 2009, Gerd Hoffmann wrote:
  Hi,

New revision of the helper macros for defining properties, going on top
of all other qdev patches posted today.

IMO this should have been part of the initial properties rework, not a followup change. Use of these macros should not be optional.

I agree we want to use macros, but the property stuff is likely going to keep evolving as we figure out how it should be used. For instance, post the property rework, there has to be a 1-1 mapping between properties and variables in state. There were a number of places in the code where that wasn't always true. A property was expanded to a different internal representation or in one case, there was no internal representation at all for a property as there was no internal state.

Conceptually, I think it's hard to figure out what the best policy is without doing a bunch of conversions.

Regards,

Anthony Liguori

Paul







reply via email to

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