dlmalloc has license incompatible with Fedora policies about wasi-libc HOT 12 OPEN

mimalloc also claims to work with wasm32. I suppose it's possible to vendor mimalloc in this tree and have some degree of testing, and there's preliminary work on allowing building without dlmalloc.

from wasi-libc.

@bjorn3 Unfortunately, IANAL also :) The patents may very well be expired and even dlmalloc be fine. However, from the Fedora PoV, this is still inclusion of new (IOW not previously included in Fedora) code with a problematic license and would need a review and exception from people that actually are lawyers.

Given the age of the code, I would probably get the exception – note however that I would not know about the expiration possibility if I did not open this issue :) Now, with the emmalloc porter, this discussion is probably just academic – I can (and will) use the malloc with non-problematic license.

With that, I'm now considering my issue to be addressed, so feel free to mark it resolved. Thanks for taking the time to do the port!

from wasi-libc.

IANAL, but dlmalloc is more than 20 years old (earliest version I could find is v2.5 from 1993, or in other words 29 year old).

There have been algorithm changes in dlmalloc much later than that.

The issue is a formal matter, we not actually expect any patent enforcement action from this particular upstream. But we don't really to divide upstreams into those who can use slightly out-of-policy licenses, and those who cannot.

from wasi-libc.

The mmap emulation code works by calling malloc, so it unfortunately won't work for implementing a malloc.

As @TerrorJack mentions, there is a build parameter for changing the malloc implementation, though the current options are dlmalloc or none.

mimalloc is a possibility. We could certainly add that to the wasi-libc tree as an option, if someone wanted to port it. It'd also be something we could consider making the default allocator, though we'd want code size and performance comparisions. Also, I am concerned about the fact that mimalloc uses spin-wait loops for some things, though I haven't studied it in depth.

Another option would be to ask Fedora for an exception for dlmalloc. It's used in a lot of places.

from wasi-libc.

Another option is emmalloc. Like dlmalloc it uses sbrk, and it's used in wasm today in emscripten as a drop-in dlmalloc replacement. It should just work in wasi-libc (as long as you don't #define the emscripten verbose logging code). License is MIT (compatible with wasi-libc).

emmalloc is, like wee_alloc, more focused on size than speed, so it's not as fast as dlmalloc on heavy malloc benchmarks. But in general usage it works very well in our experience, and it is significantly smaller than dlmalloc, about 1/3 the size (which is often significant in small programs).

As an additional benefit this would be a nice step towards convergence of wasi-libc and emscripten's libc code.

from wasi-libc.

Older versions of dlmalloc also come with an actual PD dedication.

from wasi-libc.

@sunfishcode

Another option would be to ask Fedora for an exception for dlmalloc. It's used in a lot of places.

I can certainly do that, but that assumes I asked the project their opinions about changing the implementation and got rejected, told it's a WIP etc. It was one of the reasons for opening this issue ;) However, seeing now that there are options, I would like to explore them more.

@kripken

Another option is emmalloc. Like dlmalloc it uses sbrk, and it's used in wasm today in emscripten as a drop-in dlmalloc replacement.

As an additional benefit this would be a nice step towards convergence of wasi-libc and emscripten's libc code.

Interesting, and I'll try to import this to my fork and play around with it. No experience with benchmarking mallocs etc., so an actual PR may be a bit off, but let's try it.

@fweimer-rh

Older versions of dlmalloc also come with an actual PD dedication.

Also an option, but I would rather not downgrade… I tried to reach Doug Lea with the problem description in order to see if it can be re-dedicated, but got no answer so far.

from wasi-libc.

emmalloc is, like wee_alloc, more focused on size than speed, so it's not as fast as dlmalloc on heavy malloc benchmarks. But in general usage it works very well in our experience, and it is significantly smaller than dlmalloc, about 1/3 the size (which is often significant in small programs).

wee_alloc has memory fragmentation issues and has a relatively high lower bound on memory usage as I understand it. See for example rustwasm/wee_alloc#85.

Edit: you were suggesting emmalloc, not wee_alloc. My bad.

from wasi-libc.

@bjorn3 That looks like a specific bug in wee_alloc, yeah.

We're not aware of any significant bugs on emmalloc atm. It is used in production in both large and small applications, using large and small amounts of memory.

But it is true that emmalloc (and other small allocators like wee_alloc) is definitely not as fast as dlmalloc on malloc-intensive benchmarks - dlmalloc has a lot of nice optimizations particularly for allocating huge numbers of small objects of odd sizes. Those situations are somewhat rare, though, in real-world code, in my experience, likely since developers generally try to avoid tons of malloc calls.

from wasi-libc.

As an update here, emmalloc is now in wasi-libc (#340). It currently requires a wasi-libc build-time option, MALLOC_IMPL=emmalloc.

from wasi-libc.

We switched to emmalloc as the default allocator in zig cc. The dlmalloc code is not shipped any more.

from wasi-libc.

@khardix IANAL, but dlmalloc is more than 20 years old (earliest version I could find is v2.5 from 1993, or in other words 29 year old). This means that any patents that could apply to dlmalloc have since expired and any patents filled after that could be invalidated using dlmalloc as prior art, right? Or am I misunderstanding how patent law works? Now there have been releases that are less than 20 years old, but using any 20+ year old release should be fine with respect to patents, right?

from wasi-libc.