Re: [PATCH 00/13] tools: do cleanups related to libxc python bindings

[Juergen Gross <jgross@suse.com>]

On 10/23/2015 03:04 PM, Juergen Gross wrote:
> This series is a combination of my previous patches:
>
> "libxc: remove most of tools/libxc/xc_dom_compat_linux.c"
> "tools: remove unused wrappers for python"
>
> I have split it up as requested by Ian Campbell, thus it consists of
> 13 patches instead just of 2, but the functionality is roughly the
> same. I have just kept more python bindings compared to the first
> version, as there have been reports of some out of tree uses. Asking
> for more such use case on xen-devel and xen-user didn't result in
> requests for more interfaces to be kept, so I delete them.

There have been acks and critical responses regarding this series.

What should we do?

- drop them all
- apply the first two patches only to get rid of the extra interfaces
   to the domain builder as requested by Ian Campbell
- apply all of them


Juergen

>
> Juergen Gross (13):
>    libxc: remove most of tools/libxc/xc_dom_compat_linux.c
>    libxc: remove xc_get_bit_size() from tools/libxc/xc_dom_compat_linux.c
>    python: remove flask related libxc python bindings
>    python: remove cpupool related libxc python bindings
>    python: remove cpuid related libxc python bindings
>    python: remove device related libxc python bindings
>    python: remove scheduler related libxc python bindings
>    python: remove unused memory related libxc python bindings
>    python: remove domain handling related libxc python bindings
>    python: remove vcpu related libxc python bindings
>    python: remove hvm related libxc python bindings
>    python: remove  permission related libxc python bindings
>    python: remove unused other libxc python bindings
>
>   tools/libxc/include/xc_dom.h      |    5 +
>   tools/libxc/include/xenguest.h    |   52 -
>   tools/libxc/xc_dom_compat_linux.c |  142 +-
>   tools/libxl/libxl_internal.h      |    1 +
>   tools/python/xen/lowlevel/xc/xc.c | 2624 +++----------------------------------
>   5 files changed, 170 insertions(+), 2654 deletions(-)
>