On Mon, Mar 27, 2017 at 14:30:49 -0400, John Ferlan wrote:
On 03/24/2017 11:40 AM, Jiri Denemark wrote:
> This series is a first step in refactoring migration cookies. The rest
> will take some time until it's done, but I figured the part which moves
> the code is very fragile and I don't really want to keep rebasing it
> until the refactoring is finished.
>
> Jiri Denemark (4):
> qemu: Move migration cookies to a separate file
> qemu: Fix formatting in qemu_migration_cookie.c
> qemu: Typedef migration cookie enums
> qemu: Free persistent def inside qemuMigrationCookieFree
>
> po/POTFILES.in | 1 +
> src/Makefile.am | 1 +
> src/qemu/qemu_migration.c | 1425 +-------------------------------------
> src/qemu/qemu_migration_cookie.c | 1348 ++++++++++++++++++++++++++++++++++++
> src/qemu/qemu_migration_cookie.h | 153 ++++
> 5 files changed, 1512 insertions(+), 1416 deletions(-)
> create mode 100644 src/qemu/qemu_migration_cookie.c
> create mode 100644 src/qemu/qemu_migration_cookie.h
>
I'm sure you already realize you had a conflict with the TLS series -
only need to be sure to keep the QEMU_MIGRATION_TLS_ALIAS_BASE
Yeah, this is the main reason why I wanted to push this patch before
starting to work on refactoring the code.
FWIW: while not required, the end of the new .h file should have
#endif /* __QEMU_MIGRATION_COOKIE_H__ */
Added and pushed. Thanks.
Jirka