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

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


  • To: Maximilian Heyne <mheyne@xxxxxxxxx>
  • From: Ian Jackson <ian.jackson@xxxxxxxxxx>
  • Date: Fri, 24 Apr 2020 15:50:12 +0100
  • Authentication-results: esa3.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none; spf=None smtp.pra=ian.jackson@xxxxxxxxxx; spf=Pass smtp.mailfrom=Ian.Jackson@xxxxxxxxxx; spf=None smtp.helo=postmaster@xxxxxxxxxxxxxxx
  • Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx, Paul Durrant <paul@xxxxxxx>
  • Delivery-date: Fri, 24 Apr 2020 14:50:26 +0000
  • Ironport-sdr: BWWAQSqWzSMmY6RWdU7xibLguAYzDTKSKBql1fkj2hNdp+Ec+MHlB9qXZJdptaXuzWLZOmrCwS BYdARmu9mQ2caWsjJFYpuM+OQCeGkIZdk4UTk7Gplna9UiY2Lu2EKnPNpkNhj12c4SkFE0wAou i7HqydVCcmIi2myZVtYOdp30LopaY4BRGpo1dmhbNBhWQF5t0v7dy8bff9KIAGO+OuNXKz2Mmn jGwvxKyKvstY2RrD+l5KISVCVNB1r/FGg94ncrQZvE1C2EqhRKPIw4pSXsDE+R/QvIvFn98zbT GXk=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

Maximilian Heyne writes ("Re: [PATCH 0/3] Cleanup IOREQ server on exit"):
> 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.

Thanks.  I reviewed this.

qemu is in deep freeze but the changes looked correct and are indeed
solving a regression.  I convinced myself that they were appropriately
low risk, so

Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>

for all three and I have pushed them.

In theory a backport might be appropriate since this is a bugfix but
my inclination is to leave existing releases where they are, since
anyone using qemu-trad probably wants things super-stable.  Contrary
opinions welcome.

It has been a very long time since I did an update of qemu trad so it
is possible that I have mangled the process somehow.  We will see I
guess...

Thanks also to Paul for chasing me about this.

Regards,
Ian.



 


Rackspace

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