Re: [PATCH 2/3] Consolidate documentation of virDomainMigrate{, ToURI}{, 2, 3}

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Nov 29, 2016 at 13:04:21 +0100, Peter Krempa wrote:
> On Fri, Nov 25, 2016 at 11:51:42 +0100, Jiri Denemark wrote:
> > Only the latest APIs are fully documented and the documentation of the
> > older variants (which are just limited versions of the new APIs anyway)
> > points to the newest APIs.
> > 
> > Signed-off-by: Jiri Denemark <jdenemar@xxxxxxxxxx>
> > ---
> >  src/libvirt-domain.c | 348 ++++++---------------------------------------------
> >  1 file changed, 36 insertions(+), 312 deletions(-)
> > 
> > diff --git a/src/libvirt-domain.c b/src/libvirt-domain.c
> > index f9f5a4696..e0c69dbe7 100644
> > --- a/src/libvirt-domain.c
> > +++ b/src/libvirt-domain.c
> 
> [...]
> 
> > @@ -3984,19 +3837,6 @@ virDomainMigrate2(virDomainPtr domain,
> 
> This hunk corresponds to virDomainMigrate3 documentation ... 
> 
> >   * VIR_MIGRATE_TUNNELLED and VIR_MIGRATE_PEER2PEER are not supported by this
> >   * API, use virDomainMigrateToURI3 instead.
> >   *
> > - * If you want to copy non-shared storage within migration you
> > - * can use either VIR_MIGRATE_NON_SHARED_DISK or
> > - * VIR_MIGRATE_NON_SHARED_INC as they are mutually exclusive.
> > - *
> > - * Users should note that implementation of VIR_MIGRATE_OFFLINE
> > - * flag in some drivers does not copy storage or any other file
> > - * based storage (e.g. UEFI variable storage).
> > - *
> > - * Enabling the VIR_MIGRATE_POSTCOPY flag tells libvirt to enable post-copy
> > - * migration.  Use virDomainMigrateStartPostCopy to switch migration into
> > - * the post-copy mode.  See virDomainMigrateStartPostCopy for more details
> > - * about post-copy.
> > - *
> >   * There are many limitations on migration imposed by the underlying
> >   * technology - for example it may not be possible to migrate between
> >   * different processors even with the same architecture, or between
> 
> The deletion of the stuff above is okay, but the function docs miss the
> same pointer to the fully documented function. Looks like this was a
> pre-existing problem but you could fix it with this patch.
> 
> ACK with the above tweak. Safe for freeze.

OK, I added a pointer to VIR_MIGRATE_PARAM_* typed parameters for
further documentation and pushed the series. Thanks.

Jirka

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux