Skip to content

xenopsd/xc: upstream more NUMA changes #6554

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 25, 2025

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
This reverts commit 9e6fb15

Signed-off-by: Pau Ruiz Safont <pau.ruizsafont@cloud.com>
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 <pau.ruizsafont@cloud.com>
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 <pau.ruizsafont@cloud.com>
@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
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