This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
RE: [PATCH v3 1/3] Tunables: Add tunables of spin count for pthread adaptive spin mutex
- From: "Wang, Kemi" <kemi dot wang at intel dot com>
- To: Florian Weimer <fweimer at redhat dot com>, Adhemerval Zanella <adhemerval dot zanella at linaro dot org>, Rical Jason <rj at 2c3t dot io>, Carlos Donell <carlos at redhat dot com>, Glibc alpha <libc-alpha at sourceware dot org>
- Cc: 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: Thu, 7 Jun 2018 15:02:41 +0000
- Subject: RE: [PATCH v3 1/3] Tunables: Add tunables of spin count for pthread adaptive spin mutex
- Dlp-product: dlpe-windows
- Dlp-reaction: no-action
- Dlp-version: 11.0.0.116
- References: <1527067354-13333-1-git-send-email-kemi.wang@intel.com> <783458d1-90b4-9053-acbf-cd2fc91eb92f@redhat.com>
> I think the applicable comments from the commit message should be included in here.
Sure.
BTW, I plan to make this optimization opt-in, and enable it for x86 architecture as default.
As you suggested before.
-----Original Message-----
From: Florian Weimer [mailto:fweimer@redhat.com]
Sent: Thursday, June 7, 2018 9:10 PM
To: Wang, Kemi <kemi.wang@intel.com>; Adhemerval Zanella <adhemerval.zanella@linaro.org>; Rical Jason <rj@2c3t.io>; Carlos Donell <carlos@redhat.com>; Glibc alpha <libc-alpha@sourceware.org>
Cc: 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 v3 1/3] Tunables: Add tunables of spin count for pthread adaptive spin mutex
On 05/23/2018 11:22 AM, Kemi Wang wrote:
> + mutex {
> + spin_count {
> + type: INT_32
> + minval: 0
> + maxval: 30000
> + default: 100
> + }
> + }
I think the applicable comments from the commit message should be included in here.
Thanks,
Florian