From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:57578) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bxDMI-0007fa-HK for qemu-devel@nongnu.org; Thu, 20 Oct 2016 09:26:11 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bxDME-0005wW-IC for qemu-devel@nongnu.org; Thu, 20 Oct 2016 09:26:10 -0400 Received: from mx0a-001b2d01.pphosted.com ([148.163.156.1]:48295) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1bxDME-0005vs-9s for qemu-devel@nongnu.org; Thu, 20 Oct 2016 09:26:06 -0400 Received: from pps.filterd (m0098404.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.16.0.17/8.16.0.17) with SMTP id u9KDOSum053103 for ; Thu, 20 Oct 2016 09:26:04 -0400 Received: from e06smtp13.uk.ibm.com (e06smtp13.uk.ibm.com [195.75.94.109]) by mx0a-001b2d01.pphosted.com with ESMTP id 266x1u32n5-1 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT) for ; Thu, 20 Oct 2016 09:26:03 -0400 Received: from localhost by e06smtp13.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Thu, 20 Oct 2016 14:26:01 +0100 Received: from b06cxnps3074.portsmouth.uk.ibm.com (d06relay09.portsmouth.uk.ibm.com [9.149.109.194]) by d06dlp01.portsmouth.uk.ibm.com (Postfix) with ESMTP id 51AE917D8024 for ; Thu, 20 Oct 2016 14:28:11 +0100 (BST) Received: from d06av03.portsmouth.uk.ibm.com (d06av03.portsmouth.uk.ibm.com [9.149.37.213]) by b06cxnps3074.portsmouth.uk.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id u9KDPxoC8847528 for ; Thu, 20 Oct 2016 13:25:59 GMT Received: from d06av03.portsmouth.uk.ibm.com (localhost [127.0.0.1]) by d06av03.portsmouth.uk.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id u9KDPwlu030684 for ; Thu, 20 Oct 2016 07:25:58 -0600 From: Halil Pasic Date: Thu, 20 Oct 2016 15:25:54 +0200 Message-Id: <20161020132556.67321-1-pasic@linux.vnet.ibm.com> Subject: [Qemu-devel] [PATCH v2 0/2] remove unused VMSTateField.start List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Amit Shah , Guenther Hutzl , "Dr. David Alan Gilbert" , Halil Pasic The member VMStateField.start was solely used to implement the partial data migration for VBUFFER data (basically provide migration for a sub-buffer). However the implementation of this feature is broken, but this goes unnoticed since the feature is not used at all. Let us add an unit test for VBUFFER and then remove the useless VMStateFiled.start. An additional benefit is that .start can be re introduced to be used for linked structures as proposed by Jianjun in "[QEMU PATCH v6 2/2] migration: migrate QTAILQ". Guenther Hutzl (1): tests/test-vmstate.c: add vBuffer test Halil Pasic (1): migration: drop unused VMStateField.start hw/char/exynos4210_uart.c | 2 +- hw/display/g364fb.c | 2 +- hw/dma/pl330.c | 8 ++-- hw/intc/exynos4210_gic.c | 2 +- hw/ipmi/isa_ipmi_bt.c | 4 +- hw/ipmi/isa_ipmi_kcs.c | 4 +- hw/net/vmxnet3.c | 2 +- hw/nvram/mac_nvram.c | 2 +- hw/nvram/spapr_nvram.c | 2 +- hw/sd/sdhci.c | 2 +- hw/timer/m48t59.c | 2 +- include/migration/vmstate.h | 20 +++----- migration/savevm.c | 2 +- migration/vmstate.c | 4 +- target-s390x/machine.c | 2 +- tests/test-vmstate.c | 114 ++++++++++++++++++++++++++++++++++++++++++++ util/fifo8.c | 2 +- 17 files changed, 141 insertions(+), 35 deletions(-) -- 2.8.4