749608 - Add dvm* to skiplist for Android

Status: RESOLVED DUPLICATE of bug 741315

(Socorro :: Infra, task)

Description

[James Willcox (:snorp) ([e-mail]) (he/him)]

Any time we crash on Android in the Dalvik VM, we get a trace that has a bunch of libdvm symbols (see bug 747799). Please add 'dvm' to the skiplist regex.

Comment 1

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

I believe the regex would be:
'libdvm.so@0x.*'

Is that correct?

Comment 2

[Robert Helmer [:rhelmer]]

(In reply to Naoki Hirata :nhirata from comment #1)
> I believe the regex would be:
> 'libdvm.so@0x.*'
> 
> Is that correct?

Hmm looks like we already have it:
https://github.com/mozilla/socorro/blob/master/scripts/config/processorconfig.py.dist#L134

Is it incorrect or otherwise not working?

Comment 3

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

You're right.... that should do the trick but for some reason it is not working.

Comment 4

[Scoobidiver (away)]

If every frames in the stack are in the ignore skiplist, the fallback mechanism is to show the first frame anyway. So I think it should be closed as invalid.

Comment 5

[Scoobidiver (away)]

The current libdvm ignore list works. See bp-b70404d0-75b1-49ea-ac04-4db0d2120505 (there's bug 741222 for dalvik-LinearAlloc (deleted))

Comment 6

[Robert Helmer [:rhelmer]]

(In reply to Scoobidiver from comment #4)
> If every frames in the stack are in the ignore skiplist, the fallback
> mechanism is to show the first frame anyway. So I think it should be closed
> as invalid.

Please reopen if there's any disagreement ^

Comment 7

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Turns out that we need it for bug 738198

Socorro is still showing : dvmAsmInstructionStart

It should be skipping this, but I believe that skiplist is expecting a \s so it's not skipping it properly.


'libdvm.so' should skip the correct stuff.  I had the wrong skiplist in comment 1.

Comment 8

[Scoobidiver (away)]

If the goal of this bug is to add dvm.*, it's already asked in bug 741315.

Comment 9

[Robert Kaiser]

In that light I also don't understand the reopen here.

Comment 10

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Hrm.  true.  closing; I wasn't thinking when I reopened this bug, I suppose.  I'll close it as a dup.