[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 0/3] Cleanup IOREQ server on exit


  • To: <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Maximilian Heyne <mheyne@xxxxxxxxx>
  • Date: Tue, 7 Apr 2020 11:16:09 +0200
  • Cc: Ian Jackson <ian.jackson@xxxxxxxxxx>, Paul Durrant <paul@xxxxxxx>
  • Delivery-date: Tue, 07 Apr 2020 09:16:42 +0000
  • Ironport-sdr: YsI3VwZCBvwU2QBHeIXbAZSKOJ8xt1EWbR6/hbqPjhOWoL2OIWg6+elR5bOaLMoZSuauYBKl9m Fxx0ocjm7Pug==
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

Could someone please have a look at this patch? It solves an actual issue:
Try soft-reset with qemu-xen-traditional and it will fail.

On 3/13/20 1:33 PM, Maximilian Heyne wrote:
Following up on commit 9c0eed61 ("qemu-trad: stop using the default IOREQ
server"), clean up the IOREQ server on exit. This fixes a bug with soft-reset
that shows up as "bind interdomain ioctl error 22" because the event channels
were not closed at the soft-reset and can't be bound again.

For this I used the exit notifiers from QEMU that I backported together with the
required generic notifier lists.

Anthony Liguori (1):
   Add support for generic notifier lists

Gerd Hoffmann (1):
   Add exit notifiers.

Maximilian Heyne (1):
   xen: cleanup IOREQ server on exit

  Makefile            |  1 +
  hw/xen_machine_fv.c | 11 +++++++++++
  notify.c            | 39 +++++++++++++++++++++++++++++++++++++++
  notify.h            | 43 +++++++++++++++++++++++++++++++++++++++++++
  sys-queue.h         |  5 +++++
  sysemu.h            |  5 +++++
  vl.c                | 20 ++++++++++++++++++++
  7 files changed, 124 insertions(+)
  create mode 100644 notify.c
  create mode 100644 notify.h




Amazon Development Center Germany GmbH
Krausenstr. 38
10117 Berlin
Geschaeftsfuehrung: Christian Schlaeger, Jonathan Weiss
Eingetragen am Amtsgericht Charlottenburg unter HRB 149173 B
Sitz: Berlin
Ust-ID: DE 289 237 879



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.