[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH 1/3] migration: Call blk_resume_after_migration(
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-block] [PATCH 1/3] migration: Call blk_resume_after_migration() for postcopy |
Date: |
Thu, 13 Apr 2017 20:30:50 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 13.04.2017 um 20:03 hat Eric Blake geschrieben:
> On 04/13/2017 12:54 PM, Kevin Wolf wrote:
> > Am 13.04.2017 um 19:39 hat Eric Blake geschrieben:
> >> On 04/13/2017 12:23 PM, Kevin Wolf wrote:
> >>> Commit d35ff5e6 ('block: Ignore guest dev permissions during incoming
> >>> migration') added blk_resume_after_migration() to the precopy migration
> >>> path, but neglected to add it to the duplicated code that is used for
> >>> postcopy migration. This means that the guest device doesn't request the
> >>> necessary permissions, which ultimately led to failing assertions.
> >>>
> >>> Add the missing blk_resume_after_migration() to the postcopy path.
> >>>
> >>> Signed-off-by: Kevin Wolf <address@hidden>
> >>> ---
> >>> migration/savevm.c | 8 ++++++++
> >>> 1 file changed, 8 insertions(+)
> >>
> >> Reviewed-by: Eric Blake <address@hidden>
> >>
> >> Are we targetting this for 2.9-rc5, or is it 2.10 material?
> >
> > At this point, I think it's clearly 2.10.
>
> Okay. Restating, to make sure I got your reasoning: the removed
> assertions of commit e3e0003 imply that 2.9 is not regressing in
> behavior, and at this point the worst the code can do without this patch
> applied is behave like it's done previously; therefore this patch is not
> fixing an observable 2.9 behavior and therefore not worth holding up the
> release.
Right, basically the new op blockers become ineffective for guest
devices after postcopy migration.
> But for 2.10, it's absolutely essential, as we have another patch
> pending to revert e3e0003 at which point we have a behavior break
> without this patch.
Correct.
Kevin
pgpbItRy6eLoF.pgp
Description: PGP signature
[Qemu-block] [PATCH 2/3] qemu-iotests: Filter HMP readline escape characters, Kevin Wolf, 2017/04/13
[Qemu-block] [PATCH 3/3] qemu-iotests: Test postcopy migration, Kevin Wolf, 2017/04/13