On 5/15/23 5:53 PM, Song Liu wrote:
On Mon, May 15, 2023 at 6:09 AM Yafang Shao <laoar.shao@xxxxxxxxx> wrote:After commit e21aa341785c ("bpf: Fix fexit trampoline."), the selector is only used to indicate how many times the bpf trampoline image are updated and been displayed in the trampoline ksym name. After the trampoline is freed, the selector will start from 0 again. So the selector is a useless value to the user. We can remove it. If the user want to check whether the bpf trampoline image has been updated or not, the user can compare the address. Each time the trampoline image is updated, the address will change consequently. Jiri pointed out antoher issue that perf is still using the old name "bpf_trampoline_%lu", so this change can fix the issue in perf. Fixes: e21aa341785c ("bpf: Fix fexit trampoline.") Signed-off-by: Yafang Shao <laoar.shao@xxxxxxxxx> Cc: Song Liu <song@xxxxxxxxxx> Cc: Jiri Olsa <olsajiri@xxxxxxxxx>Acked-by: Song Liu <song@xxxxxxxxxx>
Lgtm, I took this individual one for now.
