All of lore.kernel.org
 help / color / mirror / Atom feed
From: John Snow <jsnow@redhat.com>
To: Stefan Hajnoczi <stefanha@gmail.com>
Cc: kwolf@redhat.com, mst@redhat.com, qemu-devel@nongnu.org,
	armbru@redhat.com, mreitz@redhat.com, stefanha@redhat.com,
	pbonzini@redhat.com
Subject: Re: [Qemu-devel] [PATCH v4 00/17] ide: rerror/werror migration fixes for IDE/ISA and AHCI
Date: Wed, 25 Feb 2015 15:24:25 -0500	[thread overview]
Message-ID: <54EE2F79.9090404@redhat.com> (raw)
In-Reply-To: <20150225151528.GC2922@stefanha-thinkpad.redhat.com>



On 02/25/2015 10:15 AM, Stefan Hajnoczi wrote:
> On Mon, Feb 23, 2015 at 11:17:49AM -0500, John Snow wrote:
>> 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.
>
> Thanks, applied to my block tree:
> https://github.com/stefanha/qemu/commits/block
>
> Stefan
>

Ping: I don't think you did; the patches aren't present there. Is there 
a conflict I need to fix?

Thanks,
--John

  reply	other threads:[~2015-02-25 20:24 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-23 16:17 [Qemu-devel] [PATCH v4 00/17] ide: rerror/werror migration fixes for IDE/ISA and AHCI John Snow
2015-02-23 16:17 ` [Qemu-devel] [PATCH v4 01/17] ide: start extracting ide_restart_dma out of bmdma_restart_dma John Snow
2015-02-23 16:17 ` [Qemu-devel] [PATCH v4 02/17] ide: prepare to move restart to common code John Snow
2015-02-23 16:17 ` [Qemu-devel] [PATCH v4 03/17] ide: introduce ide_register_restart_cb John Snow
2015-02-23 16:17 ` [Qemu-devel] [PATCH v4 04/17] ide: do not use BMDMA in restart callback John Snow
2015-02-23 16:17 ` [Qemu-devel] [PATCH v4 05/17] ide: pass IDEBus to the restart_cb John Snow
2015-02-23 16:17 ` [Qemu-devel] [PATCH v4 06/17] ide: move restart callback to common code John Snow
2015-02-23 16:17 ` [Qemu-devel] [PATCH v4 07/17] ide: remove restart_cb callback John Snow
2015-02-23 16:17 ` [Qemu-devel] [PATCH v4 08/17] ide: replace set_unit callback with more IDEBus state John Snow
2015-02-23 16:17 ` [Qemu-devel] [PATCH v4 09/17] ide: place initial state of the current request to IDEBus John Snow
2015-02-23 16:17 ` [Qemu-devel] [PATCH v4 10/17] ide: migrate initial request state via IDEBus John Snow
2015-02-23 16:18 ` [Qemu-devel] [PATCH v4 11/17] ide: commonize io_buffer_index initialization John Snow
2015-02-23 16:18 ` [Qemu-devel] [PATCH v4 12/17] ide: make more functions static John Snow
2015-02-23 16:18 ` [Qemu-devel] [PATCH v4 13/17] ide: support PIO restart for the ISA controller John Snow
2015-02-23 16:18 ` [Qemu-devel] [PATCH v4 14/17] ahci: Migrate IDEStatus John Snow
2015-03-13  6:01   ` Amit Shah
2015-03-13  8:21     ` Kevin Wolf
2015-03-13  8:43       ` Amit Shah
2015-02-23 16:18 ` [Qemu-devel] [PATCH v4 15/17] ahci: add support for restarting non-queued commands John Snow
2015-02-23 16:18 ` [Qemu-devel] [PATCH v4 16/17] ahci: Recompute cur_cmd on migrate post load John Snow
2015-02-23 16:18 ` [Qemu-devel] [PATCH v4 17/17] qtest/ide: Test flush / retry for ISA and PCI John Snow
2015-02-25 15:15 ` [Qemu-devel] [PATCH v4 00/17] ide: rerror/werror migration fixes for IDE/ISA and AHCI Stefan Hajnoczi
2015-02-25 20:24   ` John Snow [this message]
2015-02-26 11:02     ` Stefan Hajnoczi

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=54EE2F79.9090404@redhat.com \
    --to=jsnow@redhat.com \
    --cc=armbru@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=mreitz@redhat.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@gmail.com \
    --cc=stefanha@redhat.com \
    /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.