Re: [PATCH 1/2] nptl: Remove unused members from struct pthread

[Arjun Shankar <arjun@redhat.com>]

Hi Adhemerval,

This looks good to me.

Reviewed-by: Arjun Shankar <arjun@redhat.com>

> It removes both pid_ununsed and cpuclock_offset_ununsed, saving about
> 12 bytes from struct pthread.
> ---
>  nptl/descr.h | 7 -------
>  1 file changed, 7 deletions(-)
>
> diff --git a/nptl/descr.h b/nptl/descr.h
> index bb46b5958e..b5852632e3 100644
> --- a/nptl/descr.h
> +++ b/nptl/descr.h
> @@ -172,9 +172,6 @@ struct pthread
>       therefore stack) used' flag.  */
>    pid_t tid;
>
> -  /* Ununsed.  */
> -  pid_t pid_ununsed;
> -

OK. This has been unused since c579f48edba (2016).

>    /* List of robust mutexes the thread is holding.  */
>  #if __PTHREAD_MUTEX_HAVE_PREV
>    void *robust_prev;
> @@ -342,10 +339,6 @@ struct pthread
>    /* Lock for synchronizing setxid calls.  */
>    unsigned int setxid_futex;
>
> -#if HP_TIMING_INLINE
> -  hp_timing_t cpuclock_offset_ununsed;
> -#endif
> -

OK. This has been unused since 38cc11daa43 (2019).

>    /* If the thread waits to join another one the ID of the latter is
>       stored here.
>
> --
> 2.34.1
>