Re: [Qemu-devel] [PULL 21/26] ide-test: Add FLUSH CACHE test case

[Stefan Hajnoczi]

On Fri, Jun 14, 2013 at 01:18:44PM -0500, Anthony Liguori wrote:
> Peter Maydell <address@hidden> writes:
> 
> > On 14 June 2013 18:54, Peter Maydell <address@hidden> wrote:
> >> On 7 June 2013 12:58, Stefan Hajnoczi <address@hidden> wrote:
> >>> From: Kevin Wolf <address@hidden>
> >>>
> >>> This checks in particular that BSY is set while the flush request is in
> >>> flight.
> >>
> >> This test doesn't seem to pass.
> >
> >> ERROR:/home/petmay01/linaro/qemu-from-laptop/qemu/tests/ide-test.c:460:test_flush:
> >> assertion failed ((data) & (BSY | DF | ERR | DRQ) == 0): (0x00000080
> >> == 0x00000000)
> >
> > ...fixed by http://patchwork.ozlabs.org/patch/250334/
> 
> I'm testing it myself now.  Since it's already in the block tree, I'll
> push it directly as it's blocking my queue.
> 
> Unless Kevin or Stefan are planning on sending a pull request today of
> course.

Thanks for merging mdroth's fix!

Stefan