• PatchworkReviewMeetings

Patchwork Review Meetings

The community uses a Patchwork v2 instance to track community submitted patches.

Everyone in the community is invited to attend a weekly Monday patchwork review meeting.

The meetings will be held at the following times:

• Monday at 0900h EST5EDT

Other meeting times may be added in the future to cover additional timezones.

The focus of the meeting is to review the patchwork queue here: https://patchwork.sourceware.org/project/glibc/list/

The intent is to find reviewers for patches and move patches forward along one of the states in the patchwork workflow: https://sourceware.org/glibc/wiki/Patch%20Review%20Workflow

Please set your Profile->Items per page to "500" for the meeting.

Please set your Profile->Show Patch IDs enabled to allow patch IDs to be used for meeting review.

Meetings will not be recorded, but notes from the meetings will be taken and recorded on this page.

Process:

• Show patches with: State = NEW, Delegate = Nobody

  • Use this query.

• Review newest patches looking for critical action items.
  • Record the patch at which review started.
• Delegate patches for review throughout the meeting.
• Record the patch at which review stopped.
  • This allows external observers to know which range of patches was reviewed and coordinate review.
• At the 30 minute mark go through the attendees and request items that they would to discuss for review.

Next Meeting: 2022-05-30 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and attendee requested reviews.

Meeting: 2022-05-16 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

• Carlos O'Donell sends his regrets.

Review new patches and attendee requested reviews.

• Status NEW delgate NOBODY 224 patches.
• Start at patch 54271.

• dlfcn: Move RTLD_DEFAULT/RTLD_NEXT outside USE_GNU

  • - Looks ok, Florian will review.
• linux: use statx for fstat if neither newfstatat nor fstatat64 is present
  • - Looks ok, Adhemerval will review.
• x86-64: Ignore r_addend for R_X86_64_GLOB_DAT/R_X86_64_JUMP_SLOT
  • - Fangrui state is is already done on other environments, Florian will review.
• dlsym: Make RTLD_NEXT prefer default version definition [#BZ #14932]
  • - Current status is too broken to keep a versioned symbol, change looks reasonable.
• [v11,1/6] elf: Refactor dl_new_hash so it can be tested / benchmarked
  • - Accepted by Siddhesh.
• [1/5] locale: Turn ADDC and ADDS into functions in linereader.c
  • - Not urgent, skipped for now.
• Avoid RMW of flags2 outside lock (BZ #27842)
  • - Still need some discussion that lock can be skipped if flockfile is called.
• [v2] x86_64: Implement evex512 version of strlen, strnlen, wcslen and wcsnlen
  • - Noah is reviewing it.
• [v3] Fix deadlock when pthread_atfork handler calls pthread_atfork or dlclose
  • - Accepted by Adhemerval.
• Add arc4random support
  • - Still need reviewers.
• [v3] Remove special flags of libc.5.so and libc.4.so
  • - Patch looks ok, some changers required.
• socket: Check lengths before advancing pointer in CMSG_NXTHDR
  • - Siddhesh will review it.
• [v4,3/3] linux: Add fallback for clone failure on posix_spawn (BZ #29115)
  • - Skipped, most likely will result in WONTFIX and programs will need to use
    • any new kernel time namespace fix kernel will provide.

• GLIBC LoongArch PATCHES

  • - Patchset looks ok, still requires some upstream fixes. Skipped due it will
    • require at least a v5.
• [v3] elf: Rewrite long RESOLVE_MAP macro to a debug friendly function

  • - Siddhesh is ok with the always_inline change.

• [v3] math: Add math-use-builtins-fabs (BZ#29027)
  • - The patch is ok, it does not change any code.
• stdio-common: Add the fgetln function
  • - Still need review.
• elf: Simplify version test when searching a versioned symbol
  • - Florian will review.
• Simplify ELF_RTYPE_CLASS_EXTERN_PROTECTED_DATA and revert aarch64/arm's extern protected data handle
  • - Florian will review.
• Optimize struct pthread size
  • - Skipped, Arjun will review.
• libio: Remove _IO_adjust_column, _IO_adjust_wcolumn, column tracking
  • - Adhemerval will review.
• powerpc64: small-model @got@tprel accesses
• [v3] powerpc64: Enable static-pie
  • - skipped, waiting review from powerpc maintainers.
• Stopped at 52777.
• Siddhesh
  • - Nothing to review.
• Szabolcs
  • - Nothing
• Sunil

  • - Need to check https://sourceware.org/bugzilla/show_bug.cgi?id=29127

• Noah
  • - Can we change pthread cond to use default mutex and increment the counter
    • outside? Is this a GNU extension or is related to some different mutex type?
• Patrick
  • - Nothing
• Florian
  • - Nothing

Meeting: 2022-05-16 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

• Status NEW delgate NOBODY 252 patches.
• Start at patch 53982
• [v2] Remove special flags of libc.5.so and libc.4.so
  • Adhemerval will review.
  • Florian: Cleanup is valuable.
• 53980 Enable DT_RELR automatically
  • Carlos: Make first commit message line more descriptive
• 53963 Check for ISO C compilers should also allow C++
  • Fangrui and Florian reviewed.

• 53886 fortify: Ensure that glibc_fortify condition is a constant [BZ #29141]

  • Florian noted Siddhesh is the subject matter expert.
• [v8,1/6] elf: Refactor dl_new_hash so it can be tested / benchmarked
  • Adhemerval: Is this something we can get the compiler to do?
  • Carlos: Yes, it would be better if the compiler did it, but today it doesn't.
  • Ahemerval: Not a blocking objection, but don't like the asm.
  • Siddhesh: Elevate the objection about the asm. Will comment on the list.
• 53756 Linux: Fix posix_spawn when user with time namespaces
  • Florian: Ask kernel for new way of entering time namespace
  • The kernel can't change the current behaviour.
  • Many C and C++ libraries have interfaces implemented from scratch.
  • All the interfaces would have to be changed to support this.
  • Florian: Reached out to Christian Brauner about this on the musl thread.
  • Do we need to speak with Andrei Vagin (CRIU)?
• Stopped at 53612.
• Adhemerval: Looking for arc4random review.
  • arc4random at v5 review.
  • Carlos will review kernel syscall patches.
  • Florian has not reviewed this as author of the previous attempt.
  • Carlos: We are talking about commiting to the ABI? Any objections to the ABI?
  • Florian: We should add the ABI.
• Arjun: Have two patches to reivew. New patch to post v3 for pthread_atfork.
• Florian:
  • brk change:
    • Not sure where I'm heading with the brk change.
    • Adhemerval: OK with the brk changes.
  • vfprintf rework still needs review of the buffer management.
  • UTF-8 locale source work:
    • Siddhesh can change one.
    • Carlos can change Spanish locales (22 of them)
• HJ: Nothing to review.
  • Long term goal to get rid of copy relocation in x86.
  • There are a couple of platform that support copy relocations.
  • Many don't support copy relocations.
  • From a performance and security POV we want to get rid of them.
  • Especially with protected symbols they are supposed to improve performance.
  • Semantics of copy relocations break this.
  • gcc 12 has option to avoid copy relocations on x86.
  • Explore removing copy relocations?
  • Florian: From Fedora and downstream perspective copy relocations don't matter.
    • We use LTO and the builds localize the symbols.
    • The change to copy reocations is a NOP.
    • Likewise for SUSE.

    • HJ: Note in Windows with declspec(dllimport/dllexport) we can improve code gen.

      • HJ: Anything we can do here that is similar?
  • Florian: COPY relocations may still be relevant on some architectures.
    • For example cost to access global data through the GOT
• Szabolcs: Outstanding patch was reviewed. Need to change and push.

Meeting: 2022-05-09 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

• 255 patches in NEW/NOBODY
• Started at 53612.

• GLIBC LoongArch PATCHES (caiyinyu)

  • LoongArch patch series for glibc.

  • Joseph Myers was previously reviewing this.
  • Several other developers reviewed parts of the series.
  • Review against existing v2

    • https://sourceware.org/pipermail/libc-alpha/2021-December/134811.html

  • LoongArch is not yet upstream in the kernel.

  • Need to have the port make its way into the kernel via linux-next.
  • With 5.19 slated for August, we need to consider if RC phase constants
    • are good enough for a glibc port.
  • Could we merge and then backout of the port doesn't make 5.19?
• 53573: [v3] elf: Rewrite long RESOLVE_MAP macro to a debug friendly function (guriev)
  • Fangrui was looking at earlier versions of the cleanup.
• 53613: [v2] wcrtomb: Make behavior POSIX compliant (poyarekar)
  • Looking for review for v2 from earlier reviewers.
  • Adhemerval: Patch itself is OK, but need to check if the patch has consensus.
  • Carlos: Manual should say what you have *now* in the impl.
  • Siddhesh: Main concern is bumping symbol version because it avoids backports of
    • the fix.
• 53543: [v5] nptl: Add backoff mechanism to spinlock loop (Wangyang)
  • HJ reviewing. Should be committed today.
• 53526: [v3] math: Add math-use-builtins-fabs (BZ#29027)
  • Joseph pointed out issues.
  • Followed up with suggestions. Just a code cleanup.
  • Should not change code generation.
• 53523: [PATCH v4 1/3] ia64: Always define IA64_USE_NEW_STUB as a flag macro
  • Adhemerval and Florian discussing consensus.
  • Adhemerval: Move mmap call to internal definition. So we can use the same logic.

  • https://sourceware.org/pipermail/libc-alpha/2022-May/138524.html

  • Florian to review.
• 53519: Args adjustment with ./ld.so exe [BZ #23293] (v6)
  • Need 1 and 4 patches reviewed.
  • Florian: The first patch needs reivew.
• Stopped at 53519.
• Florian: vfprintf not that high priority.
  • Someone should look at the buffer abstraction.
  • Series 7969.
  • Patch 17 implements the abstraction.
• Arjun: pthread_atfork v2.
• Adhemerval:
  • Fallback clone on posix_spawn failure.
    • Ask Christian Brauner about the vDSO issue.
    • There is conflict here with the semantics.

Meeting: 2022-05-02 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

• 259 patches in NEW/NOBODY.
• 53383 powerpc32: Remove unused HAVE_PPC_SECURE_PLT (Fangrui)
  • Reviewed by Florian.
• 53379 elf: Remove fallback to the start of DT_STRTAB for dladdr (Fangrui)
  • Needs review.
• 53375 elf: Simplify version test when searching a versioned symbol (Fangrui)
  • Needs review.
• Simplify ELF_RTYPE_CLASS_EXTERN_PROTECTED_DATA and revert aarch64/arm's extern protected data handl…
  • Ask HJ if he can provide review (worked on this before).
• elf: Remove one-default-version check when searching an unversioned symbol (Fangrui)
  • Needs review.
• S390: Enable static PIE (Stefan Liebler)
  • Needs quite a bit of kernel support.
  • Florian and Adhemerval in agreement that we should handle failures gracefully.
  • brk failures default back to mmap.
  • Should we just use mmap unconditionally?

• time: Use nonnull to avoid null pointer (Xiaoming)

  • Under review from Siddhesh.
  • Fails CI/CD for 32-bit build.
• [v4,1/6] elf: Refactor dl_new_hash so it can be tested / benchmarked (Noah)
  • Needs review for performance and timing.
  • Open question about timing from Alexander Monakov:

    • https://sourceware.org/pipermail/libc-alpha/2022-April/138275.html

• [v2] Fix deadlock when pthread_atfork handler calls pthread_atfork or dlclose (Arjun)
  • Asking Adhemerval for review since he worked on the first patch.
• Optimize struct pthread size (Adhemerval)
  • Two patches. Carlos to review.
• Add arc4random support (Adhemerval)
  • Currently at v4.
  • Siddhesh raised questions about FIPS compliance.
    • Crypto stuff we add does not get exposed publicly as API.

    • No ChaCha20 API exposed then we should be OK.

    • Document in manual that arc4random is not intended for use in secure contexts.
    • Say "Not intended for cryptographic or security purposes."
  • v5 with manual change.
• 53091 libio: Remove _IO_adjust_column, _IO_adjust_wcolumn, column tracking
  • Review after "vfprintf rework to remove vtables"
• Stopped at 53091.
• Sunil: Raised issues about backporting to release/2.32/master.
  • Problems caused by the AMD patch backport.
  • Florian: Could revert the AMD backport and then layer everything properly.
  • Consensus to revert the AMD patch, and then layer the commits correctly.
  • Florian: Can push revert to a branch for 2.32.
• Florian: Would like to backport the dlinfo fixes.
  • Carlos: Only requires the addition of a header constant. OK with me.
• Remove kernel version and distinguish library check (Adhemerval)
  • Florian to review later this week.

• linux: Use /sys/devices/system/cpu on get_nprocs_conf (BZ#28991)

  • Needs a follow-on fix because of rseq.
  • Carlos: Is malloc affected?
  • Adhemerval: No, only the exported symbol is impacted.

Meeting: 2022-04-25 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• NEW/NOBODY is at 324 patches.
• Starting at 53184
• v4 Add arc4random support (Adhemerval) 53184

  • https://sourceware.org/pipermail/libc-alpha/2022-April/138132.html

  • Discussed arc4random support and Adhemerval's implementation.
  • Looking to get consensus from Florian Weimer.
  • Florian is the author of the previous attempt at an upstreamed impl.
  • Looking to decide as a community what direction we would go in.
• 53169 elf: Replace PI_STATIC_AND_HIDDEN with opposite HIDDEN_VAR_NEEDS_DYNAMIC_RELOC
  • Adhemerval to look at this.
• 53144 elf: Move post-relocation code of _dl_start into _dl_start_final
  • Adhemerval to look at this.
  • glibc lacks some clear distinctions around the loader startup.
  • Specific barriers are needed here.
• 53127 Support DT_RELR relative relocation format (HJ)
  • Pretty much done.
  • Try to remove configure knob on this - Turn on DT_RELR by default.
• 53117 INSTALL: Rephrase -with-default-link documentation
  • Carlos to respond.
• 53094 Fix deadlock when pthread_atfork handler calls pthread_atfork or dlclose
  • Failed CI: Bad test.
  • Arjun: Reviewing Adhemerval's patches for this.
  • Arjun: What is the consensus for prepare/after handlers added during the fork
    • in a handler? Should they just become something to use later?
  • Adhemerval: When you fork, you use the list as-is.
  • Arjun to post new version.
  • Arjun to add manual to document pthread_atfork
  • Arjun to investigate what to say to the austingroup.
    • "If any handler causes a handler to be registered to deregistered then the behaviour of the handler is implementation defined"
• Stopped at 53091.
• Args adjustment with ./ld.so exe [BZ #23293] (szabolcs)

  • https://patchwork.sourceware.org/project/glibc/list/?series=8562

  • Looking for review.
  • Szabolcs to look at adding a test case.
• Sunil: Backporting code into the release branches.

  • https://sourceware.org/pipermail/libc-alpha/2022-April/137718.html

  • Commit author remains the backpot author
  • Author remains the original commit author.
  • Cherry-pick -x notes the original commit.
• [v2,1/6] elf: Refactor dl_new_hash so it can be tested / benchmarked (Noah)
  • Asking for Florian to review.
• [v5] elf: Fix DFS sorting algorithm for LD_TRACE_LOADED_OBJECTS with missing libraries (BZ #28868)
  • Siddhesh to review patch 51584.

Meeting: 2022-04-18 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

No meeting (Easter Monday).

Meeting: 2022-04-11 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• NEW/NOBODY 266 patches.
• Starting at patch 52775.
• ELF tests in Python and libio vtables/RELRO interaction (fweimer)
  • Needs review.
• [v4,1/6] x86: Remove {w}memcmp-ssse3 (noah)
  • Complaints about a specific machine was handled with reporter.
  • HJ to review.
• nptl: Handle spurious EINTR when thread cancellation is disabled (BZ#29029) (fweimer)
  • Debian has tested (Aurelien Jarno)
  • Florian wants to review this one given his state on the changes.
• [v2] math: Add math-use-builtins-fabs (BZ#29027) (adhemerval)
  • Needs review.
• manual: Avoid name collision in libm ULP table [BZ #28956] (Coldrick)
  • Carlos to push.
• [v1] sysdeps: Add 'get_fast_jitter' interace in fast-jitter.h (noah)
  • Jitter interface is for thread backoff.
  • This patch is for use by the in-progress changes to futex backoff.
• [v6] Add .clang-format style file (noah)
  • Looking for 1 review.
  • Carlos to review.
• [v3] elf: fixes compile error when both enable -Werror and -DNDEBUG (yang)
  • This needs the usual compiler pragmas to remove the unused warnings.
  • Carlos to respond.
• 52556-52525 posted fixes for fuzzing by Dmitry Chestnyh (omp.ru)
  • Discussing options for reviewing fuzz patches.
• 52304 - Superseded

• linux: Use /sys/devices/system/cpu on get_nprocs_conf (BZ#28991)

  • Still up for review (week 2).
• Refactor syslog implementation (adhemerval)
  • Just a refactor without timestamp change.
  • Improves code and testing.
• Stopped at 52087: vfprintf rework to remove vtables (fweimer).
• Florian: Discussed PT_GNU_RELRO and potential binutils issues with PT_LOAD alignment.
  • Final PT_LOAD doesn't start page aligned. Should it?
• Noah: Discussed improvements to symbol lookup.
  • Noted dl lookup function and strcmp on the hot path.
  • Florian: Pointed out we can precompute some of the math operations per linkmap.
    • Some of the math on these lookups can be stored.
  • Noah: Can we know that we aren't on a page boundary and use a distinct strcmp?
  • Carlos: You know DT_STRTAB and STRSZ and can do some figuring out.

Meeting: 2022-04-04 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• NEW/Nobody 260 patches.
• Starting review 52598
• [v3] elf: fixes compile error when both enable -Werror and -DNDEBUG
  • Needs different structure.
• Dmitry Chestnyh - Ran the static analyzer.
  • Need to follow up regarding assignment vs. DSO.
  • SVACE static analyzer results.
  • 12 patches need review.
• Discussed briefly that string tests are taking a long time.
  • What can we do to improve developer experience?
  • Note: Split the tests at the make target level with different args.
  • Note: Split the tests at the do_test level with different threads.
  • Noted that any split up should help but not solve.
• [v5] Add .clang-format style file
  • Carlos to review v5.
• Remove -z combreloc and HAVE_Z_COMBRELOC
  • Different from Adhemerval's branch to enable clang.
  • Needs review.
• elf: Simplify _dl_aux_init with inhibit_loop_to_libcall
  • Reviewed by Szabolcs. Needs fixes.
  • Adhemerval noted this can be fixed in a different way.
• 52433: [v6] Allow for unpriviledged nested containers (DJ)
  • Carlos to review.
• 52427: [v1,1/2] random-bits: Factor out entropy generating function
  • Needs review.
  • What is the status of Florian's arc4random?
• 52428: [v1,2/2] x86: Use rdtsc for generating entropy for random_bits
  • Needs review.
• Stopped at 52428.
• Florian has a vfprintf patch series reviewed.

  • https://patchwork.sourceware.org/project/glibc/list/?series=7969

  • Performance impact is very positive.
• Adhemerval wants LD_AUDIT trace fixed.
  • Florian really wants to review.

• linux: Use /sys/devices/system/cpu on get_nprocs_conf (BZ#28991)

  • Needs review.

• [v2] linux: Fix closefrom_fallback iterates until max int (BZ#28993)

  • How do we proceed? -- Should be OK to proceed to fail fast.
  • Will post v3.
  • What about ENOMEM?

Meeting: 2022-03-28 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• State NEW, delegate NOBODY is 312 patches.
• Adhemerval tries to push.

  "Enumerating objects: 1581, done.
Counting objects: 100% (1581/1581), done.
Delta compression using up to 24 threads
Compressing objects: 100% (807/807), done.
Writing objects: 100% (1093/1093), 162.32 KiB | 13.53 MiB/s, done.
Total 1093 (delta 950), reused 298 (delta 272), pack-reused 0
remote: Resolving deltas: 100% (950/950), completed with 415 local objects.
remote: *** This update introduces too many new commits (121), which would
remote: *** trigger as many emails, exceeding the current limit (100).
remote: *** Contact your repository adminstrator if you really meant
remote: *** to generate this many commit emails.
remote: error: hook declined to update refs/heads/azanella/clang"

• Carlos reviewed and it's max-commit-emails, which can be set to 1000 to avoid
  • this problem and allow you to publish a branch for review.
• [v2] nptl: Add backoff mechanism to spinlock loop (Wangyang Guo)
  • We can't use random-bits.h (30 cycles)
  • Can we use tid?
  • Can we use rdtsc? (18 cycles)
  • At least one microbench should be contributed.
• [v1,1/2] x86: Small improvements for wcscpy-ssse3 (Noah)
  • Waiting for feedback from Zhaoxin cpu discussion, see:

    • https://sourceware.org/pipermail/libc-alpha/2022-March/137354.html

• [v2,1/6] x86: Remove {w}memcmp-ssse3 (Noah)
  • HJ reviewing.
• rwlock for async application
  • This is an RFC for glibc to deviate from POSIX behaviour.
• Add HWCAP2_AFP, HWCAP2_RPRES from Linux 5.17 to AArch64 bits/hwcap.h (Joseph)
  • Accepted by Szabolcs [Arm]
• Note from Adhemvarl that POSIX will add strlcpy and strlcat
  • Prposed by Florian in 2016.

  • https://www.austingroupbugs.net/view.php?id=986

  • https://sourceware.org/glibc/wiki/strlcpy

• Stopped at 52304.
• realpath: Bring back GNU extension on ENOENT and EACCES [BZ #28996]
  • Adhemerval will review.
• [v5] Allow for unpriviledged nested containers
  • Adhemerval asking if we are going to accept this patch?
  • Carlos to review this to completion to unblock Adhemerval.

  • This blocks the syslog https://patchwork.sourceware.org/project/glibc/list/?series=8031

• Siddhesh notes that DNS tests no longer depend on proc mount.
  • However we still have tests that use DNS and can fail.
  • They need to be ported to use the in-process test DNS resolver or changed.
• Support DT_RELR relative relocation format review.
  • What is the status of DT_RELR for AArch64? Currently unimplemented.
  • Looking for review of v6 patches.
  • No technical blockers. Push April 11th if no further reviews.

Meeting: 2022-03-21 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• Carlos unable to start meeting until 930am due to technical issue.
• Discussed with Adhemerval patches with outstanding review.

Meeting: 2022-03-14 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• 255 patches with NEW/NOBODY.
• getaddrinfo facelift and fixes (Sidhesh)
  • Some feedback from Carlos regarding DNS access.
  • Feedback from Andreas about example.org.
• [v5] Allow for unpriviledged nested containers (DJ)
  • Florian reviewing.
• [v6] Support DT_RELR relative relocation format (HJ)
  • Added for x86_64, ppc64le looking at this. What about others? (Adhemerval)
  • What about aarch64 and s390x? Other arches like RSIC-V?
  • Confirmed for 2.38 we have x86_64, ppc64, ppc64le.
  • Would be nice to have aarch64.

• scripts/dso-ordering-test.py: Fix C&P error in * callrefs processing

  • Adhemerval to review.
• [v5] elf: Fix DFS sorting algorithm for LD_TRACE_LOADED_OBJECTS with missing libraries (BZ #28868)
  • Review from Florian?
• Remove kernel version and distinguish library check (Adhemerval)
  • Waiting for Florian to finish review.
  • Discussed that 4.9 is the oldest LTS kernel we could support.
  • No large benefit to move to 4.9 and it would cause downstream problems.
  • Relaxing the version check is a step forward.
• PowerPC64 static-pie (Alan Modra)
  • Tulio (IBM) to review.

• : C++20 P0482R6 and C2X N2653: support for char8_t, mbrtoc8(), and c8rtomb().

  • No review.
• [v4] localedata: Updates for Taiwanese locales [BZ #24409]
  • Carlos asking for split up and review.
• Use RFC5424 for syslog (Adhemerval)
  • Adhemerval: Need to review upstream systemd thread.
• linux: Add new syscalls
  • Added 12 syscalls.
  • Carlos: Do we need man pages for these?
  • Adhemerval: I think all of them have man pages.
  • man-pages currently slow to update.
    • Waiting for Alejandro to have access to tree?
• Predictable ELF destructor ordering
  • Florian to repost.

Meeting: 2022-03-07 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• 260 patches in NEW/NOBODY
• [v5] elf: Fix DFS sorting algorithm for LD_TRACE_LOADED_OBJECTS with missing libraries (BZ #28868)
  • Looking for review.
  • Florian will to review.
  • Backported just the fix downstream with no issues to report.
• Remove kernel version and distinguish library check
  • Adressed Florian's review.
• malloc: Exit early on test failure in tst-realloc
  • Need review.
• v5: Support DT_RELR relative relocation format (HJ)
  • No reviewer right now.
• v2: PowerPC64 static-pie
  • Tulio says IBM will review.

• C++20 P0482R6 and C2X N2653: support for char8_t, mbrtoc8(), and c8rtomb().

  • Still in the same position as last week.
• [v3] libio: Ensure output buffer for wchars (bug 28828)
  • Adhemerval reviewed.
  • Asked vor v4 with fixed tests.
  • Changes requested.
• Use RFC5424 for syslog (Adhemerval)
  • Some initial review from Paul Eggert.

  • https://github.com/systemd/systemd/issues/19251#issuecomment-817901396

    • This was the upstream conversation.
• linux: Add new syscalls (Adhemerval)
  • No reviewer yet.
• [v2] x86: Reformat code in svml_s_atanhf16_core_avx512.S
  • Noah submitted. Working with Sunil on this issue.
• v2: Predictable ELF destructor ordering (Florian)
  • Still looking for review.
  • Need to repost v3 after testing.
• [v2,1/3] x86: Optimize svml_s_tanhf_core_avx512.S (Noah)
  • Working with Sunil to reformat code before optimizing.
• Stopped at 50642.
• New DSO dependency resolver buggyness with dlclose()

  • https://sourceware.org/bugzilla/show_bug.cgi?id=28937

  • As serious as LD_TRACE_LOADED objects bug (Bug 28868) and needs fixing.
• Fixes for getnameinfo() with NI_NOFQDN (race condition)

  • https://patchwork.sourceware.org/project/glibc/list/?series=5655

  • Does any of this touch Siddhesh's getaddrinfo refactor? No. Unrelated.
  • Asked DJ for review of resource acquisition.
• Adhemeral discussed symbol redirection and usage in gcc and clang
  • Will need some changes to glibc's headers if we support clang as a compiler.
  • Need to discuss this on list with a plan.
  • Florian notes that this is a large project.

Meeting: 2022-02-28 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• NEW/NOBODY 262.
• PowerPC64 static-pie [v2 1/4-4/4]
  • Ask Tulio if he will review.

• : C++20 P0482R6 and C2X N2653: support for char8_t, mbrtoc8(), and c8rtomb().

  • Accepted by WG14. Needs review. Repost.
  • Needs a reviewer to look at this against the N2653 implementation requirements.
• [v4] localedata: Updates for Taiwanese locales [BZ #24409]
  • Carlos to look at this.
• benchtests: make compare_strings.py accept string as attribute value
  • Patch failed to apply (CI/CD caught this). Siddhesh will cleanup
• [v3] libio: Ensure output buffer for wchars (bug 28828)
  • Adhemerval reviewed the changes.
  • Adhemerval to respond again regarding test case.
• [v3] elf: Fix DFS sorting algorithm for LD_TRACE_LOADED_OBJECTS with missing libraries (BZ #28868)
  • Adhemerval need to rewrite the test case using Florian's suggestions.
• Remove distinguish library code (Adhemerval)
  • Drop kernel version check and drop arch review.
  • Florian to review.
• [v3] x86: Test wcscmp RTM in the wcsncmp overflow case [BZ #28896]
  • HJ reviewing.
• Use RFC5424 for syslog

  • https://sourceware.org/pipermail/libc-alpha/2021-November/132893.html

  • DJ needs to review.
  • New pid namespace means new proc instance.
  • Same pid namespace you can use a bind mount.
• Support DT_RELR relative relocation format
  • At v3. Latest version of binutils had DT_RELR support.
  • Note that Fedora Rawhide has binutils 2.38 and could be used to test DT_RELR.

Meeting: 2022-02-21 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• Discussed patches and how to make progress on them.

Meeting: 2022-02-14 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• 284 patches in NEW/NOBODY.
• linux: Use socket-constants-time64.h on tst-socket-timestamp-compat
  • Carlos to review.

• x86-64: Define memcmpeq in ld.so

  • Noah reviewing.
• libio: Ensure output buffer for wchars (bug 28828)
  • Adhemerval reply to resend the patch.
  • Patrick: Was a whitespace issue. Lines that are not in the patch
    • all have extra whitespace.
• resolv: Fix tst-resolv tests for 2.35 ABIs and later
  • Adhemerval said he'd look atthis.
• rtld: Add new hook to initialize dp register on hppa
  • Reviews from Florian and HJ. Dave needs a v2.
• inptl: Use 64 bit time_t stat internally for sem_open (BZ #28880)

  • Notes that running on a system with future >64-bit time fails a lot of tests.

  • Need to go fixing these issues.
• Remove bcopy and bzero optimizations

  • HJ notes that rather than remove bzero we could rename to memset_zero

  • Note that memset_zero is not equivalent to bzero because of the return type.

  • Removing bcopy and bzero and replacing with memset_zero is arch specific.

  • Each maintainer needs to consider the memset_zero optimziation.

  • Carlos: Can we notify the arch maintainers before removal?
• Stopped at 51018.
• Support DT_RELR relative relocation format
  • HJ: Answered all of Joseph's comments.
  • Joseph was original reviewer.
  • Carlos to look at this on Friday.
• Florian: Patch for aux vector changes.
  • Consensus to push but needs future work.

Meeting: 2022-02-07 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• NEW / NOBODY patches 355.
• [v2] linux: fix accuracy of get_nprocs and get_nprocs_conf [BZ #28865]
  • Adhemerval to look at.
• [v2] hppa: Fix bind-now audit (BZ #28857)
  • Adhemerval to send v3.
  • Carlos to review as hppa maintainer.
• [v1] x86: Remove SSE3 instruction for broadcast in memset.S (SSE2 Only)
  • HJ provided feedback.

• elf: Replace memcmp with memcmpeq for variable size

  • Using memcmp_eq limits the compiler optimizations for size.
    • There is a tradeoff for size vs. speed.

• x86-64: Make bcmp an alias of memcmpeq

  • HJ will drop this.
  • Carlos will make a comment about bcmp.
• [v1] x86: Improve vec generation in memset-vec-unaligned-erms.S
  • Already committed.
• Support DT_RELR relative relocation format [v2]
  • Second version of DT_RELR.
• Improve LC_MONETARY handling.
  • Ask DJ for review.
• [v2] x86: Reformat code in svml_s_atanhf16_core_avx512.S
  • Sunil will review svml cleanups for reformatting.
• Remove prelink support [v2]
  • Siddhesh has started to review.
• linux: Add new syscalls
  • Carlos to review new syscalls.
• HJ: During the 2.35 release cycle we removed a p_align check.
  • elf: Check invalid hole in PT_LOAD segments [BZ #28838]
    • Needs review.
• Patrick: [v2] Remove upper limit on tunable MALLOC_MMAP_THRESHOLD
  • Cleaned up patch tracker state.
• Siddhesh ran cleanup for patches.
  • Cleaned up to 264.
• getaddrinfo: Fix leak with AI_ALL [BZ #28852]
  • May need a v2. Florian reviwed and asked for feedback.
• [v2] setlocale: Fail if iconv module for charset is not present [BZ #27996]
  • Siddhes to post v3.

Meeting: 2022-01-31 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

With the release upcoming we've switched to reviewing blockers and desirable changes.

• NEW/NOBODY 312 patches.
• Release planned for Wednesday February 2nd.
• MIPS _r_debug issues

  • https://sourceware.org/pipermail/libc-alpha/2022-January/135820.html

  • Needs fixing properly.
• MIPS pldd support for DT_MIPS_RLD_MAP_REL

  • https://sourceware.org/pipermail/libc-alpha/2022-January/135821.html

  • Needs fixing.
• MIPS issues all in one group.

  • https://sourceware.org/pipermail/libc-alpha/2022-January/135837.html

• posix_spawn_file_actions_addtcsetpgrp_np needs review.

  • https://patchwork.sourceware.org/project/glibc/patch/20220129200620.501423-1-adhemerval.zanella@linaro.org/

  • Florian to review v2.
• C.UTF-8 locale
  • Review Andreas Schwab's comments.
• HJ - Close to fixing the loader bug for invalid p_align
  • Considering to backport the p_align change to older releases.
  • We should fix this in 2.35.
  • Need an additional check for invalid objects.
• HJ - When I run the glibc testsuite on Alderlake
  • Opened bugs for those. They are test issues.
  • Use of a privileged instruction issue.
  • Processor feature detected by cpuid but is not reported by kernel.
  • 50466, 50467 patches.
• CI/CD - tst-21269 hangs on i386

  • Discussion: https://sourceware.org/pipermail/libc-alpha/2021-July/128853.html

  • We should fix this.

Meeting: 2022-01-24 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

With the release upcoming we've switched to reviewing blockers and desirable changes.

• Review the release blockers for glibc 2.35.
• PT_LOAD fix status?
  • Committed fix e22a4557eb39d7cba9a74d70f4582c13f1a7a83a
  • Adhemerval reviewing test case inclusion.
• Fix glibc 2.34 ABI omssion.
  • HJ: Is there a missing linker feature here?
  • Discussed Solaris VER_FLG_WEAK

    • https://sourceware.org/pipermail/libc-alpha/2021-December/133796.html

• linux: Fix ancillary 64-bit time timestamp conversion (BZ #28349, BZ #28350)
  • Florian sent review.
  • Adhemerval reposted patches with fixes.
• posix: Add terminal control setting support for posix_spawn
  • Carlos to post review ASAP.
• Multiple RTLD audit fixes.
  • Adhemerval sent updated versions.
  • Carlos to look at the fixes.
• Use after free warning failures in gcc 12.
  • Carlos to follow up with Martin Sebor.
• IBM long-double issues are release blockers we aren't tracking yet.
  • Blocks Fedora Rawhide builds.
  • We need to disable the flag in more places.
  • Florian to post the patches for this today.
• Szabolcs: p_align issue.
  • Binutils switching on x86 to set p_align of load segments.
  • On AArch64 it would be nice too.
  • Old glibc would reject such a binary on a 64k page system.
  • Seems to me that I cannot switch this if there are existing 64k page sized systems.
  • Florian suggests fixing the bug in glibc and backport to old releases.
  • Carlos agrees with backport.
• Adhemerval sent prelink removal patch for glibc 2.36.
  • Carlos: Post a NEWS update with a note saying 2.35 is the last relase with prelink.

Meeting: 2022-01-17 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

With the release upcoming we've switched to reviewing blockers and desirable changes.

• NEW/NOBODY 290 patches
• [v7] elf: Properly align all PT_LOAD segments [BZ #28676]
  • Adhemerval to review alignment patch.
• linux: Fix ancillary 64-bit time timestamp conversion (BZ #28349, BZ #28350)
  • Florian to review.
• posix: Add terminal control setting support for posix_spawn
  • Carlos to finish review Monday.
• Multiple rtld-audit fixes
  • Adhemerval to rebase patch set.
  • Szabolcs looked at the aarch64 pieces and gave feedback.
• Properly handle --disable-default-pie
  • Siddhesh is going to look at this.
• Carlos and Adhemerval discussed 64-bit time_t testing.
  • No way to change library default time_t size, just testing can test it.
• Prelink discussion
  • Adhemerval: May I submit patches to remove prelink?
  • Carlos: Yes.
  • Discussed Intel, Arm, Linaro involvement in prelink.
  • Carlos: IHVs and ISVs need to invest upstream if they want to see prelink maintained.
• Paul sends his regrets and includes two items:
  • I have done a review of the accuracy of the libmvec functions (in master)
    • both with sse4.2, avx2 and avx512. Apart from a minor issue with atan2 in binary64 which was fixed (#28765), I found no error larger than 4 ulps, both in binary32 and binary64.
  • We are making good progress in the CORE-MATH project: 17 out of 26
    • functions are now available in single precision (binary32).

      See https://homepages.loria.fr/PZimmermann/CORE-MATH/. In most cases the correctly-rounded CORE-MATH code outperforms the GNU libc one (see the "perf" graphs). The only exceptions so far are the cosf function on i7, and the expf function.

Meeting: 2022-01-10 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

With the release upcoming we've switched to reviewing blockers and desirable changes.

• Discuss release blockers for glibc 2.35.
• Properly align all PT_LOAD segments with tests (v7)

  • https://patchwork.sourceware.org/project/glibc/list/?series=6100

  • Florian will look again at v7.
• [v2] math: Fix float conversion regressions with gcc-12 [BZ #28713]

  • https://patchwork.sourceware.org/project/glibc/list/?series=6114

  • Review from Paul, and Joseph. Joseph's feedback incorporated.
  • Carlos recommends pushing fix at this point.
• Fix glibc 2.34 ABI omission (missing GLIBC_2.34 in dynamic loader)

  • https://patchwork.sourceware.org/project/glibc/list/?series=5581

  • Florian to ping Andreas again pointing at previous justification.
• elf: Fix 64 time_t support for installed statically binaries

  • https://patchwork.sourceware.org/project/glibc/patch/20211229132711.3324972-1-adhemerval.zanella@linaro.org/

  • Carlos to review.
• [v4] linux: Fix ancillary 64-bit time timestamp conversion (BZ #28349, BZ #28350)

  • https://patchwork.sourceware.org/project/glibc/patch/20211222185239.1088511-1-adhemerval.zanella@linaro.org/

  • Florian to review.
• _FORTIFY_SOURCE fixes [BZ #28746]

  • https://patchwork.sourceware.org/project/glibc/list/?series=6132

  • Siddhesh to send out v3.
  • Adhemerval will review v3.
• [v5] posix: Add terminal control setting support for posix_spawn

  • https://patchwork.sourceware.org/project/glibc/patch/20211223173003.1037286-1-adhemerval.zanella@linaro.org/

  • Adhemerval: Would really like this to be released in 2.35.
  • Perfectly possible to use fork but it has issues for performance on Linux.
  • This removes the race condition.
  • Carlos to review.
• Multiple rtld-audit fixes

  • https://patchwork.sourceware.org/project/glibc/list/?series=6085

  • Adhemerval: Would be better to have this in 2.35.
  • Florian to look at the added audit version check.
  • Szabolcs to look at aarch64 audit changes.
• Desirable for the release?
  • Predictable ELF destructor ordering

    • https://patchwork.sourceware.org/project/glibc/list/?series=5764

    • Carlos to review for 2.35.
    • Suggestion from Adhemerval about ldd output being in constructor ordering.
      • Florian will review ldd output ordering.
• Go through atteendees to review patches they need review for.
• HJ: Working on DT_RELR for binutils 2.38 (end of January)
  • Compact relative relocations. Implemented for x86_64 and i386.
  • Outstanding issue is what symbol name do we want to identify the support.
  • CC'd libc-alpha mailing list with suggestion: GLIBC_ABI_DT_RELR.
    • Need a decision on symbol name for end of January for binutils 2.38.
    • Code freeze 22nd of this month.

Meeting: 2022-01-03 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

Meeting: 2021-12-27 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

Meeting: 2021-12-20 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

Meeting: 2021-12-13 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• Carlos asks what are we looking at putting into the release.
• Adhemerval:
  • Would like to get hugepage support into 2.35.
    • Siddhesh: Reviewed the last patch set.
    • Huge page support aligns with what we are doing with the text segment
      • Makes it easier to support THP with .text.
  • Carlos: What is the status of the LD_AUDIT series?
    • Adhemerval: Florian has reviewed half of the patches.
    • Florian: Ran out of time last week. I should be able to review this week.
• Florian:
  • _dl_find_object needs review.
    • Adhemerval: Will review.
  • Minor fix to rseq. No ABI implications.
    • Removes not-needed relocations.
  • Add membarrier support (syscall wrapper)
    • Need to do some digging for programming model.
    • May be an additional step required for membarrier setup.
• HJ:
  • libmvec patches are very large (42 patches)
    • Submit one function. Work with community to address concerns.
      • If the community agrees with the approach then the rest of the patches can be automated and ready.
    • Intent is to include the functions in 2.35.
  • DT_RUNPATH v3 posted.
    • Carlos to review v3.
• Sunil:
  • libmvec needs review.
    • CFI escape? Compiler generated. Encodes the DFA expression.
      • Can we remove it? What are they?
• Siddhesh:
  • No additional reviews required.
• Szabolcs:
  • No additional patches for review.
• Carlos:
  • pthread cond var fix and review. Bug fix.
  • Licked the cookie. MUST BE FIXED.
• Series 5728: Fix 64-bit time_t ancillary socket conversion
  • Posted v3. Needs review from Florian.
• Series 5725: [v4] elf: Add _dl_find_object function
  • Adhemerval to review.
• Series 5714: fix p_align on PT_LOAD segment in DSO isn't honored
  • Checked in.
• Series 2348: [v2] Use LFS and 64 bit time for installed programs (BZ #15333)
  • Needed.
• Discussed alignment issues with some architectures from the PT_LOAD changes.
  • Still need this if p_align is large.
  • We can solve this in the loader.
  • Once we fix hte linker the normal binary will have 4KiB p_align.
  • Will only get the large alignment if we have a variable that needs it.
  • For huge page we still want the large p_align even if there is no variables.
  • TODO: Must fix unmapping for 2.35.
• Series 5703: intl/plural.y: define macros to avoid multiple conflicting declarations of yyerror and yylex
  • Ask Arjun to review this change.
• Series 5688: [v3] Support target specific ALIGN for variable alignment test [BZ #28676]
  • Needs review.
• Series 5695: [v7] elf: Don't execute shared object directly [BZ #28453]
  • Needs review.
• What is the state of the OpenRISC or1k port?
  • Joseph has reviewed some of the state.
  • Is this ready for glibc 2.35?
  • State of the port is looking good.
  • Need to check build-many-glibcs and with Jospeh for the state of the port.
  • Doesn't look like there are any large blockers for 2.35.
• New ld sort algorithm is failing on OpenRISC.
  • Needs review.
• New ld sort default should be flipped to new algorithm.
  • Need to flip the new sort.

Meeting: 2021-12-06 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

Meeting: 2021-11-29 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

Meeting: 2021-11-22 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• 306 patches in NEW/NOBODY
• linux: Add generic ioctl implementation
  • Needed for ork1 port.
• linux: Add generic syscall implementation
  • Needed for ork1 port.
• Add GLIBC_PRIVATE_RELR as proxy for DT_RELR
  • What about toolchains that don't use symbol versions?
  • Why does *_RELR exist in the name?
  • Is Google going to rebase 2.27 to a more current version?
  • Ask Chrome OS developers?
  • Ask Google server branch developers?
  • What will be the generic ABI mechanism for feature lockout?
    • Does this impact the decisions we make?
    • The BSDs don't view it as an issue overall
      • Except that if we decide on this markup they may need to change things.
  • Context: Just glibc. Parallel feature lockout via symbol@version.
    • Add linker support for empty node.
    • Use dummy symbol in glibc.
  • Add concrete semantics to that version.

Meeting: 2021-11-15 @ 0900h EST5EDT

Cancelled.

Meeting: 2021-11-08 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• State NEW, Delegate Nobody, 289 patches.
• 47212: io: Refactor close_range and closefrom
  • Adhemerval: Pointed out a discrepancy for patch review and refactoring.
• 47211: powerpc: Define USE_PPC64_NOTOC iff compiler and linker also supports it
  • Adhemerval: These are further fixes for lld. Discussing with IBM.
• Discussed the current gABI issues that relate to ensuring that old ld.so
  • can safely fail when new binaries are run on old system.
  • Waiting for Cary Coutant to publish the gABI repo.
• Series 4535: Update Makefile fragments for DSO sorting tests
  • Florain: Needs changes. Post v2.
• 47161: Remove the unnecessary +make-deps
  • Needs review.
• 47065: powerpc: Use FLAG_ELF_LIBC6 for 32-bit known libraries
  • Needs review from Florian or Tulio.
• 47064: Fix memmove call in vfprintf-internal.c:group_number
  • Florian: Given that I produced the bug I should review it.
• 47061: [v3] x86: Optimize atomic_compare_and_exchange_[val|bool]_acq [BZ #28537]
  • Needs review.
  • Carlos: As x86 maintainer you could just wait and push. There is a lot of internal CPU details that this depends upon.
  • Carlos: We need a comment here to expalin the redundant read.
• 47048: [v2] elf: Use the minimal malloc on tunables_strdup
  • Siddhesh should review.

• https://sourceware.org/pipermail/libc-alpha/2021-November/132688.html

  • Florian needs review.
• Adhemerval: 64-bit time_t recvmsg review.
  • Florian: Assign that to me for review.
• Florian: May you please repost your LD_AUDIT patches? (to Adhemerval)

Meeting: 2021-11-01 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• State NEW, Delegate NOBODY, 317 patches.
• Series 4330: Enable EVEX strcmp
  • Carlos noted that with Noah and HJ working together this is enough review
    • along with the microbenchmark to commit this code.
• Series 4329: Fix lld build for armhf
  • HJ: Binary with ARM_EXIDX inbetween to relocation sections looks odd.
  • We fixed a similar issue with fa19d5c48a6b36518ce68412e3bdde6bfa8aa4a6.
  • Rewrite as cleanup and provide timing cycle information.
• Series 4318: [v1,1/5] string: Make tests birdirectional test-memcpy.c
  • HJ to review.
• Series 4294: benchtests: Add atan2f, expm1f, log1pf and tanf to bench-math (4 patches)
• Series 4292: benchtests: Add float version of math functions to bench-math (12 patches)
  • Convert double input to float input.
• Series 4208: regex: Unnest nested functions in regcomp.c
  • Carlos to look at it.

• 46656: [PiNg,9] localedata: add new locale ab_GE

• 46551: powerpc: Use the correct flag for 32-bit known libraries
  • Needs a new commit message.
• 46481: manual: remove an obsolete requirement on aligned_alloc() usage
  • Status? Suggest version the interface, return error without alignment.
• Enable hard-coded path in tests.
  • 46294: elf: Also try DT_RUNPATH for internal dlopen [BZ #28455]
  • 46295: Add and use link-test-modules-rpath-link [BZ #28455]
  • Carlos to review.
• Series 2732: malloc: Improve Huge Page support
• Maxim Kuvyrkov attended. Hello Maxim!
  • Linaro looking at benchmarking toolchains.
  • Patrick: libmicro has many mirobenchmarks.
  • Stream benchmarks were showing regression for the L3 memcpy issue we fixed before.
  • It is very important that you must find the patch that makes the change.
  • We are doing post-commit testing.
  • Pre-commit testing is the point where it is easiest to prevent a patch going.

Meeting: 2021-10-25 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• State NEW, Delegate NOBODY, 304 patches.
• Series 4140, Improve hypot(), Adhemerval to rebase.
  • Needs rebase after HJs benchtests/Makefile change.

• Series 4116, [v14,1/4] String: Add support for memcmpeq() ABI on all targets

  • Looks OK.
• Series 4105, powerpc: Use the correct flag for 32-bit known libraries
  • This looks like old a.out code is blocking this.

  • Carlos & Florian to bring up with IBM.

• Series 4101, Add missing math functions to benchtests (Sunil Pandey)
  • Adding functions to libmvec
  • Adding benchmark data for existing libmvec functions.
  • For each function we have both double and float.
  • In order to generate the microbenchmarks for float we need float and double.
  • Sunil copied the input from double to float.
• Series 4056, manual: remove an obsolete requirement on aligned_alloc() usage
  • Carlos to ask Martin Sebor.
  • Update 2021-10-25: Asked Martin for comment.

  • https://patchwork.sourceware.org/project/glibc/patch/33ec9e0c1e587813b90e8aa771c2c8e6e379dd48.camel@posteo.net/

• Series 3961, [PING,8] localedata: add new locale ab_GE
  • First issues: No feedback from Rafal and Mike (other locale maintainers).
  • Second issue: Consensus on header.
    • Carlos to respond on the second issue with new thread.

    • https://sourceware.org/pipermail/libc-alpha/2021-September/130692.html

• Series 3949, regex: fix buffer read overrun in search [BZ#28470]
  • Stuck on review. Needs v2. Marked Changes Requested given Andreas' comments.
• Series 3941, Benchtests: Improve bench-memcpy-random
  • Siddhesh to look at improvement and reveiw.
• Series 3895, [v2] elf: Support DT_RELR relative relocation format [BZ #27924]
  • Unknown dynamic tags are ignored by old glibc.
    • Old glibc must fail when running the new binary.
    • Need to add symbol dependency or bump ELF OSABI.
  • lld is there for review, but additional patches are needed to build glibc for tests.
    • More work is required in the tree to get lld working.
    • If the testsuite fails to build we can't talk about regressions easily.
    • What is the status of lld building with glibc?
      • Carlos: Ask on thread about status of lld to build glibc.
• HJ: Testing gcc with glibc 2.34 issues.
  • Sanitizer libraries has a hack that they assume dlsym malloc is always freed.
    • In glibc 2.34 in dlsym we do 2 mallocs and then do 2 frees. Hack doesn't work.

  • https://bugs.llvm.org/show_bug.cgi?id=52278

• Adhemerval: 64-bit time_t receive message fix.

  • https://patchwork.sourceware.org/project/glibc/list/?series=3280

Meeting: 2021-10-18 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• 339 NEW/NOBODY patches in queue.
• [v2] elf: Support DT_RELR relative relocation format [BZ #27924]
  • Need official gABI assignment (Carry Coutant status?)
    • Needs to be reserved verbally by Carry (Action item)
    • Required for patch acceptance.
  • Need BFD port or a released llvm/lld that supports DT_RELR to review patches.
    • Minimally a 32-bit and 64-bit port that can build glibc and passes regression testing.
    • Required for patch review.
  • Official glibc relese with feature?
    • Need BFD port to support build-many-glibcs to provide testing confidence.
  • HJ: Would like to see a BFD port before a glibc release with the features.
    • Sufficient to have a released binutils with DT_RELR also.
  • Release date of released static linkers with support should be close to glibc release with feature.
    • When glibc is released users can pick a static linker to test with (binutils or lld).
• 317 NEW/NOBODY patches in queue.
• Add and use link-test-modules-rpath-link [BZ #28455]
• elf: Also try DT_RUNPATH for internal dlopen [BZ #28455]
  • Needed for change in linker default.
• elf: Make global.out depend on reldepmod4.so [BZ #28457]
  • Existing defect.
• 46290: elf: Fix elf_get_dynamic_info() for bootstrap
  • Needs review.
• 45506: [v2] ld.so: Initialize bootstrap_map.l_ld_readonly [BZ #28340]
  • Needs review to fix valgrind issue with ppc64le.
• _FORTIFY_SOURCE=3 improvements
  • Siddhesh looking for review.
• Series 3601: Improve hypot()
  • Changes requested by Paul Zimmerman.

Meeting: 2021-10-11 @ 0900h EST5EDT

Cancelled.

Meeting: 2021-10-04 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• 398 patches in the NEW/NOBODY queue.
• H.J:
  • Most of the patches are in, there are 2 ones that need review:
    • Remove copy relocations: rquires linker, compiler, and runtime changes

      • https://patchwork.sourceware.org/project/glibc/patch/20210803215914.4170913-1-hjl.tools@gmail.com/ and https://patchwork.sourceware.org/project/glibc/patch/20210803215914.4170913-2-hjl.tools@gmail.com/

    • https://patchwork.sourceware.org/project/glibc/patch/20210924165338.2326917-2-hjl.tools@gmail.com/

      • Two issues: we don't need an API, we can use the syscall directly
      • Multiple users results in UB, so glibc acts as a gatekeeper
      • Hard to backport, so the patch adds no new symbol
      • Szabolcs is not sure if the rigth direction
        • Existing code which does something with pointer tag bits and make code incompatibility.
      • H.J pointed that a internal LAM branches fixes it.
        • These are independently of the LAM glibc API.
      • H.J will send examples on how to use this API
• Szabolcs:
  • elf: Avoid deadlock between pthread_create and ctors [BZ #28357]
  • Backport to 2.34
• We didn't make any patchwork evaluation.

Meeting: 2021-09-27 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• 365 patches in the NEW/NOBODY queue.
• 45462: [PING,5] localedata: add new locale ab_GE
  • Need to review the reviewers position on copyright/license text.

  • https://sourceware.org/pipermail/libc-alpha/2021-September/130692.html

• 45457: linux: Fix a non-constant expression in _Static_assert
  • Accepted. Under review from Paul Eggert.
• [v1,1/3] benchtests: Add medium cases and increase iters in bench-memset.c
  • Should be looked at by H.J. for correctness.
  • Results attached. Look OK.
• 45437 [powerpc] Fix unrecognized instruction errors with recent binutils
  • Is this a binutils bug?
  • Paul should commit. What review is he expecting?
• 45436 puts: indentation
  • Florian to follow up.
• 45434 pthread/tst-cancel28: Fix barrier re-init race condition
  • Adhemerval has this on his list.

• 45430 elf: Add libc_get_static_tls_bounds [BZ #16291]

  • Outstanding question from Florian.
• 45429 [v3] elf: Avoid nested functions in the loader (all ports) [BZ #27220]
  • Complexity looks reasonable.
  • No global data introduced.
  • Should we test on hppa and ia64?
  • Florian: Will test ia64.
  • Adhemerval: Will test hppa.
• Down to 344 patches.
• 45423 Disable symbol hack in libc_nonshared.a
  • Florian to ACK. Agreement that it looks correct.
• 45422 [v2] i386: Port elf_machine_{load_address,dynamic} from x86-64
  • Accepted by HJ
• 45407 powerpc: Delete unneeded ELF_MACHINE_BEFORE_RTLD_RELOC
  • Accepted by Rafael
• Break to review status from meeting participants.
• Siddhesh: Inputs to math benchmarks?
  • Paul Z. wanted to add an extra sets.
  • Agreed that workloads could be added.
• 44622 [v3] Fixed inaccuracy of j0f (BZ #28185)
  • Carlos to review.
• Patrick: Recommend that new users should login to patchwork because
  • you can't set your settings until you have a login.
  • Carlos will amend wiki with notes.
• Patrick: Is OFTC #glibc used for hte review meeting?
  • Carlos: It can be used if you want. It is the official IRC, and we will
    • discuss patch review via IRC.
• Florian: 3 things to raise.
  • opensock problem with the race.
    • 4 address families, and remove unused code.
  • recvmsg emulation code is non-functional for time64
    • systemd/qemu reported problem.
    • Some callers expect you can use a very tightly sized buffer.
    • Not enough space and this breaks.

    • See: https://sourceware.org/bugzilla/show_bug.cgi?id=28350

    • See: https://sourceware.org/bugzilla/show_bug.cgi?id=13500

  • getaddrinfo crash, dns related, reported on launchpad tracker.
    • There are no packet captures.
    • Looks like a real problem.

    • https://bugs.launchpad.net/bugs/1945072

• Adhemerval:
  • Audit patch set: Carlos: Yes, trying to get reviews.

  • sort: https://patchwork.sourceware.org/project/glibc/list/?series=2745

    • What do we do about malloc?
    • Change in algorithm?
      • Yes, the performance is different.
      • We use quicksort instead of mergesort.
      • Use intrasort to avoid quadratic worse case.

    • https://sourceware.org/pipermail/libc-alpha/2021-September/130698.html

  • Looking for comments on the series.

Meeting: 2021-09-20 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• 341 patches.
• This week is busy with GNU Tools track at LPC.
• The sysconf fixes are reviewed by Florian.
  • Change request for test case and commit.
  • Fedora 35 has downstream copy in testing.
• The pthread_kill ESRCH vs. old applications (steam)
  • Rebase before working on the sysconf changes.
• Carlos still looking at pthread cond var.
  • Next week due to LPC/GNU Tools track.
• Adhemerval reviewed dynamic loader patches.
  • Adhemerval reviewed RTLD_SHARED
  • Adhemerval reviewed loader optimization for shared libraries.
  • Carlos suggests sending Vivek and Chung-Lin direct email to discuss progress on patches.
• LD_AUDIT series is still up for review.
• Florian posted a fix for the aarch64 BTI build failure.

  • https://patchwork.sourceware.org/project/glibc/patch/87h7ef4hu2.fsf@oldenburg.str.redhat.com/

  • Szabolcs to review.
• Discussed extra build-many-glibc target for patch testing AArch64 + PAC + BTI.
• Review for pthread_mutexattr_*

  • https://patchwork.sourceware.org/project/glibc/patch/87fsuhydom.fsf@oldenburg.str.redhat.com/

  • Carlos to review.
• [v3] Fixed inaccuracy of j0f (BZ #28185)

  • https://patchwork.sourceware.org/project/glibc/patch/20210810071218.1585625-1-Paul.Zimmermann@inria.fr/

  • Put value of the input, which is wrong, and needs fixing.
• Locking issues with TLS and dlopen/pthread_create to sync TLS setup.

  • https://sourceware.org/bugzilla/show_bug.cgi?id=19329

  • Except in dlopen we need to take a lock around a particular area.
  • dlopen can exit on failure, and need to make sure lock is unlocked in failure.
    • Code for this is ugly.
  • Szabolcs to create new bug.
• HJ's r_debug patches started going into the branch.

Meeting: 2021-09-13 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• 334 patches to review.
• 44860 [v2] nptl: Fix type of pthread_mutexattr_getrobust_np, pthread_mutexattr_setrobust_np (bug 28036)
  • Carlos will review.
• Florian got feedback on pthread_cancel ESRCH changes.
  • Pushed to master and relase/2.34/master.
• sysconf revert patches?
  • Florian reviewed the patches.
  • Final patches have some issues in the test case.
  • The rest is OK.
• Siddhesh has a couple of locale realted patches that need review.
  • 44430 - v2 setlocale needs review (codonell)
  • 44607 - iconvconfig: Fix behaviour with --prefix [BZ #28199] (arjun)
• Carlos to raise locale source file header review with Mike Frysinger.
  • Attempt to get consensus on default headers to use.
• Szabolcs: No patches, but regression report of the latest glibc release has dynamic tls fixes.
  • Introduced a new lock in pthread_create.
  • Florian: Can we get a new lock?

  • The problem: Thread A: rtld lock -> user lock, B: user lock -> rtld lock. Deadlock.

  • https://sourceware.org/bugzilla/show_bug.cgi?id=15686

    • Fix for dyanmic tls bug fix regression.

  • Report: https://sourceware.org/bugzilla/show_bug.cgi?id=19329#c39

• Carlos is looking to review 2767

  • https://patchwork.sourceware.org/project/glibc/list/?series=2767

  • Reports that dotnet runtime is impacted by this on Linux.
• C.UTF-8 is in master, but will not be backported to release/2.34/master due to distribution impact.
  • Distros can either pick it up manually, or when 2.35 releases.
• RISC-V looking at extending this to 4K vector.
  • Arm SVE is passing down linked-list kind of structure that links up the extensions.
  • Kernel dumps register states on the stack.
  • Headers point to extra register state.
  • AT_MINSIGSTKSZ is used there.
  • Florian will post a note about the Arm SVE case as an example to review.
• 44926 - [v9,1/1] Extend struct r_debug to support multiple namespaces [BZ #15971]
  • Florian to review.
• 44924 - [v2] posix: Fix attribute access mode on getcwd [BZ #27476]
  • Florain to review (since he reviewed v1)
• 44897 - Add MADV_POPULATE_READ and MADV_POPULATE_WRITE from Linux 5.14 to bits/mman-linux.h

  • https://patchwork.sourceware.org/project/glibc/patch/alpine.DEB.2.22.394.2109072123510.1731265@digraph.polyomino.org.uk/

  • Asking DJ for review.
• 32-bit Arm atomic issue.

  • https://sourceware.org/pipermail/libc-help/2021-September/006004.html

  • Needs resolving in atomic.h.
• Stopped at 44884.

Meeting: 2021-09-06 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• Reviewed status of pthread_cancel ESRCH patches.
• Looked at sysconf revert. Adhemerval reviewing.

Meeting: 2021-08-30 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• 323 patches
• Source attribution cleanups, series 2700 (Carlos to review)
• [v2] setlocale: Fail if iconv module for charset is not present [BZ #27996]
  • Carlos to review.
• HJ would like to move forward with debug interface.
  • Current version is at v5.
  • Series 2714, Extend struct r_debug to support multiple namespaces
  • Noted various ld.so bugs:
    • Bug 27609.

      • https://patchwork.sourceware.org/project/glibc/patch/20210817184546.3330651-1-hjl.tools@gmail.com/

    • elf: Copy l_addr/l_ld when adding ld.so to a new namespace

      • https://patchwork.sourceware.org/project/glibc/list/?series=2675

  • All 4 patches should be reviewed by Florian.
  • Note: Do not want to change symbol size.
    • Used a symbol alias for this.
    • First: generate assembly code, include assembly, place symbol alias after.
• Do we have a template for new locales not under FSF copyright assignment?
  • Carlos: No we don't have a template. Please propose something.
• Some rtld-audit fixes (Series 2577)

  • https://patchwork.sourceware.org/project/glibc/list/?series=2577

  • Looking for reviewer.
  • v3 has the aarch64 fixes.
    • Ask Szabolcs to review aarch64.
• glibc 2.34 suffers a spurious fail for pthread_cancel/pthread_kill
  • Looking at two approaches:
    • Adhemerval's fixes for glibc 2.35
    • Florian's approach for backporting to 2.34
  • What about the pthread self-kill signal case.
    • Can the kernel deliver all the signals?
    • Why is this required?
• C.UTF-8 v8 - Carlos to review.
  • Cleanup the files to make them iso-8859-1/iso-8859-15
    • Use octal sequences

• Review: https://patchwork.sourceware.org/project/glibc/list/?series=2728

• Alredy reviewed: https://patchwork.sourceware.org/project/glibc/list/?series=2723

  • Push?

• Under review: https://patchwork.sourceware.org/project/glibc/list/?series=2716

  • HJ looking to RCA. First 4 patches OK.

• Is this needed? https://patchwork.sourceware.org/project/glibc/list/?series=2713

• Needs v4 - https://patchwork.sourceware.org/project/glibc/list/?series=2711

• Stopping at 44766

Meeting: 2021-08-23 @ 0900h EST5EDT

Cancelled.

Meeting: 2021-08-16 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• Upstream patches 271 (New/Nobody)
• Namespace patch with glibc/binutils/gdb and r_debug
  • Discussed keeping r_debug and bumping r_version
  • Export r_debug_v2 only via DT_DEBUG
• v2 version of glibc change for removing COPY relocations.

  • https://patchwork.sourceware.org/project/glibc/list/?series=2595

  • Needs review from the community.
• Add THP support in glibc's malloc

  • https://patchwork.sourceware.org/project/glibc/list/?series=2660

    • Looking for architecture feedback.
    • Looking for Distro feedback.
  • Enable madvise syscall after successful mmap
  • How do we get feedback from the workload testing?
  • What about MADV_FREE?
    • Not reviewed, but could be related.
  • What about heap_trim? malloc_trim?
  • Can have a follow-up patch to directly use huge pages.
  • On some workloads THP needs a lot of tuning.
    • Can we provide a THP pool?
    • Be as simple as possible.

Meeting: 2021-08-09 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• Upstream patches 298
• LD_AUDIT patches still need review.

  • https://patchwork.sourceware.org/project/glibc/list/?series=2577

  • 20 patches.
• NPTL fixes.

  • https://patchwork.sourceware.org/project/glibc/list/?series=2304

  • 5 patches.
• Debug Interface Problems.
  • Do not support _r_debug in executable.
  • Use DT_DEBUG in dynamic executable.
  • In static executable use _r_debug.
    • Need to remove dlopen from static exectuable
    • Would need to remove dlopen.
    • New dynamic tag.
      • DT_DEBUG_DLMOPEN
        • glibc should not support dlopen in static exectuables
          • Therefore _r_debug should not be there.
        • Have you reviewed Daniel Walker's fixes?

          • https://patchwork.sourceware.org/project/glibc/list/?series=550

          • Ask for direct feedback on MIPS vs other status.

Meeting: 2021-08-02 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• 304 patches to review.
• Audit patch series needs review:

  • https://patchwork.sourceware.org/project/glibc/list/?series=2577

• CI/CD failure that is expected to work:

  • https://patchwork.sourceware.org/project/glibc/patch/20210719163846.2954193-5-adhemerval.zanella@linaro.org/

  • Carlos to raise with DJ.
• 64-bit time_t tests improvements

  • https://patchwork.sourceware.org/project/glibc/list/?series=2510

• Next release for glibc 2.35 - Two big items.
  • Removal of _r_debug
    • _r_debug exported for debugging (Bug 28132)

      • https://patchwork.sourceware.org/project/glibc/list/?series=2585

      • Do we want to support _r_debug in static exectuables?
  • Proposal how to fix things to fix dlmopen debugging.
  • Deprecate COPY relocation?
    • Provide option to deprecate COPY relocation.
    • Issue open at the ABI level.
      • Security and performance concerns.
    • Changes for binutils, gcc, and glibc.
• Also we have dlmopen with RTLD_SHARED

  • https://patchwork.sourceware.org/project/glibc/list/?series=2451

• Copy benchtests to different machines

  • https://patchwork.sourceware.org/project/glibc/list/?series=2579

  • Carlos to review.
• Need LAM / TBI review to get API consensus
  • HJ to post for review. Florian raised issues for review.
  • Tag bits in the middle?
• Paul Zimmerman to review accuracy of glibc against other libraries.
  • Update to report for glibc 2.34.
• Static analysis fixes

  • https://patchwork.sourceware.org/project/glibc/list/?series=2555

  • Arjn to review.
• Carlos to review these for Siddhesh:

  • https://patchwork.sourceware.org/project/glibc/patch/20210722132430.1945153-1-siddhesh@sourceware.org/

  • https://patchwork.sourceware.org/project/glibc/patch/20210722132457.1945231-1-siddhesh@sourceware.org/

  • https://patchwork.sourceware.org/project/glibc/patch/20210720023658.1278155-1-siddhesh@sourceware.org/

• In glibc 2.35 Szabolcs will look at MTE and tagged address things.
  • I don't think it will be easy.
  • Some problems exist with arbitrary tagged address.
  • Pointers with arbitrary tags.
  • Pointer comparison doesn't work.
    • Pointer equality fails. User error?
  • Will start discussion upstream.
  • Will look at additional hardware features related to longjmp
    • Data versioning with notes.
• Robbie Harwood's patch needs committing

  • https://patchwork.sourceware.org/project/glibc/patch/20210728182331.1363922-1-rharwood@redhat.com/

Meeting: 2021-07-26 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• 272 patches.
• Short meeting review of the oustanding patches to review.

• https://patchwork.sourceware.org/project/glibc/list/?series=2507

  • Of the audit fixes we want to review one.
  • Regression: elf: Fix audit regression
  • Florian: Will review first patch in series.
• Florian to review C.UTF-8 patches
  • Preparing for another mass rebuild with the patches.
• HJ discovered performance regression on memmove/memcpy
  • Regression in workaround for Skylake server.
  • Implemented a fix. Fix is tested
  • Patches sent out this morning:

    • https://patchwork.sourceware.org/project/glibc/list/?series=2549

    • Carlos to look at this patch.
• Siddhesh says all his patches can wait.
• Carlos to review manual change:

  • https://patchwork.sourceware.org/project/glibc/patch/20210722162733.3185678-1-siddhesh@sourceware.org/

Meeting: 2021-07-19 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• Starting review 311 patches.
• malloc debug hooks break very old emacs binaries that rely on malloc hooks.
• New patches required to migrate malloc_set_state / malloc_get_state
• Siddhesh to work on v10 with reallocarray not included in list of interposers.
  • New v10 has malloc_*_state
• rt/tst-cpuclock2-time64 still unreliable
  • Want to clean this up for 32-bit trybot.
• Florian: Should we add the 64-bit time_t aliases?
  • 8 new symbols.
  • Don't want setsockopt and getsockopt
    • 6 new symbols.
    • Carlos: How many patches?

      • https://patchwork.sourceware.org/project/glibc/list/?series=2415

• ABI:
  • Additional time64 aliases --- Adhemerval to review.
  • v10 malloc hook removal --- Carlos to reivew.
  • x86.h addition --- Carlos to review for HJ.
• non-ABI:
  • C.UTF-8 --- What about U+FFFD? Can we use it for default_missing?
  • Adhemerval: 4 patches just fixes for the testsuite.

    • https://patchwork.sourceware.org/project/glibc/list/?series=2423 --- New version.

    • https://patchwork.sourceware.org/project/glibc/list/?series=2489

    • https://patchwork.sourceware.org/project/glibc/list/?series=2490

Meeting: 2021-07-12 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

• Limits patch (Carlos will review)

  • https://patchwork.sourceware.org/project/glibc/patch/87a6mrbsnk.fsf@oldenburg.str.redhat.com/

• x86.h patch:
  • Old patch:

    • https://patchwork.sourceware.org/project/glibc/patch/20210605135947.469959-1-hjl.tools@gmail.com/

  • New patch:

    • https://patchwork.sourceware.org/project/glibc/patch/20210605135947.469959-1-hjl.tools@gmail.com/

• lld fix is updated then it helps the most users
  • For static pie.
  • Users matter downstream.
  • Carlos will follow up with comment on patch.
• LD_AUDIT for Rice?
  • Review?
• Opt-out of security features
  • Avoiding loading PIE objects.
  • Vtable hardening.
  • Discussion on viability of patches to implement such a request.
• Need internal clone, clone2, clone3 review (Adhemerval will review).

Meeting: 2021-07-05 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

Meeting: 2021-07-05 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

Review new patches and restart review at the top.

No notes for this meeting.

Meeting: 2021-06-28 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

• Started with 243 patches.
• libanl needs review as pthread using library.
• C.UTF-8
  • Push this week.
• 43936 libanl merge
  • Review.
• Adhemerval to review libutil merge.
• Florian may need to repost libnss_files/libnss_dns.
  • Moving libresolv is an ABI change.
  • Moving the libnss_* change is important.
• clone3 review for HJ required.
• Carlos to review:

  • https://patchwork.sourceware.org/project/glibc/list/?series=2345

  • https://patchwork.sourceware.org/project/glibc/list/?series=2252

• Adhemerval to review:

  • https://patchwork.sourceware.org/project/glibc/list/?series=2363

• Siddhesh:
  • DJ reviewing malloc hooks removal for v2.
    • There is one ABI change that needs to happen.
      • Needs review.
      • Need to review this before release.

      • Series: https://patchwork.sourceware.org/project/glibc/list/?series=2369

Meeting: 2021-06-21 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

• glibc 286 patches.
• Lukasz has reviewed 64-bit time_t changes.
  • Adhemerval has review delegated.

  • https://sourceware.org/pipermail/libc-alpha/2021-June/127727.html

    • Additional fixes for release.
    • Avoids CRIO problems and optimizes usage when run on older kernels.
• SA_RESTART change can be pushed later this week.
  • Should switch to SA_ONSTACK also? Want to write a test case.
  • SA_RESTART does not fix bz for longstanding cancellation issues, but it helps.
• ABI changes in flight:
  • librt review.
  • libanl to move into libc.
  • libnss_files
  • libnss_dns
  • libutil to move into libc.
• Florian wants to Review ABI before release.
• HJ is looking at desirable for release "clone3 wrappers":

  • https://patchwork.sourceware.org/project/glibc/list/?series=2252

  • PTHREAD_STAKC_MIN fixes to mirror existing dyanmic stack size fixes.
    • HJ to work on this patch for consistency.
• gconv split out with .d directories.
  • Ask DJ to review gconv split out .d patches?
• HWCAP things on Arm had a regression in glibc 2.34
  • This is for july when the ABI is frozen and look at regression bugs.

  • https://sourceware.org/bugzilla/show_bug.cgi?id=27046

• Carlos needs to review:

  • https://patchwork.sourceware.org/project/glibc/patch/20210125135619.9695-1-hjl.tools@gmail.com/

  • https://patchwork.sourceware.org/project/glibc/patch/20210202145733.545020-1-hjl.tools@gmail.com/

• Need review for Florian:

  • https://sourceware.org/pipermail/libc-alpha/2021-June/127727.html

  • https://sourceware.org/pipermail/libc-alpha/2021-June/127727.html

• Carlos to post to libc-alpha that wiki page is up for blockers.
• Review starting at 43935
  • libutil (small, self contained)

    • https://patchwork.sourceware.org/project/glibc/list/?series=2334

There was a quick discussion about the public LD_AUDIT request after patch review.

• Any detailed discussions need to happen on libc-alpha in public.
• Discussion of the Rice University post here:

  • https://sourceware.org/pipermail/libc-alpha/2021-June/127636.html

• LD_BIND_NOW? How do we handle it.
  • Input how to fix it?
  • We need BIND_NOW to have something to work with SVE.
    • What do we do with SVE
  • NEON support broken on aarch64.
    • Bump the LA_VERSION on Arm64
    • Only supports version 2.
    • Don't know how to fix this for SVE.
    • Fixing it for NEON and SVE both is desirable.
  • Is BIND_NOW just an optimization or semantic issue?
  • Bump LAV_CURRENT to 2 on all arches.
    • Start calling la_symbind in more cases.
    • Call early in the bind now scenario.
      • For the bindnow call sites we add a check afterwards that the callback didn't request PLT_ENTER/PLT_EXIT hooks.
      • If the hook check fails, we report a dlsym error or relocation error.
      • Can we call the callbacks that early?
      • Yes, already relocated.

Meeting: 2021-06-14 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on OFTC.

• Start at 291 patches.
• 64-bit time_t patch review.
• Process startup issue. Seen by HPC developers.
• Question: Related to unversioned library installs.

  • https://patchwork.sourceware.org/project/glibc/list/?series=2293

  • Haven't talked to Gentoo/Debian yet.
• Start review at 43844

• https://patchwork.sourceware.org/project/glibc/patch/87czsook3i.fsf@oldenburg.str.redhat.com/

• Carlos to create 2.34 release page. (Now done).
• Patchwork has a way to update only one patch?

  • https://patchwork.sourceware.org/project/glibc/patch/20210611172439.248245-1-adhemerval.zanella@linaro.org/

• Follow up upstream with patchwork to post a new patch, just one v3 for a series.

• https://patchwork.sourceware.org/project/glibc/list/?series=2304

  • Change how pthreads synchronize with detach/join
  • No ABI, can review into July.
• ABI changes:
  • Add close_range support on Linux
  • New POSIX spawn file actions.
  • _Fork() from POSIX.
• 43819 gconv configuration parsing cleanups. No ABI change.
• glibc 2.34 release blocker tracking bug.
  • Link release page to tree-view.
• 43815 - Consolidated by Siddhesh working on them.
• 43803 - Do not install shared objects under versioned.

Meeting: 2021-06-07 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on freenode.

Review new patches and restart review at the top.

Notes:

• HJ raised upstream issue requiring review: Removal of COPY relocs.
  • Brief discussion about COPY relocs and where the conversation will take place.
  • Carlos suggested we need an upstream meeting between ABI stakeholders.
  • Discussed status of Non-PIE issues briefly and directed conversation to upstream.
• Discussed briefly GNU Tools Cauldron at LPC since this would present an opportunity to discuss feature and patch status.
• Suggested DJ review the iconv split up patches from Siddhesh.
• Adhemerval noted that for 43664 that Szabolcs may want to double check compiler builtin status for Wilco's patch.
• Stopped at 43663 which will be reviewed by Florian since v4 fixes all previous comments.

Meeting: 2021-05-31 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on freenode.

Review new patches and restart review at the top.

Notes:

• Start at top of list.
• Current patch queue count is 251 patches needding review.
• Started at 43622.
• Series 2241 "nptl: pthread cancellation refactor" needs review before the thread stack changes can get reworked.
• 43558 to review for Carlos.
• 43497, Carlos to ask Jonathan Wakely to repost v3 with a different title and the same in-reply-to chain.
• 43389, 33388 - Superseded and removed.
• Stopped at 41649. 197 patches in queue.

Meeting: 2021-05-24 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on freenode.

Review new patches and restart review at the top.

Notes:

• Reviewed libpthread merge. Two patches left to review. Adhemerval doing review.
• Two x86 patches for misaligned stack need review.
• 64-bit time_t patches should be reviewed this week, 25 patches + 1 patch for header export.
• One RFC patch for slow tls after dlopen, should review before 2.34 release.
• Two 64-bit time_t test patches need possible rework, to look at after 64_bit time_t changes.

Meeting: 2021-05-17 @ 0900h EST5EDT

Video/Audio: https://bluejeans.com/9093064454

IRC: #glibc on freenode.

Review new patches and restart review at the top.

Notes:

• 308 patches in queue.
• Meeting URL has changed to avoid Jitsi server compatibility issues.
• Lukasz's task is to provided y2038 support for 32-bit processors.
• Last year some customers were now requiring to have a y2038 solution.
• Some system designs are so ahead they need to support y2038.
• Some problems with the refactor with ABI symbols.
• Newest version is compilable is on github.

  • https://github.com/lmajewski/y2038_glibc

  • https://github.com/lmajewski/y2038_glibc/commits/y2038_edge

• Not fixing utmp.
• Review Adhemerval's changes.
• 7 weeks.
  • 20 patches for 64-bit time_t
  • 8 patches for libpthread removal.
    • Deal with librt removal.
    • Some impact on timer handling.

Meeting: 2021-05-10 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Review new patches and restart review at the top.

Notes:

• Total of 313 patches in the queue (new/nobody).
• Discussed making 64-bit time_t (15-16 patches) a serious 2.34 feature.
• Reviewed up to patch 42023.
• Paul would like a quick discussion about the match component.

Meeting: 2021-05-03 @ 0900h EST5EDT

Cancelled.

Meeting: 2021-04-26 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Notes:

• 333 - state new, delegate nobody.
• 43062 execveat system call wrapper.
• 42197 fix italian locale capitalization
• Paul wants to discuss plans for math component.
• Szabolcs - reviews complete for dynamic tls fixes, need comment reviews.

Meeting: 2021-04-19 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Meeting: 2021-04-12 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Review new patches and restart review at 42561.

Meeting: 2021-04-05 @ 0900h EST5EDT

Cancelled.

Meeting: 2021-03-29 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Review new patches and restart review at 42244.

Notes:

• Total patch list is at 388 patches, so approximately +30 patches from last time.
• Discussed libpthread merge and the status of the patches. Moving forward well.

Meeting: 2021-03-22 @ 0900h EST5EDT

Cancelled.

Meeting: 2021-03-15 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Review new patches and restart review at 42023.

Notes:

• 352 patches in State=New Delegate=Nobody.
• Discussed auto-posting committed patches to list using patchwork to drive that.

• Asked DJ to look at https://patchwork.sourceware.org/project/glibc/list/?series=1817

• 42282 - 42334: Any review? Some sparse review. Settle the filesystem 2038 issue? On disk file format issue for out-of-range dates. Repost test as a distinct series.
• 42277 - Review x86-64 Avoid RTM abort inside a RTM region. Still need reviewer.
• 42244. Stopped here.

Meeting: 2021-03-08 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Review new patches and restart review at 41481.

Notes:

• Manually review State=Under Review, Delegate=Nobody (15 patches).
• Reviewed the patches without delegate and moved back to NEW for review.
• 356 patches in queue for current state=new delegate=nobody.
• 15 patches were delegate=nobody state=under review.
• Stopped at 42023 "powerpc: backtrace cleanup."

• Guest made a comment "https://sourceware.org/glibc/wiki/ThreadPropertiesAPI" which sparked a discussion about TLS issues that need fixing.

Meeting: 2021-03-01 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Review new patches and restart review at 41785.

Notes:

• IBM: Talk to Tulio and Paul Murphy about patch review for IBM.
• Changes requested for 41576, 41572 - Raise with Fangrui Song discuss coverage and what support looks like.
• Stopped at 41481.

Meeting: 2021-02-22 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Review new patches and restart review at 41628.

Notes:

• No reviewers found for TLS P&C issues yet, review next week.

• Next week the 64-bit time_t patches for final support will be posted for review.

Meeting: 2021-02-15 @ 0900h EST5EDT

Cancelled.

Meeting: 2021-02-08 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Notes:

• Patch queue was 430 patches at the start of the meeting.
• Closed the meeting with 412 patches. Up +5 patches from last week after review.
• Ended meeting at 41628 (though Szabolcs was going to close that patch).

Meeting: 2021-02-01 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Notes:

• All release blockers reviewed. There were none left.
• Started reviewing the queue.
• Stopped at patch ID 41767. There were 407 patches left in the queue.

Meeting: 2021-01-25 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Will review release blocker patches and desirable patches.

Notes:

• All release blockers reviewed.
• Decided that hard freeze should happen.
• Moving SIGSTKSZ issues to be pushed as soon as 2.34 opens. Needs warning NEWS item for upcoming deprecation.
• Discussed upcoming features.

Meeting: 2021-01-18 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Will review release blocker patches and desirable patches.

Notes:

• Moved DSO sorting to glibc 2.34 to resolve issues with dual sorting transition.
• Carlos to review _SC_MINSIGSTKSZ and deprecate _SC_SIGSTKSZ_SOURCE.
• Florian to review COMMON_CPUID_INDEX_MAX handshake.
• Florian to review Fix getdents64 fallback on mips64-n32
• Fix static PIE ifunc resolvers.
• Adhemerval to email HJ and Szabolcs on status.
• Florian Weimer to ping Martin Sebor on PR/98512.
• Florian to raise issue around check for x86 ISA v2 and v3 and virt.
• What about deadlines for the release?
  • Keep only those issues that are ABI regressions.
• How do we shorten the release window?
  • Discuss on libc-alpha with new thread topic.

Meeting: 2021-01-11 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Will review release blocker patches and desirable patches.

Notes:

• Hosted by Siddhesh
• Adhemerval to run the call as Release Manager
• AArch64 static PIE needs a fix and patch reviewer should be found.

• Adhemerval Notes

Meeting: 2021-01-04 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

Queue is clean and we are reviewing patches.

Notes:

• Currently reviewing patches starting at patch ID 41361
• Last meeting NEW queue size was 424 patches.
• Reviewed patches for the release.
  • Release blockers patchwork patch review complete.
  • Desirable for release pathcwork patch review complete.
  • All patches have delegates for review or are in progress.
• Marking LibOS RFC patchwork patches as dropped
  • GNU Cauldron 2019 feedback was the patches need updating.
  • LibOS implementation needs reworking to be supported by upstream.
  • Marking all patches as dropped.
• Build failures with gcc 11 / glibc mainline need fixing before release.

Meeting: 2020-12-14 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

The goal is to review the entire queue for a clean and manageable state before returning to start tackling the patches. This ensures that we can run automation across the queue without worrying about patches that don't need to be handled, and to make people aware of our efforts.

Notes:

• Currently at patch ID 33866, with only 558 patches in the queue.
• We are 72% of the way through the full patch queue.
• After we walk the full queue once doing cleanup we'll start handling patch-by-patch review.
• Completed the full queue review.
• Started working the patch queue from the top of the list downward. Made it to patch ID 41361.

Meeting: 2020-12-07 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

The goal is to review the entire queue for a clean and manageable state before returning to start tackling the patches. This ensures that we can run automation across the queue without worrying about patches that don't need to be handled, and to make people aware of our efforts.

Notes:

• Currently at patch ID 40823, which is 10% of the way through the patch list.
• Estimate that in 9 more meetings we get through the queue. Completion on 2021-02-15 (after glibc 2.33 release).
• No meetings on December 21st, or December 28th, taking two weeks off for the holidays.
• Ended meeting at patch ID 33866.

Meeting: 2020-11-30 @ 0900h EST5EDT

Video/Audio: https://jitsi.member.fsf.org/glibc-patch-review

IRC: #glibc on freenode.

The initial goal of several of the first meetings is to get the patchwork queue down to a clean manageable state. Along with writing automation to keep it in a clean state.

Notes:

• Added "Patch IDs" to the list of things to enable in patchwork for review.
• Discussed the process and the problems with using patchwork.
• Started reviewing the list of patches and got the queue down to 1239.
• Process is to skip patches from people who are known to actively maintain their queue.
• Dug deeper into queues from submitters with fewer patches or people who are not expected to be deeply involved in the process.
• Delegated some patches to reviewers that were on the call.
• Ended meeting at patch ID 40823, reviewed 125 of 1239 (10% review).
• Discussed patch sending problems and issues with patches sent to the list which are not understood by patchwork.

• Arjun noted git imap-send to send patches to your imap drafts folder and then send them on to the list e.g. git format-patch --stdout HEAD^ | git imap-send