[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/2] block: fix spurious DEVICE_TRAY_MOVED event
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH 0/2] block: fix spurious DEVICE_TRAY_MOVED events on shutdown |
Date: |
Thu, 25 Apr 2013 16:29:45 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Apr 25, 2013 at 09:51:47AM -0400, Luiz Capitulino wrote:
> On Mon, 22 Apr 2013 15:53:43 +0200
> Stefan Hajnoczi <address@hidden> 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? :)
Nice try :)
We've never gotten media change right. I really would appreciate a
second pair of eyes. There are still a couple of days until hard
freeze.
Holding off until then.
Stefan