Re: [Qemu-devel] [PATCH v1 22/22] petalogix-ml605: Make the LMB visible only to the CPU

[Andreas Färber]

Am 16.12.2013 14:29, schrieb Peter Maydell:
> On 16 December 2013 12:46, Andreas Färber <address@hidden> wrote:
>> TCG loads/saves should always have a CPU[Arch]State associated. Would it
>> work to always alias the system MemoryRegion again at cpu.c level with
>> lowest priority for range [0,UINT64_MAX] and let derived CPUs do per-CPU
>> MemoryRegions by adding MemoryRegions with higher priority?
> 
> I think that we should definitely not have individual CPUs
> looking at the system memory region directly.

Well, overall the whole MemoryRegion API is an abstraction, and the
system MemoryRegion is where SysBus maps its devices, so we can't get
rid of it tomorrow just yet. I had compatibility with today's code in
mind, Edgar apparently even more so, and I think we do need some
container for memory accessible to all CPUs to not force adding all
devices per CPU. The PC is still the most widely used machine in QEMU
and per-CPU devices would probably be APICs whereas I imagine "all the
rest" still in the big bucket whatever we name it.

Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg