Kornilios Kourtis
Kornilios Kourtis
Hello, > Actually, we had evaluated reducing the map size, but the challenge for this [specific map](https://github.com/cilium/tetragon/blob/main/pkg/policyfilter/map.go#L195) (policy_id → cgroup_id inner maps) is that its contents are dynamic: cgroup IDs...
> For BPF_F_NO_PREALLOC, I just wanted to clarify one point. Please feel free to correct me if I'm wrong, from my understanding, even when this flag is used, the map...
> It seems to me that settings large numbers to make these issues disappear is like kicking the can down the road. Ideally these maps should be sized for having...
I believe we should consider adding a flag anyway. It will allow us to easily compare the two approaches, and act as a backup in case there are unwanted side-effects...
> [@mtardy](https://github.com/mtardy) Thanks for the feedback! > ### On the flag approach: > > > Yeah maybe we can have a flag in common for all those changes so that...
Also, it does not have to be one flag per map. We can have something like: ``` --bpf-maps-prealloc=true --bpf-map-prealloc-disable=map1,map2 ``` Where we enable prealloc on all maps, and disable on...
Above makes sense to me. A few notes: * Presumably, we will support setting this flag only for a subset of maps (at least in the first PR). It would...