[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 8/8] tests: add dbus-vmstate-test
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v6 8/8] tests: add dbus-vmstate-test |
Date: |
Mon, 16 Dec 2019 09:58:12 +0000 |
User-agent: |
Mutt/1.12.1 (2019-06-15) |
On Fri, Dec 13, 2019 at 06:20:15PM +0000, Dr. David Alan Gilbert wrote:
> * Marc-André Lureau (address@hidden) wrote:
> > Signed-off-by: Marc-André Lureau <address@hidden>
>
> > +static gboolean
> > +vmstate_save(VMState1 *object, GDBusMethodInvocation *invocation,
> > + gpointer user_data)
> > +{
> > + TestServer *h = user_data;
> > + GVariant *var;
> > +
> > + var = g_variant_new_fixed_array(G_VARIANT_TYPE_BYTE,
> > + h->id->data, h->id->size,
> > sizeof(char));
> > + g_dbus_method_invocation_return_value(invocation,
> > + g_variant_new("(@ay)", var));
> > + h->save_called = true;
> > +
> > + return TRUE;
> > +}
> > +
> > +static gboolean
> > +wait_for_migration_complete(gpointer user_data)
>
> It's a shame we don't have a way to share this with migration-test.c;
> we occasionally add more debug/cases in there.
Easy enough to create a tests/migration-helpers.{c,h} file to share
code between tests.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- Re: [PATCH v6 5/8] Add dbus-vmstate object, (continued)