Patchwork [3/3] hw/onenand.c: fix migration of dynamically allocated buffer "otp"

login
register
mail settings
Submitter Igor Mitsyanko
Date March 10, 2013, 1:47 p.m.
Message ID <1362923278-4080-4-git-send-email-i.mitsyanko@gmail.com>
Download mbox | patch
Permalink /patch/226437/
State New
Headers show

Comments

Igor Mitsyanko - March 10, 2013, 1:47 p.m.
VMSTATE_BUFFER_UNSAFE should be used for buffers inlined in device state, not
for buffers allocated dynamically. Change to VMSTATE_BUFFER_POINTER_UNSAFE macro,
which will do migration right.

Signed-off-by: Igor Mitsyanko <i.mitsyanko@gmail.com>
---
 hw/onenand.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

Patch

diff --git a/hw/onenand.c b/hw/onenand.c
index 00a8738..12ccb67f 100644
--- a/hw/onenand.c
+++ b/hw/onenand.c
@@ -185,7 +185,8 @@  static const VMStateDescription vmstate_onenand = {
         VMSTATE_UINT8(ecc.cp, OneNANDState),
         VMSTATE_UINT16_ARRAY(ecc.lp, OneNANDState, 2),
         VMSTATE_UINT16(ecc.count, OneNANDState),
-        VMSTATE_BUFFER_UNSAFE(otp, OneNANDState, 0, ((64 + 2) << PAGE_SHIFT)),
+        VMSTATE_BUFFER_POINTER_UNSAFE(otp, OneNANDState, 0,
+            ((64 + 2) << PAGE_SHIFT)),
         VMSTATE_END_OF_LIST()
     }
 };