On 2/19/20 4:02 AM, Alexei Starovoitov wrote:
The motivation is clear, but I think the users shouldn't be made aware of such implementation details. I think instead of filter_in/out it's better to do 'full or safe' mode of probing. By default it can do all the probing that doesn't cause extra dmesgs and in 'full' mode it can probe everything.
Alright, then I will send later v2 where the "internal" implementation (filtering out based on regex) stays similar (filter_out will stay in the code without being exposed to users, filter_in will be removed). And the exposed option of "safe" probing will just apply the "(trace|write_user)" filter_out pattern. Does it sound good?