Re: [PATCH] serial: Fix double migration data

2020-03-31 Thread Paolo Bonzini
On 30/03/20 19:56, Marc-André Lureau wrote: >>> Why do you make it a sub-state? >> Because it's consistent with serial-isa and it's simple. > ok, lgtm then > > Reviewed-by: Marc-André Lureau Queued, thanks. Paolo

Re: [PATCH] serial: Fix double migration data

2020-03-30 Thread Marc-André Lureau
Hi On Mon, Mar 30, 2020 at 7:41 PM Dr. David Alan Gilbert wrote: > > * Marc-André Lureau (marcandre.lur...@gmail.com) wrote: > > Hi > > > > On Mon, Mar 30, 2020 at 6:47 PM Dr. David Alan Gilbert (git) > > wrote: > > > > > > From: "Dr. David Alan Gilbert" > > > > > > After c9808d60281 we have bo

Re: [PATCH] serial: Fix double migration data

2020-03-30 Thread Dr. David Alan Gilbert
* Marc-André Lureau (marcandre.lur...@gmail.com) wrote: > Hi > > On Mon, Mar 30, 2020 at 6:47 PM Dr. David Alan Gilbert (git) > wrote: > > > > From: "Dr. David Alan Gilbert" > > > > After c9808d60281 we have both an object representing the serial-isa > > device and a separate object representing

Re: [PATCH] serial: Fix double migration data

2020-03-30 Thread Marc-André Lureau
Hi On Mon, Mar 30, 2020 at 6:47 PM Dr. David Alan Gilbert (git) wrote: > > From: "Dr. David Alan Gilbert" > > After c9808d60281 we have both an object representing the serial-isa > device and a separate object representing the underlying common serial > uart. Both of these have vmsd's associate

[PATCH] serial: Fix double migration data

2020-03-30 Thread Dr. David Alan Gilbert (git)
From: "Dr. David Alan Gilbert" After c9808d60281 we have both an object representing the serial-isa device and a separate object representing the underlying common serial uart. Both of these have vmsd's associated with them and thus the migration stream ends up with two copies of the migration d