[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] nvdimm: release the correct device list
From: |
address@hidden |
Subject: |
Re: [PATCH] nvdimm: release the correct device list |
Date: |
Mon, 13 Sep 2021 06:40:01 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 |
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 <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,
>>>
>>>
>>
>>
>>
>
- Re: [PATCH] nvdimm: release the correct device list,
address@hidden <=