<div dir="ltr"><div><div dir="auto">Thanks, it sounds reasonable :)<br></div><br><div class="gmail_quote"><div>On Tue, May 1, 2018 at 1:47 Geoffrey Garen <<a href="mailto:ggaren@apple.com" target="_blank">ggaren@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">If we have just a few allocations, we should use the mmap based allocator. This preserves the invariant that bmalloc can be used as a general-purpose malloc implementation.<br>
<br>
If we have lots of small allocations, we should probably reconsider the design.<br>
<br>
I’m not familiar with the new uses of std::vector inside bmalloc. That’s not something I would recommend.<br>
<br>
Geoff<br>
<br>
> On Apr 30, 2018, at 3:35 AM, Yusuke SUZUKI <<a href="mailto:utatane.tea@gmail.com" target="_blank">utatane.tea@gmail.com</a>> wrote:<br>
> <br>
> Hi, WebKittens,<br>
> <br>
> IIRC, bmalloc uses mmap based page allocator for internal memory use. For example, bmalloc::Vector uses it instead of calling malloc.<br>
> But recent changes start using std::vector, which means it uses std malloc under the hood.<br>
> <br>
> So my question is, if we want some internal memory allocation in bmalloc, shoud we use std::malloc? Or should we use mmap based allocator?<br>
> <br>
> Best regards,<br>
> Yusuke Suzuki<br>
> -- <br>
> Regards,<br>
> Yusuke Suzuki<br>
> _______________________________________________<br>
> webkit-dev mailing list<br>
> <a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
> <a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" rel="noreferrer" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
<br>
</blockquote></div></div></div>