From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46547) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dVCXk-0002br-QZ for qemu-devel@nongnu.org; Wed, 12 Jul 2017 03:58:45 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dVCXj-0002uo-Uo for qemu-devel@nongnu.org; Wed, 12 Jul 2017 03:58:44 -0400 Received: from mx1.redhat.com ([209.132.183.28]:51098) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dVCXj-0002u4-OY for qemu-devel@nongnu.org; Wed, 12 Jul 2017 03:58:43 -0400 Date: Wed, 12 Jul 2017 09:58:37 +0200 From: Cornelia Huck Message-ID: <20170712095837.69d1446f@dhcp-192-215.str.redhat.com> In-Reply-To: <20170711145441.33925-1-pasic@linux.vnet.ibm.com> References: <20170711145441.33925-1-pasic@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v3 0/6] migration: s390x css migration List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Halil Pasic Cc: Christian Borntraeger , "Dr. David Alan Gilbert" , "Jason J . Herne" , Juan Quintela , Cornelia Huck , Dong Jia Shi , Thomas Huth , qemu-devel@nongnu.org On Tue, 11 Jul 2017 16:54:35 +0200 Halil Pasic wrote: > Like for v2 the scope of this patch series is now limited to decoupling > channel subsystem migration from the migration of virtio-ccw proxies. > > There wasn't a whole lot of criticism regarding v2, so very little > changed since then. All issues identified in the previous round were > addressed as agreed upon. I hope this series is now good for 2.10. I did a quick readthrough again and I agree that this series is good for 2.10. > > @Dave You helped me quite a lot with these, thanks a lot! I could not > find any r-b's or similar form you so if you would like to take the > well deserved credit for your work... > > v2 --> v3: > * rebased onto current master, since 's390x: vmstatify config migration > for virtio-ccw' is already there we don't need it here any more > * added ack's and r-b' > * got rid of loads of nits (thanks Connie and Thomas) > * added explanation why certain members are not migrated > > v1 --> v2: > * Split out the vmystatify virtio-ccw part, reorganize > * Use WITH_TMP instead of one-shot VMStateInfo's > > Halil Pasic (6): > s390x: add helper get_machine_class > s390x: add css_migration_enabled to machine class > s390x/css: add missing css state conditionally > s390x/css: add ORB to SubchDev > s390x/css: activate ChannelSubSys migration > s390x/css: use SubchDev.orb > > hw/intc/s390_flic.c | 20 ++++++ > hw/s390x/css.c | 144 +++++++++++++++++++++++++++++++++---- > hw/s390x/s390-virtio-ccw.c | 58 +++++++++------ > include/hw/s390x/css.h | 11 ++- > include/hw/s390x/s390-virtio-ccw.h | 7 ++ > 5 files changed, 199 insertions(+), 41 deletions(-) >