[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/9] migration: Export qemu-file-channel.c funct
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH 3/9] migration: Export qemu-file-channel.c functions in its own file |
Date: |
Thu, 18 May 2017 15:26:23 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Peter Xu <address@hidden> wrote:
> On Wed, May 17, 2017 at 05:47:50PM +0200, Juan Quintela wrote:
>> Signed-off-by: Juan Quintela <address@hidden>
>> ---
>> include/migration/migration.h | 1 +
>> include/migration/qemu-file.h | 4 ----
>> migration/channel.c | 1 +
>> migration/colo.c | 1 +
>> migration/migration.c | 1 +
>> migration/qemu-file-channel.c | 1 +
>> migration/qemu-file-channel.h | 21 +++++++++++++++++++++
>> migration/rdma.c | 1 +
>> migration/savevm.c | 1 +
>> tests/test-vmstate.c | 1 +
>> 10 files changed, 29 insertions(+), 4 deletions(-)
>> create mode 100644 migration/qemu-file-channel.h
>>
>> diff --git a/include/migration/migration.h b/include/migration/migration.h
>> index e831259..8280df1 100644
>> --- a/include/migration/migration.h
>> +++ b/include/migration/migration.h
>> @@ -19,6 +19,7 @@
>> #include "qemu/thread.h"
>> #include "qemu/notify.h"
>> #include "migration/vmstate.h"
>> +#include "io/channel.h"
>
> Could I ask why we add this line here? I thought one of the main goals
> of this series is removing things from migration.h...
I remove from include/migration/qemu-file.h
-#include "io/channel.h"
Because all the QIOChannel functions in qemu-file.h are moved to
qemu-file-channel.h.
Great!
But migration/vmstate.h includes qemu-file.h
And migration.h includes vmstate.h
And migration.h has this functions:
void qemu_start_incoming_migration(const char *uri, Error **errp);
QIOChannel *ioc,
Error **errp);
void migration_tls_channel_connect(MigrationState *s,
QIOChannel *ioc,
const char *hostname,
Error **errp);
And nothing else declares the QIOChannel.
So, the easy solution so far is to include this by now to maintain
compilation.
BTW, good catch.
So, I think that there is no other good solution to untangle the mess.
Later, Juan.
[Qemu-devel] [PATCH 4/9] migration: Remove migration.h from colo.h, Juan Quintela, 2017/05/17
[Qemu-devel] [PATCH 5/9] migration: Move qjson.h to migration/, Juan Quintela, 2017/05/17
[Qemu-devel] [PATCH 7/9] migration: Remove qemu-file.h from vmstate.h, Juan Quintela, 2017/05/17