[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] make windows notice media change
From: |
Gleb Natapov |
Subject: |
Re: [Qemu-devel] [PATCH v2] make windows notice media change |
Date: |
Wed, 29 Jul 2009 23:14:44 +0300 |
On Wed, Jul 29, 2009 at 09:05:18PM +0100, Paul Brook wrote:
> > >> BTW, why would there be the version parameter in the first place if it
> > >> wasn't supposed to load older versions?!
> > >
> > > Like I already said: it's there to prevent an old version being loaded
> > > accidentally. Without this an incompatible change will result in
> > > anything from a crash to corrupt/inconsistent guest state. Versioning
> > > allows us to reject the snapshot and fail safely.
> >
> > If that was the case then the if (version != x) return -EINVAL check could
> > have been in the generic code and there would be no need for the version
> > parameter in the load function.
>
> Preventing loading bad snapshots is the primary goal.
>
> Allowing loading old snapshots is a secondary feature. Personally I think
> it's
> not worth the effort, and in practice is unlikely to be feasible for whole
> machines over any significant length of time. However I don't feel strongly
> enough to actually rip the code out.
>
Agree. All this "migrate from older version" just produce code that is
never tested in practice.
--
Gleb.
- Re: [Qemu-devel] [PATCH v2] make windows notice media change, (continued)
- Re: [Qemu-devel] [PATCH v2] make windows notice media change, Filip Navara, 2009/07/29
- Re: [Qemu-devel] [PATCH v2] make windows notice media change, Paul Brook, 2009/07/29
- Re: [Qemu-devel] [PATCH v2] make windows notice media change, Filip Navara, 2009/07/29
- Re: [Qemu-devel] [PATCH v2] make windows notice media change, Filip Navara, 2009/07/29
- Re: [Qemu-devel] [PATCH v2] make windows notice media change, Anthony Liguori, 2009/07/29
- Re: [Qemu-devel] [PATCH v2] make windows notice media change, Paul Brook, 2009/07/29
- Re: [Qemu-devel] [PATCH v2] make windows notice media change, Filip Navara, 2009/07/29
- Re: [Qemu-devel] [PATCH v2] make windows notice media change, Paul Brook, 2009/07/29
- Re: [Qemu-devel] [PATCH v2] make windows notice media change,
Gleb Natapov <=
- Re: [Qemu-devel] [PATCH v2] make windows notice media change, Stefano Stabellini, 2009/07/30
- Re: [Qemu-devel] [PATCH v2] make windows notice media change, Paul Brook, 2009/07/30
- Re: [Qemu-devel] [PATCH v2] make windows notice media change, Anthony Liguori, 2009/07/29
- [Qemu-devel] Re: [PATCH v2] make windows notice media change, Juan Quintela, 2009/07/29
- [Qemu-devel] Re: [PATCH v2] make windows notice media change, Anthony Liguori, 2009/07/29
- [Qemu-devel] Re: [PATCH v2] make windows notice media change, Juan Quintela, 2009/07/29