On Fri, Dec 14, 2012 at 9:34 PM, Blue Swirl <blauwir...@gmail.com> wrote: > On Fri, Dec 14, 2012 at 6:00 PM, Stefan Hajnoczi <stefa...@redhat.com> wrote: >> +typedef struct { >> + void *host_addr; >> + hwaddr guest_addr; >> + uint64_t size; >> + bool readonly; >> +} HostmemRegion; > > This should be HostMemRegion. > >> + >> +typedef struct { >> + /* The listener is invoked when regions change and a new list of >> regions is >> + * built up completely before they are installed. >> + */ >> + MemoryListener listener; >> + HostmemRegion *new_regions; >> + size_t num_new_regions; >> + >> + /* Current regions are accessed from multiple threads either to lookup >> + * addresses or to install a new list of regions. The lock protects the >> + * pointer and the regions. >> + */ >> + QemuMutex current_regions_lock; >> + HostmemRegion *current_regions; >> + size_t num_current_regions; >> +} Hostmem; > > And this HostMem.
Okay, will fix both. Stefan