Re: [PATCH v2 0/4] nptl: Add pthread_clockjoin_np

[Adhemerval Zanella <adhemerval dot zanella at linaro dot org>]

On 18/09/2019 09:30, Mike Crowe wrote:
> The aim of this series is to add pthread_clockjoin_np, which works
> like pthread_timedjoin_np except it accepts an additional parameter to
> indicate which clock the timeout should be measured against.
> 
> The naming matches the pthread_cond_clockwait,
> pthread_mutex_clocklock, sem_clockwait, etc. functions added in glibc
> v2.30 and documented at http://austingroupbugs.net/view.php?id=1216.
> 
> The series also includes some minimal documentation for
> pthread_tryjoin_np and pthread_timedjoin_np, along with some new
> tests.
> 
> Thanks to everyone who provided feedback for the first version[1] of
> this series.
> 
> [1] https://sourceware.org/ml/libc-alpha/2019-06/msg00911.html

Hi Mike,

I think your patchset it ready to push upstream.  I have fixed the small
nits I brought on the review along with some more specific changes below:

  - Move the NEWS entry for pthread_clockjoin_np to its own bullet on
    2.31.

  - Moved __pthread_clockjoin_ex to a hidden definition and removed the
    hidden_{proto,def}.

  - Added some missing one line file description.

I also pushed the set on a personal branch [1]. If you are ok with this 
set, I would like to push it upstream.

[1] https://sourceware.org/git/?p=glibc.git;a=shortlog;h=refs/heads/azanella/pthread_clockjoin_np