[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: |
Fri, 17 Sep 2021 14:59:33 +0200 |
On Mon, 13 Sep 2021 06:40:01 +0000
"lizhijian@fujitsu.com" <lizhijian@fujitsu.com> wrote:
> ping again
Michael,
can you include this in your next pull req, please?
>
>
>
> 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 <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,
> >>>
> >>>
> >>
> >>
> >>
> >