<?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>197152</bug_id>
          
          <creation_ts>2019-04-22 00:05:25 -0700</creation_ts>
          <short_desc>[JSC] Use volatile load to populate backing page in MarkedBlock::Footer instead of using holdLock</short_desc>
          <delta_ts>2019-04-22 11:36:27 -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>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Yusuke Suzuki">ysuzuki</reporter>
          <assigned_to name="Yusuke Suzuki">ysuzuki</assigned_to>
          <cc>ews-watchlist</cc>
    
    <cc>keith_miller</cc>
    
    <cc>mark.lam</cc>
    
    <cc>msaboff</cc>
    
    <cc>saam</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1529191</commentid>
    <comment_count>0</comment_count>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2019-04-22 00:05:25 -0700</bug_when>
    <thetext>[JSC] Use volatile load to populate backing page in MarkedBlock::Footer instead of using holdLock</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1529192</commentid>
    <comment_count>1</comment_count>
      <attachid>367925</attachid>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2019-04-22 00:05:55 -0700</bug_when>
    <thetext>Created attachment 367925
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1529323</commentid>
    <comment_count>2</comment_count>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2019-04-22 11:35:13 -0700</bug_when>
    <thetext>Committed r244506: &lt;https://trac.webkit.org/changeset/244506&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1529325</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2019-04-22 11:36:27 -0700</bug_when>
    <thetext>&lt;rdar://problem/50101020&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>367925</attachid>
            <date>2019-04-22 00:05:55 -0700</date>
            <delta_ts>2019-04-22 11:28:13 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-197152-20190422000554.patch</filename>
            <type>text/plain</type>
            <size>2522</size>
            <attacher name="Yusuke Suzuki">ysuzuki</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjQ0NDk3CmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlw
dENvcmUvQ2hhbmdlTG9nIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwppbmRleCBl
MTFhNzcwZGJlYzZlYjI0NTkyOTEzYzIyMzUwNTQ1NjY2NjlkNmY2Li44YmZmZDc2ZWM3OWRmMTIw
Yzk1OTIzZTMyZjBhM2YxNDM0MmQ2NmRlIDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENv
cmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKQEAgLTEs
MyArMSwxNyBAQAorMjAxOS0wNC0yMiAgWXVzdWtlIFN1enVraSAgPHlzdXp1a2lAYXBwbGUuY29t
PgorCisgICAgICAgIFtKU0NdIFVzZSB2b2xhdGlsZSBsb2FkIHRvIHBvcHVsYXRlIGJhY2tpbmcg
cGFnZSBpbiBNYXJrZWRCbG9jazo6Rm9vdGVyIGluc3RlYWQgb2YgdXNpbmcgaG9sZExvY2sKKyAg
ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE5NzE1MgorCisg
ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEVtaXQgdm9sYXRp
bGUgbG9hZCBpbnN0ZWFkIG9mIHVzaW5nIGhvbGRMb2NrIHRvIHBvcHVsYXRlIGJhY2tpbmcgcGFn
ZSBpbiBNYXJrZWRCbG9jazo6Rm9vdGVyLgorCisgICAgICAgICogaGVhcC9CbG9ja0RpcmVjdG9y
eS5jcHA6CisgICAgICAgIChKU0M6OkJsb2NrRGlyZWN0b3J5Ojppc1BhZ2VkT3V0KToKKyAgICAg
ICAgKiBoZWFwL01hcmtlZEJsb2NrLmg6CisgICAgICAgIChKU0M6Ok1hcmtlZEJsb2NrOjpwb3B1
bGF0ZVBhZ2UgY29uc3QpOgorCiAyMDE5LTA0LTIwICBZdXN1a2UgU3V6dWtpICA8eXN1enVraUBh
cHBsZS5jb20+CiAKICAgICAgICAgVW5yZXZpZXdlZCwgc3VwcHJlc3Mgd2FybmluZ3MgaW4gbm9u
IERhcndpbiBlbnZpcm9ubWVudHMKZGlmZiAtLWdpdCBhL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9o
ZWFwL0Jsb2NrRGlyZWN0b3J5LmNwcCBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9oZWFwL0Jsb2Nr
RGlyZWN0b3J5LmNwcAppbmRleCBmMDc2OTc5N2QxNzI3ZTcwODQ3MmJiOTJmMzc5ZjEzOWU0NTVi
MTZlLi43ZWVlNGE4NzQ2YjZiZDE4ODUyYTdhYjYyODc0YmJhZjU3YWJlMjg5IDEwMDY0NAotLS0g
YS9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvaGVhcC9CbG9ja0RpcmVjdG9yeS5jcHAKKysrIGIvU291
cmNlL0phdmFTY3JpcHRDb3JlL2hlYXAvQmxvY2tEaXJlY3RvcnkuY3BwCkBAIC02MSwxMSArNjEs
OCBAQCBib29sIEJsb2NrRGlyZWN0b3J5Ojppc1BhZ2VkT3V0KE1vbm90b25pY1RpbWUgZGVhZGxp
bmUpCiB7CiAgICAgdW5zaWduZWQgaXRlcnNTaW5jZUxhc3RUaW1lQ2hlY2sgPSAwOwogICAgIGZv
ciAoYXV0byogYmxvY2sgOiBtX2Jsb2NrcykgewotICAgICAgICBpZiAoYmxvY2spIHsKLSAgICAg
ICAgICAgIC8vIFdlIHRha2UgYW5kIGRyb3AgdGhlIGxvY2sgYXMgYSB3YXkgb2YgdG91Y2hpbmcg
dGhhdCBwYWdlIG9mIG1lbW9yeS4KLSAgICAgICAgICAgIC8vIEZJWE1FOiBzaG91bGQgd2UgaW5z
dGVhZCBkbyBhIGNoZWFwZXIgdGhpbmcgbGlrZSBhIHZvbGF0aWxlIGxvYWQgaW4gdGhlIHBhZ2U/
Ci0gICAgICAgICAgICAodm9pZCkgaG9sZExvY2soYmxvY2stPmJsb2NrKCkubG9jaygpKTsKLSAg
ICAgICAgfQorICAgICAgICBpZiAoYmxvY2spCisgICAgICAgICAgICBibG9jay0+YmxvY2soKS5w
b3B1bGF0ZVBhZ2UoKTsKICAgICAgICAgKytpdGVyc1NpbmNlTGFzdFRpbWVDaGVjazsKICAgICAg
ICAgaWYgKGl0ZXJzU2luY2VMYXN0VGltZUNoZWNrID49IEhlYXA6OnNfdGltZUNoZWNrUmVzb2x1
dGlvbikgewogICAgICAgICAgICAgTW9ub3RvbmljVGltZSBjdXJyZW50VGltZSA9IE1vbm90b25p
Y1RpbWU6Om5vdygpOwpkaWZmIC0tZ2l0IGEvU291cmNlL0phdmFTY3JpcHRDb3JlL2hlYXAvTWFy
a2VkQmxvY2suaCBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9oZWFwL01hcmtlZEJsb2NrLmgKaW5k
ZXggYTM1NjYwYmNiZWY0MTFhMzZhOWM3ZjZkOGQxODViZTU3YzA0OTg3ZS4uNDM5MjkzMmFlYzcw
ZmNmMjIwODQ4YWZlNjU3ZTgwZjU0YjU3OTlhMSAxMDA2NDQKLS0tIGEvU291cmNlL0phdmFTY3Jp
cHRDb3JlL2hlYXAvTWFya2VkQmxvY2suaAorKysgYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvaGVh
cC9NYXJrZWRCbG9jay5oCkBAIC0zNzQsNiArMzc0LDExIEBAIGNsYXNzIE1hcmtlZEJsb2NrIHsK
ICAgICBDb3VudGluZ0xvY2smIGxvY2soKSB7IHJldHVybiBmb290ZXIoKS5tX2xvY2s7IH0KICAg
ICAKICAgICBTdWJzcGFjZSogc3Vic3BhY2UoKSBjb25zdCB7IHJldHVybiBmb290ZXIoKS5tX3N1
YnNwYWNlOyB9CisKKyAgICB2b2lkIHBvcHVsYXRlUGFnZSgpIGNvbnN0CisgICAgeworICAgICAg
ICAqYml0d2lzZV9jYXN0PHZvbGF0aWxlIHVpbnQ4X3QqPigmZm9vdGVyKCkpOworICAgIH0KICAg
ICAKICAgICBzdGF0aWMgY29uc3RleHByIHNpemVfdCBvZmZzZXRPZkZvb3RlciA9IGVuZEF0b20g
KiBhdG9tU2l6ZTsKIAo=
</data>
<flag name="review"
          id="384250"
          type_id="1"
          status="+"
          setter="saam"
    />
          </attachment>
      

    </bug>

</bugzilla>