Skip to content
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

Re-enable NUMA memory binding on newer kernels #1207

Merged
merged 1 commit into from
Mar 5, 2019
Merged

Conversation

wingo
Copy link

@wingo wingo commented Feb 21, 2019

  • lib/ljsyscall/syscall/linux/syscalls.lua: More robustly create
    multi-word bitmasks.

  • src/lib/numa.lua: Remove facility for blacklisting kernel versions.
    Re-enable bind_to_numa_node generally.

* lib/ljsyscall/syscall/linux/syscalls.lua: More robustly create
  multi-word bitmasks.

* src/lib/numa.lua: Remove facility for blacklisting kernel versions.
  Re-enable bind_to_numa_node generally.
@wingo
Copy link
Author

wingo commented Feb 21, 2019

WDYT @takikawa, @eugeneia, and @dpino ?

@eugeneia
Copy link
Collaborator

eugeneia commented Feb 22, 2019

I hate that this is so complicated, but this PR seems to make it simpler overall. 👍 Since this touches ljsyscall, maybe @justincormack has more insight.

@eugeneia
Copy link
Collaborator

I also wonder about our relation with upstream ljsyscall, we have snabbco#1122 lying around. Would be nice to sync with upstream (both directions.)

@wingo
Copy link
Author

wingo commented Feb 22, 2019

Regarding upstream, there's justincormack/ljsyscall#193 sitting around because I am a bad person and haven't done the last-mile there; so on the up side of things there won't be conflicts with upstream :P

@justincormack
Copy link

One problem about upstream is that I was reluctant to switch to LuaJit 2.1 branch as there has never actually been a proper release, and some of the PRs were not compatible due to syntactic differences. There are a couple of options, 1. fix the differences with some (probably a bit ugly) workarounds 2. drop support for the 2.0 branch 3. switch to raptorjit as the upstream...

@wingo
Copy link
Author

wingo commented Feb 22, 2019

I will try to get in the workarounds upstream.

@wingo wingo merged commit a5ddf2f into lwaftr Mar 5, 2019
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.

4 participants