Revision f8494fa3dd10b52eab47a9666a8bc34719a129aa authored by Joel Fernandes (Google) on 26 June 2018, 00:08:22 UTC, committed by Steven Rostedt (VMware) on 12 July 2018, 23:56:25 UTC
Currently ftrace displays data in trace output like so: _-----=> irqs-off / _----=> need-resched | / _---=> hardirq/softirq || / _--=> preempt-depth ||| / delay TASK-PID CPU TGID |||| TIMESTAMP FUNCTION | | | | |||| | | bash-1091 [000] ( 1091) d..2 28.313544: sched_switch: However Android's trace visualization tools expect a slightly different format due to an out-of-tree patch patch that was been carried for a decade, notice that the TGID and CPU fields are reversed: _-----=> irqs-off / _----=> need-resched | / _---=> hardirq/softirq || / _--=> preempt-depth ||| / delay TASK-PID TGID CPU |||| TIMESTAMP FUNCTION | | | | |||| | | bash-1091 ( 1091) [002] d..2 64.965177: sched_switch: From kernel v4.13 onwards, during which TGID was introduced, tracing with systrace on all Android kernels will break (most Android kernels have been on 4.9 with Android patches, so this issues hasn't been seen yet). From v4.13 onwards things will break. The chrome browser's tracing tools also embed the systrace viewer which uses the legacy TGID format and updates to that are known to be difficult to make. Considering this, I suggest we make this change to the upstream kernel and backport it to all Android kernels. I believe this feature is merged recently enough into the upstream kernel that it shouldn't be a problem. Also logically, IMO it makes more sense to group the TGID with the TASK-PID and the CPU after these. Link: http://lkml.kernel.org/r/20180626000822.113931-1-joel@joelfernandes.org Cc: jreck@google.com Cc: tkjos@google.com Cc: stable@vger.kernel.org Fixes: 441dae8f2f29 ("tracing: Add support for display of tgid in trace output") Signed-off-by: Joel Fernandes (Google) <joel@joelfernandes.org> Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
1 parent 0fc8c35
File | Mode | Size |
---|---|---|
altera-stapl | ||
c2port | ||
cardreader | ||
cb710 | ||
cxl | ||
echo | ||
eeprom | ||
genwqe | ||
ibmasm | ||
lis3lv02d | ||
lkdtm | ||
mei | ||
mic | ||
ocxl | ||
sgi-gru | ||
sgi-xp | ||
ti-st | ||
vmw_vmci | ||
Kconfig | -rw-r--r-- | 17.9 KB |
Makefile | -rw-r--r-- | 2.2 KB |
ad525x_dpot-i2c.c | -rw-r--r-- | 2.6 KB |
ad525x_dpot-spi.c | -rw-r--r-- | 2.8 KB |
ad525x_dpot.c | -rw-r--r-- | 20.2 KB |
ad525x_dpot.h | -rw-r--r-- | 7.9 KB |
apds9802als.c | -rw-r--r-- | 7.6 KB |
apds990x.c | -rw-r--r-- | 33.9 KB |
aspeed-lpc-ctrl.c | -rw-r--r-- | 7.5 KB |
aspeed-lpc-snoop.c | -rw-r--r-- | 7.1 KB |
atmel-ssc.c | -rw-r--r-- | 6.1 KB |
atmel_tclib.c | -rw-r--r-- | 4.5 KB |
bh1770glc.c | -rw-r--r-- | 37.5 KB |
cs5535-mfgpt.c | -rw-r--r-- | 10.2 KB |
ds1682.c | -rw-r--r-- | 7.1 KB |
dummy-irq.c | -rw-r--r-- | 1.5 KB |
enclosure.c | -rw-r--r-- | 18.9 KB |
fsa9480.c | -rw-r--r-- | 12.8 KB |
hmc6352.c | -rw-r--r-- | 4.0 KB |
hpilo.c | -rw-r--r-- | 21.6 KB |
hpilo.h | -rw-r--r-- | 5.5 KB |
ibmvmc.c | -rw-r--r-- | 60.7 KB |
ibmvmc.h | -rw-r--r-- | 5.2 KB |
ics932s401.c | -rw-r--r-- | 13.2 KB |
ioc4.c | -rw-r--r-- | 14.3 KB |
isl29003.c | -rw-r--r-- | 11.6 KB |
isl29020.c | -rw-r--r-- | 5.5 KB |
kgdbts.c | -rw-r--r-- | 30.8 KB |
lattice-ecp3-config.c | -rw-r--r-- | 5.9 KB |
pch_phub.c | -rw-r--r-- | 28.0 KB |
pci_endpoint_test.c | -rw-r--r-- | 16.0 KB |
phantom.c | -rw-r--r-- | 13.2 KB |
pti.c | -rw-r--r-- | 26.4 KB |
qcom-coincell.c | -rw-r--r-- | 4.0 KB |
spear13xx_pcie_gadget.c | -rw-r--r-- | 19.9 KB |
sram-exec.c | -rw-r--r-- | 3.6 KB |
sram.c | -rw-r--r-- | 10.8 KB |
sram.h | -rw-r--r-- | 1.2 KB |
tifm_7xx1.c | -rw-r--r-- | 11.1 KB |
tifm_core.c | -rw-r--r-- | 8.3 KB |
tsl2550.c | -rw-r--r-- | 10.7 KB |
vexpress-syscfg.c | -rw-r--r-- | 7.1 KB |
vmw_balloon.c | -rw-r--r-- | 33.8 KB |
![swh spinner](/static/img/swh-spinner.gif)
Computing file changes ...