Skip to content

Conversation

@qlndzt
Copy link
Collaborator

@qlndzt qlndzt commented Dec 26, 2025

This PR is for:

Add lua disabled config

Checklist

  • Added unit test.

Backport to branches

@yinjiping
Copy link
Contributor

注意一下提交的格式
以 feat: / fix: 开头 后面加一个修改模块如 agent/eBPF ...

@yinjiping
Copy link
Contributor

yinjiping commented Dec 27, 2025

image

oncpu对 php/python/lua/node.js 是否是依赖dwarf功能,
如果说dwarf功能没有开启,那么这个时候 php/python/lua/node.js 即使是开启了,从上面代码看也不会生效。

这个符合预期吗? 也就上说 oncpu的 php/python/lua/node.js 是独立的还是依赖 dwarf?

@kylewanginchina @qlndzt

@qlndzt qlndzt changed the title Add lua disabled config feat: agent/eBPF: Add lua disabled config Dec 28, 2025
@qlndzt
Copy link
Collaborator Author

qlndzt commented Dec 28, 2025

image oncpu对 php/python/lua/node.js 是否是依赖dwarf功能, 如果说dwarf功能没有开启,那么这个时候 php/python/lua/node.js 即使是开启了,从上面代码看也不会生效。

这个符合预期吗? 也就上说 oncpu的 php/python/lua/node.js 是独立的还是依赖 dwarf?

@kylewanginchina @qlndzt

lua的话是不依赖dwarf的

@kylewanginchina
Copy link
Contributor

image oncpu对 php/python/lua/node.js 是否是依赖dwarf功能, 如果说dwarf功能没有开启,那么这个时候 php/python/lua/node.js 即使是开启了,从上面代码看也不会生效。

这个符合预期吗? 也就上说 oncpu的 php/python/lua/node.js 是独立的还是依赖 dwarf?

@kylewanginchina @qlndzt

@yinjiping 是依赖dwarf的,和python是一致的。因为代码会调用C库,C库的部分会用到dwarf,如果dwarf部分不enable,拿到的栈数据中就会是unknown的地址

@kylewanginchina
Copy link
Contributor

image oncpu对 php/python/lua/node.js 是否是依赖dwarf功能, 如果说dwarf功能没有开启,那么这个时候 php/python/lua/node.js 即使是开启了,从上面代码看也不会生效。 这个符合预期吗? 也就上说 oncpu的 php/python/lua/node.js 是独立的还是依赖 dwarf? @kylewanginchina @qlndzt

lua的话是不依赖dwarf的

@qlndzt 如果lua不依赖,一方面就不应该在unwind_events_handle中的判断后面添加lua相关的初始逻辑
image
另一方面,lua不依赖的话,那lua中涉及dwarf unwind的逻辑是否有实现上的功能重复?

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