diff options
| author | John Snow <jsnow@redhat.com> | 2015-02-05 12:41:24 -0500 |
|---|---|---|
| committer | Stefan Hajnoczi <stefanha@redhat.com> | 2015-02-16 15:07:17 +0000 |
| commit | ea41deb6022c1468de094dac12610de74220e4b0 (patch) | |
| tree | 1993897fee79be43da7c0df2364e0f076559ac50 /tests/ahci-test.c | |
| parent | 64a5a272e31c99cfb348f908d71c98d2eb83ba28 (diff) | |
libqos/ahci: add ahci command verify
Helps to verify that a command completed successfully.
Signed-off-by: John Snow <jsnow@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Message-id: 1423158090-25580-14-git-send-email-jsnow@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'tests/ahci-test.c')
| -rw-r--r-- | tests/ahci-test.c | 16 |
1 files changed, 2 insertions, 14 deletions
diff --git a/tests/ahci-test.c b/tests/ahci-test.c index 08340207b3..6e7b76533f 100644 --- a/tests/ahci-test.c +++ b/tests/ahci-test.c @@ -662,7 +662,6 @@ static void ahci_test_identify(AHCIQState *ahci) unsigned i; int rc; AHCICommand *cmd; - uint8_t cx; g_assert(ahci != NULL); @@ -702,20 +701,9 @@ static void ahci_test_identify(AHCIQState *ahci) * so we should find that there are no pending interrupts yet. */ g_assert_cmphex(ahci_px_rreg(ahci, i, AHCI_PX_IS), ==, 0); - /* Issue Command #cx via PxCI */ + /* Issue command and sanity check response. */ ahci_command_issue(ahci, cmd); - cx = ahci_command_slot(cmd); - - /* Check registers for post-command consistency */ - ahci_port_check_error(ahci, i); - /* BUG: we expect AHCI_PX_IS_DPS to be set. */ - ahci_port_check_interrupts(ahci, i, AHCI_PX_IS_DHRS | AHCI_PX_IS_PSS); - ahci_port_check_nonbusy(ahci, i, cx); - /* Investigate the CMD, assert that we read 512 bytes */ - ahci_port_check_cmd_sanity(ahci, i, cx, 512); - /* Investigate FIS responses */ - ahci_port_check_d2h_sanity(ahci, i, cx); - ahci_port_check_pio_sanity(ahci, i, cx, 512); + ahci_command_verify(ahci, cmd); /* Last, but not least: Investigate the IDENTIFY response data. */ memread(data_ptr, &buff, 512); |