<?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>111441</bug_id>
          
          <creation_ts>2013-03-05 07:50:09 -0800</creation_ts>
          <short_desc>Cache flush problem on ARMv7 JSC</short_desc>
          <delta_ts>2013-03-08 01:29:52 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>JavaScriptCore</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>108645</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Gabor Rapcsanyi">rgabor</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>barraclough</cc>
    
    <cc>benjamin</cc>
    
    <cc>cmarcelo</cc>
    
    <cc>fpizlo</cc>
    
    <cc>hausmann</cc>
    
    <cc>laszlo.gombos</cc>
    
    <cc>ojan.autocc</cc>
    
    <cc>oliver</cc>
    
    <cc>ossy</cc>
    
    <cc>psychon</cc>
    
    <cc>webkit.review.bot</cc>
    
    <cc>zherczeg</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>847678</commentid>
    <comment_count>0</comment_count>
    <who name="Gabor Rapcsanyi">rgabor</who>
    <bug_when>2013-03-05 07:50:09 -0800</bug_when>
    <thetext>The ARMv7 random crash problem is caused by cache flushing. https://bugs.webkit.org/show_bug.cgi?id=108632 just hide the real bug.

The problem is similar to this bug:
https://bugs.webkit.org/show_bug.cgi?id=77712

After changing the cache flush mechanism in ARMv7 to ARM traditional the problem has gone.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>847683</commentid>
    <comment_count>1</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2013-03-05 07:58:00 -0800</bug_when>
    <thetext>*** Bug 108632 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>847685</commentid>
    <comment_count>2</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2013-03-05 07:59:32 -0800</bug_when>
    <thetext>Could you revert Simon&apos;s workaround ( http://trac.webkit.org/changeset/144170 ) after or with this patch?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>848784</commentid>
    <comment_count>3</comment_count>
      <attachid>191737</attachid>
    <who name="Gabor Rapcsanyi">rgabor</who>
    <bug_when>2013-03-06 06:07:16 -0800</bug_when>
    <thetext>Created attachment 191737
proposed fix

In another patch we should make it more sophisticated but for now it solves the problem.
I&apos;m also reverting Simon&apos;s patch with this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>848968</commentid>
    <comment_count>4</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2013-03-06 09:43:20 -0800</bug_when>
    <thetext>Yay, with this patch there are only 171 remaining inspector crashes on Thumb2,
but they are unrelated to this bug - http://build.webkit.sed.hu/builders/ARMv7%20Linux%20Qt5%20Release%20%28Test%29/builds/7987</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>850949</commentid>
    <comment_count>5</comment_count>
      <attachid>191737</attachid>
    <who name="Zoltan Herczeg">zherczeg</who>
    <bug_when>2013-03-08 00:27:34 -0800</bug_when>
    <thetext>Comment on attachment 191737
proposed fix

r=me.

But I think the current flush mechanism invalidates a large amount of unnecessary, so we should improve with it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>850984</commentid>
    <comment_count>6</comment_count>
      <attachid>191737</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2013-03-08 01:29:48 -0800</bug_when>
    <thetext>Comment on attachment 191737
proposed fix

Clearing flags on attachment: 191737

Committed r145194: &lt;http://trac.webkit.org/changeset/145194&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>850985</commentid>
    <comment_count>7</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2013-03-08 01:29:52 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>191737</attachid>
            <date>2013-03-06 06:07:16 -0800</date>
            <delta_ts>2013-03-08 01:29:48 -0800</delta_ts>
            <desc>proposed fix</desc>
            <filename>armv7_random_crash.patch</filename>
            <type>text/plain</type>
            <size>4304</size>
            <attacher name="Gabor Rapcsanyi">rgabor</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cgYi9Tb3VyY2UvSmF2
YVNjcmlwdENvcmUvQ2hhbmdlTG9nCmluZGV4IDE0MzI0YWEuLjZhMWI5NzIgMTAwNjQ0Ci0tLSBh
L1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL0phdmFTY3JpcHRD
b3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDEzLTAzLTA2ICBHYWJvciBSYXBjc2Fu
eWkgIDxyZ2Fib3JAd2Via2l0Lm9yZz4KKworICAgICAgICBDYWNoZSBmbHVzaCBwcm9ibGVtIG9u
IEFSTXY3IEpTQworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9MTExNDQxCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAg
ICAgTm90IHByb3BlciBjYWNoZSBmbHVzaCBjYXVzaW5nIHJhbmRvbSBjcmFzaGVzIG9uIEFSTXY3
IExpbnV4IHdpdGggVjggdGVzdHMuCisgICAgICAgIFRoZSBwcm9ibGVtIGlzIHNpbWlsYXIgdG8g
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTc3NzEyLgorICAgICAgICBD
aGFuZ2UgdGhlIGNhY2hlIGZ1bHNoIG1lY2hhbmlzbSBzaW1pbGFyIHRvIEFSTSB0cmFkaXRpbmFs
IGFuZCByZXZlcnQgdGhlCisgICAgICAgIHRlbXBvcmFyeSBmaXguCisKKyAgICAgICAgKiBhc3Nl
bWJsZXIvQVJNdjdBc3NlbWJsZXIuaDoKKyAgICAgICAgKEpTQzo6QVJNdjdBc3NlbWJsZXI6OmNh
Y2hlRmx1c2gpOgorCiAyMDEzLTAzLTA1ICBGaWxpcCBQaXpsbyAgPGZwaXpsb0BhcHBsZS5jb20+
CiAKICAgICAgICAgR2V0IHJpZCBvZiB0aGUgaW52ZXJ0IGFyZ3VtZW50IHRvIFNwZWN1bGF0aXZl
SklUOjpqdW1wU2xvd0ZvclVud2FudGVkQXJyYXlNb2RlCmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2
YVNjcmlwdENvcmUvYXNzZW1ibGVyL0FSTXY3QXNzZW1ibGVyLmggYi9Tb3VyY2UvSmF2YVNjcmlw
dENvcmUvYXNzZW1ibGVyL0FSTXY3QXNzZW1ibGVyLmgKaW5kZXggOGE1Mjk5Ny4uZTNmYzU5MyAx
MDA2NDQKLS0tIGEvU291cmNlL0phdmFTY3JpcHRDb3JlL2Fzc2VtYmxlci9BUk12N0Fzc2VtYmxl
ci5oCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9hc3NlbWJsZXIvQVJNdjdBc3NlbWJsZXIu
aApAQCAtMjIxOCwxOCArMjIxOCwyMyBAQCBwdWJsaWM6CiAjaWYgT1MoSU9TKQogICAgICAgICBz
eXNfY2FjaGVfY29udHJvbChrQ2FjaGVGdW5jdGlvblByZXBhcmVGb3JFeGVjdXRpb24sIGNvZGUs
IHNpemUpOwogI2VsaWYgT1MoTElOVVgpCi0gICAgICAgIGFzbSB2b2xhdGlsZSgKLSAgICAgICAg
ICAgICJwdXNoICAgIHtyN31cbiIKLSAgICAgICAgICAgICJtb3YgICAgIHIwLCAlMFxuIgotICAg
ICAgICAgICAgIm1vdiAgICAgcjEsICUxXG4iCi0gICAgICAgICAgICAibW92dyAgICByNywgIzB4
MlxuIgotICAgICAgICAgICAgIm1vdnQgICAgcjcsICMweGZcbiIKLSAgICAgICAgICAgICJtb3Zz
ICAgIHIyLCAjMHgwXG4iCi0gICAgICAgICAgICAic3ZjICAgICAweDBcbiIKLSAgICAgICAgICAg
ICJwb3AgICAgIHtyN31cbiIKLSAgICAgICAgICAgIDoKLSAgICAgICAgICAgIDogInIiIChjb2Rl
KSwgInIiIChyZWludGVycHJldF9jYXN0PGNoYXIqPihjb2RlKSArIHNpemUpCi0gICAgICAgICAg
ICA6ICJyMCIsICJyMSIsICJyMiIpOworICAgICAgICB1aW50cHRyX3QgY3VycmVudFBhZ2UgPSBy
ZWludGVycHJldF9jYXN0PHVpbnRwdHJfdD4oY29kZSkgJiB+KHBhZ2VTaXplKCkgLSAxKTsKKyAg
ICAgICAgdWludHB0cl90IGxhc3RQYWdlID0gKHJlaW50ZXJwcmV0X2Nhc3Q8dWludHB0cl90Pihj
b2RlKSArIHNpemUpICYgfihwYWdlU2l6ZSgpIC0gMSk7CisgICAgICAgIGRvIHsKKyAgICAgICAg
ICAgIGFzbSB2b2xhdGlsZSgKKyAgICAgICAgICAgICAgICAicHVzaCAgICB7cjd9XG4iCisgICAg
ICAgICAgICAgICAgIm1vdiAgICAgcjAsICUwXG4iCisgICAgICAgICAgICAgICAgIm1vdiAgICAg
cjEsICUxXG4iCisgICAgICAgICAgICAgICAgIm1vdncgICAgcjcsICMweDJcbiIKKyAgICAgICAg
ICAgICAgICAibW92dCAgICByNywgIzB4ZlxuIgorICAgICAgICAgICAgICAgICJtb3ZzICAgIHIy
LCAjMHgwXG4iCisgICAgICAgICAgICAgICAgInN2YyAgICAgMHgwXG4iCisgICAgICAgICAgICAg
ICAgInBvcCAgICAge3I3fVxuIgorICAgICAgICAgICAgICAgIDoKKyAgICAgICAgICAgICAgICA6
ICJyIiAoY3VycmVudFBhZ2UpLCAiciIgKGN1cnJlbnRQYWdlICsgcGFnZVNpemUoKSkKKyAgICAg
ICAgICAgICAgICA6ICJyMCIsICJyMSIsICJyMiIpOworICAgICAgICAgICAgY3VycmVudFBhZ2Ug
Kz0gcGFnZVNpemUoKTsKKyAgICAgICAgfSB3aGlsZSAobGFzdFBhZ2UgPj0gY3VycmVudFBhZ2Up
OwogI2VsaWYgT1MoV0lOQ0UpCiAgICAgICAgIENhY2hlUmFuZ2VGbHVzaChjb2RlLCBzaXplLCBD
QUNIRV9TWU5DX0FMTCk7CiAjZWxpZiBPUyhRTlgpCmRpZmYgLS1naXQgYS9Tb3VyY2UvV1RGL0No
YW5nZUxvZyBiL1NvdXJjZS9XVEYvQ2hhbmdlTG9nCmluZGV4IDI0MGYxMzkuLjVhMzg0MzEgMTAw
NjQ0Ci0tLSBhL1NvdXJjZS9XVEYvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XVEYvQ2hhbmdlTG9n
CkBAIC0xLDMgKzEsMTkgQEAKKzIwMTMtMDMtMDYgIEdhYm9yIFJhcGNzYW55aSAgPHJnYWJvckB3
ZWJraXQub3JnPgorCisgICAgICAgIENhY2hlIGZsdXNoIHByb2JsZW0gb24gQVJNdjcgSlNDCisg
ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMTE0NDEKKwor
ICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBOb3QgcHJvcGVy
IGNhY2hlIGZsdXNoIGNhdXNpbmcgcmFuZG9tIGNyYXNoZXMgb24gQVJNdjcgTGludXggd2l0aCBW
OCB0ZXN0cy4KKyAgICAgICAgVGhlIHByb2JsZW0gaXMgc2ltaWxhciB0byBodHRwczovL2J1Z3Mu
d2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Nzc3MTIuCisgICAgICAgIENoYW5nZSB0aGUgY2Fj
aGUgZnVsc2ggbWVjaGFuaXNtIHNpbWlsYXIgdG8gQVJNIHRyYWRpdGluYWwgYW5kIHJldmVydCB0
aGUKKyAgICAgICAgdGVtcG9yYXJ5IGZpeC4KKworICAgICAgICAqIHd0Zi9PU0FsbG9jYXRvclBv
c2l4LmNwcDoKKyAgICAgICAgKFdURjo6T1NBbGxvY2F0b3I6OnJlc2VydmVVbmNvbW1pdHRlZCk6
CisgICAgICAgIChXVEY6Ok9TQWxsb2NhdG9yOjpkZWNvbW1pdCk6CisKIDIwMTMtMDMtMDUgIFJ5
dWFuIENob2kgIDxyeXVhbi5jaG9pQHNhbXN1bmcuY29tPgogCiAgICAgICAgIFtFRkxdIEJ1aWxk
IGJyZWFrIHdpdGggbGF0ZXN0IEVGTCBsaWJyYXJpZXMKZGlmZiAtLWdpdCBhL1NvdXJjZS9XVEYv
d3RmL09TQWxsb2NhdG9yUG9zaXguY3BwIGIvU291cmNlL1dURi93dGYvT1NBbGxvY2F0b3JQb3Np
eC5jcHAKaW5kZXggODcxNWI4MS4uN2IyYTU1YyAxMDA2NDQKLS0tIGEvU291cmNlL1dURi93dGYv
T1NBbGxvY2F0b3JQb3NpeC5jcHAKKysrIGIvU291cmNlL1dURi93dGYvT1NBbGxvY2F0b3JQb3Np
eC5jcHAKQEAgLTQzLDcgKzQzLDcgQEAgdm9pZCogT1NBbGxvY2F0b3I6OnJlc2VydmVVbmNvbW1p
dHRlZChzaXplX3QgYnl0ZXMsIFVzYWdlIHVzYWdlLCBib29sIHdyaXRhYmxlLAogICAgIHZvaWQq
IHJlc3VsdCA9IG1tYXAoMCwgYnl0ZXMsIFBST1RfTk9ORSwgTUFQX0xBWlkgfCBNQVBfUFJJVkFU
RSB8IE1BUF9BTk9OLCAtMSwgMCk7CiAgICAgaWYgKHJlc3VsdCA9PSBNQVBfRkFJTEVEKQogICAg
ICAgICBDUkFTSCgpOwotI2VsaWYgT1MoTElOVVgpICYmICFDUFUoQVJNKQorI2VsaWYgT1MoTElO
VVgpCiAgICAgVU5VU0VEX1BBUkFNKHVzYWdlKTsKICAgICBVTlVTRURfUEFSQU0od3JpdGFibGUp
OwogICAgIFVOVVNFRF9QQVJBTShleGVjdXRhYmxlKTsKQEAgLTE2NSw3ICsxNjUsNyBAQCB2b2lk
IE9TQWxsb2NhdG9yOjpkZWNvbW1pdCh2b2lkKiBhZGRyZXNzLCBzaXplX3QgYnl0ZXMpCiAjaWYg
T1MoUU5YKQogICAgIC8vIFVzZSBQUk9UX05PTkUgYW5kIE1BUF9MQVpZIHRvIGRlY29tbWl0IHRo
ZSBwYWdlcy4KICAgICBtbWFwKGFkZHJlc3MsIGJ5dGVzLCBQUk9UX05PTkUsIE1BUF9GSVhFRCB8
IE1BUF9MQVpZIHwgTUFQX1BSSVZBVEUgfCBNQVBfQU5PTiwgLTEsIDApOwotI2VsaWYgT1MoTElO
VVgpICYmICFDUFUoQVJNKQorI2VsaWYgT1MoTElOVVgpCiAgICAgbWFkdmlzZShhZGRyZXNzLCBi
eXRlcywgTUFEVl9ET05UTkVFRCk7CiAgICAgaWYgKG1wcm90ZWN0KGFkZHJlc3MsIGJ5dGVzLCBQ
Uk9UX05PTkUpKQogICAgICAgICBDUkFTSCgpOwo=
</data>

          </attachment>
      

    </bug>

</bugzilla>