Re: [Qemu-devel] [PATCH] migration/colo.c: Fix compilation issue when disable replication

[Dr. David Alan Gilbert]

* Dr. David Alan Gilbert (address@hidden) wrote:
> * Zhang Chen (address@hidden) wrote:
> > This compilation issue will occur when user use --disable-replication
> > to config Qemu.
> > 
> > Reported-by: Thomas Huth <address@hidden>
> > Signed-off-by: Zhang Chen <address@hidden>
> 
> Queued.

Unqueued; this still isn't building for me with replication disabled;
I'm hitting one problem where the compiler is complaining that
colo_send_message_value is unused, and then I'm hitting a bunch of link
time errors.

This needs a few more fixes first.

Dave

> > ---
> >  migration/colo.c | 18 +++++++++++++++++-
> >  1 file changed, 17 insertions(+), 1 deletion(-)
> > 
> > diff --git a/migration/colo.c b/migration/colo.c
> > index 956ac236b7..b81e6ec5bb 100644
> > --- a/migration/colo.c
> > +++ b/migration/colo.c
> > @@ -59,6 +59,8 @@ static bool colo_runstate_is_stopped(void)
> >  
> >  static void secondary_vm_do_failover(void)
> >  {
> > +/* COLO needs enable block-replication */
> > +#ifdef CONFIG_REPLICATION
> >      int old_state;
> >      MigrationIncomingState *mis = migration_incoming_get_current();
> >      Error *local_err = NULL;
> > @@ -121,10 +123,14 @@ static void secondary_vm_do_failover(void)
> >      if (mis->migration_incoming_co) {
> >          qemu_coroutine_enter(mis->migration_incoming_co);
> >      }
> > +#else
> > +    abort();
> > +#endif
> >  }
> >  
> >  static void primary_vm_do_failover(void)
> >  {
> > +#ifdef CONFIG_REPLICATION
> >      MigrationState *s = migrate_get_current();
> >      int old_state;
> >      Error *local_err = NULL;
> > @@ -165,6 +171,9 @@ static void primary_vm_do_failover(void)
> >  
> >      /* Notify COLO thread that failover work is finished */
> >      qemu_sem_post(&s->colo_exit_sem);
> > +#else
> > +    abort();
> > +#endif
> >  }
> >  
> >  COLOMode get_colo_mode(void)
> > @@ -373,6 +382,7 @@ static int 
> > colo_do_checkpoint_transaction(MigrationState *s,
> >                                            QIOChannelBuffer *bioc,
> >                                            QEMUFile *fb)
> >  {
> > +#ifdef CONFIG_REPLICATION
> >      Error *local_err = NULL;
> >      int ret = -1;
> >  
> > @@ -483,6 +493,9 @@ out:
> >          error_report_err(local_err);
> >      }
> >      return ret;
> > +#else
> > +    abort();
> > +#endif
> >  }
> >  
> >  static void colo_compare_notify_checkpoint(Notifier *notifier, void *data)
> > @@ -790,13 +803,16 @@ void *colo_process_incoming_thread(void *opaque)
> >              qemu_mutex_unlock_iothread();
> >              goto out;
> >          }
> > +#ifdef CONFIG_REPLICATION
> >          /* discard colo disk buffer */
> >          replication_do_checkpoint_all(&local_err);
> >          if (local_err) {
> >              qemu_mutex_unlock_iothread();
> >              goto out;
> >          }
> > -
> > +#else
> > +        abort();
> > +#endif
> >          /* Notify all filters of all NIC to do checkpoint */
> >          colo_notify_filters_event(COLO_EVENT_CHECKPOINT, &local_err);
> >  
> > -- 
> > 2.17.GIT
> > 
> --
> Dr. David Alan Gilbert / address@hidden / Manchester, UK
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK