On Fri, Apr 07, 2017 at 10:46:55 +0200, Peter Krempa wrote:
On Wed, Apr 05, 2017 at 15:10:01 +0200, Jiri Denemark wrote:
> This new API is supposed to reset all migration parameters to make sure
> future migrations won't accidentally use them. This patch makes the
> first step and moves qemuMigrationResetTLS call inside
> qemuMigrationReset.
>
> Signed-off-by: Jiri Denemark <jdenemar(a)redhat.com>
> ---
> src/qemu/qemu_migration.c | 31 +++++++++++++++++++++++--------
> src/qemu/qemu_migration.h | 5 +++++
> src/qemu/qemu_process.c | 4 ++--
> 3 files changed, 30 insertions(+), 10 deletions(-)
[...]
> diff --git a/src/qemu/qemu_migration.h b/src/qemu/qemu_migration.h
> index 5248f399d..28eb55056 100644
> --- a/src/qemu/qemu_migration.h
> +++ b/src/qemu/qemu_migration.h
> @@ -285,4 +285,9 @@ qemuMigrationResetTLS(virQEMUDriverPtr driver,
> char *in_tlsAlias,
> char *in_secAlias);
>
> +void
> +qemuMigrationReset(virQEMUDriverPtr driver,
> + virDomainObjPtr vm,
> + qemuDomainAsyncJob job);
> +
This does not conform to the prevailing coding style in the file.
It does since the style is pretty inconsistent there. Anyway, see patch
10/8 :-)
Jirka