On Mon, Jun 16, 2014 at 05:46:24PM +1000, Peter Crosthwaite wrote: > On Mon, Jun 16, 2014 at 5:40 PM, Peter Maydell <peter.mayd...@linaro.org> > wrote: > > On 10 June 2014 02:32, Alistair Francis <alistair.fran...@xilinx.com> wrote: > >> This patch adds the Cortex-A9 ARM CPU to the A9MPCore. > > > > I think this is in general the right way to go. > > > >> + /* Properties for the A9 CPU */ > >> + DEFINE_PROP_UINT32("midr", A9MPPrivState, midr, 0), > >> + DEFINE_PROP_UINT64("reset-cbar", A9MPPrivState, reset_cbar, 0), > > > > It seems a bit error-prone if every container object has to manually > > mirror the properties of the individual CPUs out like this; maybe > > there's a better way we could come up with? > > > > Paolo, weren't you looking at passthrough properties for the > > virtio devices? > > > > Stefan added support for QDEV property aliasing. > > http://lists.gnu.org/archive/html/qemu-devel/2014-05/msg06489.html > > The bigger issue though is how do you do an N:1 mapping. The container > should only have 1 "midr" prop, but it should mirror to all contained > CPUs. Should we add multiplicity to the aliasing feature?
If we'll need 1:N alias properties in other places too. Stefan
pgpv9ITmywk38.pgp
Description: PGP signature