[PATCH] sysconf: Use conservative default for _SC_NPROCESSORS_ONLN [BZ #21542]

Florian Weimer fweimer@redhat.com
Fri Jun 9 13:37:00 GMT 2017


"H.J. Lu" <hjl.tools@gmail.com> writes:

> On Fri, Jun 9, 2017 at 5:35 AM, Florian Weimer <fweimer@redhat.com> wrote:
>> On 06/09/2017 02:32 PM, H.J. Lu wrote:
>>> On Fri, Jun 9, 2017 at 5:28 AM, Florian Weimer <fweimer@redhat.com> wrote:
>>>> On 06/09/2017 02:26 PM, H.J. Lu wrote:
>>>>> On x86, we compute the number of threads which share L3
>>>>> cache in a package via CPUID.  We should use this info as
>>>>> default on x86.
>>>>
>>>> That would only allow us to return a larger value than 2 in this case,
>>>> wouldn't it?
>>>>
>>>> I don't know if the additional complexity is worth it.  The 1/2
>>>
>>> The information is there.  We just need to store the result in
>>> cpu_features and return it in __default_nprocs ().
>>
>> It still needs another sysdeps override for __default_nprocs ().  This
>> is what I meant.

> Just a suggestion.  I have no strong opinion.

Okay, in this case, I will wait a bit for other comments and check in
what I have eventually.

Thanks,
Florian



More information about the Libc-alpha mailing list