You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2022/03/28 22:33:14 UTC

[GitHub] [trafficserver] pbchou opened a new pull request #8764: Fixes Issue #8728 - Lua plugin memory leak on remap configuration reloads

pbchou opened a new pull request #8764:
URL: https://github.com/apache/trafficserver/pull/8764


   This fix adds reference counting for the Lua plugin remap instance handles. The reference counting
   allows us to eliminate an existing memory leak of the instance handles. In addition, this means
   that the old Lua memory allocated by LuaJIT may also be freed via LuaJIT garbage collection.
   
   This fix also adds the '--ljgc' remap instance plugin parameter to the Lua plugin. This paramter
   enables on-demand LuaJIT garbage collection while the remap instances are created and deleted.
   This is useful when operating close to the LuaJIT memory limit, which is currently 2GB on Linux
   using LuaJIT v2.1.0-beta3 from 2017.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficserver] randall commented on pull request #8764: Lua plugin memory leak on remap configuration reloads

Posted by GitBox <gi...@apache.org>.
randall commented on pull request #8764:
URL: https://github.com/apache/trafficserver/pull/8764#issuecomment-1082006974


   @ezelkow1 Can you add pbchou to the allowlist for CI?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficserver] randall merged pull request #8764: Lua plugin memory leak on remap configuration reloads

Posted by GitBox <gi...@apache.org>.
randall merged pull request #8764:
URL: https://github.com/apache/trafficserver/pull/8764


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficserver] bneradt commented on pull request #8764: Lua plugin memory leak on remap configuration reloads

Posted by GitBox <gi...@apache.org>.
bneradt commented on pull request #8764:
URL: https://github.com/apache/trafficserver/pull/8764#issuecomment-1082016483


   > @ezelkow1 Can you add pbchou to the allowlist for CI?
   
   Added.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficserver] ezelkow1 commented on pull request #8764: Lua plugin memory leak on remap configuration reloads

Posted by GitBox <gi...@apache.org>.
ezelkow1 commented on pull request #8764:
URL: https://github.com/apache/trafficserver/pull/8764#issuecomment-1082030717


   [approve ci]


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org