Skip to content

Commit

Permalink
migration: unexport migrate_fd_error()
Browse files Browse the repository at this point in the history
The only migrate_fd_error() call sites are in "migration/migration.c",
which is also where we define migrate_fd_error(). Make the function
static, and remove its declaration from "migration/migration.h".

Cc: Juan Quintela <quintela@redhat.com> (maintainer:Migration)
Cc: Leonardo Bras <leobras@redhat.com> (reviewer:Migration)
Cc: Peter Xu <peterx@redhat.com> (reviewer:Migration)
Cc: qemu-trivial@nongnu.org
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2018404
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Michael Tokarev <mjt@tls.msk.ru>
Reviewed-by: Peter Xu <peterx@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
  • Loading branch information
lersek authored and Michael Tokarev committed Jul 8, 2023
1 parent 8c69ae9 commit aaf26bd
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion migration/migration.c
Original file line number Diff line number Diff line change
Expand Up @@ -1220,7 +1220,7 @@ static void migrate_error_free(MigrationState *s)
}
}

void migrate_fd_error(MigrationState *s, const Error *error)
static void migrate_fd_error(MigrationState *s, const Error *error)
{
trace_migrate_fd_error(error_get_pretty(error));
assert(s->to_dst_file == NULL);
Expand Down
1 change: 0 additions & 1 deletion migration/migration.h
Original file line number Diff line number Diff line change
Expand Up @@ -466,7 +466,6 @@ bool migration_has_all_channels(void);
uint64_t migrate_max_downtime(void);

void migrate_set_error(MigrationState *s, const Error *error);
void migrate_fd_error(MigrationState *s, const Error *error);

void migrate_fd_connect(MigrationState *s, Error *error_in);

Expand Down

0 comments on commit aaf26bd

Please sign in to comment.