[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [patch 2/3] Add support for live block copy
From: |
Marcelo Tosatti |
Subject: |
Re: [Qemu-devel] [patch 2/3] Add support for live block copy |
Date: |
Mon, 28 Feb 2011 16:09:02 -0300 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Sat, Feb 26, 2011 at 07:45:44AM -0600, Anthony Liguori wrote:
> >>>+- "commit_filename": target commit filename (json-string, optional)
> >>I think we should drop this.
> >Why? Sorry but this can't wait for non-config persistent storage. This
> >mistake was made in the past with irqchip for example, lets not repeat
> >it.
> >
> >Its OK to deprecate "commit_filename" in favour of its location in
> >non-config persistent storage.
> >
> >Its not the end of the world for a mgmt app to handle change (not saying
> >its not a good principle) such as this.
>
> Even as a one off, it's not a very good solution to the problem.
> We'd be way better of just having nothing here than using the commit
> file. What are the semantics of a half written file? How does a
> management tool detect a half written file?
If the commit file contains the full commit message, it can be
considered valid. Otherwise, it should be considered invalid.
Stopping the guest and waiting for mgmt to issue a continue command is a
solution, but it has drawbacks introduced by reliance on mgmt app (what
if mgmt app crashes, latency, etc).
But it seems that it is preferred over a commit file.
Addressing the other comments in the meantime, thanks for input.
[Qemu-devel] Re: [patch 2/3] Add support for live block copy, Anthony Liguori, 2011/02/22
Re: [Qemu-devel] [patch 2/3] Add support for live block copy, Anthony Liguori, 2011/02/22
Re: [Qemu-devel] [patch 2/3] Add support for live block copy, Anthony Liguori, 2011/02/23
Re: [Qemu-devel] [patch 2/3] Add support for live block copy, Anthony Liguori, 2011/02/26
[Qemu-devel] [patch 3/3] do not allow migration if block copy in progress, Marcelo Tosatti, 2011/02/22
[Qemu-devel] [patch 1/3] add migration_active function, Marcelo Tosatti, 2011/02/22