[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 00/30] AHCI test suite framework
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v2 00/30] AHCI test suite framework |
Date: |
Mon, 11 Aug 2014 17:14:51 +0100 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Mon, Aug 04, 2014 at 05:11:01PM -0400, John Snow wrote:
> This patch series introduces a number of small fixes and tweaks to
> help support an AHCI test suite that in the future I hope to expand
> to a fuller regression suite to help guide the development of the
> AHCI device support under, in particular, the Q35 machine type in QEMU.
>
> Paolo Bonzini has contributed a number of cleanup and refactoring patches
> that support changes to the PIO setup FIS packet construction code, which
> is necessary for testing ths specification adherence of the IDENTIFY command,
> which issues its data exclusively via PIO mechanisms.
>
> The ahci-test code being checked in represents a minimum of functionality
> needed in order to issue and receive commands from the AHCI HBA under the
> libqos / qtest environment.
>
> In V2, as detailed below, these tests are not currently expected to pass.
> I will post a complementary patch outside of this set that highlights
> the exact set of tests that will not pass, which can help verify at least
> the portions of these tests that do work correctly.
>
> Assertions that currently fail:
> - Ordering of PCI capabilities as defined by either AHCI or Intel ICH9
> - Boot-time values of the PxTFD register, which should not have valid
> data until after a D2H FIS is received, but does in Qemu 2.1
> - Boot-time values of the PxSIG register, which should have a specific
> placeholder signature until the first D2H FIS is received, but is
> currently blank.
> - The "Descriptor Processed" interrupt is expected after the IDENTIFY
> command exhausts the given PRDT, but is not seen.
Thanks, I have merged patches up to and including Patch 24 onto my block
tree:
https://github.com/stefanha/qemu/commits/block
This should make it easier to manage the next revision of the series
where we can focus on the qtest test cases.
Stefan
pgpj6urVvnhiE.pgp
Description: PGP signature
- [Qemu-devel] [PATCH v2 27/30] ahci: add test_pci_enable to ahci-test., (continued)
- [Qemu-devel] [PATCH v2 27/30] ahci: add test_pci_enable to ahci-test., John Snow, 2014/08/04
- [Qemu-devel] [PATCH v2 28/30] ahci: Add test_hba_spec to ahci-test., John Snow, 2014/08/04
- [Qemu-devel] [PATCH v2 22/30] libqos: Fixes a small memory leak., John Snow, 2014/08/04
- [Qemu-devel] [PATCH v2 23/30] libqos: allow qpci_iomap to return BAR mapping size, John Snow, 2014/08/04
- [Qemu-devel] [PATCH v2 25/30] ahci: Adding basic functionality qtest., John Snow, 2014/08/04
- Re: [Qemu-devel] [PATCH v2 00/30] AHCI test suite framework, Stefan Hajnoczi, 2014/08/06
- Re: [Qemu-devel] [PATCH v2 00/30] AHCI test suite framework,
Stefan Hajnoczi <=