On Thu, Sep 07, 2023 at 06:04:06AM -0700, Mattias Nissler wrote: > @@ -3105,6 +3105,9 @@ void address_space_init(AddressSpace *as, MemoryRegion > *root, const char *name) > as->ioeventfds = NULL; > QTAILQ_INIT(&as->listeners); > QTAILQ_INSERT_TAIL(&address_spaces, as, address_spaces_link); > + as->bounce.in_use = false; > + qemu_mutex_init(&as->map_client_list_lock); > + QLIST_INIT(&as->map_client_list); > as->name = g_strdup(name ? name : "anonymous"); > address_space_update_topology(as); > address_space_update_ioeventfds(as);
Missing the counterpart in do_address_space_destroy()? Perhaps we should assert on having no one using the buffer, or on the client list too. Thanks, -- Peter Xu