From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55328) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zvvvm-0002ff-4l for qemu-devel@nongnu.org; Mon, 09 Nov 2015 18:32:59 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zvvvl-0000WG-3G for qemu-devel@nongnu.org; Mon, 09 Nov 2015 18:32:58 -0500 From: John Snow Date: Mon, 9 Nov 2015 18:32:42 -0500 Message-Id: <1447111968-15268-4-git-send-email-jsnow@redhat.com> In-Reply-To: <1447111968-15268-1-git-send-email-jsnow@redhat.com> References: <1447111968-15268-1-git-send-email-jsnow@redhat.com> Subject: [Qemu-devel] [PATCH 3/9] libqos/ahci: ATAPI identify List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-block@nongnu.org Cc: John Snow , qemu-devel@nongnu.org We need to say "hello!" to our ATAPI friends in a slightly different manner. Signed-off-by: John Snow --- tests/ahci-test.c | 8 +++++++- tests/libqos/ahci.c | 5 +++++ tests/libqos/ahci.h | 1 + 3 files changed, 13 insertions(+), 1 deletion(-) diff --git a/tests/ahci-test.c b/tests/ahci-test.c index 8370fa3..58d0545 100644 --- a/tests/ahci-test.c +++ b/tests/ahci-test.c @@ -204,6 +204,7 @@ static AHCIQState *ahci_boot_and_enable(const char *cli, ...) va_list ap; uint16_t buff[256]; uint8_t port; + uint8_t hello; if (cli) { va_start(ap, cli); @@ -218,7 +219,12 @@ static AHCIQState *ahci_boot_and_enable(const char *cli, ...) /* Initialize test device */ port = ahci_port_select(ahci); ahci_port_clear(ahci, port); - ahci_io(ahci, port, CMD_IDENTIFY, &buff, sizeof(buff), 0); + if (is_atapi(ahci, port)) { + hello = CMD_PACKET_ID; + } else { + hello = CMD_IDENTIFY; + } + ahci_io(ahci, port, hello, &buff, sizeof(buff), 0); return ahci; } diff --git a/tests/libqos/ahci.c b/tests/libqos/ahci.c index 59bf893..81edf34 100644 --- a/tests/libqos/ahci.c +++ b/tests/libqos/ahci.c @@ -114,6 +114,11 @@ void ahci_free(AHCIQState *ahci, uint64_t addr) qfree(ahci->parent, addr); } +bool is_atapi(AHCIQState *ahci, uint8_t port) +{ + return ahci_px_rreg(ahci, port, AHCI_PX_SIG) == AHCI_SIGNATURE_CDROM; +} + /** * Locate, verify, and return a handle to the AHCI device. */ diff --git a/tests/libqos/ahci.h b/tests/libqos/ahci.h index 9ffd415..705fbd6 100644 --- a/tests/libqos/ahci.h +++ b/tests/libqos/ahci.h @@ -596,5 +596,6 @@ void ahci_command_adjust(AHCICommand *cmd, uint64_t lba_sect, uint64_t gbuffer, /* Command Misc */ uint8_t ahci_command_slot(AHCICommand *cmd); +bool is_atapi(AHCIQState *ahci, uint8_t port); #endif -- 2.4.3