On Thu, Apr 15, 2021 at 11:56 PM Jiri Olsa [off-list ref] wrote:

On Thu, Apr 15, 2021 at 04:45:24PM -0700, Alexei Starovoitov wrote:

quoted

On Wed, Apr 14, 2021 at 12:52 PM Jiri Olsa [off-list ref] wrote:

quoted

hi,
while adding test for pinning the module while there's
trampoline attach to it, I noticed that we don't allow
link detach and following re-attach for trampolines.
Adding that for tracing and lsm programs.

You need to have patch [1] from bpf tree for test module
attach test to pass.

v5 changes:
  - fixed missing hlist_del_init change
  - fixed several ASSERT calls
  - added extra patch for missing ';'
  - added ASSERT macros to lsm test
  - added acks

It doesn't work:

hi,
I got the same warning when running test without the
patch [1] I mentioned:
  861de02e5f3f bpf: Take module reference for trampoline in module

I still don't see it in bpf-next/master

Finally applied to bpf-next.
Thank you and sorry for the delay.