diff options
| author | Pavel Butsykin <pbutsykin@virtuozzo.com> | 2016-04-12 18:48:15 -0400 |
|---|---|---|
| committer | John Snow <jsnow@redhat.com> | 2016-04-12 18:48:15 -0400 |
| commit | 502356eeeb5fd2bdd92b2d5156e511626c1c3814 (patch) | |
| tree | 8aed873d0e5859db34e6a8d335367e26f15efb4a /backends/hostmem-file.c | |
| parent | 9a41826f3843bf11bf8f52f176071f8c1fb0aa7e (diff) | |
| download | focaccia-qemu-502356eeeb5fd2bdd92b2d5156e511626c1c3814.tar.gz focaccia-qemu-502356eeeb5fd2bdd92b2d5156e511626c1c3814.zip | |
ide: really restart pending and in-flight atapi dma
Restart of ATAPI DMA used to be unreachable, because the request to do so wasn't indicated in bus->error_status due to the lack of spare bits, and ide_restart_bh() would return early doing nothing. This patch makes use of the observation that not all bit combinations were possible in ->error_status. In particular, IDE_RETRY_READ only made sense together with IDE_RETRY_DMA or IDE_RETRY_PIO. This allows to re-use IDE_RETRY_READ alone as an indicator of ATAPI DMA restart request. To makes things more uniform, ATAPI DMA gets its own value for ->dma_cmd. As a means against confusion, macros are added to test the state of ->error_status. The patch fixes the restart of both in-flight and pending ATAPI DMA, following the scheme similar to that of IDE DMA. [Including a fixup patch: Message-id: 1460465594-15777-1-git-send-email-pbutsykin@virtuozzo.com --js] Signed-off-by: Pavel Butsykin <pbutsykin@virtuozzo.com> Signed-off-by: Denis V. Lunev <den@openvz.org> Reviewed-by: Roman Kagan <rkagan@virtuozzo.com> Reviewed-by: John Snow <jsnow@redhat.com> Message-id: 1459924806-306-4-git-send-email-den@openvz.org Signed-off-by: John Snow <jsnow@redhat.com>
Diffstat (limited to 'backends/hostmem-file.c')
0 files changed, 0 insertions, 0 deletions