On 25/06/2021 08:38, Mark Cave-Ayland wrote:

I noticed whilst testing the previous dp8393x patchset that I would always
get a segfault whilst attempting to migrate the MIPS magnum machine.

A bit of detective work shows that the problem is an incorrect 
VMStateDescription
in the g364fb device which expects a G364State but instead receives a
G364SysBusState.

Looking back through the git history suggests that migration for this device
(and also the MIPS magnum machines) has been broken for several years, so patch
1 takes the opportunity to improve the migration stream for the framebuffer
first whilst patch 2 contains the actual fix.

Note that I don't use the MIPS magnum machines on a regular basis but the
patchset fixes the migration error and survives some light testing here.

Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk>


Mark Cave-Ayland (2):
   g364fb: use RAM memory region for framebuffer
   g364fb: add VMStateDescription for G364SysBusState

  hw/display/g364fb.c | 27 ++++++++++++++++++---------
  1 file changed, 18 insertions(+), 9 deletions(-)

Thanks for the review Phil, I've just sent v2 on its way. Does it make sense for this to be merged via the MIPS tree?


ATB,

Mark.

Reply via email to