diff mbox series

xen: cleanup IOREQ server on exit

Message ID 1564428563-1006-1-git-send-email-igor.druzhinin@citrix.com
State New
Headers show
Series xen: cleanup IOREQ server on exit | expand

Commit Message

Igor Druzhinin July 29, 2019, 7:29 p.m. UTC
Device model is supposed to destroy IOREQ server for itself.

Signed-off-by: Igor Druzhinin <igor.druzhinin@citrix.com>
---
 hw/i386/xen/xen-hvm.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Paul Durrant July 30, 2019, 8:34 a.m. UTC | #1
> -----Original Message-----
> From: Igor Druzhinin <igor.druzhinin@citrix.com>
> Sent: 29 July 2019 20:29
> To: xen-devel@lists.xenproject.org; qemu-devel@nongnu.org
> Cc: sstabellini@kernel.org; Anthony Perard <anthony.perard@citrix.com>; Paul Durrant
> <Paul.Durrant@citrix.com>; mst@redhat.com; Igor Druzhinin <igor.druzhinin@citrix.com>
> Subject: [PATCH] xen: cleanup IOREQ server on exit
> 
> Device model is supposed to destroy IOREQ server for itself.
> 
> Signed-off-by: Igor Druzhinin <igor.druzhinin@citrix.com>

Acked-by: Paul Durrant <paul.durrant@citrix.com>

> ---
>  hw/i386/xen/xen-hvm.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/hw/i386/xen/xen-hvm.c b/hw/i386/xen/xen-hvm.c
> index e8e79e0..30a5948 100644
> --- a/hw/i386/xen/xen-hvm.c
> +++ b/hw/i386/xen/xen-hvm.c
> @@ -1242,6 +1242,8 @@ static void xen_exit_notifier(Notifier *n, void *data)
>  {
>      XenIOState *state = container_of(n, XenIOState, exit);
> 
> +    xen_destroy_ioreq_server(xen_domid, state->ioservid);
> +
>      xenevtchn_close(state->xce_handle);
>      xs_daemon_close(state->xenstore);
>  }
> --
> 2.7.4
diff mbox series

Patch

diff --git a/hw/i386/xen/xen-hvm.c b/hw/i386/xen/xen-hvm.c
index e8e79e0..30a5948 100644
--- a/hw/i386/xen/xen-hvm.c
+++ b/hw/i386/xen/xen-hvm.c
@@ -1242,6 +1242,8 @@  static void xen_exit_notifier(Notifier *n, void *data)
 {
     XenIOState *state = container_of(n, XenIOState, exit);
 
+    xen_destroy_ioreq_server(xen_domid, state->ioservid);
+
     xenevtchn_close(state->xce_handle);
     xs_daemon_close(state->xenstore);
 }