On Thu, 8 Jan 2026 at 01:41, Philippe Mathieu-Daudé <[email protected]> wrote:
> machine.c doesn't use any target-specific macro defined by
> the "migration/cpu.h" header. Use the minimum header requiered:
* requiered -> required
> "migration/qemu-file-types.h" and "migration/vmstate.h", which are not
> target-specific.
* They are included via migration/cpu.h too; Still the change looks fine.
> diff --git a/target/openrisc/machine.c b/target/openrisc/machine.c
> index f2853674f0f..2d5ca16c511 100644
> --- a/target/openrisc/machine.c
> +++ b/target/openrisc/machine.c
> @@ -19,7 +19,8 @@
>
> #include "qemu/osdep.h"
> #include "cpu.h"
> -#include "migration/cpu.h"
> +#include "migration/qemu-file-types.h"
> +#include "migration/vmstate.h"
>
> static const VMStateDescription vmstate_tlb_entry = {
> .name = "tlb_entry",
Reviewed-by: Prasad Pandit <[email protected]>
Thank you.
---
- Prasad
