From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43608) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1f1E46-0005vB-3b for qemu-devel@nongnu.org; Wed, 28 Mar 2018 12:36:46 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1f1E42-0007lv-Vy for qemu-devel@nongnu.org; Wed, 28 Mar 2018 12:36:46 -0400 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:59091 helo=mx1.redhat.com) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1f1E42-0007lV-RC for qemu-devel@nongnu.org; Wed, 28 Mar 2018 12:36:42 -0400 From: "Dr. David Alan Gilbert (git)" Date: Wed, 28 Mar 2018 17:36:24 +0100 Message-Id: <20180328163630.48576-1-dgilbert@redhat.com> Subject: [Qemu-devel] [PATCH v2 0/6] e1000 migration changes for 2.12 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org, eswierk@skyportsystems.com, jasowang@redhat.com, pbonzini@redhat.com Cc: quintela@redhat.com, peterx@redhat.com From: "Dr. David Alan Gilbert" Hi, This set of patches change the e1000 migration code to make it easier to keep with compatibility with older versions in backwards migration. I think the first 3 patches are fairly uncontrovercial and I would like them for 2.12; it would be nice to have the lot since changing them after we've shipped is much more difficult. v2 Ed and Paolo answered my question that I asked in the cover letter; and I think I've followed the advice - although my testing has been very light. The new patches do two things: a) When we receive a stream without the subsection we duplicate the received pops state into both props and tso_props. b) When we send without the subsection we decide which set to send in the main part of the state based on which state was last changed. Dave Dr. David Alan Gilbert (6): e1000: Convert v3 fields to subsection e1000: Dupe offload data on reading old stream e1000: wire new subsection to property e1000: Migrate props via a temporary structure e1000: Choose which set of props to migrate e1000: Old machine types, turn new subsection off hw/net/e1000.c | 103 ++++++++++++++++++++++++++++++++++++++++------------ include/hw/compat.h | 4 ++ 2 files changed, 84 insertions(+), 23 deletions(-) -- 2.14.3