On Fri, 7 Jun 2024 12:56:43 +0100
Salil Mehta <salil.me...@huawei.com> wrote:
> CPU ctrl-dev MMIO region length could be used in ACPI GED and various other
> architecture specific places. Move ACPI_CPU_HOTPLUG_REG_LEN macro to more
> appropriate common header file.
>
> Signed-off-by: Salil Mehta <salil.me...@huawei.com>
> Reviewed-by: Alex Bennée <alex.ben...@linaro.org>
> Reviewed-by: Jonathan Cameron <jonathan.came...@huawei.com>
> Reviewed-by: Gavin Shan <gs...@redhat.com>
> Reviewed-by: David Hildenbrand <da...@redhat.com>
> Reviewed-by: Shaoqin Huang <shahu...@redhat.com>
> Tested-by: Vishnu Pajjuri <vis...@os.amperecomputing.com>
> Tested-by: Xianglai Li <lixiang...@loongson.cn>
> Tested-by: Miguel Luis <miguel.l...@oracle.com>
> Tested-by: Zhao Liu <zhao1....@intel.com>
> Reviewed-by: Zhao Liu <zhao1....@intel.com>
> ---
> hw/acpi/cpu.c | 2 +-
> include/hw/acpi/cpu_hotplug.h | 2 ++
> 2 files changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/hw/acpi/cpu.c b/hw/acpi/cpu.c
> index 2d81c1e790..69aaa563db 100644
> --- a/hw/acpi/cpu.c
> +++ b/hw/acpi/cpu.c
> @@ -1,13 +1,13 @@
> #include "qemu/osdep.h"
> #include "migration/vmstate.h"
> #include "hw/acpi/cpu.h"
> +#include "hw/acpi/cpu_hotplug.h"
> #include "hw/core/cpu.h"
> #include "qapi/error.h"
> #include "qapi/qapi-events-acpi.h"
> #include "trace.h"
> #include "sysemu/numa.h"
>
> -#define ACPI_CPU_HOTPLUG_REG_LEN 12
> #define ACPI_CPU_SELECTOR_OFFSET_WR 0
> #define ACPI_CPU_FLAGS_OFFSET_RW 4
> #define ACPI_CPU_CMD_OFFSET_WR 5
> diff --git a/include/hw/acpi/cpu_hotplug.h b/include/hw/acpi/cpu_hotplug.h
> index 3b932abbbb..48b291e45e 100644
> --- a/include/hw/acpi/cpu_hotplug.h
> +++ b/include/hw/acpi/cpu_hotplug.h
this file has deps on x86 machine and contains mainly
legacy CPU hotplug API for x86.
> @@ -19,6 +19,8 @@
> #include "hw/hotplug.h"
> #include "hw/acpi/cpu.h"
>
> +#define ACPI_CPU_HOTPLUG_REG_LEN 12
the better place for it would be include/hw/acpi/cpu.h
> typedef struct AcpiCpuHotplug {
> Object *device;
> MemoryRegion io;
