Re: [Qemu-devel] [PATCH 3/6] e1000e: fix for migration compatibility

2016-08-18 Thread Paolo Bonzini
On 18/08/2016 15:25, Cao jin wrote: > > > On 08/18/2016 09:04 PM, Paolo Bonzini wrote: >> >> >> On 18/08/2016 15:11, Cao jin wrote: >>> >>> >>> On 08/18/2016 06:47 PM, Paolo Bonzini wrote: On 17/08/2016 16:39, Cao jin wrote: > commit 66bf7d58 removed internal msi state flag E

Re: [Qemu-devel] [PATCH 3/6] e1000e: fix for migration compatibility

2016-08-18 Thread Cao jin
On 08/18/2016 09:04 PM, Paolo Bonzini wrote: On 18/08/2016 15:11, Cao jin wrote: On 08/18/2016 06:47 PM, Paolo Bonzini wrote: On 17/08/2016 16:39, Cao jin wrote: commit 66bf7d58 removed internal msi state flag E1000E_USE_MSI, but it is used by intr_state which exists in vmstate. Restor

Re: [Qemu-devel] [PATCH 3/6] e1000e: fix for migration compatibility

2016-08-18 Thread Paolo Bonzini
On 18/08/2016 15:11, Cao jin wrote: > > > On 08/18/2016 06:47 PM, Paolo Bonzini wrote: >> >> >> On 17/08/2016 16:39, Cao jin wrote: >>> commit 66bf7d58 removed internal msi state flag E1000E_USE_MSI, but it >>> is used by intr_state which exists in vmstate. Restore it for migration >>> to older

Re: [Qemu-devel] [PATCH 3/6] e1000e: fix for migration compatibility

2016-08-18 Thread Cao jin
On 08/18/2016 06:47 PM, Paolo Bonzini wrote: On 17/08/2016 16:39, Cao jin wrote: commit 66bf7d58 removed internal msi state flag E1000E_USE_MSI, but it is used by intr_state which exists in vmstate. Restore it for migration to older QEMU versions CC: Dmitry Fleytman CC: Jason Wang CC: Mar

Re: [Qemu-devel] [PATCH 3/6] e1000e: fix for migration compatibility

2016-08-18 Thread Paolo Bonzini
On 17/08/2016 16:39, Cao jin wrote: > commit 66bf7d58 removed internal msi state flag E1000E_USE_MSI, but it > is used by intr_state which exists in vmstate. Restore it for migration > to older QEMU versions > > CC: Dmitry Fleytman > CC: Jason Wang > CC: Markus Armbruster > CC: Marcel Apfelba

Re: [Qemu-devel] [PATCH 3/6] e1000e: fix for migration compatibility

2016-08-17 Thread Dmitry Fleytman
Acked-by: Dmitry Fleytman > On 17 Aug 2016, at 17:39, Cao jin wrote: > > commit 66bf7d58 removed internal msi state flag E1000E_USE_MSI, but it > is used by intr_state which exists in vmstate. Restore it for migration > to older QEMU versions > > CC: Dmitry Fleytman > CC: Jason Wang > CC: Ma

[Qemu-devel] [PATCH 3/6] e1000e: fix for migration compatibility

2016-08-17 Thread Cao jin
commit 66bf7d58 removed internal msi state flag E1000E_USE_MSI, but it is used by intr_state which exists in vmstate. Restore it for migration to older QEMU versions CC: Dmitry Fleytman CC: Jason Wang CC: Markus Armbruster CC: Marcel Apfelbaum CC: Michael S. Tsirkin Signed-off-by: Cao jin --