[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Remove unnecessary minimum_version_id_old fields
From: |
Peter Maydell |
Subject: |
Re: [PATCH] Remove unnecessary minimum_version_id_old fields |
Date: |
Tue, 18 Jan 2022 11:26:49 +0000 |
On Tue, 18 Jan 2022 at 10:44, Peter Maydell <peter.maydell@linaro.org> wrote:
>
> The migration code will not look at a VMStateDescription's
> minimum_version_id_old field unless that VMSD has set the
> load_state_old field to something non-NULL. (The purpose of
> minimum_version_id_old is to specify what migration version is needed
> for the code in the function pointed to by load_state_old to be able
> to handle it on incoming migration.)
>
> We have exactly one VMSD which still has a load_state_old,
> in the PPC CPU; every other VMSD which sets minimum_version_id_old
> is doing so unnecessarily. Delete all the unnecessary ones.
...and 20220118104150.1899661-1-clg@kaod.org/">https://patchew.org/QEMU/20220118104150.1899661-1-clg@kaod.org/
proposes removing that PPC usage, so once both this patch and that
one go in we can delete the migration/ code relating to load_state_old.
-- PMM