|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [PATCH 07/12] tools/libxc: untangle libxenctrl from libxenguest
Ian Jackson writes ("[PATCH 07/12] tools/libxc: untangle libxenctrl from
libxenguest"):
> From: Juergen Gross <jgross@xxxxxxxx>
>
> Sources of libxenctrl and libxenguest are completely entangled. In
> practice libxenguest is a user of libxenctrl, so don't let any source
> libxenctrl include xg_private.h.
>
> This can be achieved by moving all definitions used by libxenctrl from
> xg_private.h to xc_private.h. Additionally xc_dom.h needs to be
> exported, so rename it to xenctrl_dom.h.
>
> Rename all libxenguest sources from xc_* to xg_*.
>
> Move xc_[un]map_domain_meminfo() fnctions to new source xg_domain.c as
> they are defined in include/xenguest.h and should be in libxenguest.
>
> Remove xc_efi.h and xc_elf.h as they aren't used anywhere.
Reviewing this is quite difficult. Is there any way it could be split
up ? Perhaps some of it could be generated automatically ? (Eg you
could send a patch whose commit message had the perl rune you used to
generate it, which would enable a reviewer to see what was supposed to
be going on.)
Thanks,
Ian.
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |