[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/14] nvdimm acpi: rename nvdimm_plugged_device
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH 07/14] nvdimm acpi: rename nvdimm_plugged_device_list |
Date: |
Wed, 9 Nov 2016 17:39:47 +0100 |
On Mon, 7 Nov 2016 19:13:42 +0800
Xiao Guangrong <address@hidden> wrote:
> Its behavior has been changed as the nvdimm device which is being
> realized also will be handled in this function, so rename it to
> reflect the fact
>
> Suggested-by: Igor Mammedov <address@hidden>
> Signed-off-by: Xiao Guangrong <address@hidden>
Reviewed-by: Igor Mammedov <address@hidden>
> ---
> hw/acpi/nvdimm.c | 17 ++++++++---------
> 1 file changed, 8 insertions(+), 9 deletions(-)
>
> diff --git a/hw/acpi/nvdimm.c b/hw/acpi/nvdimm.c
> index 65eb6c9..f2c0659 100644
> --- a/hw/acpi/nvdimm.c
> +++ b/hw/acpi/nvdimm.c
> @@ -33,7 +33,7 @@
> #include "hw/nvram/fw_cfg.h"
> #include "hw/mem/nvdimm.h"
>
> -static int nvdimm_plugged_device_list(Object *obj, void *opaque)
> +static int nvdimm_device_list(Object *obj, void *opaque)
> {
> GSList **list = opaque;
>
> @@ -41,23 +41,22 @@ static int nvdimm_plugged_device_list(Object *obj, void
> *opaque)
> *list = g_slist_append(*list, DEVICE(obj));
> }
>
> - object_child_foreach(obj, nvdimm_plugged_device_list, opaque);
> + object_child_foreach(obj, nvdimm_device_list, opaque);
> return 0;
> }
>
> /*
> - * inquire plugged NVDIMM devices and link them into the list which is
> + * inquire NVDIMM devices and link them into the list which is
> * returned to the caller.
> *
> * Note: it is the caller's responsibility to free the list to avoid
> * memory leak.
> */
> -static GSList *nvdimm_get_plugged_device_list(void)
> +static GSList *nvdimm_get_device_list(void)
> {
> GSList *list = NULL;
>
> - object_child_foreach(qdev_get_machine(), nvdimm_plugged_device_list,
> - &list);
> + object_child_foreach(qdev_get_machine(), nvdimm_device_list, &list);
> return list;
> }
>
> @@ -215,7 +214,7 @@ static uint32_t nvdimm_slot_to_dcr_index(int slot)
> static NVDIMMDevice *nvdimm_get_device_by_handle(uint32_t handle)
> {
> NVDIMMDevice *nvdimm = NULL;
> - GSList *list, *device_list = nvdimm_get_plugged_device_list();
> + GSList *list, *device_list = nvdimm_get_device_list();
>
> for (list = device_list; list; list = list->next) {
> NVDIMMDevice *nvd = list->data;
> @@ -346,7 +345,7 @@ static void nvdimm_build_structure_dcr(GArray
> *structures, DeviceState *dev)
>
> static GArray *nvdimm_build_device_structure(void)
> {
> - GSList *device_list = nvdimm_get_plugged_device_list();
> + GSList *device_list = nvdimm_get_device_list();
> GArray *structures = g_array_new(false, true /* clear */, 1);
>
> for (; device_list; device_list = device_list->next) {
> @@ -1279,7 +1278,7 @@ void nvdimm_build_acpi(GArray *table_offsets, GArray
> *table_data,
> nvdimm_build_ssdt(table_offsets, table_data, linker, state->dsm_mem,
> ram_slots);
>
> - device_list = nvdimm_get_plugged_device_list();
> + device_list = nvdimm_get_device_list();
> /* no NVDIMM device is plugged. */
> if (!device_list) {
> return;
- Re: [Qemu-devel] [PATCH 02/14] nvdimm acpi: drop the lock of fit buffer, (continued)
- [Qemu-devel] [PATCH 04/14] pc: memhp: stop handling nvdimm hotplug in pc_dimm_unplug, Xiao Guangrong, 2016/11/07
- [Qemu-devel] [PATCH 05/14] nvdimm acpi: clean up nvdimm_build_acpi, Xiao Guangrong, 2016/11/07
- [Qemu-devel] [PATCH 07/14] nvdimm acpi: rename nvdimm_plugged_device_list, Xiao Guangrong, 2016/11/07
- [Qemu-devel] [PATCH 06/14] docs: improve the doc of Read FIT method, Xiao Guangrong, 2016/11/07
- [Qemu-devel] [PATCH 08/14] nvdimm acpi: cleanup nvdimm_build_fit, Xiao Guangrong, 2016/11/07
- [Qemu-devel] [PATCH 10/14] nvdimm acpi: define DSM return codes, Xiao Guangrong, 2016/11/07
- [Qemu-devel] [PATCH 09/14] nvdimm acpi: rename nvdimm_acpi_hotplug, Xiao Guangrong, 2016/11/07