From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50269) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YPvi5-0008Dk-1D for qemu-devel@nongnu.org; Mon, 23 Feb 2015 11:18:21 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YPvhz-00052i-Pf for qemu-devel@nongnu.org; Mon, 23 Feb 2015 11:18:16 -0500 Received: from mx1.redhat.com ([209.132.183.28]:52136) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YPvhz-00052R-Dt for qemu-devel@nongnu.org; Mon, 23 Feb 2015 11:18:11 -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 t1NGIALx028059 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Mon, 23 Feb 2015 11:18:10 -0500 From: John Snow Date: Mon, 23 Feb 2015 11:17:49 -0500 Message-Id: <1424708286-16483-1-git-send-email-jsnow@redhat.com> Subject: [Qemu-devel] [PATCH v4 00/17] ide: rerror/werror migration fixes for IDE/ISA and AHCI 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 This series fixes rerror/werror support for IDE/ISA and implements it in a migratable way for AHCI. This series also fixes AHCI migration. This series was written mostly by Paolo to unify the restart mechanics of IDE/ISA and IDE/BMDMA, moving much of the restart logic into common code. Many of the earlier patches make more sense considering the end goal of eliminating BMDMA specific restart code to be shared with ISA and AHCI codepaths. Migration for halted commands is fixed for ISA and AHCI. As a consequence, operations halted via rerror=stop or werror=stop should be able to be successfully migrated and resumed when using ISA, PCI, or AHCI. This series includes tests for ISA and PCI/BMDMA, but does not yet include tests for AHCI, which require some more qtest work to be upstreamed first. Regardless, the AHCI tests have been written and can be observed at: https://github.com/jnsnow/qemu/commits/ahci-devel See "ahci: add migrate dma test" and "ahci-test: add flush migrate test" for the WIP versions of the AHCI test that I used to exercise this patchset. V4: - Rebased. - Return -1 instead of assert(). V3: - Rebased, including Dave Gilbert's ATAPI migration workaround - Some rephrasing for the comment in patch #15 - Added assertion that the busy_slot variable is within range John Snow (3): ahci: Migrate IDEStatus ahci: Recompute cur_cmd on migrate post load qtest/ide: Test flush / retry for ISA and PCI Paolo Bonzini (14): ide: start extracting ide_restart_dma out of bmdma_restart_dma ide: prepare to move restart to common code ide: introduce ide_register_restart_cb ide: do not use BMDMA in restart callback ide: pass IDEBus to the restart_cb ide: move restart callback to common code ide: remove restart_cb callback ide: replace set_unit callback with more IDEBus state ide: place initial state of the current request to IDEBus ide: migrate initial request state via IDEBus ide: commonize io_buffer_index initialization ide: make more functions static ide: support PIO restart for the ISA controller ahci: add support for restarting non-queued commands hw/ide/ahci.c | 43 +++++++++++--------- hw/ide/atapi.c | 3 +- hw/ide/cmd646.c | 3 +- hw/ide/core.c | 118 ++++++++++++++++++++++++++++++++++++++++++++++++------ hw/ide/internal.h | 16 +++++--- hw/ide/isa.c | 3 +- hw/ide/macio.c | 6 --- hw/ide/pci.c | 109 +++++++------------------------------------------ hw/ide/pci.h | 12 +++--- hw/ide/piix.c | 3 +- hw/ide/via.c | 3 +- tests/ide-test.c | 20 ++++++--- 12 files changed, 180 insertions(+), 159 deletions(-) -- 1.9.3