Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate

2014-08-28 Thread Paolo Bonzini
Il 28/08/2014 10:31, Pavel Dovgaluk ha scritto: >> From: Paolo Bonzini [mailto:paolo.bonz...@gmail.com] On Behalf Of Paolo >> Bonzini >> Il 27/08/2014 12:48, Pavel Dovgaluk ha scritto: From: Paolo Bonzini [mailto:pbonz...@redhat.com] Il 27/08/2014 12:30, Pavel Dovgaluk ha scritto: >>

Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate

2014-08-28 Thread Pavel Dovgaluk
> From: Paolo Bonzini [mailto:paolo.bonz...@gmail.com] On Behalf Of Paolo > Bonzini > Il 27/08/2014 12:48, Pavel Dovgaluk ha scritto: > >> From: Paolo Bonzini [mailto:pbonz...@redhat.com] > >> Il 27/08/2014 12:30, Pavel Dovgaluk ha scritto: > From: Paolo Bonzini [mailto:pbonz...@redhat.com] >

Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate

2014-08-27 Thread Paolo Bonzini
Il 27/08/2014 12:48, Pavel Dovgaluk ha scritto: >> From: Paolo Bonzini [mailto:pbonz...@redhat.com] >> Il 27/08/2014 12:30, Pavel Dovgaluk ha scritto: From: Paolo Bonzini [mailto:pbonz...@redhat.com] Il 27/08/2014 12:15, Pavel Dovgaluk ha scritto: >>> Again, this is only needed in you

Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate

2014-08-27 Thread Pavel Dovgaluk
> From: Paolo Bonzini [mailto:pbonz...@redhat.com] > Il 27/08/2014 12:30, Pavel Dovgaluk ha scritto: > >> From: Paolo Bonzini [mailto:pbonz...@redhat.com] > >> Il 27/08/2014 12:15, Pavel Dovgaluk ha scritto: > > Again, this is only needed in your record/replay system (and you haven't > > ye

Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate

2014-08-27 Thread Paolo Bonzini
Il 27/08/2014 12:30, Pavel Dovgaluk ha scritto: >> From: Paolo Bonzini [mailto:pbonz...@redhat.com] >> Il 27/08/2014 12:15, Pavel Dovgaluk ha scritto: > Again, this is only needed in your record/replay system (and you haven't > yet quite explained why the design has this limitation), so it

Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate

2014-08-27 Thread Pavel Dovgaluk
> From: Paolo Bonzini [mailto:pbonz...@redhat.com] > Il 27/08/2014 12:15, Pavel Dovgaluk ha scritto: > >> > Again, this is only needed in your record/replay system (and you haven't > >> > yet quite explained why the design has this limitation), so it should > >> > not be a part of this series. > >

Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate

2014-08-27 Thread Paolo Bonzini
Il 27/08/2014 12:15, Pavel Dovgaluk ha scritto: >> > Again, this is only needed in your record/replay system (and you haven't >> > yet quite explained why the design has this limitation), so it should >> > not be a part of this series. > I see. Updating s->timer and s->TimerExpire in post_load wi

Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate

2014-08-27 Thread Pavel Dovgaluk
> From: Paolo Bonzini [mailto:paolo.bonz...@gmail.com] On Behalf Of Paolo > Bonzini > Il 26/08/2014 09:15, Pavel Dovgalyuk ha scritto: > > This patch adds virtual clock-dependent timers to VMState to allow correct > > saving and restoring the state of RTL8139 network controller. > > > > Signed-off

Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate

2014-08-26 Thread Paolo Bonzini
Il 26/08/2014 09:15, Pavel Dovgalyuk ha scritto: > This patch adds virtual clock-dependent timers to VMState to allow correct > saving and restoring the state of RTL8139 network controller. > > Signed-off-by: Pavel Dovgalyuk > --- > hw/net/rtl8139.c | 50 +++