ACPR/android_system_core
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Hash only warning "file:line" in warning collector.

Browse source 
Before this change, the signature of a warning is computed from
a string such as this one:

/mnt/host/source/src/third_party/kernel/files/drivers/gpu/drm/i915/intel_dp.c:351 intel_dp_check_edp+0x6b/0xb9()

This change removes all text after the first space, i.e. the function name
and IP offset.  The latter varies across architectures and can vary across
builds, giving different signatures for the same warning.

BUG=chromium:227080
TEST=manual

Change-Id: Ied65dbd8edd734bd152a9dd35aaa48f1e8f1b9e8
Reviewed-on: https://gerrit.chromium.org/gerrit/49505
Commit-Queue: Luigi Semenzato <semenzato@chromium.org>
Reviewed-by: Luigi Semenzato <semenzato@chromium.org>
Tested-by: Luigi Semenzato <semenzato@chromium.org>
This commit is contained in:
Luigi Semenzato 2013-04-29 12:38:36 -07:00 • committed by ChromeBot
parent 6fdc0b48cb
commit 5eb471c416
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
crash_reporter/warn_collector.l
View file

@ -48,7 +48,7 @@ END_TRACE {PREFIX}" ---[ end trace".*
^{CUT_HERE}\n{WARNING} BEGIN(PRE_WARN);
.|\n /* ignore all other input in state 0 */
<PRE_WARN>.*\n if (WarnStart()) {
<PRE_WARN>[^ ]+\n if (WarnStart()) { /* yytext is file:line */
BEGIN(WARN); ECHO;
} else {
BEGIN(0);