-
Notifications
You must be signed in to change notification settings - Fork 29
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
fix systrack synchronization #199
base: master
Are you sure you want to change the base?
Conversation
lneto
commented
Sep 24, 2024
- change luaprobe to sleep=false
- split systrack into two runtimes (driver and probes)
@endersonmaia can you check if it fixes #96 properly? Thank you! |
b787286
to
4c2d54c
Compare
examples/systrack.lua
Outdated
probe.new(address, {pre = handler, post = nop}) | ||
end | ||
|
||
driver.probes = lunatik.runtime("systrack/probes", false) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn’t it be "examples/systrack/probes"
? As is, I get the message Cannot open /lib/modules/lua/systrack/probes.lua
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah! Good catch.. thanks! I'll fix it.. I was using Lua root for testing.. sorry about that
Just tried, and for me, it doesn’t work:
|
Hmm.. ungood.. can you give more info about your system? Arch and kernel version? |
Here you are: ❯ uname -a
❯ lscpu
❯ cat -p /etc/lsb-release
|
* change luaprobe to sleep=false * split systrack into two runtimes (driver and probes)
4c2d54c
to
329433b
Compare
I've updated the branch with the correct path.
"unfortunately", I could reproduce the issue.. heh.. it's almost clean to me that there's a deadlock being reached in luaprobe.. it would need more effort to fix.. so, I will put this PR on hold.. as it has low prio =/ |