On 24/07/17 23:00, Cédric Le Goater wrote: >>> +#include "qemu/osdep.h" >>> +#include "qemu/log.h" >>> +#include "qapi/error.h" >>> +#include "target/ppc/cpu.h" >>> +#include "sysemu/cpus.h" >>> +#include "sysemu/dma.h" >>> +#include "monitor/monitor.h" >>> +#include "hw/ppc/xive.h" >>> + >>> +#include "xive-internal.h" >>> + >>> +/* >>> + * Main XIVE object >> >> As with XICs, does it really make sense for there to be a "main" XIVE >> object, or should be an interface attached to the machine? > > yes. There are internal tables which are very specific to the controller > and I don't think they belong to the machine.
These tables belong to a CPU chip (die?) and we do not emulate these now (machines and cores are the closest) and since we do not want (do we?) to treat a core as a chip, the machine is the most obvious owner for these tables. -- Alexey