<?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>147777</bug_id>
          
          <creation_ts>2015-08-07 02:59:42 -0700</creation_ts>
          <short_desc>[GTK] WTF unit tests are timing out in the bots</short_desc>
          <delta_ts>2015-08-12 08:20:52 -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>Tools / Tests</component>
          <version>528+ (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>Gtk</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Carlos Garcia Campos">cgarcia</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>fpizlo</cc>
    
    <cc>ggaren</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1115706</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-08-07 02:59:42 -0700</bug_when>
    <thetext>It&apos;s because WTF_Lock tests, they actually pass, but they are slower than our timeout.

$ time bin/TestWebKitAPI/WTF/TestWTF --gtest_filter=WTF_Lock.*
**PASS** WTF_Lock.UncontentedShortSection
**PASS** WTF_Lock.UncontentedLongSection
**PASS** WTF_Lock.ContentedShortSection
**PASS** WTF_Lock.ContentedLongSection
**PASS** WTF_Lock.ManyContentedShortSections
**PASS** WTF_Lock.ManyContentedLongSections

real	0m15.656s
user	0m51.736s
sys	0m36.848s

We need a way to mark tests as slow and use a longer timeout like WTR does. I don&apos;t know if these tests are supposed to be so slow, though. Filip?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1115713</commentid>
    <comment_count>1</comment_count>
      <attachid>258479</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-08-07 03:29:32 -0700</bug_when>
    <thetext>Created attachment 258479
Patch

Marking WTF_Lock.ContentedShortSection as slow seems to be enough</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1115749</commentid>
    <comment_count>2</comment_count>
      <attachid>258479</attachid>
    <who name="Filip Pizlo">fpizlo</who>
    <bug_when>2015-08-07 07:44:29 -0700</bug_when>
    <thetext>Comment on attachment 258479
Patch

Alternatively you could just make the tests run for less time. In each test, the number of iterations is the last argument to the runTest template.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1115763</commentid>
    <comment_count>3</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-08-07 08:28:22 -0700</bug_when>
    <thetext>Committed r188125: &lt;http://trac.webkit.org/changeset/188125&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1116183</commentid>
    <comment_count>4</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-08-10 01:03:13 -0700</bug_when>
    <thetext>Unfortunately this was not enough for the bots, that seem to be slower than my laptop:

$ time (../../Tools/jhbuild/jhbuild-wrapper --gtk run xvfb-run bin/TestWebKitAPI/WTF/TestWTF --gtest_filter=WTF_Lock.*)
**PASS** WTF_Lock.UncontentedShortSection
**PASS** WTF_Lock.UncontentedLongSection
**PASS** WTF_Lock.ContentedShortSection
**PASS** WTF_Lock.ContentedLongSection
**PASS** WTF_Lock.ManyContentedShortSections
**PASS** WTF_Lock.ManyContentedLongSections

real	0m43.051s
user	3m32.349s
sys	1m44.799s</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1116952</commentid>
    <comment_count>5</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-08-12 08:20:52 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; Unfortunately this was not enough for the bots, that seem to be slower than
&gt; my laptop:
&gt; 
&gt; $ time (../../Tools/jhbuild/jhbuild-wrapper --gtk run xvfb-run
&gt; bin/TestWebKitAPI/WTF/TestWTF --gtest_filter=WTF_Lock.*)
&gt; **PASS** WTF_Lock.UncontentedShortSection
&gt; **PASS** WTF_Lock.UncontentedLongSection
&gt; **PASS** WTF_Lock.ContentedShortSection
&gt; **PASS** WTF_Lock.ContentedLongSection
&gt; **PASS** WTF_Lock.ManyContentedShortSections
&gt; **PASS** WTF_Lock.ManyContentedLongSections
&gt; 
&gt; real	0m43.051s
&gt; user	3m32.349s
&gt; sys	1m44.799s

Increased the timeout for slow tests to timeout * 5 and it stills times out . . . Maybe we should just try to make the test faster.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>258479</attachid>
            <date>2015-08-07 03:29:32 -0700</date>
            <delta_ts>2015-08-07 07:44:29 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>rgt-slow-tests.diff</filename>
            <type>text/plain</type>
            <size>1838</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCA5NmVj
NjNlLi5jZjE0ODY3IDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDUgKzEsMjAgQEAKIDIwMTUtMDgtMDcgIENhcmxvcyBHYXJjaWEgQ2FtcG9z
ICA8Y2dhcmNpYUBpZ2FsaWEuY29tPgogCisgICAgICAgIFtHVEtdIFdURiB1bml0IHRlc3RzIGFy
ZSB0aW1pbmcgb3V0IGluIHRoZSBib3RzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn
L3Nob3dfYnVnLmNnaT9pZD0xNDc3NzcKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9P
UFMhKS4KKworICAgICAgICBBZGQgYSB3YXkgdG8gbWFyayBnb29nbGUgdW5pdCB0ZXN0cyBhcyBz
bG93IGFuZCBhZGQKKyAgICAgICAgV1RGX0xvY2suQ29udGVudGVkU2hvcnRTZWN0aW9uIHRvIHRo
ZSBsaXN0LiBJbiBjYXNlIG9mIHNsb3cgdGVzdAorICAgICAgICB3ZSB1c2UgdGhlIGRvdWJsZSBv
ZiB0aGUgZ2l2ZW4gdGltZW91dCBmb3IgdGhhdCBwYXJ0aWN1bGFyIHRlc3QuCisKKyAgICAgICAg
KiBTY3JpcHRzL3J1bi1ndGstdGVzdHM6CisgICAgICAgIChUZXN0UnVubmVyKToKKyAgICAgICAg
KFRlc3RSdW5uZXIuX3J1bl9nb29nbGVfdGVzdCk6CisKKzIwMTUtMDgtMDcgIENhcmxvcyBHYXJj
aWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29tPgorCiAgICAgICAgIFtHVEtdIHJ1bi1ndGst
dGVzdHMgc2hvdWxkIG5vdCBzdG9wIHdoZW4gYSBnb29nbGUgdGVzdCBjYXNlIGZhaWxzCiAgICAg
ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNDc3NzgKIApkaWZm
IC0tZ2l0IGEvVG9vbHMvU2NyaXB0cy9ydW4tZ3RrLXRlc3RzIGIvVG9vbHMvU2NyaXB0cy9ydW4t
Z3RrLXRlc3RzCmluZGV4IDcyMDVlNDQuLjIyZDlhZDIgMTAwNzU1Ci0tLSBhL1Rvb2xzL1Njcmlw
dHMvcnVuLWd0ay10ZXN0cworKysgYi9Ub29scy9TY3JpcHRzL3J1bi1ndGstdGVzdHMKQEAgLTgw
LDYgKzgwLDEwIEBAIGNsYXNzIFRlc3RSdW5uZXI6CiAgICAgICAgIFNraXBwZWRUZXN0KCJXZWJL
aXQyL1Rlc3RXZWJLaXQyIiwgIldlYktpdDIuR2VvbG9jYXRpb25UcmFuc2l0aW9uVG9Mb3dBY2N1
cmFjeSIsICJUZXN0IGNhdXNlcyBjcmFzaCBvbiB0aGUgbmV4dCB0ZXN0IiwgMTI1MDY4KSwKICAg
ICBdCiAKKyAgICBTTE9XID0gWworICAgICAgICAiV1RGX0xvY2suQ29udGVudGVkU2hvcnRTZWN0
aW9uIiwKKyAgICBdCisKICAgICBkZWYgX19pbml0X18oc2VsZiwgb3B0aW9ucywgdGVzdHM9W10p
OgogICAgICAgICBzZWxmLl9vcHRpb25zID0gb3B0aW9ucwogCkBAIC0zNDAsNyArMzQ0LDEwIEBA
IGNsYXNzIFRlc3RSdW5uZXI6CiAKICAgICBkZWYgX3J1bl9nb29nbGVfdGVzdChzZWxmLCB0ZXN0
X3Byb2dyYW0sIHN1YnRlc3QpOgogICAgICAgICB0ZXN0X2NvbW1hbmQgPSBbdGVzdF9wcm9ncmFt
LCAnLS1ndGVzdF9maWx0ZXI9JXMnICUgKHN1YnRlc3QpXQotICAgICAgICByZXR1cm4gc2VsZi5f
cnVuX3Rlc3RfY29tbWFuZCh0ZXN0X2NvbW1hbmQsIHNlbGYuX29wdGlvbnMudGltZW91dCkKKyAg
ICAgICAgdGltZW91dCA9IHNlbGYuX29wdGlvbnMudGltZW91dAorICAgICAgICBpZiBzdWJ0ZXN0
IGluIFRlc3RSdW5uZXIuU0xPVzoKKyAgICAgICAgICAgIHRpbWVvdXQgKj0gMgorICAgICAgICBy
ZXR1cm4gc2VsZi5fcnVuX3Rlc3RfY29tbWFuZCh0ZXN0X2NvbW1hbmQsIHRpbWVvdXQpCiAKICAg
ICBkZWYgX3J1bl9nb29nbGVfdGVzdF9zdWl0ZShzZWxmLCB0ZXN0X3Byb2dyYW0pOgogICAgICAg
ICByZXRjb2RlID0gMAo=
</data>
<flag name="review"
          id="283655"
          type_id="1"
          status="+"
          setter="fpizlo"
    />
          </attachment>
      

    </bug>

</bugzilla>