OSDN Git Service

hw/xen: don't clear map_track[] in xen_gnttab_reset()
authorDavid Woodhouse <dwmw@amazon.co.uk>
Wed, 18 Oct 2023 12:31:20 +0000 (13:31 +0100)
committerDavid Woodhouse <dwmw@amazon.co.uk>
Mon, 6 Nov 2023 10:03:45 +0000 (10:03 +0000)
commit3de75ed352411899dbc9222e82fe164890c77e78
tree2f4497c55b690a9df3bdb9b162439176a1cb5c47
parent18e83f28bf39ffd2784aeb2e4e229096a86d349b
hw/xen: don't clear map_track[] in xen_gnttab_reset()

The refcounts actually correspond to 'active_ref' structures stored in a
GHashTable per "user" on the backend side (mostly, per XenDevice).

If we zero map_track[] on reset, then when the backend drivers get torn
down and release their mapping we hit the assert(s->map_track[ref] != 0)
in gnt_unref().

So leave them in place. Each backend driver will disconnect and reconnect
as the guest comes back up again and reconnects, and it all works out OK
in the end as the old refs get dropped.

Cc: qemu-stable@nongnu.org
Fixes: de26b2619789 ("hw/xen: Implement soft reset for emulated gnttab")
Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
Reviewed-by: Paul Durrant <paul@xen.org>
hw/i386/kvm/xen_gnttab.c