<div dir="ltr">done, r196097.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Dec 2, 2013 at 5:13 PM, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
On 02/12/2013 13:08, Kostya Serebryany wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
<br>
<br>
<br>
On Mon, Dec 2, 2013 at 4:30 PM, Alp Toker <<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a> <mailto:<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>>> wrote:<br>
<br>
<br>
    On 02/12/2013 11:05, Kostya Serebryany wrote:<br>
<br>
<br>
<br>
<br>
        On Mon, Dec 2, 2013 at 2:55 PM, İsmail Dönmez<br>
        <<a href="mailto:ismail@donmez.ws" target="_blank">ismail@donmez.ws</a> <mailto:<a href="mailto:ismail@donmez.ws" target="_blank">ismail@donmez.ws</a>><br></div><div class="im">
        <mailto:<a href="mailto:ismail@donmez.ws" target="_blank">ismail@donmez.ws</a> <mailto:<a href="mailto:ismail@donmez.ws" target="_blank">ismail@donmez.ws</a>>>> wrote:<br>
<br>
            This is under kvm with 3GB RAM (-m 3000)<br>
<br>
        This test tests the ability to allocate 4.1Gb RAM.<br>
        It will have to be disabled on small RAM machines somehow.<br>
        Or you can run it with 6Gb RAM. This test has been there for<br>
        quite a while<br>
        and you are the first to complain<br>
<br>
<br>
    I imagine there aren't more complaints because people are just<br>
    disabling asan on their builders to avoid the cost.<br>
<br>
    Regardless of whether it's a low-end or high-end system with<br>
    hundreds of gigabytes RAM, the 4.1 Gb a single test allocates<br>
    could probably be put to better uses such as additional builders.<br>
<br>
<br>
        , so maybe running with 6Gb is the right approach.<br>
<br>
<br>
    How about gating this behind ASAN_AVOID_EXPENSIVE_TESTS?<br>
<br>
<br>
Like this?<br>
--- asan/tests/asan_test.cc     (revision 196066)<br>
+++ asan/tests/asan_test.cc     (working copy)<br>
@@ -299,7 +299,7 @@<br>
 }<br>
 TEST(AddressSanitizer, HugeMallocTest) {<br>
-  if (SANITIZER_WORDSIZE != 64) return;<br>
+  if (SANITIZER_WORDSIZE != 64 || ASAN_AVOID_EXPENSIVE_TESTS) return;<br>
   size_t n_megs = 4100;<br>
   TestLargeMalloc(n_megs << 20);<br>
 }<br>
</div></blockquote>
<br>
Brilliant, that should do the trick.<br>
<br>
Thanks kcc<span class="HOEnZb"><font color="#888888"><br>
<br>
Alp.</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
<br>
-- <br>
<a href="http://www.nuanti.com" target="_blank">http://www.nuanti.com</a><br>
the browser experts<br>
<br>
</div></div></blockquote></div><br></div>