Re: [Xen-devel] [XEN PATCH for-4.13 0/6] Drop/deprecate libxl_get_required_*_memory

[Jürgen Groß <jgross@xxxxxxxx>]

On 04.10.19 17:17, Ian Jackson wrote:
> libxl_get_required_shadow_memory has always been anomalous.  libxl
> ought to default these things itself.  Recently, another analogous
> setting, iommu_memkb, was introduced, along with another function
> along the same lines.
>
> This API is not very good.  Fixing it was not entirely trivial.
> (Thanks to Paul for explaining some of the difficulties and Anthony
> for in-principle review of my proposal.)
>
> Ian Jackson (6):
>    libxl: Offer API versions 0x040700 and 0x040800
>    xl: Pass libxl_domain_config to freemem(), instead of b_info
>    libxl: libxl__domain_config_setdefault: New function
>    libxl: libxl_domain_need_memory: Make it take a domain_config
>    libxl: Move shadow_memkb and iommu_memkb defaulting into libxl
>    libxl: Remove/deprecate libxl_get_required_*_memory from the API
>
>   tools/libxl/libxl.h          | 24 ++++++++++-
>   tools/libxl/libxl_create.c   | 95 ++++++++++++++++++++++++++++++++++++++------
>   tools/libxl/libxl_dom.c      |  7 ++--
>   tools/libxl/libxl_internal.h | 10 +++++
>   tools/libxl/libxl_mem.c      | 70 ++++++++++++++++++++++++++------
>   tools/libxl/libxl_utils.c    | 15 -------
>   tools/libxl/libxl_utils.h    |  2 +-
>   tools/xl/xl_parse.c          | 15 +------
>   tools/xl/xl_vmcontrol.c      |  6 +--
>   9 files changed, 183 insertions(+), 61 deletions(-)
For the series:

Release-acked-by: Juergen Gross <jgross@xxxxxxxx>


Juergen

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel