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(-)
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. @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(-) -- 2.11.2
On Tue, 11 Jul 2017 16:54:35 +0200 Halil Pasic <pasic@linux.vnet.ibm.com> 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(-) >
* Halil Pasic (pasic@linux.vnet.ibm.com) 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. > > @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... No problem, as long as we've got a happy migration that's enough, and we've finally got VMSTATE_QTAILQ used as well. Dave > 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(-) > > -- > 2.11.2 > -- Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
On 07/11/2017 04:54 PM, 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. > > @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(-) > applied to s390-next. Thanks
On 07/11/2017 04:54 PM, 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. > > @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(-) Can you clarify what test you have done regarding compatibility migrations to 2.9 and earlier?
On 07/12/2017 10:01 AM, Christian Borntraeger wrote: > On 07/11/2017 04:54 PM, 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. >> >> @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(-) > > Can you clarify what test you have done regarding compatibility migrations to 2.9 and earlier? > Hi Christian! Again the same. I was migrating a guests with just some virtio-blk devices between my new qemu (with this patch set on top of master) and itself with default machine. And then between qemu with machine virtio-ccw-2.5 and a 2.5 binary with machine virtio-ccw-2.5 (2.5 the binary is some old "internal driver"). The old and the new binary where tested in both (source and target) roles. I've verified that the disks are still working after migration. Given the nature of the changes I didn't do a great number of repeated tests, as was doing all the migrations within the same host via HMP command. If one has an appropriate automated setup (I don't) I would welcome some more testing (and be happy to add a Tested-by), but I don't think it is strictly necessary. Regards, Halil
On 07/12/2017 01:06 PM, Halil Pasic wrote: > > > On 07/12/2017 10:01 AM, Christian Borntraeger wrote: >> On 07/11/2017 04:54 PM, 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. >>> >>> @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(-) >> >> Can you clarify what test you have done regarding compatibility migrations to 2.9 and earlier? >> > > Hi Christian! > > Again the same. I was migrating a guests with just some virtio-blk devices > between my new qemu (with this patch set on top of master) and itself with > default machine. And then between qemu with machine virtio-ccw-2.5 and a > 2.5 binary with machine virtio-ccw-2.5 (2.5 the binary is some old > "internal driver"). The old and the new binary where tested in both (source > and target) roles. I've verified that the disks are still working after > migration. > > Given the nature of the changes I didn't do a great number of repeated tests, > as was doing all the migrations within the same host via HMP command. > > If one has an appropriate automated setup (I don't) I would welcome some > more testing (and be happy to add a Tested-by), but I don't think it is > strictly necessary. No thats fine. I just want to have this kind of testing done by the developer. Thank you for doing so. If you have done it, you can always say it in your cover letter
© 2016 - 2024 Red Hat, Inc.