[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] nvdimm: release the correct device list
From: |
Igor Mammedov |
Subject: |
Re: [PATCH] nvdimm: release the correct device list |
Date: |
Tue, 29 Jun 2021 16:05:34 +0200 |
On Thu, 24 Jun 2021 19:04:15 +0800
Li Zhijian <lizhijian@cn.fujitsu.com> wrote:
> Signed-off-by: Li Zhijian <lizhijian@cn.fujitsu.com>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
> ---
> hw/acpi/nvdimm.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/hw/acpi/nvdimm.c b/hw/acpi/nvdimm.c
> index e3d5fe19392..ff317263e85 100644
> --- a/hw/acpi/nvdimm.c
> +++ b/hw/acpi/nvdimm.c
> @@ -355,10 +355,10 @@ nvdimm_build_structure_caps(GArray *structures,
> uint32_t capabilities)
>
> static GArray *nvdimm_build_device_structure(NVDIMMState *state)
> {
> - GSList *device_list = nvdimm_get_device_list();
> + GSList *device_list, *list = nvdimm_get_device_list();
> GArray *structures = g_array_new(false, true /* clear */, 1);
>
> - for (; device_list; device_list = device_list->next) {
> + for (device_list = list; device_list; device_list = device_list->next) {
> DeviceState *dev = device_list->data;
>
> /* build System Physical Address Range Structure. */
> @@ -373,7 +373,7 @@ static GArray *nvdimm_build_device_structure(NVDIMMState
> *state)
> /* build NVDIMM Control Region Structure. */
> nvdimm_build_structure_dcr(structures, dev);
> }
> - g_slist_free(device_list);
> + g_slist_free(list);
>
> if (state->persistence) {
> nvdimm_build_structure_caps(structures, state->persistence);
> @@ -1339,9 +1339,9 @@ static void nvdimm_build_ssdt(GArray *table_offsets,
> GArray *table_data,
>
> void nvdimm_build_srat(GArray *table_data)
> {
> - GSList *device_list = nvdimm_get_device_list();
> + GSList *device_list, *list = nvdimm_get_device_list();
>
> - for (; device_list; device_list = device_list->next) {
> + for (device_list = list; device_list; device_list = device_list->next) {
> AcpiSratMemoryAffinity *numamem = NULL;
> DeviceState *dev = device_list->data;
> Object *obj = OBJECT(dev);
> @@ -1356,7 +1356,7 @@ void nvdimm_build_srat(GArray *table_data)
> build_srat_memory(numamem, addr, size, node,
> MEM_AFFINITY_ENABLED | MEM_AFFINITY_NON_VOLATILE);
> }
> - g_slist_free(device_list);
> + g_slist_free(list);
> }
>
> void nvdimm_build_acpi(GArray *table_offsets, GArray *table_data,