This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Aarch64 machine maintainership (was: Add ifunc memcpy and memmove for aarch64)
- From: Siddhesh Poyarekar <siddhesh at gotplt dot org>
- To: Steve Ellcey <sellcey at caviumnetworks dot com>, Szabolcs Nagy <szabolcs dot nagy at arm dot com>, Andrew Pinski <pinskia at gmail dot com>
- Cc: nd at arm dot com, Adhemerval Zanella <adhemerval dot zanella at linaro dot org>, Wilco Dijkstra <Wilco dot Dijkstra at arm dot com>, "libc-alpha at sourceware dot org" <libc-alpha at sourceware dot org>, Marcus Shawcroft <Marcus dot Shawcroft at arm dot com>
- Date: Wed, 22 Mar 2017 11:07:59 +0530
- Subject: Aarch64 machine maintainership (was: Add ifunc memcpy and memmove for aarch64)
- Authentication-results: sourceware.org; auth=none
- References: <AM5PR0802MB2610A11854986B9A53EBC68783430@AM5PR0802MB2610.eurprd08.prod.outlook.com> <c69e79db-4c42-0a14-dd11-d4825612b351@gotplt.org> <c0e44c88-244d-e327-de4b-02470c3a78cf@linaro.org> <1486509609.2866.59.camel@caviumnetworks.com> <805d8c52-1397-ddd0-71cc-93d019f8a7f2@gotplt.org> <1486598532.2866.66.camel@caviumnetworks.com> <589C4992.10505@arm.com> <CA+=Sn1k7BYS4G1Xwpeteif_JK4tdh0O9wCg9WOez4L=C3hMaYQ@mail.gmail.com> <1486688083.2866.82.camel@caviumnetworks.com> <1487809656.2866.171.camel@caviumnetworks.com> <54d21a51-1156-879c-7847-1b19eaca9b3e@gotplt.org> <1487866836.2866.173.camel@caviumnetworks.com> <eabb56f8-bea6-7375-019e-5a6365120a50@gotplt.org> <1487868166.2866.176.camel@caviumnetworks.com> <da742e76-0702-fb09-a1c3-86aadea5179e@gotplt.org> <1488394091.2866.259.camel@caviumnetworks.com> <58C83A67.4060806@arm.com> <1489621977.2451.46.camel@caviumnetworks.com>
On Thursday 16 March 2017 05:22 AM, Steve Ellcey wrote:
> I checked this patch in after moving the HWCAP_CPUID to bits/hwcap.h
> and adding an include of sys/auxv.h to cpu-features.c to get the value.
> When I added an include of bits/hwcap.h directly I got an error about
> including auxv.h instead of hwcap.h.
Technically you still needed to get an ack from Marcus who is the only
machine maintainer for aarch64. Practically though, the patch is fine
and it appears that Marcus hasn't had the time to review the patch and
it doesn't make sense to wait too long for something that has had
all-round consensus.
That said, it would be nice to have one of two things happen going forward:
Either Marcus names one or more machine maintainers for aarch64 that
have the bandwidth to review and approve aarch64 patches for glibc. I
would like to propose Adhemerval as an aarch64 machine maintainer too
since he has the necessary experience in glibc and also the involvement
in aarch64 work.
Alternatively, make aarch64 patch review open like x86. This might be
the way forward given that aarch64 development interests are getting
increasingly diverse (probably more so than x86) and it may become quite
difficult for a single maintainer from ARM to gate everything that goes in.
Siddhesh