Skip to content

Add support for LuaJIT's GC64 mode #1019

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

Closed
agentzh opened this issue Mar 25, 2017 · 7 comments
Closed

Add support for LuaJIT's GC64 mode #1019

agentzh opened this issue Mar 25, 2017 · 7 comments

Comments

@agentzh
Copy link
Member

agentzh commented Mar 25, 2017

LuaJIT has recently added support for the new GC64 mode which allows the full low 47-bit address space to be used by LuaJIT's GC-managed memory.

It seems that this module requires some changes regarding light userdata to actually support this GC64 mode:

http://www.freelists.org/post/luajit/fix-lightud-type-for-48bit-virtual-address

LuaJIT/LuaJIT#230 (comment)

@alyoshark
Copy link

Appreciate the effort! Would love to know roughly which version is going to include the fix. Thank you.

@spacewander
Copy link
Member

Is there a plan to support GC64 mode? Can't wait to have the ability to use more memory.

@agentzh
Copy link
Member Author

agentzh commented Sep 8, 2017

@spacewander Patches welcome.

@spacewander
Copy link
Member

Let me repost the discussion in elsewhere:

  1. With LuaJIT in GC64 mode, all test cases pass in x64 platform.
  2. However, there is a problem with lightuserdata in ARM64 and simillary platform. See the detail in Lightuserdata on ARM64 #1152.
  3. GC64 mode slightly drops performance for CPU-bound task.

@agentzh
Copy link
Member Author

agentzh commented Sep 12, 2017

We already have #1152 for ARM64 + GC64. We can close this.

@agentzh agentzh closed this as completed Sep 12, 2017
@agentzh
Copy link
Member Author

agentzh commented Sep 15, 2017

Noted one segmentation fault when running lua-resty-core's test suite with LuaJIT GC64:

openresty/lua-resty-core#144

@agentzh
Copy link
Member Author

agentzh commented Sep 26, 2017

@pavelnemirovsky Please do not cross post. It's annoying. Thanks for your cooperation.

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

No branches or pull requests

3 participants