Re: [Qemu-devel] [PATCHv5 11/12] tests: Don't assume structure of PCI IO base in ahci-test

2016-10-25 Thread Greg Kurz
On Mon, 24 Oct 2016 16:00:00 +1100 David Gibson wrote: > In a couple of places ahci-test makes assumptions about how the tokens > returned from qpci_iomap() are formatted in ways it probably shouldn't. > > First in verify_state() it uses a non-NULL token to indicate that the AHCI > device has be

Re: [Qemu-devel] [PATCHv5 11/12] tests: Don't assume structure of PCI IO base in ahci-test

2016-10-24 Thread John Snow
On 10/24/2016 01:00 AM, David Gibson wrote: In a couple of places ahci-test makes assumptions about how the tokens returned from qpci_iomap() are formatted in ways it probably shouldn't. First in verify_state() it uses a non-NULL token to indicate that the AHCI device has been enabled (part of

[Qemu-devel] [PATCHv5 11/12] tests: Don't assume structure of PCI IO base in ahci-test

2016-10-23 Thread David Gibson
In a couple of places ahci-test makes assumptions about how the tokens returned from qpci_iomap() are formatted in ways it probably shouldn't. First in verify_state() it uses a non-NULL token to indicate that the AHCI device has been enabled (part of enabling is to iomap()). This changes it to us