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

Ian Jackson posted 6 patches 4 years, 5 months ago
Failed in applying to current master (apply log)
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(-)
[Xen-devel] [XEN PATCH for-4.13 0/6] Drop/deprecate libxl_get_required_*_memory
Posted by Ian Jackson 4 years, 5 months ago
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(-)

-- 
2.11.0


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [XEN PATCH for-4.13 0/6] Drop/deprecate libxl_get_required_*_memory
Posted by Jürgen Groß 4 years, 5 months ago
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@suse.com>


Juergen

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