Closed
Bug 675568
Opened 13 years ago
Closed 13 years ago
SIMD of libjpeg turbo is disabled on Windows
Categories
(Firefox Build System :: General, defect)
Tracking
(firefox5 unaffected, firefox6+ fixed, firefox7+ fixed, firefox8+ fixed)
RESOLVED
FIXED
mozilla8
People
(Reporter: m_kato, Assigned: m_kato)
References
Details
(Keywords: regression, Whiteboard: [qa-])
Attachments
(2 files, 1 obsolete file)
(deleted),
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jpr
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
SKIP_LIBRARY_CHECKS is "yes" on Windows, so SIMD of libjpeg turbo is turned off. We should turn on.
Assignee | ||
Comment 1•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
Attachment #549720 -
Flags: review?(khuey)
Assignee | ||
Updated•13 years ago
|
Component: ImageLib → Build Config
QA Contact: imagelib → build-config
Assignee | ||
Updated•13 years ago
|
Keywords: regression
Comment 2•13 years ago
|
||
Do we have any idea when this changed?
(In reply to comment #2)
> Do we have any idea when this changed?
According to Bug 652399, for Gecko 6.
Comment 4•13 years ago
|
||
Setting tracking-? for FF6-8. This is a major (factor of 2?) jpeg decoding regression. This matters particularly much now that we aggressively discard images in background tabs.
Risk of taking this even on beta is low, because we had libjpeg-turbo enabled in FF4 and FF5.
status-firefox6:
--- → affected
status-firefox7:
--- → affected
status-firefox8:
--- → affected
tracking-firefox6:
--- → ?
tracking-firefox7:
--- → ?
tracking-firefox8:
--- → ?
Comment on attachment 549720 [details] [diff] [review]
fix
We should fix this by moving the added check down to the libjpeg-turbo section.
We should also make it so that ./configure --with-system-jpeg --enable-libjpeg-turbo errors out.
Attachment #549720 -
Flags: review?(khuey) → review-
Comment 6•13 years ago
|
||
I backed out bug 652399 on beta: http://hg.mozilla.org/releases/mozilla-beta/rev/f83956d23621
I didn't back out on Aurora, so we'll want to take this fix on Aurora.
Updated•13 years ago
|
Comment 7•13 years ago
|
||
The push from comment 6 was to the relbranch, not the default branch. Properly backed out on beta: http://hg.mozilla.org/releases/mozilla-beta/rev/eeabcaa28b68
status-firefox5:
--- → unaffected
Assignee | ||
Comment 8•13 years ago
|
||
Attachment #549720 -
Attachment is obsolete: true
Assignee | ||
Updated•13 years ago
|
Attachment #550975 -
Flags: review?(khuey)
Comment on attachment 550975 [details] [diff] [review]
fix v2
>@@ -6317,16 +6316,20 @@ if test -n "$MOZ_LIBJPEG_TURBO"; then
>+ if test "$SYSTEM_JPEG" = 1; then
>+ AC_MSG_ERROR([cannot use --with-system-jpeg with --enable-libjpeg-turbo.])
>+ fi
>+
I think this should be right after the MOZ_ARG_DISABLE_BOOL for libjpeg-turbo.
r=me with that.
Attachment #550975 -
Flags: review?(khuey) → review+
Assignee | ||
Comment 10•13 years ago
|
||
landed with khuey's comment
http://hg.mozilla.org/integration/mozilla-inbound/rev/5f5c92568eb8
Whiteboard: [inbound]
Assignee | ||
Comment 11•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
Attachment #551389 -
Flags: approval-mozilla-aurora?
Status: NEW → RESOLVED
Closed: 13 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
Updated•13 years ago
|
Attachment #551389 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 13•13 years ago
|
||
Comment 14•13 years ago
|
||
Mozilla/5.0 (Windows NT 5.1; rv:7.0) Gecko/20100101 Firefox/7.0
Can anyone please help me with a test case or with STR / guidelines that I can use to verify this fix?
Thanks!
Comment 15•13 years ago
|
||
Find a very large JPEG image (not a solid color). Load it in a tab. In a new tab, load about:memory and click "minimize memory usage" -- this should cause the image to be discarded. Now switch back to the tab with the image. Time with a stopwatch how long it takes before the image is displayed onscreen. It should be significantly faster after this change than before.
Alternatively, you could profile a release build under a workload which spends most of its time decoding JPEGs, and examine the stack traces to see whether it's spending time in C or ASM routines.
You could also run a build in a debugger and see whether it's spending time in an ASM routine, executing SSE2 code.
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•