All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Jackson <ian.jackson@citrix.com>
To: Maximilian Heyne <mheyne@amazon.de>
Cc: xen-devel@lists.xenproject.org, Paul Durrant <paul@xen.org>
Subject: Re: [PATCH 0/3] Cleanup IOREQ server on exit
Date: Fri, 24 Apr 2020 15:50:12 +0100	[thread overview]
Message-ID: <24226.64676.86981.838902@mariner.uk.xensource.com> (raw)
In-Reply-To: <6f476505-5e85-8a8a-d6d7-db56ea921637@amazon.de>

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@eu.citrix.com>

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.


      parent reply	other threads:[~2020-04-24 14:50 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-13 12:33 [Xen-devel] [PATCH 0/3] Cleanup IOREQ server on exit Maximilian Heyne
2020-03-13 12:33 ` [Xen-devel] [PATCH 1/3] Add support for generic notifier lists Maximilian Heyne
2020-04-08 14:58   ` Paul Durrant
2020-03-13 12:33 ` [Xen-devel] [PATCH 2/3] Add exit notifiers Maximilian Heyne
2020-04-08 14:57   ` Paul Durrant
2020-03-13 12:33 ` [Xen-devel] [PATCH 3/3] xen: cleanup IOREQ server on exit Maximilian Heyne
2020-04-08 14:56   ` Paul Durrant
2020-04-07  9:16 ` [PATCH 0/3] Cleanup " Maximilian Heyne
2020-04-08  8:40   ` Paul Durrant
2020-04-21 16:35   ` Paul Durrant
2020-04-24 14:50   ` Ian Jackson [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=24226.64676.86981.838902@mariner.uk.xensource.com \
    --to=ian.jackson@citrix.com \
    --cc=mheyne@amazon.de \
    --cc=paul@xen.org \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.