Skip to content

Conversation

psafont
Copy link
Member

@psafont psafont commented Jun 25, 2025

These commits allow to minimize the differences between the versions that use a xen with the NUMA-enabled claim_pages, and the ones that are not.

Now it's a single patch with 4 lines of code:

  • 2 lines to change the C binding for domain_claim_pages (ocaml/xenopsd/c_stubs/xenctrlext_stubs.c)
  • 2 lines to remove the raising of exception to try to use the C binding with a single NUMA node (ocaml/xenopsd/xc/xenctrlext.ml)

psafont added 3 commits June 25, 2025 13:21
On xen versions that don't support this call yet, xenctrlext will simply fail
and continue to behave like before.

Signed-off-by: Pau Ruiz Safont <[email protected]>
This interface is not yet available in xen, so fail before doing the hypercall.
This patch is meant to be reverted on system that provide the new interface for
easily test it.

Signed-off-by: Pau Ruiz Safont <[email protected]>
@psafont psafont requested a review from mg12 June 25, 2025 12:27
| [Node node] ->
Some (Xenctrlext.NumaNode.from node, node)
| [] | _ :: _ :: _ ->
D.debug
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would suggest to elevate this to info

@psafont psafont added this pull request to the merge queue Jun 25, 2025
Merged via the queue into xapi-project:master with commit 1a68634 Jun 25, 2025
16 checks passed
@psafont psafont deleted the private/paus/numa-share branch October 9, 2025 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants