On Wed, Aug 07, 2013 at 05:02:42PM -0700, Kais Belgaied wrote:
> proposed changes look good.
> comment needed regarding the reason for internally managed per-thread data
> (limitation of Posix' PTHREAD_KEYS_MAX)

Hmm, I thought that was more about patch 3.  I updated that patch's
commit message to:

    ovs-thread: New function ovsthread_id_self().
    
    I foresee a need for possibly large numbers of instances of "struct
    seq" (which is introduced in an upcoming patch).  Each struct seq
    needs some per-thread data.  POSIX has pthread_key_t for this, but
    the number of keys can be fairly limited, to as few as 128.  It is
    reasonable to work around this by using a hash table indexed on the
    current thread.  That only works if one can get a thread identifier
    that is hashable (pthread_t is not).  This patch introduces a
    hashable thread identifier.
    
    Signed-off-by: Ben Pfaff <b...@nicira.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to