ping again
On 30/08/2021 09:04, Li Zhijian wrote: > ping > > > On 03/08/2021 12:00, Li, Zhijian wrote: >> ping >> >> Any body could help to review/queue this patch ? >> >> >> >> On 2021/6/29 22:05, Igor Mammedov wrote: >>> On Thu, 24 Jun 2021 19:04:15 +0800 >>> Li Zhijian <lizhij...@cn.fujitsu.com> wrote: >>> >>>> Signed-off-by: Li Zhijian <lizhij...@cn.fujitsu.com> >>> Reviewed-by: Igor Mammedov <imamm...@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, >>> >>> >> >> >> >