Re: [PATCH 2/7] shared_memory: Define the APIs to get/set shared memory parameters

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

 



On 09/10/2012 06:08 AM, Osier Yang wrote:
> * include/libvirt/libvirt.h.in: (Add macros for the param fields,
>   declare the APIs).
> * src/driver.h: (New methods for the driver struct)
> * src/libvirt.c: (Implement the public APIs)
> * src/libvirt_public.syms: (Export the public symbols)
> ---
>  include/libvirt/libvirt.h.in |   63 ++++++++++++++++++++++
>  python/generator.py          |    2 +
>  src/driver.h                 |   14 +++++
>  src/libvirt.c                |  121 ++++++++++++++++++++++++++++++++++++++++++
>  src/libvirt_public.syms      |    2 +
>  5 files changed, 202 insertions(+), 0 deletions(-)

Looks good in general, once you squash in the renames mentioned in your
followup patch, and once we settle the bikeshed question of whether to
drop 'Shared' from the API names.

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