关于 lua-resty-lock 的一个问题

wp.4163196 · 2014-10-08T10:34:42+00:00

Hello! 2014-10-01 17:25 GMT-07:00 sunjun : > 我的需求是resty.upload 模块读取数据&#x524D...

关于 lua-resty-lock 的一个问题

wp.4163196

hi，大家好：

我们的一个项目在使用ngx_lua，然后在lua-resty-lock中遇到了一个问题：

185 function _M.unlock(self)

186 local dict = self.dict

187 local cdata = self.cdata

188 unref_obj(tonumber(cdata.dict_id))

189 local key_id = tonumber(cdata.key_id)

190 if key_id <= 0 then

191 return nil, "unlocked"

192 end

在188行，我不是很明白为什么在 unlock 的时候就去执行 unref_obj(cdata.dict_id)

不是应该是在gc_lock 时才去释放dict_id吗？

在我们的项目中，经常会出现lua crash 问题, 然后我把188行注释掉后，问题就不再重现了

doujiang24

如果显式调用了 unlock，gc_lock 就不再释放了

只有没有显式调用 unlock 的，才会在 gc 的时候自动释放

注释掉 188 行，是很危险的，会使 memo 不断增大，导致内存泄露（如果你有显式调用 unlock 的话）

建议：

1. 把使用方式贴出来

2. 用 gdb 分析 core dump

http://wiki.nginx.org/Debugging

2014-10-08 10:34 GMT+08:00 Peng Wu <wp.4...@gmail.com>:

hi，大家好：

我们的一个项目在使用ngx_lua，然后在lua-resty-lock中遇到了一个问题：

185 function _M.unlock(self)

186 local dict = self.dict

187 local cdata = self.cdata

188 unref_obj(tonumber(cdata.dict_id))

189 local key_id = tonumber(cdata.key_id)

190 if key_id <= 0 then

191 return nil, "unlocked"

192 end

在188行，我不是很明白为什么在 unlock 的时候就去执行 unref_obj(cdata.dict_id)
不是应该是在gc_lock 时才去释放dict_id吗？
在我们的项目中，经常会出现lua crash 问题, 然后我把188行注释掉后，问题就不再重现了

--

wp.4163196

谢谢回复，我又仔细看了一下lock.lua的代码，确实直接注释掉是不对的。

但我仍然觉得在188行有问题，对于创建的一个lock对象，多次调用lock/unlock方法则每次都会去执行unref_obj(cdata.dict_id)

假设cdata.dict_id = 3, 那每次执行unlock方法都会将memo[3]放到FREE_LIST中去，如果memo[3]当前正被其他lock对象引用，是不是会造成混乱？

我现在采取的办法是在188行下再加一行cdata.dict_id = -1，保证多次调用lock/unlock方法时只在第一次执行unref_obj(cdata.dict_id)

虽然反复看代码确认这个方案应该可行，但仍需要向大家确认一下，求指导，谢谢。

在 2014年10月8日下午4:40，DeJiang Zhu <douj...@gmail.com>写道：

如果显式调用了 unlock，gc_lock 就不再释放了

只有没有显式调用 unlock 的，才会在 gc 的时候自动释放

注释掉 188 行，是很危险的，会使 memo 不断增大，导致内存泄露（如果你有显式调用 unlock 的话）

建议：

1. 把使用方式贴出来
2. 用 gdb 分析 core dump

http://wiki.nginx.org/Debugging

2014-10-08 10:34 GMT+08:00 Peng Wu <wp.4...@gmail.com>:
hi，大家好：

我们的一个项目在使用ngx_lua，然后在lua-resty-lock中遇到了一个问题：

185 function _M.unlock(self)

186 local dict = self.dict

187 local cdata = self.cdata

188 unref_obj(tonumber(cdata.dict_id))

189 local key_id = tonumber(cdata.key_id)

190 if key_id <= 0 then

191 return nil, "unlocked"

192 end

在188行，我不是很明白为什么在 unlock 的时候就去执行 unref_obj(cdata.dict_id)
不是应该是在gc_lock 时才去释放dict_id吗？
在我们的项目中，经常会出现lua crash 问题, 然后我把188行注释掉后，问题就不再重现了

--

--

doujiang24

在 2014年10月9日上午10:11，Peng Wu <wp.4...@gmail.com>写道：

假设cdata.dict_id = 3, 那每次执行unlock方法都会将memo[3]放到FREE_LIST中去，如果memo[3]当前正被其他lock对象引用，是不是会造成混乱？

嗯，重复调用 lock / unlock，确实会有问题

我现在采取的办法是在188行下再加一行cdata.dict_id = -1，保证多次调用lock/unlock方法时只在第一次执行unref_obj(cdata.dict_id)

这样也不够安全，比如：多次调用 lock / unlock，但是最后一次没有 unlock，进入 gc_lock 的时候，会拿不到 dict

虽然反复看代码确认这个方案应该可行，但仍需要向大家确认一下，求指导，谢谢。

或许在 gc 的时候，释放 dict 更加合适一些

顺手提了一个 pull request

https://github.com/openresty/lua-resty-lock/pull/3/files

在 2014年10月8日下午4:40，DeJiang Zhu <douj...@gmail.com>写道：

如果显式调用了 unlock，gc_lock 就不再释放了

只有没有显式调用 unlock 的，才会在 gc 的时候自动释放

注释掉 188 行，是很危险的，会使 memo 不断增大，导致内存泄露（如果你有显式调用 unlock 的话）

建议：

1. 把使用方式贴出来
2. 用 gdb 分析 core dump

http://wiki.nginx.org/Debugging

2014-10-08 10:34 GMT+08:00 Peng Wu <wp.4...@gmail.com>:
hi，大家好：

我们的一个项目在使用ngx_lua，然后在lua-resty-lock中遇到了一个问题：

185 function _M.unlock(self)

186 local dict = self.dict

187 local cdata = self.cdata

188 unref_obj(tonumber(cdata.dict_id))

189 local key_id = tonumber(cdata.key_id)

190 if key_id <= 0 then

191 return nil, "unlocked"

192 end

在188行，我不是很明白为什么在 unlock 的时候就去执行 unref_obj(cdata.dict_id)
不是应该是在gc_lock 时才去释放dict_id吗？
在我们的项目中，经常会出现lua crash 问题, 然后我把188行注释掉后，问题就不再重现了

--

--

--

agentzh

Hello!

2014-10-09 2:52 GMT-07:00 DeJiang Zhu:
> 或许在 gc 的时候，释放 dict 更加合适一些
>
> 顺手提了一个 pull request
>
> https://github.com/openresty/lua-resty-lock/pull/3/files
>

补丁已合并到 git master，多谢！

Best regards,
-agentzh