Re: [PATCHv3 06/12] conf: Add helper for listing domains on drivers supporting virDomainObj

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

 



On 06/11/2012 04:34 AM, Peter Krempa wrote:
> 
> This patch adds common code to list domains in fashion used by
> virListAllDomains with all currently supported flags. The header file
> also contains macros that group filters together that are used to
> shorten filter conditions.
> ---
> Diff to v2:
> -added macros grouping filter flags
> -Added support for managed save filtering, now that virDomainObj holds this property
> - Removed the now unneeded filtering function
> - Changed allocation of the return list, now It's allocated before for the full
>  length and it's trimmed afterwards
> - Added newline after the DOMAIN_LIST_SOURCES definition in Makefile.am
> ---
>  src/Makefile.am          |    8 ++-
>  src/conf/virdomainlist.c |  182 ++++++++++++++++++++++++++++++++++++++++++++++
>  src/conf/virdomainlist.h |   66 +++++++++++++++++
>  src/libvirt_private.syms |    4 +
>  4 files changed, 259 insertions(+), 1 deletions(-)
>  create mode 100644 src/conf/virdomainlist.c
>  create mode 100644 src/conf/virdomainlist.h

I see why you needed the new file, after all, since I hit the same link
problem when trying to add a helper for snapshot listing.  But since I
am now proposing to use it for snapshots, maybe we should think about
naming it virobjlist.[hc] (a file for all sorts of virObjects:
virDomainPtr, virDomainSnapshotPtr, ...).  Or maybe not; right now,
domain snapshots are mostly handled by domain_conf.c instead of being
split into a separate file.

> +
> +    /* managed save filter function is provided from the driver */
> +    if (MATCH(VIR_CONNECT_LIST_FILTERS_MANAGEDSAVE) &&

Comment is outdated.

> +    /* filter by snapshot existence */
> +    if (MATCH(VIR_CONNECT_LIST_FILTERS_SNAPSHOT)) {
> +        int nsnap = virDomainSnapshotObjListNum(&vm->snapshots, 0);

I've got a pending patch which will conflict with this; I'll post my v2
series as if this patch had already been applied.

> +
> +# define VIR_CONNECT_LIST_FILTERS_ACTIVE           \
> +                (VIR_CONNECT_LIST_DOMAINS_ACTIVE | \
> +                 VIR_CONNECT_LIST_DOMAINS_INACTIVE)

Nice.  I should probably add my snapshot filters into this header, too.

ACK with comment fix.

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
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]