Re: [Qemu-devel] [PATCH] macio: Use blk_drain instead of blk_drain_all

[Fam Zheng <famz@redhat.com>]

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 <famz@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 <jsnow@redhat.com>
> 
> Shall I take this through my tree?

I think that MAINTAINERS says so. :)

Fam