<div dir="ltr"><div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 25 Aug 2020, 17:28 Pablo Polvorin, <<a href="mailto:pablo.polvorin@nextroll.com" target="_blank">pablo.polvorin@nextroll.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space"><br><div><br><blockquote type="cite"><div>On 25 Aug 2020, at 09:05, Lukas Larsson <<a href="mailto:lukas@erlang.org" rel="noreferrer" target="_blank">lukas@erlang.org</a>> wrote:</div><br><div><div dir="ltr" style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><div>Hello,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 25, 2020 at 9:26 AM Pablo Polvorin <<a href="mailto:pablo.polvorin@nextroll.com" rel="noreferrer" target="_blank">pablo.polvorin@nextroll.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Hi,<br>we recently migrated our systems from OTP21 to OTP22.   On one of them, performance noticeable degraded when switched to OTP22 (20% drop).<br>After some investigation, we found the apparent cause was the madvice() call when returning the carriers to the carrier pool, that lead to lots of<br>minor page faults, ultimately killing our performance.  We run in a virtualised cloud, not sure how much that affects the minor page fault overhead.<br></blockquote><div><br></div><div>Do you know if you have access to MADV_FREE or use MADV_DONTNEED?</div><div><br></div></div></div></div></blockquote><div>Looks like we don’t :/ </div><div><div><br></div><div>#include <sys/mman.h></div><div>#include <stdio.h></div><div>int main() {</div><div>   #ifdef MADV_FREE</div><div><span style="white-space:pre-wrap">      </span>   printf("Have it\n");</div><div>   #else</div><div><span style="white-space:pre-wrap">      </span>   printf("Dont have it\n");</div><div>   #endif</div><div>   return 0;</div><div>}</div></div><div>> <span style="font-family:Menlo">Dont have it</span></div><div><br></div><div>This is on amazon linux,   4.14.186-110.268.amzn1.x86_64 .</div><div></div></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">That may explain it. Maybe we should not use madvise when FREE is not available.</div><div dir="auto"><br></div><div>Have you tried do delete this line: <a href="https://github.com/erlang/otp/blob/7ad81c674d1aa705ae41743b343043d05ea1944b/erts/emulator/sys/common/erl_mmap.h#L215">https://github.com/erlang/otp/blob/7ad81c674d1aa705ae41743b343043d05ea1944b/erts/emulator/sys/common/erl_mmap.h#L215</a> and see what happens then?</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space"><div><div><br></div><br><blockquote type="cite"><div><div dir="ltr" style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><br>So we spend some time tweaking the allocator' settings (that haven't been touched in many years, with the system itself evolved a lot since that time) and got<br>to a good improvement. But for carrier migration ultimately the thing that worked best for us was just disable it entirely.  Is this a terrible idea?<br>our load is a fairly stable flow of homogenous requests, that lead to several short-lived (milliseconds) processes being spawned. Have plenty of memory, so  I'm not too worried about a badly utilised carrier being stuck within a scheduler.<br><br>Got a few questions regarding this:<br><br>* Wonder, it's something common out there to disable carrier migration?  Feels a bit strange that nobody hit the same problem when updating to OTP22, I’m<br>assuming there are lots of not-so-great allocator settings out there, like was our case.  (disabling it was our last try actually, we tried the settings suggested by<br>erts_alloc_config, and then make the +M<S>acul  and +M<S>acfml settings significantly stricter as well, and while that helps, still had too many page faults).<br><br></blockquote><div><br></div><div>Carrier migration can help a lot to deal with memory fragmentation issues. It is however not free as you have noticed. I know that other people have disabled it with some success, but as far as I remember that was to work around bugs in the migration logic, not because of the performance overhead.</div></div></div></div></blockquote><div>Given the frequency at which we where abandoning and taking carriers from the pool,  smells something fishy on our settings and allocation pattern.  But so far couldn’t really figure out how to bring that down to a low enough level that the madvice() won’t affect us much.</div></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Yes, that does seem odd. Carriers are not meant to be pushed in and put of the pool at a rapid pace.</div><div dir="auto"><br></div><div>I don't suppose you have a relatively small example that will reproduce the behaviour? Or if not, then maybe a couple of recon_alloc snapshots?</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space"><div><br><blockquote type="cite"><div><div dir="ltr" style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">* What's the tradeoff on having large multiblock carriers,  other than the memory overhead when they aren't fully used?.<br><br>* Do it make sense to make a config flag to allow carrier migration but disable the madvice() on free blocks?<br></blockquote><div><br></div><div>Given your experiences with it, I think that would make sense. We did not notice any degradation when testing madvise ourselves, but it is not possible to test all scenarios in all environments.</div><div><br></div></div></div></div></blockquote><div>Might work on this,  although I guess it would require to re-learn autoconf sorcery so probably will not happen soon </div></div></div></blockquote><div><br></div><div>No need to do any autoconf sorcery, I was thinking that this could probably be a start flag passed to erl?</div></div></div></div>
</div>