Skip to content

AHCI migration

Kevin Wolf qemu-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v4 Latest3 patches 4 repliesIn review

Patches in v4 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3ahci: Remove unused AHCIDevice fieldshw/ide/ahci.c hw/ide/ahci.h+2 -80 replies
2/3ahci: Change data types in preparation for migrationhw/ide/ahci.c hw/ide/ahci.h+8 -81 reply
3/3ahci: Add migration supporthw/ide/ahci.c hw/ide/ahci.h hw/ide/ich.c+99 -51 reply

Reviews trailers per patch

All 3 patches have a Reviewed-by or Acked-by.

PatchJuan Quintela
1/3R-b
2/3R-b
3/3R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v4 thread

Cover letter [Qemu-devel] [PATCH v4 0/3] AHCI migration

Let's get Jason's patches merged while they still apply. I addressed the review comments (mostly my own) that came up during the v3 review, otherwise this is unchanged.

Please note that in my tests it didn't work entirely reliably and I saw guest lockups and kernel crashes in like one of ten cases. I confirmed that the same kind of bugs occurs with v3 of the series, so my changes are likely innocent. Someone will have to debug this some more, but what I did took about the time that I'm willing to spend on it right now.

Jason Baron (2):
ahci: Remove unused AHCIDevice fields
ahci: Add migration support

Kevin Wolf (1):
ahci: Change data types in preparation for migration

 hw/ide/ahci.c |   97 ++++++++++++++++++++++++++++++++++++++++++++++++++------
 hw/ide/ahci.h |   20 ++++++++---
 hw/ide/ich.c  |   13 +++++--
 3 files changed, 109 insertions(+), 21 deletions(-)
Signature · 1 line
1.7.6.5