Closed Bug 1154080 Opened 5 years ago Closed 5 years ago
Long hangs with Adblock Plus in latest Nightly
Starting today I've been experiencing long hangs when loading pages. The Gecko Profiler eventually (after hanging indefinitely on the first try) pointed out ABP as the culprit, and after disabling the add-on completely indeed the hangs are gone. All the time is below this line: exports.FilterStorage._generateFilterData() @ filterStorage.js:479 Adblock Plus There is no single culprit, however - just a very long list of unsymbolicated things like 0xfffc0000344637b0L, which then branch a few times before stopping. I assume this is JITcode, but I can't tell more than that. I've attached the profile in case it's useful. The filter subscriptions I'm using are EasyList, EasyList Dutch and Peter Lowe's list. I don't have a lot of custom rules. I have "Allow some non-intrusive advertising" checked, though I doubt it matters - I was seeing hangs on pages for which I'd disabled ABP. Regression range is somewhere between the 2014-04-11 Nightly and the 2014-04-13 Nightly - the 2014-04-12 Nightly was very late, so I pretty much skipped it. I'll try to narrow it down more, and see if it shows up on a fresh profile. I noticed some other people complaining about Nightly being slow on Mozillazine  so I think this is a common problem.  http://forums.mozillazine.org/viewtopic.php?f=23&t=2928017
Oh, I should mention that I'm not using e10s, because LastPass still doesn't work half the time with e10s enabled. Inbound regression range: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=7de210d3a493&tochange=4d09658bd866 Probably bug 1049290, those cycle collector changes are pretty innocent I think. ting, could you have a look?
A website I can easily reproduce this on is tweakers.net, just loading the front page or various news articles. I have Adblock Plus turned off for tweakers.net, in case that matters.
I'm also getting infinite hangs that don't use any CPU - I don't know if those have the same cause. Unfortunately I can't reproduce those on command, so not much hope of finding a regression range.
Sorry about the bugspam, I keep thinking of more information to add after the fact. I wanted to add that the version of Adblock Plus I am using is 188.8.131.5230, which I think is the latest development build.
[Tracking Requested - why for this release]: Addon compat regression, even assuming there is no functionality change to boot.
djvj, what tools can I use to debug this?
Flags: needinfo?(janus926) → needinfo?(kvijayan)
The offending patch was backed out. Can somebody confirm in next nightly this is fixed? https://hg.mozilla.org/integration/mozilla-inbound/rev/2c35e6083ede
Clear NI, as I am fixing it in bug 1049290.
I can confirm that the hangs are gone with the build from  (based on 2c35e6083ede), thanks! The other hangs I was seeing, where the process hangs indefinitely while not using any CPU, are still there. So those are unrelated - I'll try to sample a stack for them or something.  https://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/mozilla-inbound-win64/1429009329/
I filed the other hangs as bug 1154263.
(In reply to Ting-Yu Chou [:ting] from comment #6) > djvj, what tools can I use to debug this? The issue seems to be that the hang is caused by repeated generation of new ICInNativeDoesNotExist stubs. Chances are that we're repeatedly generating new stubs because the previous stubs' condition code has some flaw that prevents it from successfully recognizing the inputs it gets. The best way to handle this is to trace the issue using a debugger to single-step the jitcode in the generated stub, and figure out WHY the conditions aren't hitting. There's probably some subtle error there that is not easy to spot.
Disregard that last comment, I was looking at the |getprop| instead of the |in| op. I'll post a comment about fixing this on the main bug.
Fixed in the latest Nightly by backout of bug 1049290.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Looks like this was fixed by a backout; tracking the regressing bug instead of this one.
5 years ago
You need to log in before you can comment on or make changes to this bug.