Message ID | 1436951080-12390-1-git-send-email-quintela@redhat.com |
---|---|
State | New |
Headers | show |
On (Wed) 15 Jul 2015 [11:04:34], Juan Quintela wrote: > Hi > > This pull requset: > - small optimization by Li > - fixes global state with savevm (reported by John Snow) > - get migration traces for migration state changes even if migration > events are not enabled > - Change state after migration has finished (reported by Christian) > - add documentation for events > > Please, Apply. Peter, can you fix the typos in my name in patches 5 and 6, or do you want a new pull req? Thanks, Amit
Self-Nack Misspelled Amit name, resending with name fixed. Sorry. Juan Quintela <quintela@redhat.com> wrote: > Hi > > This pull requset: > - small optimization by Li > - fixes global state with savevm (reported by John Snow) > - get migration traces for migration state changes even if migration > events are not enabled > - Change state after migration has finished (reported by Christian) > - add documentation for events > > Please, Apply. > > PD. Yes, it had to pass: > make check > virt-test > savevm by hand > > > The following changes since commit 661725da09f47eb92d356fac10a4cf3b7ad1f61d: > > Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20150714' into staging (2015-07-14 18:50:17 +0100) > > are available in the git repository at: > > git://github.com/juanquintela/qemu.git tags/migration/20150715 > > for you to fetch changes up to 4915a5ad93e6c8ac77e2e5a007be178d50c7b019: > > migration: We also want to store the global state for savevm (2015-07-15 10:51:05 +0200) > > ---------------------------------------------------------------- > migration/next for 20150715 > > ---------------------------------------------------------------- > Juan Quintela (5): > migration: Only change state after migration has finished > migration: Trace event and migration event are different things > migration: Write documetation for events capabilites > migration: Register global state section before loadvm > migration: We also want to store the global state for savevm > > Liang Li (1): > migration: reduce the count of strlen call > > include/migration/migration.h | 1 + > migration/migration.c | 52 ++++++++++++++++++++++++------------------- > migration/ram.c | 10 ++++----- > migration/savevm.c | 6 +++++ > qmp-commands.hx | 1 + > vl.c | 2 +- > 6 files changed, 43 insertions(+), 29 deletions(-)
On 15 July 2015 at 11:16, Amit Shah <amit.shah@redhat.com> wrote: > On (Wed) 15 Jul 2015 [11:04:34], Juan Quintela wrote: >> Hi >> >> This pull requset: >> - small optimization by Li >> - fixes global state with savevm (reported by John Snow) >> - get migration traces for migration state changes even if migration >> events are not enabled >> - Change state after migration has finished (reported by Christian) >> - add documentation for events >> >> Please, Apply. > > Peter, can you fix the typos in my name in patches 5 and 6, or do you > want a new pull req? I've applied Juan's fixed pull. For the record, I always need a new pullreq -- it's not possible to edit the commits in a pull, I can only edit if I'm applying patches individually to master. thanks -- PMM
Peter Maydell <peter.maydell@linaro.org> wrote: > On 15 July 2015 at 11:16, Amit Shah <amit.shah@redhat.com> wrote: >> On (Wed) 15 Jul 2015 [11:04:34], Juan Quintela wrote: >>> Hi >>> >>> This pull requset: >>> - small optimization by Li >>> - fixes global state with savevm (reported by John Snow) >>> - get migration traces for migration state changes even if migration >>> events are not enabled >>> - Change state after migration has finished (reported by Christian) >>> - add documentation for events >>> >>> Please, Apply. >> >> Peter, can you fix the typos in my name in patches 5 and 6, or do you >> want a new pull req? > > I've applied Juan's fixed pull. For the record, I always need a > new pullreq -- it's not possible to edit the commits in a pull, > I can only edit if I'm applying patches individually to master. I used a different algorithm. What is easier for Peter? O:-) Thanks, Juan.