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 <[email protected]>
Juergen
_______________________________________________
Xen-devel mailing list
[email protected]
https://lists.xenproject.org/mailman/listinfo/xen-devel