Re: [musl] Re: sysconf(_SC_NPROCESSORS_CONF) broken.

[enh <enh@google.com>]

done: https://android-review.googlesource.com/c/platform/bionic/+/2168844

what's the worst that could happen?


On Thu, Jul 28, 2022 at 10:27 AM enh <enh@google.com> wrote:
>
> interesting... in case anyone's wondering why bionic didn't use
> /sys/devices/system/cpu/possible, i didn't know it existed.
>
> i'm pretty tempted to move us over to that. not because of the
> distinction between "present" and "possible" -- i'm assuming over here
> in mobile those two should always be the same anyway -- but because i
> can just share code with "online" and do slightly less work than the
> current directory scan...
>
> On Wed, Jul 27, 2022 at 4:00 PM Jonathan Rajotte-Julien
> <jonathan.rajotte-julien@efficios.com> wrote:
> >
> > Hi,
> >
> > >
> > > i think the most recent thread on the topic:
> > > https://www.openwall.com/lists/musl/2021/07/06/2
> > >
> > > an older discussion is at
> > > https://www.openwall.com/lists/musl/2019/03/15/5
> >
> >
> > FIY, we recently had to revisit our handling of all this.
> >
> > See [1].
> >
> > Which seems to be similar to what Vincent Donnefort, from [2], proposed.
> >
> > A follow up to unify our approach is also available here[3].
> >
> > [1] https://github.com/lttng/lttng-ust/commit/66dbdc3448a77043d0fd59f47b17e77a8d59fddb
> > [2] https://www.openwall.com/lists/musl/2021/07/06/2
> > [3] https://review.lttng.org/c/lttng-ust/+/8619
> >
> > Cheers