[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH] macio: Use blk_drain instead of bl
From: |
John Snow |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH] macio: Use blk_drain instead of blk_drain_all |
Date: |
Tue, 14 Jun 2016 16:26:22 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.0 |
On 06/13/2016 09:21 PM, Fam Zheng wrote:
> On Mon, 06/13 17:33, John Snow wrote:
>>
>>
>> On 06/12/2016 02:56 AM, Fam Zheng wrote:
>>> We only care about the associated backend, so blk_drain is more
>>> appropriate here.
>>>
>>> Signed-off-by: Fam Zheng <address@hidden>
>>> ---
>>> hw/ide/macio.c | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/hw/ide/macio.c b/hw/ide/macio.c
>>> index 78c10a0..a8c7321 100644
>>> --- a/hw/ide/macio.c
>>> +++ b/hw/ide/macio.c
>>> @@ -400,7 +400,7 @@ static void pmac_ide_flush(DBDMA_io *io)
>>> IDEState *s = idebus_active_if(&m->bus);
>>>
>>> if (s->bus->dma->aiocb) {
>>> - blk_drain_all();
>>> + blk_drain(s->blk);
>>> }
>>> }
>>>
>>>
>>
>> Reviewed-by: John Snow <address@hidden>
>>
>> Shall I take this through my tree?
>
> I think that MAINTAINERS says so. :)
>
> Fam
>
For now, then:
Thanks, applied to my IDE tree:
https://github.com/jnsnow/qemu/commits/ide
https://github.com/jnsnow/qemu.git
--js