Am 16.12.2013 14:29, schrieb Peter Maydell: > On 16 December 2013 12:46, Andreas Färber <afaer...@suse.de> 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
