Kyle Dong
Kyle Dong
This per-CPU map memory growth issue is also a scalability concern and should be considered part of the broader work tracked in #4191.
Hi @mtardy, just wanted to follow up on this issue to see if there’s been any update or thoughts from the team. I’ve been revisiting this recently and still seeing...
@kkourt, thanks for confirming, and that makes sense. I'd like to prepare a draft to implement what I mentioned in the proposed solution. Does that sound good to you?
Thanks for taking time to review this PR @mtardy! Let's wait @kkourt for his comment about this [idea](https://github.com/cilium/tetragon/issues/4249#issuecomment-3559821386). If you all agree to have a generic flag to enable/disable `BPF_F_NO_PREALLOC`,...
@mtardy @kkourt I have modified the implementation regarding to the [discussion](https://github.com/cilium/tetragon/issues/4249#issuecomment-3569948894) in #4249 It's ready for re-review. Thanks!
>I need to check the issue properly, but did we discuss performance implications? I guess policy_filter_maps is fine, I'm not sure override_task is that straight forward It has been discussed...
@mtardy and @kkourt, would you please review this PR when you get a chance. Thanks!
@mtardy Thank you so much for taking time to review and discuss this PR. I have edit the comment as you suggested. Please take another look when you have time....
Hi @mtardy, thank you so much for the thoughtful guidance on `BPF_F_NO_PREALLOC` and map sizing. Yes, I had read your comment in https://github.com/cilium/tetragon/issues/4191#issuecomment-3411447022 very carefully. Actually, we had evaluated reducing...
Hi @mtardy and @kkourt just checking in to see if there’s been any update or thoughts on this one. I’ve been thinking about a potential workaround: we could consider using...