perf tool jit profiling broken 20.04

Bug #2020197 reported by Younes Manton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Fix Released
Undecided
Unassigned
Focal
Confirmed
Undecided
Unassigned
Jammy
Confirmed
Undecided
Unassigned

Bug Description

The following kernel commit:

[babd04386b1df8c364cdaa39ac0e54349502e1e5] perf jit: Include program header in ELF files

introduced a bug in perf that causes samples to be attributed to the wrong instructions in
the annotated assembly output of `perf report` and `perf annotate`.

The following commit:

[89b15d00527b7825ff19130ed83478e80e3fae99] perf inject: Fix GEN_ELF_TEXT_OFFSET for jit

fixes the bug. Can it be backported to 20.04 and wherever else it's needed? The output looks very
confusing when parts of the code one expects to accumulate ticks don't and other parts
that shouldn't be executed at all accumulate ticks.

The above patch depends on parts of d6e6a27d960f9f07aef0b979c49c6736ede28f75, which depends on something else, etc. etc.

Revision history for this message
Ubuntu Kernel Bot (ubuntu-kernel-bot) wrote : Missing required logs.

This bug is missing log files that will aid in diagnosing the problem. While running an Ubuntu kernel (not a mainline or third-party kernel) please enter the following command in a terminal window:

apport-collect 2020197

and then change the status of the bug to 'Confirmed'.

If, due to the nature of the issue you have encountered, you are unable to run this command, please add a comment stating that fact and change the bug status to 'Confirmed'.

This change has been made by an automated script, maintained by the Ubuntu Kernel Team.

Changed in linux (Ubuntu):
status: New → Incomplete
Revision history for this message
Younes Manton (younes-m) wrote :

Log files not relevant.

Changed in linux (Ubuntu):
status: Incomplete → Confirmed
Revision history for this message
Timo Aaltonen (tjaalton) wrote :

fixed in 6.1-rc1 and up, and not found in upstream stable@ so please send it there?

Changed in linux (Ubuntu):
status: Confirmed → Fix Released
Revision history for this message
Timo Aaltonen (tjaalton) wrote :

the breaking commit was added upstream to 5.4.215, 5.15.71

Changed in linux (Ubuntu Focal):
status: New → Confirmed
Changed in linux (Ubuntu Jammy):
status: New → Confirmed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.