<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>30293</bug_id>
          
          <creation_ts>2009-10-11 19:33:41 -0700</creation_ts>
          <short_desc>[Qt] performance regressions on loading page after ImageDecoderQt changes</short_desc>
          <delta_ts>2010-09-23 11:24:40 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebCore Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Performance, Qt, Regression</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>34208</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Holger Freyther">zecke</reporter>
          <assigned_to name="QtWebKit Unassigned">webkit-qt-unassigned</assigned_to>
          <cc>benjamin</cc>
    
    <cc>hausmann</cc>
    
    <cc>jturcotte</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>153960</commentid>
    <comment_count>0</comment_count>
    <who name="Holger Freyther">zecke</who>
    <bug_when>2009-10-11 19:33:41 -0700</bug_when>
    <thetext>The old and new code has been executed on the same data and 30 iterations each. The majority of cases are faster, some show a regression we should need to understand. The point of this bug is to understand the cause of this regression.

loading test
row   - slower in msecs on 30 iterations

row01 - 3msec
row02 - 6msec
row14 - 844msec
row16 - 1031msec
row19 - 538msec
row22 - 49msec
row24 - 73msec
row25 - 693msec
row26 - 705msec
row30 - 3msec
row32 - 1785msec
row37 - 35msec
row39 - 261msec


cycling: slower in msecs on 30 iterations

row09 -  9msec
row11 - 39msec
row17 - 1msec
row27 - 10msec
row30 - 8msec
row32 - 114msec
row38 - 1msec
row39 - 1051msec</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>154000</commentid>
    <comment_count>1</comment_count>
    <who name="Holger Freyther">zecke</who>
    <bug_when>2009-10-12 01:16:40 -0700</bug_when>
    <thetext>(In reply to comment #0)


&gt; row32 - 1785msec

This does not happen anymore. I will have to redo tests and verify the /etc/hosts.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>154004</commentid>
    <comment_count>2</comment_count>
    <who name="Holger Freyther">zecke</who>
    <bug_when>2009-10-12 01:46:33 -0700</bug_when>
    <thetext>Okay, new regression list:

 RESULT : tst_Loading::load():&quot;row06&quot;:
-     576.83 msec per iteration (total: 17305, iterations: 30)
+     623.29 msec per iteration (total: 18699, iterations: 30)
 RESULT : tst_Loading::load():&quot;row09&quot;:
-     3.33 msec per iteration (total: 100, iterations: 30)
+     5.23 msec per iteration (total: 157, iterations: 30)
 RESULT : tst_Loading::load():&quot;row10&quot;:
-     872.39 msec per iteration (total: 26172, iterations: 30)
+     875.20 msec per iteration (total: 26256, iterations: 30)
 RESULT : tst_Loading::load():&quot;row12&quot;:
-     411.80 msec per iteration (total: 12354, iterations: 30)
+     412.33 msec per iteration (total: 12370, iterations: 30)
 RESULT : tst_Loading::load():&quot;row14&quot;:
-     215.7 msec per iteration (total: 6472, iterations: 30)
+     304.4 msec per iteration (total: 9134, iterations: 30)
 RESULT : tst_Loading::load():&quot;row18&quot;:
-     1,000.6 msec per iteration (total: 30020, iterations: 30)
+     1,019.8 msec per iteration (total: 30597, iterations: 30)
 RESULT : tst_Loading::load():&quot;row19&quot;:
-     23.6 msec per iteration (total: 710, iterations: 30)
+     30.1 msec per iteration (total: 905, iterations: 30)
 RESULT : tst_Loading::load():&quot;row20&quot;:
-     80.29 msec per iteration (total: 2409, iterations: 30)
+     98.70 msec per iteration (total: 2961, iterations: 30)
 RESULT : tst_Loading::load():&quot;row22&quot;:
-     2.7 msec per iteration (total: 83, iterations: 30)
+     3.2 msec per iteration (total: 96, iterations: 30)
 RESULT : tst_Loading::load():&quot;row25&quot;:
-     49.03 msec per iteration (total: 1471, iterations: 30)
+     49.23 msec per iteration (total: 1477, iterations: 30)
 RESULT : tst_Loading::load():&quot;row26&quot;:
-     138.1 msec per iteration (total: 4144, iterations: 30)
+     152.2 msec per iteration (total: 4567, iterations: 30)
 RESULT : tst_Loading::load():&quot;row30&quot;:
-     5.36 msec per iteration (total: 161, iterations: 30)
+     7.36 msec per iteration (total: 221, iterations: 30)
 RESULT : tst_Loading::load():&quot;row31&quot;:
-     40.66 msec per iteration (total: 1220, iterations: 30)
+     44.93 msec per iteration (total: 1348, iterations: 30)
 RESULT : tst_Loading::load():&quot;row32&quot;:
-     327.3 msec per iteration (total: 9820, iterations: 30)
+     354.60 msec per iteration (total: 10638, iterations: 30)
 RESULT : tst_Loading::load():&quot;row37&quot;:
-     44.33 msec per iteration (total: 1330, iterations: 30)
+     62.53 msec per iteration (total: 1876, iterations: 30)
 RESULT : tst_Loading::load():&quot;row38&quot;:
-     238.1 msec per iteration (total: 7145, iterations: 30)
+     259.9 msec per iteration (total: 7798, iterations: 30)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>154015</commentid>
    <comment_count>3</comment_count>
      <attachid>41029</attachid>
    <who name="Holger Freyther">zecke</who>
    <bug_when>2009-10-12 03:12:01 -0700</bug_when>
    <thetext>Created attachment 41029
Current loading speed regressions

Current update. This is coming where everything is locally served. We are seeing 12 cases that seem to be slower than before. The problem is QBENCHMARK is not really good for statistics. If one loading took extremely long our whole result is screwed..</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>220732</commentid>
    <comment_count>4</comment_count>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2010-05-04 13:31:31 -0700</bug_when>
    <thetext>Do you remember what you are comparing against?
I don&apos;t have regressions compared to 4.6.2.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>283795</commentid>
    <comment_count>5</comment_count>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2010-09-23 11:24:40 -0700</bug_when>
    <thetext>The benchmarks are now run regularily. Image loading in with 4.7 if faster than with 4.6.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>41029</attachid>
            <date>2009-10-12 03:12:01 -0700</date>
            <delta_ts>2009-10-12 03:12:01 -0700</delta_ts>
            <desc>Current loading speed regressions</desc>
            <filename>result</filename>
            <type>application/octet-stream</type>
            <size>1980</size>
            <attacher name="Holger Freyther">zecke</attacher>
            
              <data encoding="base64">LS0tIG5ldy1sb2cucHJlLWltZwkyMDA5LTEwLTEyIDEyOjA2OjIzLjAwMDAwMDAwMCArMDIwMAor
KysgbmV3LWxvZy5pbWcJMjAwOS0xMC0xMiAxMTo1Nzo0OS4wMDAwMDAwMDAgKzAyMDAKQEAgLTEs
NDEgKzEsNDcgQEAKICoqKioqKioqKiBTdGFydCB0ZXN0aW5nIG9mIHRzdF9Mb2FkaW5nICoqKioq
KioqKgogQ29uZmlnOiBVc2luZyBRVGVzdCBsaWJyYXJ5IDQuNy4wLCBRdCA0LjcuMAogUEFTUyAg
IDogdHN0X0xvYWRpbmc6OmluaXRUZXN0Q2FzZSgpCi1RV0FSTiAgOiB0c3RfTG9hZGluZzo6bG9h
ZChyb3cwMSkgUFJFIEltYWdlIFdvcmsKK1FXQVJOICA6IHRzdF9Mb2FkaW5nOjpsb2FkKHJvdzAx
KSBJbWFnZSBEZWNvZGVyIGNoYW5nZXMKIFJFU1VMVCA6IHRzdF9Mb2FkaW5nOjpsb2FkKCk6InJv
dzAzIjoKLSAgICAgNjIwLjc5IG1zZWMgcGVyIGl0ZXJhdGlvbiAodG90YWw6IDE4NjI0LCBpdGVy
YXRpb25zOiAzMCkKKyAgICAgNjYyLjIzIG1zZWMgcGVyIGl0ZXJhdGlvbiAodG90YWw6IDE5ODY3
LCBpdGVyYXRpb25zOiAzMCkKIFJFU1VMVCA6IHRzdF9Mb2FkaW5nOjpsb2FkKCk6InJvdzA2IjoK
LSAgICAgNDYzLjIzIG1zZWMgcGVyIGl0ZXJhdGlvbiAodG90YWw6IDEzODk3LCBpdGVyYXRpb25z
OiAzMCkKKyAgICAgNDgwLjAwIG1zZWMgcGVyIGl0ZXJhdGlvbiAodG90YWw6IDE0NDAwLCBpdGVy
YXRpb25zOiAzMCkKIFJFU1VMVCA6IHRzdF9Mb2FkaW5nOjpsb2FkKCk6InJvdzA5IjoKLSAgICAg
NC40MyBtc2VjIHBlciBpdGVyYXRpb24gKHRvdGFsOiAxMzMsIGl0ZXJhdGlvbnM6IDMwKQorICAg
ICA1LjE2IG1zZWMgcGVyIGl0ZXJhdGlvbiAodG90YWw6IDE1NSwgaXRlcmF0aW9uczogMzApCiBS
RVNVTFQgOiB0c3RfTG9hZGluZzo6bG9hZCgpOiJyb3cxNiI6Ci0gICAgIDY5NS4yMyBtc2VjIHBl
ciBpdGVyYXRpb24gKHRvdGFsOiAyMDg1NywgaXRlcmF0aW9uczogMzApCisgICAgIDY5Ni4yMCBt
c2VjIHBlciBpdGVyYXRpb24gKHRvdGFsOiAyMDg4NiwgaXRlcmF0aW9uczogMzApCiBSRVNVTFQg
OiB0c3RfTG9hZGluZzo6bG9hZCgpOiJyb3cyNCI6Ci0gICAgIDMuNzAgbXNlYyBwZXIgaXRlcmF0
aW9uICh0b3RhbDogMTExLCBpdGVyYXRpb25zOiAzMCkKKyAgICAgNS40NiBtc2VjIHBlciBpdGVy
YXRpb24gKHRvdGFsOiAxNjQsIGl0ZXJhdGlvbnM6IDMwKQogUkVTVUxUIDogdHN0X0xvYWRpbmc6
OmxvYWQoKToicm93MjYiOgotICAgICAxNzkuNiBtc2VjIHBlciBpdGVyYXRpb24gKHRvdGFsOiA1
MzkwLCBpdGVyYXRpb25zOiAzMCkKKyAgICAgMTc5LjggbXNlYyBwZXIgaXRlcmF0aW9uICh0b3Rh
bDogNTM5NiwgaXRlcmF0aW9uczogMzApCiBSRVNVTFQgOiB0c3RfTG9hZGluZzo6bG9hZCgpOiJy
b3cyOSI6Ci0gICAgIDQ5Ljc5IG1zZWMgcGVyIGl0ZXJhdGlvbiAodG90YWw6IDE0OTQsIGl0ZXJh
dGlvbnM6IDMwKQorICAgICA1NS4yMCBtc2VjIHBlciBpdGVyYXRpb24gKHRvdGFsOiAxNjU2LCBp
dGVyYXRpb25zOiAzMCkKIFJFU1VMVCA6IHRzdF9Mb2FkaW5nOjpsb2FkKCk6InJvdzMwIjoKLSAg
ICAgMy4zMyBtc2VjIHBlciBpdGVyYXRpb24gKHRvdGFsOiAxMDAsIGl0ZXJhdGlvbnM6IDMwKQor
ICAgICA2LjA5IG1zZWMgcGVyIGl0ZXJhdGlvbiAodG90YWw6IDE4MywgaXRlcmF0aW9uczogMzAp
CiBSRVNVTFQgOiB0c3RfTG9hZGluZzo6bG9hZCgpOiJyb3czMSI6Ci0gICAgIDI5LjggbXNlYyBw
ZXIgaXRlcmF0aW9uICh0b3RhbDogODk3LCBpdGVyYXRpb25zOiAzMCkKKyAgICAgNDQuMDYgbXNl
YyBwZXIgaXRlcmF0aW9uICh0b3RhbDogMTMyMiwgaXRlcmF0aW9uczogMzApCiBSRVNVTFQgOiB0
c3RfTG9hZGluZzo6bG9hZCgpOiJyb3czNCI6Ci0gICAgIDEsNzMxLjUgbXNlYyBwZXIgaXRlcmF0
aW9uICh0b3RhbDogNTE5NDUsIGl0ZXJhdGlvbnM6IDMwKQorICAgICAxLDc5NS41IG1zZWMgcGVy
IGl0ZXJhdGlvbiAodG90YWw6IDUzODY3LCBpdGVyYXRpb25zOiAzMCkK
</data>

          </attachment>
      

    </bug>

</bugzilla>