Re: [Qemu-devel] [RFC v1 1/2] arm: Add the cortex-a9 CPU to the a9mpcore device

[Stefan Hajnoczi <stefanha@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1246 bytes --]

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.maydell@linaro.org> wrote:
> > On 10 June 2014 02:32, Alistair Francis <alistair.francis@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

[-- Attachment #2: Type: application/pgp-signature, Size: 473 bytes --]