__init_tp: Initialize TID to non-zero value

[yamt]

__init_tp: Initialize TID to non-zero value

This fixes TID-based locking used within libc.

Also, initialize detach_state.

cf. WebAssembly/wasi-threads#16

[loganek]

Initialize the TID to a large value, which is hopefully
less likely to conflict with host-allocated TIDs

So, that assumption is correct only if the host assigns the ID incrementally, which might not be the case (host can decide the implementation).

[yamt]

Initialize the TID to a large value, which is hopefully
less likely to conflict with host-allocated TIDs

So, that assumption is correct only if the host assigns the ID incrementally, which might not be the case (host can decide the implementation).

right. this is a kludge at best.

i think a real fix involves spec changes. see WebAssembly/wasi-threads#15

[loganek]

I'd suggest let's wait with this PR then until the WebAssembly/wasi-threads#15 is resolved

[loganek]

I think we can update the PR based on the discussion in WebAssembly/wasi-threads#15 @yamt ?

[yamt]

I think we can update the PR based on the discussion in WebAssembly/wasi-threads#15 @yamt ?

done

[yamt]

i updated the comment after WebAssembly/wasi-threads#16

[abrown]

I think this is fine; I would want to link to the wasi-threads heading where this is discussed, but that's personal preference.