<?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>172143</bug_id>
          
          <creation_ts>2017-05-15 16:49:08 -0700</creation_ts>
          <short_desc>bmalloc: Bump the size of the deallocator log to 512</short_desc>
          <delta_ts>2017-05-15 18:29:41 -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>New Bugs</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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Geoffrey Garen">ggaren</reporter>
          <assigned_to name="Geoffrey Garen">ggaren</assigned_to>
          <cc>msaboff</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1308863</commentid>
    <comment_count>0</comment_count>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2017-05-15 16:49:08 -0700</bug_when>
    <thetext>bmalloc: Bump the size of the deallocator log to 512</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1308865</commentid>
    <comment_count>1</comment_count>
      <attachid>310186</attachid>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2017-05-15 16:50:17 -0700</bug_when>
    <thetext>Created attachment 310186
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1308866</commentid>
    <comment_count>2</comment_count>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2017-05-15 16:51:15 -0700</bug_when>
    <thetext>Benchmark results from Mac Pro:

~/OpenSource/Source/bmalloc&gt; ~/OpenSource/PerformanceTests/MallocBench/run-malloc-benchmarks Baseline:~/OpenSource/WebKitBuildBaseline/Release/ Patch:~/OpenSource/WebKitBuild/Release/
                                                                                
                                                      Baseline                          Patch                              Δ
Execution Time:
    churn                                                 71ms                           71ms                               
    list_allocate                                         68ms                           68ms                               
    tree_allocate                                         65ms                           65ms                               
    tree_churn                                            76ms                           76ms                               
    fragment                                              61ms                           61ms                               
    fragment_iterate                                      50ms                           49ms                 ^ 1.02x faster
    medium                                               147ms                          145ms                 ^ 1.01x faster
    big                                                  122ms                          124ms                 ! 1.02x slower
    facebook                                             182ms                          180ms                 ^ 1.01x faster
    reddit                                               103ms                          100ms                 ^ 1.03x faster
    flickr                                               106ms                          104ms                 ^ 1.02x faster
    theverge                                             133ms                          135ms                 ! 1.02x slower
    nimlang                                              118ms                          117ms                 ^ 1.01x faster
    message_one                                          172ms                          174ms                 ! 1.01x slower
    message_many                                         935ms                          920ms                 ^ 1.02x faster
    churn --parallel                                      42ms                           32ms                 ^ 1.31x faster
    list_allocate --parallel                             145ms                          143ms                 ^ 1.01x faster
    tree_allocate --parallel                             751ms                          770ms                 ! 1.03x slower
    tree_churn --parallel                                917ms                          904ms                 ^ 1.01x faster
    fragment --parallel                                   78ms                           77ms                 ^ 1.01x faster
    fragment_iterate --parallel                           12ms                           11ms                 ^ 1.09x faster
    medium --parallel                                    131ms                          127ms                 ^ 1.03x faster
    big --parallel                                       114ms                          115ms                 ! 1.01x slower
    facebook --parallel                                4,850ms                        4,760ms                 ^ 1.02x faster
    reddit --parallel                                  3,472ms                        3,495ms                 ! 1.01x slower
    flickr --parallel                                  3,497ms                        3,460ms                 ^ 1.01x faster
    theverge --parallel                                4,709ms                        4,684ms                 ^ 1.01x faster

    &lt;geometric mean&gt;                                     198ms                          195ms                 ^ 1.02x faster
    &lt;arithmetic mean&gt;                                    782ms                          777ms                 ^ 1.01x faster
    &lt;harmonic mean&gt;                                       89ms                           85ms                 ^ 1.05x faster

Peak Memory:
    churn                                              1,032kB                        1,020kB                ^ 1.01x smaller
    list_allocate                                      2,312kB                        2,312kB                               
    tree_allocate                                      5,672kB                        5,672kB                               
    tree_churn                                         4,944kB                        4,944kB                               
    fragment                                           7,212kB                        7,224kB                  ! 1.0x bigger
    fragment_iterate                                  25,804kB                       25,812kB                  ! 1.0x bigger
    medium                                         1,182,668kB                    1,182,680kB                  ! 1.0x bigger
    big                                            1,083,860kB                    1,083,872kB                  ! 1.0x bigger
    facebook                                          73,112kB                       73,036kB                 ^ 1.0x smaller
    reddit                                            14,192kB                       14,040kB                ^ 1.01x smaller
    flickr                                            26,716kB                       26,568kB                ^ 1.01x smaller
    theverge                                          27,068kB                       27,024kB                 ^ 1.0x smaller
    nimlang                                          173,944kB                      173,804kB                 ^ 1.0x smaller
    message_one                                        4,288kB                        4,448kB                 ! 1.04x bigger
    message_many                                       2,716kB                        3,040kB                 ! 1.12x bigger
    churn --parallel                                   1,652kB                        1,688kB                 ! 1.02x bigger
    list_allocate --parallel                           2,972kB                        3,076kB                 ! 1.03x bigger
    tree_allocate --parallel                          12,704kB                       13,396kB                 ! 1.05x bigger
    tree_churn --parallel                             12,968kB                       12,956kB                 ^ 1.0x smaller
    fragment --parallel                                7,232kB                        7,392kB                 ! 1.02x bigger
    fragment_iterate --parallel                       26,452kB                       26,264kB                ^ 1.01x smaller
    medium --parallel                              1,147,656kB                    1,124,716kB                ^ 1.02x smaller
    big --parallel                                 1,043,872kB                    1,037,744kB                ^ 1.01x smaller
    facebook --parallel                            1,594,592kB                    1,596,080kB                  ! 1.0x bigger
    reddit --parallel                                289,932kB                      292,100kB                 ! 1.01x bigger
    flickr --parallel                                553,680kB                      553,156kB                 ^ 1.0x smaller
    theverge --parallel                              604,364kB                      599,460kB                ^ 1.01x smaller

    &lt;geometric mean&gt;                                  35,459kB                       35,743kB                 ! 1.01x bigger
    &lt;arithmetic mean&gt;                                293,838kB                      292,723kB                 ^ 1.0x smaller
    &lt;harmonic mean&gt;                                    6,737kB                        6,850kB                 ! 1.02x bigger

Memory at End:
    churn                                                592kB                          580kB                ^ 1.02x smaller
    list_allocate                                        596kB                          596kB                               
    tree_allocate                                        620kB                          620kB                               
    tree_churn                                           616kB                          616kB                               
    fragment                                             632kB                          644kB                 ! 1.02x bigger
    fragment_iterate                                     792kB                          800kB                 ! 1.01x bigger
    medium                                            12,188kB                       12,200kB                  ! 1.0x bigger
    big                                                1,376kB                        1,388kB                 ! 1.01x bigger
    facebook                                           2,972kB                        2,880kB                ^ 1.03x smaller
    reddit                                             2,028kB                        1,876kB                ^ 1.08x smaller
    flickr                                             3,032kB                        2,884kB                ^ 1.05x smaller
    theverge                                           2,984kB                        2,940kB                ^ 1.01x smaller
    nimlang                                           58,348kB                       58,320kB                 ^ 1.0x smaller
    message_one                                          932kB                        1,020kB                 ! 1.09x bigger
    message_many                                       1,320kB                        1,740kB                 ! 1.32x bigger
    churn --parallel                                   1,216kB                        1,252kB                 ! 1.03x bigger
    list_allocate --parallel                           1,508kB                        1,528kB                 ! 1.01x bigger
    tree_allocate --parallel                           2,196kB                        2,520kB                 ! 1.15x bigger
    tree_churn --parallel                              2,780kB                        2,836kB                 ! 1.02x bigger
    fragment --parallel                                2,328kB                        2,836kB                 ! 1.22x bigger
    fragment_iterate --parallel                        1,668kB                        1,808kB                 ! 1.08x bigger
    medium --parallel                                 31,124kB                       38,864kB                 ! 1.25x bigger
    big --parallel                                    78,712kB                       77,840kB                ^ 1.01x smaller
    facebook --parallel                               38,840kB                       36,432kB                ^ 1.07x smaller
    reddit --parallel                                 30,104kB                       30,552kB                 ! 1.01x bigger
    flickr --parallel                                 33,588kB                       35,976kB                 ! 1.07x bigger
    theverge --parallel                               31,916kB                       29,700kB                ^ 1.07x smaller

    &lt;geometric mean&gt;                                   3,524kB                        3,637kB                 ! 1.03x bigger
    &lt;arithmetic mean&gt;                                 12,778kB                       13,009kB                 ! 1.02x bigger
    &lt;harmonic mean&gt;                                    1,580kB                        1,621kB                 ! 1.03x bigger

=====
~/OpenSource/Source/bmalloc&gt; ~/OpenSource/PerformanceTests/MallocBench/run-malloc-benchmarks --memory_warning Baseline:~/OpenSource/WebKitBuildBaseline/Release/ Patch:~/OpenSource/WebKitBuild/Release/
                                                                                     
                                                                Baseline                               Patch                                   Δ
Peak Memory:
    reddit_memory_warning --runs 0                              14,036kB                            14,020kB                      ^ 1.0x smaller
    flickr_memory_warning --runs 0                              26,732kB                            26,700kB                      ^ 1.0x smaller
    theverge_memory_warning --runs 0                            27,016kB                            26,976kB                      ^ 1.0x smaller

    &lt;geometric mean&gt;                                            21,642kB                            21,615kB                      ^ 1.0x smaller
    &lt;arithmetic mean&gt;                                           22,595kB                            22,565kB                      ^ 1.0x smaller
    &lt;harmonic mean&gt;                                             20,595kB                            20,569kB                      ^ 1.0x smaller

Memory at End:
    reddit_memory_warning --runs 0                               8,980kB                             8,964kB                      ^ 1.0x smaller
    flickr_memory_warning --runs 0                              12,280kB                            12,248kB                      ^ 1.0x smaller
    theverge_memory_warning --runs 0                            21,048kB                            21,008kB                      ^ 1.0x smaller

    &lt;geometric mean&gt;                                            13,240kB                            13,212kB                      ^ 1.0x smaller
    &lt;arithmetic mean&gt;                                           14,103kB                            14,073kB                      ^ 1.0x smaller
    &lt;harmonic mean&gt;                                             12,484kB                            12,458kB                      ^ 1.0x smaller</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1308906</commentid>
    <comment_count>3</comment_count>
      <attachid>310186</attachid>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2017-05-15 17:46:28 -0700</bug_when>
    <thetext>Comment on attachment 310186
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1308924</commentid>
    <comment_count>4</comment_count>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2017-05-15 18:29:41 -0700</bug_when>
    <thetext>Committed r216895: &lt;http://trac.webkit.org/changeset/216895&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>310186</attachid>
            <date>2017-05-15 16:50:17 -0700</date>
            <delta_ts>2017-05-15 17:46:28 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-172143-20170515165017.patch</filename>
            <type>text/plain</type>
            <size>1321</size>
            <attacher name="Geoffrey Garen">ggaren</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9ibWFsbG9jL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvYm1h
bGxvYy9DaGFuZ2VMb2cJKHJldmlzaW9uIDIxNjg4OCkKKysrIFNvdXJjZS9ibWFsbG9jL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE0IEBACisyMDE3LTA1LTE1ICBHZW9mZnJl
eSBHYXJlbiAgPGdnYXJlbkBhcHBsZS5jb20+CisKKyAgICAgICAgYm1hbGxvYzogQnVtcCB0aGUg
c2l6ZSBvZiB0aGUgZGVhbGxvY2F0b3IgbG9nIHRvIDUxMgorICAgICAgICBodHRwczovL2J1Z3Mu
d2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTcyMTQzCisKKyAgICAgICAgUmV2aWV3ZWQgYnkg
Tk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVGhpcyBpcyBhIHNwZWVkdXAgb24gcGFyYWxsZWwg
d29ya2xvYWRzIGZvciBtYWNoaW5lcyB3aXRoIGxvdHMgb2YgQ1BVcy4KKworICAgICAgICAqIGJt
YWxsb2MvU2l6ZXMuaDoKKwogMjAxNy0wNS0xMiAgTWljaGFlbCBTYWJvZmYgIDxtc2Fib2ZmQGFw
cGxlLmNvbT4KIAogICAgICAgICBbaU9TXSBVc2UgbWVtb3J5IGZvb3RwcmludCB0byBkeW5hbWlj
YWxseSBhZGp1c3QgYmVoYXZpb3Igb2YgYWxsb2NhdG9ycwpJbmRleDogU291cmNlL2JtYWxsb2Mv
Ym1hbGxvYy9TaXplcy5oCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9ibWFsbG9jL2JtYWxsb2MvU2l6
ZXMuaAkocmV2aXNpb24gMjE2ODI5KQorKysgU291cmNlL2JtYWxsb2MvYm1hbGxvYy9TaXplcy5o
CSh3b3JraW5nIGNvcHkpCkBAIC02NSw3ICs2NSw3IEBAIG5hbWVzcGFjZSBTaXplcyB7CiAgICAg
c3RhdGljIGNvbnN0IHNpemVfdCBsYXJnZUFsaWdubWVudCA9IHNtYWxsTWF4IC8gcGFnZVNpemVX
YXN0ZUZhY3RvcjsKICAgICBzdGF0aWMgY29uc3Qgc2l6ZV90IGxhcmdlQWxpZ25tZW50TWFzayA9
IGxhcmdlQWxpZ25tZW50IC0gMTsKIAotICAgIHN0YXRpYyBjb25zdCBzaXplX3QgZGVhbGxvY2F0
b3JMb2dDYXBhY2l0eSA9IDI1NjsKKyAgICBzdGF0aWMgY29uc3Qgc2l6ZV90IGRlYWxsb2NhdG9y
TG9nQ2FwYWNpdHkgPSA1MTI7CiAgICAgc3RhdGljIGNvbnN0IHNpemVfdCBidW1wUmFuZ2VDYWNo
ZUNhcGFjaXR5ID0gMzsKICAgICAKICAgICBzdGF0aWMgY29uc3Qgc3RkOjpjaHJvbm86Om1pbGxp
c2Vjb25kcyBtYXhTY2F2ZW5nZVNsZWVwRHVyYXRpb24gPSBzdGQ6OmNocm9ubzo6bWlsbGlzZWNv
bmRzKDI1MCk7Cg==
</data>
<flag name="review"
          id="331284"
          type_id="1"
          status="+"
          setter="msaboff"
    />
          </attachment>
      

    </bug>

</bugzilla>