[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/24] ide: wrap start_dma callback
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH 09/24] ide: wrap start_dma callback |
Date: |
Wed, 30 Oct 2013 13:29:32 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 28.10.2013 um 17:43 hat Paolo Bonzini geschrieben:
> Make it optional and prepare for the next patches.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> hw/ide/atapi.c | 6 ++----
> hw/ide/core.c | 15 ++++++++-------
> hw/ide/internal.h | 1 +
> 3 files changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/hw/ide/atapi.c b/hw/ide/atapi.c
> index 05e60b1..a7688bf 100644
> --- a/hw/ide/atapi.c
> +++ b/hw/ide/atapi.c
> @@ -255,8 +255,7 @@ static void ide_atapi_cmd_reply(IDEState *s, int size,
> int max_size)
> if (s->atapi_dma) {
> bdrv_acct_start(s->bs, &s->acct, size, BDRV_ACCT_READ);
> s->status = READY_STAT | SEEK_STAT | DRQ_STAT;
> - s->bus->dma->ops->start_dma(s->bus->dma, s,
> - ide_atapi_cmd_read_dma_cb);
> + ide_start_dma(s, ide_atapi_cmd_read_dma_cb);
I was wondering whether the s->status update should be moved into
ide_start_dma(). Then I noticed that the value is different here,
because it's lacking BSY. Probably an inconsistency that wouldn't hurt
to get rid of? (The spec says that during a DMA operation BSY or DRQ or
both must be set.)
Kevin
- Re: [Qemu-devel] [PATCH 02/24] libqtest: return progress from qmp/qmpv, (continued)
- [Qemu-devel] [PATCH 03/24] libqtest: add QTEST_LOG for debugging qtest testcases, Paolo Bonzini, 2013/10/28
- [Qemu-devel] [PATCH 04/24] ide-test: add test for werror=stop, Paolo Bonzini, 2013/10/28
- [Qemu-devel] [PATCH 05/24] ide: simplify reset callbacks, Paolo Bonzini, 2013/10/28
- [Qemu-devel] [PATCH 06/24] ide: simplify set_inactive callbacks, Paolo Bonzini, 2013/10/28
- [Qemu-devel] [PATCH 07/24] ide: simplify async_cmd_done callbacks, Paolo Bonzini, 2013/10/28
- [Qemu-devel] [PATCH 08/24] ide: simplify start_transfer callbacks, Paolo Bonzini, 2013/10/28
- [Qemu-devel] [PATCH 09/24] ide: wrap start_dma callback, Paolo Bonzini, 2013/10/28
- Re: [Qemu-devel] [PATCH 09/24] ide: wrap start_dma callback,
Kevin Wolf <=
- [Qemu-devel] [PATCH 10/24] ide: add trigger_irq callback, Paolo Bonzini, 2013/10/28
[Qemu-devel] [PATCH 11/24] ide: fold add_status callback into set_inactive, Paolo Bonzini, 2013/10/28
[Qemu-devel] [PATCH 12/24] ide: move BM_STATUS bits to pci.[ch], Paolo Bonzini, 2013/10/28
[Qemu-devel] [PATCH 13/24] ide: move retry constants out of BM_STATUS_* namespace, Paolo Bonzini, 2013/10/28