From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54752) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Y13Xo-00088C-86 for qemu-devel@nongnu.org; Tue, 16 Dec 2014 20:36:58 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Y13Xf-0002Y9-Gc for qemu-devel@nongnu.org; Tue, 16 Dec 2014 20:36:52 -0500 Received: from mx1.redhat.com ([209.132.183.28]:52915) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Y13Xf-0002Y0-9K for qemu-devel@nongnu.org; Tue, 16 Dec 2014 20:36:43 -0500 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id sBH1agoN002480 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Tue, 16 Dec 2014 20:36:42 -0500 From: John Snow Date: Tue, 16 Dec 2014 20:36:06 -0500 Message-Id: <1418780167-16231-17-git-send-email-jsnow@redhat.com> In-Reply-To: <1418780167-16231-1-git-send-email-jsnow@redhat.com> References: <1418780167-16231-1-git-send-email-jsnow@redhat.com> Subject: [Qemu-devel] [PATCH v2 16/17] ahci: Recompute cur_cmd on migrate post load List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: kwolf@redhat.com, mst@redhat.com, armbru@redhat.com, mreitz@redhat.com, stefanha@redhat.com, pbonzini@redhat.com, John Snow When the AHCI HBA device is migrated, all of the information that led to the request being created is stored in the AHCIDevice structures, except for pointers into guest data where return information needs to be stored. The "cur_cmd" field is usually responsible for this. To rebuild the cur_cmd pointer post-migration, we can utilize the busy_slot index to figure out where the command header we are still processing is. This allows a machine in a halted state from rerror=stop or werror=stop to be migrated and resume operations without issue. Signed-off-by: John Snow --- hw/ide/ahci.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/hw/ide/ahci.c b/hw/ide/ahci.c index c153228..8078d3e 100644 --- a/hw/ide/ahci.c +++ b/hw/ide/ahci.c @@ -1373,6 +1373,10 @@ static int ahci_state_post_load(void *opaque, int version_id) */ if (ad->busy_slot == -1) { check_cmd(s, i); + } else { + /* We are in the middle of a command, and may need to access + * the command header in guest memory again. */ + ad->cur_cmd = &((AHCICmdHdr *)ad->lst)[ad->busy_slot]; } } -- 1.9.3