Re: [Qemu-devel] [Qemu-block] [PATCH v3 0/9] ahci: enable migration

[John Snow <jsnow@redhat.com>]

On 05/04/2015 08:29 AM, Kevin Wolf wrote:
> Am 30.04.2015 um 20:07 hat John Snow geschrieben:
>> The day we all feared is here, and I am proposing we allow the migration
>> of the AHCI device tentatively for the 2.4 development window.
>>
>> There are some more NCQ migration tests are needed, but I felt that it was
>> important to get migration enabled as close to the start of the 2.4
>> development window as possible.
>>
>> If the NCQ patches don't pan out by the time the 2.4 freeze occurs, we can
>> revert the migration boolean and add a conditional around the ahci tests
>> that rely on the migration feature being enabled.
>>
>> I am justifying this checkin based on a series of ping-pong
>> migration tests I ran under heavy load (using google's stressapptest)
>> and saw over 300 successful migrations without a single failure.
>>
>> This series does a few things:
>> (1) Add migration facilities to libqos
>> (2) Enable AHCI and ICH9 migration
>> (3) Add a series of migration tests to ahci-test
>
> Reviewed-by: Kevin Wolf <kwolf@redhat.com>
>
> I think besides the NCQ tests, we'll definitely also want to test
> migration with READ DMA in flight (this series tests only WRITE DMA and
> FLUSH CACHE). Probably also discard.
>

Added to the todo list, thanks.

> Nice to have, but less important, would be the other variants that
> exist, like the EXT version of each command, and READ/WRITE SECTOR.
>
> But all of that can be added in a follow-up series, it's not a reason to
> hold up what's already there.
>
> Kevin
>

Thanks!

--js