On Thu, 2013-10-03 at 20:46 +0200, Paolo Bonzini wrote: > Il 03/10/2013 17:39, Alex Williamson ha scritto: > > +static const MemoryRegionOps vfio_rom_ops = { > > + .read = vfio_rom_read, > > + .endianness = DEVICE_LITTLE_ENDIAN, > > +}; > > + > > I think you need to define a write callback too (unless you're sure for > some other reason that the area will never be loaded). Ok, I was under the impression that the memory API would handle lack of an accessor as not supporting that access. I can add a follow on patch to make a dummy write function if my assumption isn't true. Thanks, Alex -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html