FR: Output Folded Stacks Format

Aperta
  • Qualcosa non va?
#658 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

  • #669 di @mhansen — chiusa senza merge

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
25/100
Tipo di issue
Funzionalità
Chiarezza
Da chiarire
Stato di attività
Ferma
Stack tecnologico
go
Ambito
performance

Direzione di ricerca

Inizia esaminando i percorsi di output esistenti di pprof per -callgrind e -dot e il modo in cui selezionano un valore del profilo per i pesi del grafo. Confronta l’output proposto con difffolded.pl e flamegraph.pl di Brendan Gregg e con il formato folded di pprofutils di Felixge. Il lavoro sarebbe completato dopo aver deciso il formato e il comportamento dell’opzione, inclusa la scelta se emettere un valore o più colonne e come gestire labels e gli altri dati del profilo.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Priority: p3 type: feat

I think if we built an exporter to Brendan Gregg's folded stacks format, we could:

  1. Enable people to do differential flamegraphs using difffolded.pl and flamegraph.pl, perhaps addressing the user need in #483
  2. Allow post-processing the profiles using text tools like grep & sed
  3. Hook into some more profile analysis tools: see profilerpedia, folded stacks is a very well-connected format.

I'm picturing an option like -callgrind and -dot, perhaps -folded.

The trouble might be that folded stacks format only accepts one output number per stack, but pprof can have multiple numbers per stack. So we'd probably have to choose the number to output with a flag, if it was ambiguous. But perhaps parts of pprof already handle this? e.g. the -dot output probably chooses an event to use as the weight of the graph size?

Prior art in this space: @felixge's pprofutils folded: https://github.com/felixge/pprofutils#folded. Felix has an extension to the folded stacks format to have numbers with multiple columns. I think that's a neat idea for representing pprof and allows roundtripping back to pprof format. It would be nice if this extension caught on, but I wonder if it might limit the interoperability with other folded stacks tools.

Another idea might be to output only one number column per-file (e.g. pprof -folded=cpu-cycles and pprof -folded=instructions), and then to roundtrip, we could output multiple files, convert both of them back to pprof, then concatenate the files with pprof -proto <profile1> <profile2>.

There are plenty of other challenges with roundtripping too (labels, filenames, mappings...) but perhaps just solving part of the problem would still be useful and enable more analyses. Anyway, I mostly wanted to open a discussion about this. Thoughts?

Lingua principale
Go
Stelle
9.3k
Fork
687
Merge medio
2g 2h
PR unite (30g)
9

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di google/pprof

Tutte le issue di google/pprof

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.