On Sat, Oct 27, 2018 at 09:40:13AM +0200, Duy Nguyen wrote:
> > We expect to be able to store a void pointer here and get it back, which
> > should work even for a single thread. Do we need something like:
> >
> > extern void *pthread_specific_data;
> >
> > #define pthread_setspecific(key, data) do { \
> > pthread_specific_data = data; \
> > } while(0)
> >
> > void pthread_getspecific(key) pthread_specific_data
>
> The data is per key though so a correct implementation may involve a
> hashmap or a list.
Ah, yeah, you're right, I was mixing up the thread id and the key in my
head. I think it would just be an array of void pointers, with
pthread_key_create() returning an static index.
> It does simplify index-pack which has to fall back to nothread_data
> when pthreads is not available. But with index-pack being the only
> call site that can take advantage of this (run-command.c probably will
> use real pthreads library anyway), I'm not sure if it's worth really
> implementing these functions.
Yeah, you might be right.
-Peff