On Mon, 22 Apr 2013 15:53:43 +0200
Stefan Hajnoczi <stefa...@gmail.com> wrote:

> On Fri, Apr 19, 2013 at 01:47:17PM -0400, Luiz Capitulino wrote:
> > Hi,
> > 
> > This fixes a regression introduced by commit 9ca111544, as detailed in
> > patch 2/2, by moving bdrv_dev_change_media_cb() calls to callers of
> > bdrv_close() that need it, as suggested by Kevin.
> > 
> > Luiz Capitulino (2):
> >   block: make bdrv_dev_change_media_cb() public
> >   block: move bdrv_dev_change_media_cb() to callers that really need it
> > 
> >  block.c               | 5 +----
> >  blockdev.c            | 2 ++
> >  include/block/block.h | 1 +
> >  3 files changed, 4 insertions(+), 4 deletions(-)
> 
> Looks okay but I'll wait for Markus or Kevin to review too.  The media
> change code is subtle, we've had a long history of fixes :).

I wouldn't say this is hugely important, but I'm targeting 1.5.

So, maybe lack of review means you could apply it? :)

Reply via email to