Skip to content

Commit

Permalink
bcc/tools: remove unused signal handlers
Browse files Browse the repository at this point in the history
Several top tools defined signal handler, but not used.
They work well without signal handler, so just remove it.

Signed-off-by: Hengqi Chen <[email protected]>
  • Loading branch information
chenhengqi authored and yonghong-song committed Jul 15, 2021
1 parent 8b09996 commit b057641
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 15 deletions.
5 changes: 0 additions & 5 deletions tools/biotop.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
from bcc import BPF
from time import sleep, strftime
import argparse
import signal
from subprocess import call

# arguments
Expand Down Expand Up @@ -52,10 +51,6 @@
loadavg = "/proc/loadavg"
diskstats = "/proc/diskstats"

# signal handler
def signal_ignore(signal_value, frame):
print()

# load BPF program
bpf_text = """
#include <uapi/linux/ptrace.h>
Expand Down
5 changes: 0 additions & 5 deletions tools/filetop.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@
from bcc import BPF
from time import sleep, strftime
import argparse
import signal
from subprocess import call

# arguments
Expand Down Expand Up @@ -59,10 +58,6 @@
# linux stats
loadavg = "/proc/loadavg"

# signal handler
def signal_ignore(signal_value, frame):
print()

# define BPF program
bpf_text = """
#include <uapi/linux/ptrace.h>
Expand Down
5 changes: 0 additions & 5 deletions tools/slabratetop.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
from bcc.utils import printb
from time import sleep, strftime
import argparse
import signal
from subprocess import call

# arguments
Expand Down Expand Up @@ -54,10 +53,6 @@
# linux stats
loadavg = "/proc/loadavg"

# signal handler
def signal_ignore(signal, frame):
print()

# define BPF program
bpf_text = """
#include <uapi/linux/ptrace.h>
Expand Down

0 comments on commit b057641

Please sign in to comment.