This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
RE: [PATCH v2 5/5] Manual: Add manual for pthread mutex
- From: "Wang, Kemi" <kemi dot wang at intel dot com>
- To: Rical Jasan <rj at 2c3t dot io>
- Cc: Adhemerval Zanella <adhemerval dot zanella at linaro dot org>, Florian Weimer <fweimer at redhat dot com>, Carlos Donell <carlos at redhat dot com>, Glibc alpha <libc-alpha at sourceware dot org>, Dave Hansen <dave dot hansen at linux dot intel dot com>, "Chen, Tim C" <tim dot c dot chen at intel dot com>, "Kleen, Andi" <andi dot kleen at intel dot com>, "Huang, Ying" <ying dot huang at intel dot com>, "Lu, Aaron" <aaron dot lu at intel dot com>, "Li, Aubrey" <aubrey dot li at intel dot com>
- Date: Wed, 18 Jul 2018 07:55:45 +0000
- Subject: RE: [PATCH v2 5/5] Manual: Add manual for pthread mutex
- References: <1531464752-18830-1-git-send-email-kemi.wang@intel.com> <1531464752-18830-6-git-send-email-kemi.wang@intel.com> <ba8c7061-580d-5bb8-5076-1093ebd86dfa@2c3t.io>
OK. I will take a look at it.
-----Original Message-----
From: Rical Jasan [mailto:rj@2c3t.io]
Sent: Wednesday, July 18, 2018 3:34 PM
To: Wang, Kemi <kemi.wang@intel.com>
Cc: Adhemerval Zanella <adhemerval.zanella@linaro.org>; Florian Weimer <fweimer@redhat.com>; Carlos Donell <carlos@redhat.com>; Glibc alpha <libc-alpha@sourceware.org>; Dave Hansen <dave.hansen@linux.intel.com>; Chen, Tim C <tim.c.chen@intel.com>; Kleen, Andi <andi.kleen@intel.com>; Huang, Ying <ying.huang@intel.com>; Lu, Aaron <aaron.lu@intel.com>; Li, Aubrey <aubrey.li@intel.com>
Subject: Re: [PATCH v2 5/5] Manual: Add manual for pthread mutex
On 07/12/2018 11:52 PM, Kemi Wang wrote:
> Pthread mutex is not described in the documentation, so I started to
> document pthread mutex with PTHREAD_MUTEX_QUEUESPINNER type here at least.
>
> Signed-off-by: Kemi Wang <kemi.wang@intel.com>
> ---
> manual/Makefile | 2 +-
> manual/mutex.texi | 68
> +++++++++++++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 69 insertions(+), 1 deletion(-) create mode 100644
> manual/mutex.texi
I don't think this patch should create a new chapter. There is already threads.texi---woefully incomplete as it may be (one and a half pages in the PDF and over 100 functions in the .texi file documented in comments as undocumented). There are currently two sections in threads.texi:
Thread-specific Data and Non-POSIX Extensions. I think this patch should introduce a new section in that chapter; something like Mutexes.
This issue is independent of the content, which I'll review separately.
Rical