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 <f...@redhat.com>
>>> ---
>>> 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 <js...@redhat.com>
>>
>> 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
