From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45740) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YIlJ1-0001gJ-SP for qemu-devel@nongnu.org; Tue, 03 Feb 2015 16:46:52 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YIlJ0-0002cO-VQ for qemu-devel@nongnu.org; Tue, 03 Feb 2015 16:46:47 -0500 Received: from mx1.redhat.com ([209.132.183.28]:57459) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YIlJ0-0002bl-Ei for qemu-devel@nongnu.org; Tue, 03 Feb 2015 16:46:46 -0500 Received: from int-mx13.intmail.prod.int.phx2.redhat.com (int-mx13.intmail.prod.int.phx2.redhat.com [10.5.11.26]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id t13Lkj3i014783 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Tue, 3 Feb 2015 16:46:45 -0500 From: John Snow Date: Tue, 3 Feb 2015 16:46:23 -0500 Message-Id: <1422999999-25868-4-git-send-email-jsnow@redhat.com> In-Reply-To: <1422999999-25868-1-git-send-email-jsnow@redhat.com> References: <1422999999-25868-1-git-send-email-jsnow@redhat.com> Subject: [Qemu-devel] [PATCH v2 03/19] qtest/ahci: rename 'Command' to 'CommandHeader' List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: famz@redhat.com, mst@redhat.com, armbru@redhat.com, stefanha@redhat.com, pbonzini@redhat.com, John Snow The structure name is a bit of a misnomer; the structure currently named command is actually the commandheader. A future patch in this series will add an actual "Command" structure, so we'll rename it now before the rest of the functions in this series try to use it. In addition, rename the "b1" and "b2" fields to be a unified uint16_t named "flags." Signed-off-by: John Snow --- tests/ahci-test.c | 8 ++++---- tests/libqos/ahci.h | 7 +++---- 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/tests/ahci-test.c b/tests/ahci-test.c index 90647f2..d420e5f 100644 --- a/tests/ahci-test.c +++ b/tests/ahci-test.c @@ -660,7 +660,7 @@ static void ahci_test_identify(AHCIQState *ahci) RegD2HFIS *d2h = g_malloc0(0x20); RegD2HFIS *pio = g_malloc0(0x20); RegH2DFIS fis; - AHCICommand cmd; + AHCICommandHeader cmd; PRD prd; uint32_t reg, table, data_ptr; uint16_t buff[256]; @@ -703,9 +703,9 @@ static void ahci_test_identify(AHCIQState *ahci) /* Copy the existing Command #0 structure from the CLB into local memory, * and build a new command #0. */ memread(ahci->port[i].clb, &cmd, sizeof(cmd)); - cmd.b1 = 5; /* reg_h2d_fis is 5 double-words long */ - cmd.b2 = 0x04; /* clear PxTFD.STS.BSY when done */ - cmd.prdtl = cpu_to_le16(1); /* One PRD table entry. */ + cmd.flags = cpu_to_le16(5); /* reg_h2d_fis is 5 double-words long */ + cmd.flags |= cpu_to_le16(0x400); /* clear PxTFD.STS.BSY when done */ + cmd.prdtl = cpu_to_le16(1); /* One PRD table entry. */ cmd.prdbc = 0; cmd.ctba = cpu_to_le32(table); cmd.ctbau = 0; diff --git a/tests/libqos/ahci.h b/tests/libqos/ahci.h index 49ddee5..e3ada6e 100644 --- a/tests/libqos/ahci.h +++ b/tests/libqos/ahci.h @@ -329,15 +329,14 @@ typedef struct RegH2DFIS { * Command List entry structure. * The command list contains between 1-32 of these structures. */ -typedef struct AHCICommand { - uint8_t b1; - uint8_t b2; +typedef struct AHCICommandHeader { + uint16_t flags; /* Cmd-Fis-Len, PMP#, and flags. */ uint16_t prdtl; /* Phys Region Desc. Table Length */ uint32_t prdbc; /* Phys Region Desc. Byte Count */ uint32_t ctba; /* Command Table Descriptor Base Address */ uint32_t ctbau; /* '' Upper */ uint32_t res[4]; -} __attribute__((__packed__)) AHCICommand; +} __attribute__((__packed__)) AHCICommandHeader; /** * Physical Region Descriptor; pointed to by the Command List Header, -- 1.9.3