We should not read this unless requested. Make it conditional on the
option being provided.

Add some debugging to show the state being written.

Signed-off-by: Simon Glass <s...@chromium.org>
---

(no changes since v1)

 arch/sandbox/cpu/start.c | 8 +++++---
 arch/sandbox/cpu/state.c | 1 +
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/arch/sandbox/cpu/start.c b/arch/sandbox/cpu/start.c
index 8582f05162fa..2c8a72590b55 100644
--- a/arch/sandbox/cpu/start.c
+++ b/arch/sandbox/cpu/start.c
@@ -522,9 +522,11 @@ int sandbox_main(int argc, char *argv[])
                state->ram_buf_fname = NULL;
        }
 
-       ret = sandbox_read_state(state, state->state_fname);
-       if (ret)
-               goto err;
+       if (state->read_state && state->state_fname) {
+               ret = sandbox_read_state(state, state->state_fname);
+               if (ret)
+                       goto err;
+       }
 
        if (state->handle_signals) {
                ret = os_setup_signal_handlers();
diff --git a/arch/sandbox/cpu/state.c b/arch/sandbox/cpu/state.c
index fcc0028ff4a2..e38bb248b7ff 100644
--- a/arch/sandbox/cpu/state.c
+++ b/arch/sandbox/cpu/state.c
@@ -521,6 +521,7 @@ int state_uninit(void)
                        printf("Failed to write sandbox state\n");
                        return -1;
                }
+               log_debug("Wrote state to file '%s'\n", state->ram_buf_fname);
        }
 
        /* Delete this at the last moment so as not to upset gdb too much */
-- 
2.42.0.515.g380fc7ccd1-goog

Reply via email to