<?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>173734</bug_id>
          
          <creation_ts>2017-06-22 14:04:25 -0700</creation_ts>
          <short_desc>ARM64: doubled executable memory limit from 32MiB to 64MiB</short_desc>
          <delta_ts>2017-06-25 21:59:33 -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>JavaScriptCore</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="JF Bastien">jfbastien</reporter>
          <assigned_to name="JF Bastien">jfbastien</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>fpizlo</cc>
    
    <cc>jfbastien</cc>
    
    <cc>keith_miller</cc>
    
    <cc>mark.lam</cc>
    
    <cc>msaboff</cc>
    
    <cc>oliver</cc>
    
    <cc>saam</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1322043</commentid>
    <comment_count>0</comment_count>
    <who name="JF Bastien">jfbastien</who>
    <bug_when>2017-06-22 14:04:25 -0700</bug_when>
    <thetext>Some WebAssembly programs stress the amount of memory we have available, especially when we consider tiering (BBQ never dies, and is bigger that OMG). Tiering to OMG just piles on more memory, and we&apos;re also competing with JavaScript.

Separately, I&apos;ll try to find places where we allocate too much, and I&apos;ll make sure OMG executable allocation failure fails gracefully.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1322046</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2017-06-22 14:05:30 -0700</bug_when>
    <thetext>&lt;rdar://problem/32932407&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1322050</commentid>
    <comment_count>2</comment_count>
      <attachid>313659</attachid>
    <who name="JF Bastien">jfbastien</who>
    <bug_when>2017-06-22 14:10:14 -0700</bug_when>
    <thetext>Created attachment 313659
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1322079</commentid>
    <comment_count>3</comment_count>
      <attachid>313659</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-06-22 15:12:29 -0700</bug_when>
    <thetext>Comment on attachment 313659
patch

Clearing flags on attachment: 313659

Committed r218723: &lt;http://trac.webkit.org/changeset/218723&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1322080</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-06-22 15:12:31 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1322093</commentid>
    <comment_count>5</comment_count>
    <who name="Saam Barati">saam</who>
    <bug_when>2017-06-22 15:38:45 -0700</bug_when>
    <thetext>Let&apos;s also make it so that we throw an OOM instead of crashing when running out of X memory.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1322096</commentid>
    <comment_count>6</comment_count>
    <who name="JF Bastien">jfbastien</who>
    <bug_when>2017-06-22 15:43:55 -0700</bug_when>
    <thetext>(In reply to Saam Barati from comment #5)
&gt; Let&apos;s also make it so that we throw an OOM instead of crashing when running
&gt; out of X memory.

Oh right I don’t know why I thought we’d want otherwise! I’ll do that after my investigation.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1322757</commentid>
    <comment_count>7</comment_count>
    <who name="JF Bastien">jfbastien</who>
    <bug_when>2017-06-25 21:59:33 -0700</bug_when>
    <thetext>(In reply to JF Bastien from comment #6)
&gt; (In reply to Saam Barati from comment #5)
&gt; &gt; Let&apos;s also make it so that we throw an OOM instead of crashing when running
&gt; &gt; out of X memory.
&gt; 
&gt; Oh right I don’t know why I thought we’d want otherwise! I’ll do that after
&gt; my investigation.

That&apos;s happening here: https://bugs.webkit.org/show_bug.cgi?id=171537</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>313659</attachid>
            <date>2017-06-22 14:10:14 -0700</date>
            <delta_ts>2017-06-22 15:12:29 -0700</delta_ts>
            <desc>patch</desc>
            <filename>0001-ARM64-doubled-executable-memory-limit-from-32MiB-to-.patch</filename>
            <type>text/plain</type>
            <size>2019</size>
            <attacher name="JF Bastien">jfbastien</attacher>
            
              <data encoding="base64">RnJvbSA2ODI0ZTQ0ZDY5ODFjNDFiYTQ5ZjE0NTVmNTcxYWVjNTU0MDI2ZjEwIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBKRiBCYXN0aWVuIDxqZmJhc3RpZW5AYXBwbGUuY29tPgpEYXRl
OiBUaHUsIDIyIEp1biAyMDE3IDE0OjA5OjI4IC0wNzAwClN1YmplY3Q6IFtQQVRDSF0gQVJNNjQ6
IGRvdWJsZWQgZXhlY3V0YWJsZSBtZW1vcnkgbGltaXQgZnJvbSAzMk1pQiB0byA2NE1pQgoKLS0t
CiBTb3VyY2UvSmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nICAgICAgICAgICAgICAgICB8IDE1ICsr
KysrKysrKysrKysrKwogU291cmNlL0phdmFTY3JpcHRDb3JlL2ppdC9FeGVjdXRhYmxlQWxsb2Nh
dG9yLmggfCAgMiArLQogMiBmaWxlcyBjaGFuZ2VkLCAxNiBpbnNlcnRpb25zKCspLCAxIGRlbGV0
aW9uKC0pCgpkaWZmIC0tZ2l0IGEvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZyBiL1Nv
dXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKaW5kZXggZDRjYzcwYi4uZDJjMzY5YyAxMDA2
NDQKLS0tIGEvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZworKysgYi9Tb3VyY2UvSmF2
YVNjcmlwdENvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTggQEAKKzIwMTctMDYtMjIgIEpGIEJh
c3RpZW4gIDxqZmJhc3RpZW5AYXBwbGUuY29tPgorCisgICAgICAgIEFSTTY0OiBkb3VibGVkIGV4
ZWN1dGFibGUgbWVtb3J5IGxpbWl0IGZyb20gMzJNaUIgdG8gNjRNaUIKKyAgICAgICAgaHR0cHM6
Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE3MzczNAorICAgICAgICA8cmRhcjov
L3Byb2JsZW0vMzI5MzI0MDc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgU29tZSBXZWJBc3NlbWJseSBwcm9ncmFtcyBzdHJlc3MgdGhlIGFtb3VudCBv
ZiBtZW1vcnkgd2UgaGF2ZQorICAgICAgICBhdmFpbGFibGUsIGVzcGVjaWFsbHkgd2hlbiB3ZSBj
b25zaWRlciB0aWVyaW5nIChCQlEgbmV2ZXIgZGllcywKKyAgICAgICAgYW5kIGlzIGJpZ2dlciB0
aGF0IE9NRykuIFRpZXJpbmcgdG8gT01HIGp1c3QgcGlsZXMgb24gbW9yZSBtZW1vcnksCisgICAg
ICAgIGFuZCB3ZSdyZSBhbHNvIGNvbXBldGluZyB3aXRoIEphdmFTY3JpcHQuCisKKyAgICAgICAg
KiBqaXQvRXhlY3V0YWJsZUFsbG9jYXRvci5oOgorCiAyMDE3LTA2LTIyICBKb3NlcGggUGVjb3Jh
cm8gIDxwZWNvcmFyb0BhcHBsZS5jb20+CiAKICAgICAgICAgV2ViIEluc3BlY3RvcjogUmVtb3Zl
IHN0YWxlICJyYXdTY29wZXMiIGNvbmNlcHQgdGhhdCB3YXMgbmV2ZXIgYXZhaWxhYmxlIGluIEpT
QwpkaWZmIC0tZ2l0IGEvU291cmNlL0phdmFTY3JpcHRDb3JlL2ppdC9FeGVjdXRhYmxlQWxsb2Nh
dG9yLmggYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvaml0L0V4ZWN1dGFibGVBbGxvY2F0b3IuaApp
bmRleCA1N2M4YjA1Li5hNzJlZmJlIDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENvcmUv
aml0L0V4ZWN1dGFibGVBbGxvY2F0b3IuaAorKysgYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvaml0
L0V4ZWN1dGFibGVBbGxvY2F0b3IuaApAQCAtNjYsNyArNjYsNyBAQCBzdGF0aWMgY29uc3Qgc2l6
ZV90IGZpeGVkRXhlY3V0YWJsZU1lbW9yeVBvb2xTaXplID0gRklYRURfRVhFQ1VUQUJMRV9NRU1P
UllfUE9PTAogI2VsaWYgQ1BVKEFSTSkKIHN0YXRpYyBjb25zdCBzaXplX3QgZml4ZWRFeGVjdXRh
YmxlTWVtb3J5UG9vbFNpemUgPSAxNiAqIDEwMjQgKiAxMDI0OwogI2VsaWYgQ1BVKEFSTTY0KQot
c3RhdGljIGNvbnN0IHNpemVfdCBmaXhlZEV4ZWN1dGFibGVNZW1vcnlQb29sU2l6ZSA9IDMyICog
MTAyNCAqIDEwMjQ7CitzdGF0aWMgY29uc3Qgc2l6ZV90IGZpeGVkRXhlY3V0YWJsZU1lbW9yeVBv
b2xTaXplID0gNjQgKiAxMDI0ICogMTAyNDsKICNlbGlmIENQVShYODZfNjQpCiBzdGF0aWMgY29u
c3Qgc2l6ZV90IGZpeGVkRXhlY3V0YWJsZU1lbW9yeVBvb2xTaXplID0gMTAyNCAqIDEwMjQgKiAx
MDI0OwogI2Vsc2UKLS0gCjIuMTAuMQoK
</data>

          </attachment>
      

    </bug>

</bugzilla>