Re: [Qemu-devel] [PATCH v2 1/1] s390x: vmstatify config migration for virtio-ccw

2017-06-06 Thread Halil Pasic
On 06/06/2017 08:07 PM, Christian Borntraeger wrote: > On 06/06/2017 08:02 PM, Halil Pasic wrote: >> >> >> On 06/06/2017 10:21 AM, Christian Borntraeger wrote: >>> On 06/02/2017 04:05 PM, Halil Pasic wrote: Let's vmstatify virtio_ccw_save_config and virtio_ccw_load_config for flexibilit

Re: [Qemu-devel] [PATCH v2 1/1] s390x: vmstatify config migration for virtio-ccw

2017-06-06 Thread Christian Borntraeger
On 06/06/2017 08:02 PM, Halil Pasic wrote: > > > On 06/06/2017 10:21 AM, Christian Borntraeger wrote: >> On 06/02/2017 04:05 PM, Halil Pasic wrote: >>> Let's vmstatify virtio_ccw_save_config and virtio_ccw_load_config for >>> flexibility (extending using subsections) and for fun. >>> >>> To achie

Re: [Qemu-devel] [PATCH v2 1/1] s390x: vmstatify config migration for virtio-ccw

2017-06-06 Thread Halil Pasic
On 06/06/2017 10:21 AM, Christian Borntraeger wrote: > On 06/02/2017 04:05 PM, Halil Pasic wrote: >> Let's vmstatify virtio_ccw_save_config and virtio_ccw_load_config for >> flexibility (extending using subsections) and for fun. >> >> To achieve this we need to hack the config_vector, which is Vi

Re: [Qemu-devel] [PATCH v2 1/1] s390x: vmstatify config migration for virtio-ccw

2017-06-06 Thread Christian Borntraeger
On 06/02/2017 04:05 PM, Halil Pasic wrote: > Let's vmstatify virtio_ccw_save_config and virtio_ccw_load_config for > flexibility (extending using subsections) and for fun. > > To achieve this we need to hack the config_vector, which is VirtIODevice > (that is common virtio) state, in the middle of

Re: [Qemu-devel] [PATCH v2 1/1] s390x: vmstatify config migration for virtio-ccw

2017-06-05 Thread Dong Jia Shi
* Eric Blake [2017-06-05 07:19:14 -0500]: Hi Eric, > On 06/04/2017 10:09 PM, Dong Jia Shi wrote: > > * Halil Pasic [2017-06-02 16:05:31 +0200]: > > > > Hi Halil, > > > > Sorry for the late show up. I just found some nits, which could be > > ignored for me. > > > >> Let's vmstatify virtio_ccw

Re: [Qemu-devel] [PATCH v2 1/1] s390x: vmstatify config migration for virtio-ccw

2017-06-05 Thread Eric Blake
On 06/04/2017 10:09 PM, Dong Jia Shi wrote: > * Halil Pasic [2017-06-02 16:05:31 +0200]: > > Hi Halil, > > Sorry for the late show up. I just found some nits, which could be > ignored for me. > >> Let's vmstatify virtio_ccw_save_config and virtio_ccw_load_config for >> flexibility (extending us

Re: [Qemu-devel] [PATCH v2 1/1] s390x: vmstatify config migration for virtio-ccw

2017-06-04 Thread Dong Jia Shi
* Halil Pasic [2017-06-02 16:05:31 +0200]: Hi Halil, Sorry for the late show up. I just found some nits, which could be ignored for me. > Let's vmstatify virtio_ccw_save_config and virtio_ccw_load_config for > flexibility (extending using subsections) and for fun. > > To achieve this we need t

[Qemu-devel] [PATCH v2 1/1] s390x: vmstatify config migration for virtio-ccw

2017-06-02 Thread Halil Pasic
Let's vmstatify virtio_ccw_save_config and virtio_ccw_load_config for flexibility (extending using subsections) and for fun. To achieve this we need to hack the config_vector, which is VirtIODevice (that is common virtio) state, in the middle of the VirtioCcwDevice state representation. This is s