From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56241) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aTQTD-0007r3-6Y for qemu-devel@nongnu.org; Wed, 10 Feb 2016 03:49:56 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aTQTA-0003b7-0z for qemu-devel@nongnu.org; Wed, 10 Feb 2016 03:49:55 -0500 Received: from mailhub.sw.ru ([195.214.232.25]:26510 helo=relay.sw.ru) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aTQT9-0003af-Ks for qemu-devel@nongnu.org; Wed, 10 Feb 2016 03:49:51 -0500 From: Vladimir Sementsov-Ogievskiy Date: Wed, 10 Feb 2016 11:49:25 +0300 Message-Id: <1455094166-7381-5-git-send-email-vsementsov@virtuozzo.com> In-Reply-To: <1455094166-7381-1-git-send-email-vsementsov@virtuozzo.com> References: <1455094166-7381-1-git-send-email-vsementsov@virtuozzo.com> Subject: [Qemu-devel] [PATCH 4/5] virtio-balloon: rewrite get_current_ram_size() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: vsementsov@virtuozzo.com, guangrong.xiao@linux.intel.com, mst@redhat.com, armbru@redhat.com, lcapitulino@redhat.com, stefanha@redhat.com, imammedo@redhat.com, cornelia.huck@de.ibm.com, den@openvz.org Use pc_dimm_built_list() instead of qmp_pc_dimm_device_list() Actually, Qapi is not related to this internal helper. Signed-off-by: Vladimir Sementsov-Ogievskiy --- hw/virtio/virtio-balloon.c | 23 ++++++----------------- 1 file changed, 6 insertions(+), 17 deletions(-) diff --git a/hw/virtio/virtio-balloon.c b/hw/virtio/virtio-balloon.c index 6a4c4d2..14aab77 100644 --- a/hw/virtio/virtio-balloon.c +++ b/hw/virtio/virtio-balloon.c @@ -296,26 +296,15 @@ static void virtio_balloon_get_config(VirtIODevice *vdev, uint8_t *config_data) static ram_addr_t get_current_ram_size(void) { - MemoryDeviceInfoList *info_list = NULL; - MemoryDeviceInfoList **prev = &info_list; - MemoryDeviceInfoList *info; + GSList *list = NULL, *item; ram_addr_t size = ram_size; - qmp_pc_dimm_device_list(qdev_get_machine(), &prev); - for (info = info_list; info; info = info->next) { - MemoryDeviceInfo *value = info->value; - - if (value) { - switch (value->type) { - case MEMORY_DEVICE_INFO_KIND_DIMM: - size += value->u.dimm->size; - break; - default: - break; - } - } + pc_dimm_build_list(qdev_get_machine(), &list); + for (item = list; item; item = g_slist_next(item)) { + Object *obj = OBJECT(item->data); + size += object_property_get_int(obj, PC_DIMM_SIZE_PROP, &error_abort); } - qapi_free_MemoryDeviceInfoList(info_list); + g_slist_free(list); return size; } -- 1.8.3.1