ide: Turn debug messages into assertions

These printfs aren't really debug messages, but clearly indicate a bug if they
ever become effective. Noone uses DEBUG_IDE, let's re-enable the check
unconditionally and make it an assertion instead of printfs in the device
emulation.

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
This commit is contained in:
Kevin Wolf 2011-05-09 11:42:03 +02:00
parent 1e71db3087
commit 2860e3eb96

View File

@ -296,12 +296,8 @@ void bmdma_cmd_writeb(void *opaque, uint32_t addr, uint32_t val)
*/ */
if (bm->bus->dma->aiocb) { if (bm->bus->dma->aiocb) {
qemu_aio_flush(); qemu_aio_flush();
#ifdef DEBUG_IDE assert(bm->bus->dma->aiocb == NULL);
if (bm->bus->dma->aiocb) assert((bm->status & BM_STATUS_DMAING) == 0);
printf("ide_dma_cancel: aiocb still pending\n");
if (bm->status & BM_STATUS_DMAING)
printf("ide_dma_cancel: BM_STATUS_DMAING still pending\n");
#endif
} }
} else { } else {
bm->cur_addr = bm->addr; bm->cur_addr = bm->addr;