<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Dec 2, 2013 at 4:30 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><br>
On 02/12/2013 11:05, Kostya Serebryany wrote:<div class="im"><br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
<br>
<br>
On Mon, Dec 2, 2013 at 2:55 PM, İsmail Dönmez <<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 quite a while<br>
and you are the first to complain<br>
</blockquote>
<br></div>
I imagine there aren't more complaints because people are just disabling asan on their builders to avoid the cost.<br>
<br>
Regardless of whether it's a low-end or high-end system with hundreds of gigabytes RAM, the 4.1 Gb a single test allocates could probably be put to better uses such as additional builders.<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
, so maybe running with 6Gb is the right approach.<br>
</blockquote>
<br></div>
How about gating this behind ASAN_AVOID_EXPENSIVE_TESTS?<br></blockquote><div><br></div><div>Like this? </div><div><div><div>--- asan/tests/asan_test.cc     (revision 196066)</div><div>+++ asan/tests/asan_test.cc     (working copy)</div>
<div>@@ -299,7 +299,7 @@</div><div> }</div><div> </div><div> TEST(AddressSanitizer, HugeMallocTest) {</div><div>-  if (SANITIZER_WORDSIZE != 64) return;</div><div>+  if (SANITIZER_WORDSIZE != 64 || ASAN_AVOID_EXPENSIVE_TESTS) return;</div>
<div>   size_t n_megs = 4100;</div><div>   TestLargeMalloc(n_megs << 20);</div><div> }</div></div></div><div><br></div><div> </div></div></div></div>