dlmalloc locks seems to be not enabled in pthread builds about wasi-libc HOT 7 OPEN

I thought that _REENTRANT got set and then USE_LOCKS will be defined?

from wasi-libc.

Ah sorry, I missed the dlmalloc.c file! Yes, USE_LOCKS does get defined. Though it seems that currently spin locks is used in dlmalloc, which looks not ideal compared to futex based atomics.wait instruction?

from wasi-libc.

Yes we probably want to make sure USE_SPIN_LOCKS is not defined, so it will fallback to pthread locks.

from wasi-libc.

Here is where emscripten does that: https://github.com/emscripten-core/emscripten/blob/e64e3701c9c6d9fc2622fb34b92065971c326ed1/system/lib/dlmalloc.c#L29-L32

from wasi-libc.

I think there's an even bigger problem there; IIUIC pthread_mutex_lock in wasi-libc currently uses spinlock. We do have a futex-based lock at libc-top-half/musl/src/thread/__lock.c which gets used in some places, but not pthread lock itself. And also that futex lock doesn't take a timeout argument.

from wasi-libc.

As far as I can tell pthread_mutex_lock uses __wasilibc_futex_wait just like __wait.

Its just a rather indirect path:

pthread_mutex_lock -> __pthread_mutex_timedlock
__pthread_mutex_timedlock -> __timedwait
__timedwait -> __timedwait_cp
__timedwait_cp -> __futex4_cp
__futex4_cp -> __wasilibc_futex_wait

Is your analysis different to mine?

from wasi-libc.

Ah, that makes sense to me now! Thanks a lot for the clarification.

from wasi-libc.