On 24/8/23 15:38, Fiona Ebner wrote:
Fails without the previous commit "hw/ide: reset: cancel async DMA
operation before reseting state".
I haven't ever written such a test before, but I wanted something to
expose the problem more easily. It hardcodes the behavior that the
pending write actually is done during reset, which might not be ideal.
It could just check that the first sector is still intact instead.
If I should make this a proper test, I'd be happy about some guidance,
but not sure if required for such a specific one-off issue. After all,
a different variation of the bug might have written to some other
sector not covered by this test.
Signed-off-by: Fiona Ebner <f.eb...@proxmox.com>
---
tests/qtest/ahci-test.c | 81 +++++++++++++++++++++++++++++++++++++++++
1 file changed, 81 insertions(+)
Per our style, variables are allocated on function prologue,
otherwise the test LGTM:
Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>
