On 27/04/2015 13:19, Gal Hammer wrote: > +static void vmgenid_init(Object *obj) > +{ > + SysBusDevice *sbd = SYS_BUS_DEVICE(obj); > + VmGenIdState *s = VMGENID(obj); > + > + memory_region_init_io(&s->iomem, obj, &vmgenid_ram_ops, s, "vgid", 16);
This is a small problem. The spec says the memory of the VMGENID should not be uncacheable, but MMIO _should_ be uncacheable. This is running on a VM so we maybe not care, but it's worth pointing it out. Paolo > + sysbus_init_mmio(sbd, &s->iomem); > + > + object_property_add_str(obj, PROPERTY_UUID, NULL, vmgenid_set_uuid, > NULL); > +} > +