Jordan Rome

Results 346 comments of Jordan Rome

@danobi Thanks for the explanation! And you're right, while I was seeing a lot of -1 returns for `__sys_bpf` (when only printing if the retval was less than 0), I...

Sounds useful. The syntax also looks good. @viktormalik I'm also happy to pick it up if no one gets to it first (though similarly I have other things I need...

Just to note, we should probably test deletion/insertion into the same map during iteration to make sure there aren't weird side effects.

@thediveo Yes, it's in 0.21.0: https://github.com/bpftrace/bpftrace/releases/tag/v0.21.0

I do agree that option 2 seems more idiomatic to bpftrace but with user functions on the horizon, maybe option 1 is more re-usable. Is there a consideration for anonymous...

All makes sense to me. Maybe because I just worked on the config syntax, I'm considering the "floating" element of these type annotations e.g. will they be forced to be...

@JakeHillion Thanks for picking this up. As discussed offline, I'm curious if dropping both the "global" and the "let" is possible to improve the conciseness of this new syntax e.g....

Just detailing some inconsistencies: ``` sudo bpftrace -e 'BEGIN { @ = count(); print((@)); exit(); }' Attaching 1 probe... @: 1 @: 1 ``` ``` sudo bpftrace -e 'BEGIN {...

> Will put in a quick fix to leak memory for now - better to run out of stack space than return wrong results. I'm also in this camp.

@danobi Sure, would be happy to help. Let me know if one of those items above would be good to start with or if you want to hop on a...