[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH 1/3] block: implement bdrv_snapshot_goto for blk
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-block] [PATCH 1/3] block: implement bdrv_snapshot_goto for blkreplay |
Date: |
Mon, 27 Feb 2017 18:35:25 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 27.02.2017 um 13:07 hat Pavel Dovgalyuk geschrieben:
> > From: Kevin Wolf [mailto:address@hidden
> > Am 31.01.2017 um 12:57 hat Pavel Dovgalyuk geschrieben:
> > > This patch enables making snapshots with blkreplay used in
> > > block devices.
> > >
> > > Signed-off-by: Pavel Dovgalyuk <address@hidden>
> >
> > Specifically, I think it avoids the blkreplay_open/close sequence. Is
> > this what is needed to make it work?
>
> Then I'll need to implement bdrv_open, because there is only bdrv_file_open
> for blkreplay now.
>
> Which way is better?
I was just checking whether I understood the reason for this correctly.
If I did, then I think your solution is fine and we should just make the
commit message a bit more explicit.
Kevin
> > We should probably mention in the commit message the exact reason why
> > implementing .bdrv_snapshot_goto, but not the other snapshot related
> > callbacks, fixes things. If you confirm my assumption, I can add that
> > while applying.
>
> Pavel Dovgalyuk
>