From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:55686) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RPsXW-0007NJ-D9 for qemu-devel@nongnu.org; Mon, 14 Nov 2011 04:09:19 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RPsXV-00047O-Bj for qemu-devel@nongnu.org; Mon, 14 Nov 2011 04:09:18 -0500 Received: from mx1.redhat.com ([209.132.183.28]:28295) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RPsXV-000477-3O for qemu-devel@nongnu.org; Mon, 14 Nov 2011 04:09:17 -0500 Message-ID: <4EC0DB75.7070306@redhat.com> Date: Mon, 14 Nov 2011 10:12:21 +0100 From: Kevin Wolf MIME-Version: 1.0 References: <1321113420-3252-1-git-send-email-aliguori@us.ibm.com> In-Reply-To: <1321113420-3252-1-git-send-email-aliguori@us.ibm.com> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 1/7] migrate: add migration blockers List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: Lucas Meneghel Rodrigues , Stefan Hajnoczi , Juan Quintela , qemu-devel@nongnu.org, Avi Kivity Am 12.11.2011 16:56, schrieb Anthony Liguori: > This lets different subsystems register an Error that is thrown whenever > migration is attempted. This works nicely because it gracefully supports > things like hotplug. > > Right now, if multiple errors are registered, only one of them is reported. > I expect that for 1.1, we'll extend query-migrate to return all of the reasons > why migration is disabled at any given point in time. > > Signed-off-by: Anthony Liguori > Signed-off-by: Anthony Liguori > --- > migration.c | 18 ++++++++++++++++++ > migration.h | 15 +++++++++++++++ > 2 files changed, 33 insertions(+), 0 deletions(-) There doesn't seem to be a patch 0/n, so it goes here... The series looks good in general. I commented on two patches, and there are a few more image formats that must block migration (basically everything that is writable, I would be surprised if there was one that didn't cache anything). Kevin