On 27/2/26 21:36, Brian Cain wrote:
> From: Brian Cain <bcain@quicinc.com>
>
> Reviewed-by: Taylor Simpson <ltaylorsimpson@gmail.com>
> Signed-off-by: Brian Cain <brian.cain@oss.qualcomm.com>
> ---
> target/hexagon/internal.h | 4 ++++
> target/hexagon/cpu.c | 3 +++
> target/hexagon/machine.c | 23 +++++++++++++++++++++++
> 3 files changed, 30 insertions(+)
> create mode 100644 target/hexagon/machine.c
>
> diff --git a/target/hexagon/internal.h b/target/hexagon/internal.h
> index 5fc837ae229..cd06ff41d4f 100644
> --- a/target/hexagon/internal.h
> +++ b/target/hexagon/internal.h
> @@ -31,4 +31,8 @@ void hexagon_debug(CPUHexagonState *env);
>
> extern const char * const hexagon_regnames[TOTAL_PER_THREAD_REGS];
>
> +#ifndef CONFIG_USER_ONLY
> +extern const VMStateDescription vmstate_hexagon_cpu;
> +#endif
> +
> #endif
> diff --git a/target/hexagon/cpu.c b/target/hexagon/cpu.c
> index c4df7cbe30b..a8f2f9e238a 100644
> --- a/target/hexagon/cpu.c
> +++ b/target/hexagon/cpu.c
> @@ -386,6 +386,9 @@ static void hexagon_cpu_class_init(ObjectClass *c, const void *data)
> cc->gdb_stop_before_watchpoint = true;
> cc->gdb_core_xml_file = "hexagon-core.xml";
> cc->disas_set_info = hexagon_cpu_disas_set_info;
> +#ifndef CONFIG_USER_ONLY
> + dc->vmsd = &vmstate_hexagon_cpu;
> +#endif
> cc->tcg_ops = &hexagon_tcg_ops;
> }
>
> diff --git a/target/hexagon/machine.c b/target/hexagon/machine.c
> new file mode 100644
> index 00000000000..89a2ca0bd94
> --- /dev/null
> +++ b/target/hexagon/machine.c
> @@ -0,0 +1,23 @@
> +/*
> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> + *
> + * SPDX-License-Identifier: GPL-2.0-or-later
> + */
> +
> +#include "qemu/osdep.h"
> +#include "migration/cpu.h"
> +#include "cpu.h"
> +
> +const VMStateDescription vmstate_hexagon_cpu = {
> + .name = "cpu",
> + .version_id = 1,
> + .minimum_version_id = 1,
> + .fields = (VMStateField[]) {
> + VMSTATE_STRUCT(parent_obj, HexagonCPU, 0, vmstate_cpu_common, CPUState),
We shouldn't need that (the parent should be migrated by himself), but
I'll defer to migration experts here.
> + VMSTATE_UINTTL_ARRAY(env.gpr, HexagonCPU, TOTAL_PER_THREAD_REGS),
> + VMSTATE_UINTTL_ARRAY(env.pred, HexagonCPU, NUM_PREGS),
> + VMSTATE_UINTTL_ARRAY(env.t_sreg, HexagonCPU, NUM_SREGS),
> + VMSTATE_UINTTL_ARRAY(env.greg, HexagonCPU, NUM_GREGS),
VMSTATE_UINTTL_ARRAY() is about to be removed (as it doesn't scale
for heterogeneous emulation). Please use VMSTATE_UINT32_ARRAY() instead.
> + VMSTATE_END_OF_LIST()
> + },
> +};