Closed Bug 601222 Opened 14 years ago Closed 14 years ago

Blacklist tracejit and methodjit on samsung i9000 devices


(Core :: JavaScript Engine, defect)

Not set



Tracking Status
fennec 2.0b1+ ---


(Reporter: cjones, Assigned: cjones)



(Keywords: relnote)


(1 file)

+++ This bug was initially created as a clone of Bug #600488 +++

Spun off from bug 600596 and bug 600488, which now need to block betaN instead of beta1 (I don't have those powers).
On my vibrant, this makes -j testComparisons.js crash go away.  With

    os.environ['JS_IGNORE_JIT_BROKENNESS'] = '1'
    args = [
        '--jitflags', 'j',

it comes back.

This is a gross hack, but the precedent is 

                       && JSC::MacroAssemblerX86Common::getSSEState() >=

To my knowledge, we don't have a cleaner way of checking the host CPU.
Assignee: general → jones.chris.g
Attachment #480221 - Flags: review?(dvander)
tracking-fennec: --- → 2.0b1+
Attachment #480221 - Flags: review?(dvander) → review+ (relbranch)

Closed: 14 years ago
Resolution: --- → FIXED
It struck me while I was walking to dinner that it might have been a good idea to write something to logcat when we disable JIT.  Ah well, maybe can get that in if we respin.
It's really too bad that this went in with an env var instead of a pref :(  The jit issue seems fixed with Android 2.2, but it's very difficult to enable the jit now.
Maybe we could change the blacklist to also depend on the OS version, or the kernel version.  The leaked Android 2.2 image for Captivate (I assume this is the one that Vlad is testing) uses kernel according to this page (compared to 2.6.29 in the shipping Android 2.1 images):
(In reply to comment #4)
> The
> jit issue seems fixed with Android 2.2, but it's very difficult to enable the
> jit now.

How did you test the JIT?  (It's easy to enable for test with the script here

If it's indeed fixed, yes, we should pass 2.2 through.
Well, I tested by running sunspider and kraken, and browsing for about 20-30 minutes with no hangs.  I've never been able to browse that long before without issues.  I'll give the tests a try when I get a chance.
Blocks: 602393
The blacklist in the patch doesn't include the string "GT-I9000" which is what my (UK/European, non-carrier-model) Galaxy S has for 'Hardware' according to /proc/cpuinfo. Should it?
If there's a significant number of these running 2.1 (eclair), then yes.
Samsung have not yet officially released Froyo firmware for the device; besides those using leaked or community builds, every Galaxy S runs 2.1-update1. Also, note that updates are via Samsung's "Kies" software rather than OTA, so uptake of 2.2 (rumoured to arrive later this month) will be slow. The device is on the majority of UK carriers and is reasonably popular.
Blocks: 603624
Samsung has officially realeased froyo for Galaxy S phones, most carriers in the US have not decided to release it yet. Although fully legit leaked versions of froyo for galaxy s phones are available.
Blocks: 638963
You need to log in before you can comment on or make changes to this bug.