Skip to content

ahci: fis decomposition improvements

John Snow qemu-devel 1 version Applied

Status from this message: Thanks, applied to my block tree:

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest5 patches 2 repliesApplied

Patches in v1 Applied

#Subject and filesSizeRepliesApplied
1/5ahci: add is_ncq predicate helperhw/ide/ahci.c hw/ide/ahci.h+27 -40 replies
2/5ahci: Fix FIS decompositionhw/ide/ahci.c+26 -380 replies
3/5ahci: Reorder error cases in handle_cmdhw/ide/ahci.c+14 -150 replies
4/5ahci: Check cmd_fis[1] more explicitlyhw/ide/ahci.c+12 -110 replies
5/5ahci: factor out FIS decomposition from handle_cmdhw/ide/ahci.c+86 -830 replies

Reviews trailers per patch

None of the 5 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchPaolo Bonzini
SeriesR-b
1/5
2/5
3/5
4/5
5/5

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

Read the v1 thread

Cover letter [Qemu-devel] [PATCH 0/5] ahci: fis decomposition improvements

This patch series may require, and is based on another AHCI series pending on-list:
http://lists.gnu.org/archive/html/qemu-devel/2014-10/msg04143.html

This series aims to improve the way that handle_cmd reads and behaves in interpreting the FIS packets received by the AHCI HBA. This series used to be part of a larger RFC, but has since been pared down to a more focused series.

Previous RFC:
http://lists.gnu.org/archive/html/qemu-devel/2014-09/msg02694.html

Patch 1 is a subset of the previously posted "AHCI: Rename NCQFIS structure fields" patch. This subset is just aimed at improving the readability of the FIS decomposition in handle_cmd for now.

Patch 2 is nearly the same as "AHCI: Fix FIS decomposition" which has yet to be reviewed.

Patches 3, 4, and 5 have been inspected previously, and should be identical.
"ide/ahci: Reorder error cases in handle_cmd" "ahci: Check cmd_fis[1] more explicitly"
"ahci: factor out FIS decomposition"

John Snow (5):
ahci: add is_ncq predicate helper
AHCI: Fix FIS decomposition
ahci: Reorder error cases in handle_cmd
ahci: Check cmd_fis[1] more explicitly
ahci: factor out FIS decomposition from handle_cmd

 hw/ide/ahci.c | 233 ++++++++++++++++++++++++++++++----------------------------
 hw/ide/ahci.h |   3 +
 2 files changed, 125 insertions(+), 111 deletions(-)
Signature · 1 line
1.9.3