iovisor/bcc

update killsnoop to use tracepoints

Open

#3 592 ouverte le 27 août 2021

Voir sur GitHub
 (7 commentaires) (0 réactions) (1 assigné)C (4 051 forks)batch import
help wanted

Métriques du dépôt

Stars
 (22 409 stars)
Métriques de merge PR
 (Merge moyen 12j 6h) (12 PRs mergées en 30 j)

Description

This is a request for help.

I wrote killsnoop back in 2015 before tracepoint support, and so I kprobe'd sys_kill(). It still does some derivation of that. But now there's a report it no longer works on Linux 5.11: https://github.com/iovisor/bcc/pull/3572#issuecomment-900357032 CC @chenhengqi

Can someone please update killsnoop (both Python and libbpf-tools) to use tracepoints instead of kprobes (if it works as expected). All of these:

  syscalls:sys_enter_kill                            [Tracepoint event]
  syscalls:sys_enter_tgkill                          [Tracepoint event]
  syscalls:sys_enter_tkill                           [Tracepoint event]
  syscalls:sys_exit_kill                             [Tracepoint event]
  syscalls:sys_exit_tgkill                           [Tracepoint event]
  syscalls:sys_exit_tkill                            [Tracepoint event]

Guide contributeur

Direction de recherche
Mettez à jour killsnoop (à la fois Python et libbpf tools) pour utiliser des tracepoints au lieu de kprobes. Examinez l'implémentation existante basée sur kprobe et remplacez la par les tracepoints de syscall listés.
Stack technique
cpython
Domaine
observability
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
BPFtracepointsC
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50