iovisor/bcc

Reduce CPU overhead of per-event tools by allowing more buffering

Open

#1033 aperta il 8 mar 2017

Vedi su GitHub
 (12 commenti) (0 reazioni) (0 assegnatari)C (4051 fork)batch import
help wantedprio:high

Metriche repository

Star
 (22.409 star)
Metriche merge PR
 (Merge medio 12g 6h) (12 PR mergiate in 30 g)

Descrizione

Not a big problem yet, but at some point we should look into this...

The normal perf_output-consuming loop we've been using is:

while 1:
    b.kprobe_poll()

This wakes up as quickly as it can, and can consume noticeable CPU. I had to tune it for my cpuunclaimed tool:

while 1:
    # allow some buffering by calling sleep(), to reduce the context switch
    # rate and lower overhead.
    try:
        if not exiting:
            sleep(wakeup_s)
    except KeyboardInterrupt:
        exiting = 1
    b.kprobe_poll()
    slept += wakeup_s

This ticket is to explore doing this for more tools. Should b.kprobe_poll() have a built-in sleep, that can be tuned? Eg, 100ms by default?

This is more practical now we have #997, and can increase the buffer size.

Guida contributor

Direzione di ricerca
Esplora l'aggiunta di un sleep configurabile nel loop kprobe poll per ridurre l'uso della CPU. Guarda strumenti esistenti come cpuunclaimed e implementa un pattern simile.
Tech stack
c
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
BPFCPython
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50