On Fri, Jun 09, 2023 at 04:08:36PM +0200, Niklas Cassel wrote: > From: Niklas Cassel <niklas.cas...@wdc.com> > > Hello John, > > Here comes some misc AHCI cleanups. > > Most are related to error handling. > > Please review. > > Changes since v2: > -Squashed in the test commits that were sent out as a separate series into > the patch "hw/ide/ahci: PxCI should not get cleared when ERR_STAT is set", > and reordered some of the patches, such that each and every commit passes > the ahci test suite as a separate unit. This way it will be possible to > perform a git bisect without seeing any failures in the ahci test suite. > > > Kind regards, > Niklas > > Niklas Cassel (8): > hw/ide/ahci: remove stray backslash > hw/ide/core: set ERR_STAT in unsupported command completion > hw/ide/ahci: write D2H FIS when processing NCQ command > hw/ide/ahci: simplify and document PxCI handling > hw/ide/ahci: PxSACT and PxCI is cleared when PxCMD.ST is cleared > hw/ide/ahci: PxCI should not get cleared when ERR_STAT is set > hw/ide/ahci: fix ahci_write_fis_sdb() > hw/ide/ahci: fix broken SError handling > > hw/ide/ahci.c | 112 +++++++++++++++++++++++++++----------- > hw/ide/core.c | 2 +- > tests/qtest/libqos/ahci.c | 106 +++++++++++++++++++++++++++--------- > tests/qtest/libqos/ahci.h | 8 +-- > 4 files changed, 164 insertions(+), 64 deletions(-) > > -- > 2.40.1 > >
Hello Philippe, Considering that you picked up my patch, "hw/ide/ahci: remove stray backslash" (patch 1/8 in this series), and since John seems to have gone silent for 40+ days, could you please consider taking this series through your misc tree? The series still applies, you just need to skip patch 1/8 (which has already landed in qemu master). Kind regards, Niklas
