Message ID | 20230601134434.519805-1-nks@flawful.org |
---|---|
Headers | show |
Series | misc AHCI cleanups | expand |
On Thu, Jun 1, 2023 at 9:45 AM Niklas Cassel <nks@flawful.org> wrote: > > From: Niklas Cassel <niklas.cassel@wdc.com> > > Hello John, > > Here comes some misc AHCI cleanups. > > Most are related to error handling. > > Please review. > > (I'm also working on a second series which will add support for > READ LOG EXT and READ LOG DMA EXT, but I will send that one out > once it is ready. (It might take a couple of weeks still, since > I've been a bit busy lately.)) > > > Changes since v1: > -Picked up Reviewed-by tags. > (I did not convert your ACK to explicit Acked-by tags, since I assume > that the patches will go via your tree). Guess so! I haven't been involved with IDE for a minute so I left the ACKs in case I wandered off to signify that I hadn't reviewed them thoroughly, but they *looked* good. Since I haven't wandered off, guess I will actually take this and send an MR. I'll try to do this Friday, June 2nd. Thanks again for the very detailed commit messages, which make this easy. :) --js > -Rebased on master in order to fix a conflict in patch > "hw/ide/ahci: simplify and document PxCI handling". > -Dropped patch "hw/ide/ahci: trigger either error IRQ or regular IRQ, not both" > for now, as it caused a boot time regression in SeaBIOS. > This appears to be a bug in SeaBIOS, for more info see: > https://mail.coreboot.org/hyperkitty/list/seabios@seabios.org/thread/RIHV3FZ4EVMAJA4TEDPASKNYV7V72O4C/ > I will resend the QEMU patch separately once the SeaBIOS patch has been > merged, and once QEMU has updated to a SeaBIOS tag that includes the fix. > > > 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 on when processing NCQ command > hw/ide/ahci: simplify and document PxCI handling > hw/ide/ahci: PxCI should not get cleared when ERR_STAT is set > hw/ide/ahci: PxSACT and PxCI is cleared when PxCMD.ST is cleared > 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 +- > 2 files changed, 81 insertions(+), 33 deletions(-) > > -- > 2.40.1 >
From: Niklas Cassel <niklas.cassel@wdc.com> Hello John, Here comes some misc AHCI cleanups. Most are related to error handling. Please review. (I'm also working on a second series which will add support for READ LOG EXT and READ LOG DMA EXT, but I will send that one out once it is ready. (It might take a couple of weeks still, since I've been a bit busy lately.)) Changes since v1: -Picked up Reviewed-by tags. (I did not convert your ACK to explicit Acked-by tags, since I assume that the patches will go via your tree). -Rebased on master in order to fix a conflict in patch "hw/ide/ahci: simplify and document PxCI handling". -Dropped patch "hw/ide/ahci: trigger either error IRQ or regular IRQ, not both" for now, as it caused a boot time regression in SeaBIOS. This appears to be a bug in SeaBIOS, for more info see: https://mail.coreboot.org/hyperkitty/list/seabios@seabios.org/thread/RIHV3FZ4EVMAJA4TEDPASKNYV7V72O4C/ I will resend the QEMU patch separately once the SeaBIOS patch has been merged, and once QEMU has updated to a SeaBIOS tag that includes the fix. 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 on when processing NCQ command hw/ide/ahci: simplify and document PxCI handling hw/ide/ahci: PxCI should not get cleared when ERR_STAT is set hw/ide/ahci: PxSACT and PxCI is cleared when PxCMD.ST is cleared 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 +- 2 files changed, 81 insertions(+), 33 deletions(-)