Re: [Qemu-devel] [PATCH 16/16] ahci: fix sdb fis semantics

2015-06-26 Thread John Snow
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 06/26/2015 12:11 PM, Stefan Hajnoczi wrote: > On Mon, Jun 22, 2015 at 08:21:15PM -0400, John Snow wrote: >> @@ -682,19 +680,22 @@ static void ahci_write_fis_sdb(AHCIState >> *s, int port, uint32_t finished) >> >> sdb_fis->type = SATA_FIS_TYPE_S

Re: [Qemu-devel] [PATCH 16/16] ahci: fix sdb fis semantics

2015-06-26 Thread Stefan Hajnoczi
On Mon, Jun 22, 2015 at 08:21:15PM -0400, John Snow wrote: > @@ -682,19 +680,22 @@ static void ahci_write_fis_sdb(AHCIState *s, int port, > uint32_t finished) > > sdb_fis->type = SATA_FIS_TYPE_SDB; > /* Interrupt pending & Notification bit */ > -sdb_fis->flags = (ad->hba->control_r

[Qemu-devel] [PATCH 16/16] ahci: fix sdb fis semantics

2015-06-22 Thread John Snow
There are two things to fix here: The first one is subtle: the PxSACT register in the AHCI HBA has different semantics from the field it is shadowing, the ACT field in the Set Device Bits FIS. In the HBA register, PxSACT acts as a bitfield indicating outstanding NCQ commands where a set bit indic