Pre-release review of glibc 2.17: 0 P1 bugs, 14 P2 bugs, 0 P3 bugs.

["Carlos O'Donell" <carlos at systemhalted dot org>]

Dave,

I've gone through the bugs marked with milestone 2.17.

There are 14 P2 bugs in the tracker with milestone 2.17.

Could you please comment on the list?

The following 2 issues should go in immediately:

"Document __chk_fail ABI in test case"
http://sourceware.org/bugzilla/show_bug.cgi?id=14197
- Comment fixup should go in immediately we delayed it last time
- It's less of an issue if this goes in ASAP.

"hppa: Fix two integer to pointer warnings."
http://sourceware.org/bugzilla/show_bug.cgi?id=6677
- Testing this fix now.
- Restricted to hppa.

The following bug was something Roland said was "unacceptable" and
should be fixed.

"_{POSIX_V7|_POSIX_V6|XBS5}_ILP32_OFFBIG are wrong for -mx32 build"
http://sourceware.org/bugzilla/show_bug.cgi?id=14200
- No patch and no review.
- This is too subtle for me to understand.

The other 11 should have their milestone bumped to indicate
we are not accepting them for 2.17:

"QoI regression: strstr() slowed from O(n) to O(n^2) on SSE4 machines"
http://sourceware.org/bugzilla/show_bug.cgi?id=12100
- Patch exists.
- IMO this is too risky to go in for 2.17.

"fclose violates POSIX 2008 on seekable input streams"
http://sourceware.org/bugzilla/show_bug.cgi?id=12724
- No patch exists, and too risky.

"assertion error in res_query.c"
http://sourceware.org/bugzilla/show_bug.cgi?id=13013
- Patch exists.
- Insufficient review.

"Building libc.pot has a lot of warnings."
http://sourceware.org/bugzilla/show_bug.cgi?id=13853
- Patches exist but need serious review.
- Risky to change these scripts without more time.

"Malloc can deadlock in retry paths"
http://sourceware.org/bugzilla/show_bug.cgi?id=13939
- Insufficient review.
- Risky.

"Cleanup reversal of sunrpc obsoletion code"
http://sourceware.org/bugzilla/show_bug.cgi?id=14084
- Low priority.
- Hack working well.

"Clean up shlib-versions"
http://sourceware.org/bugzilla/show_bug.cgi?id=14171
- Sweeping change not valid for freeze.

"--localedir arg to configure is ignored"
http://sourceware.org/bugzilla/show_bug.cgi?id=14259
- No update from patch author.

"fprintf() function is multithread-unsafe"
http://sourceware.org/bugzilla/show_bug.cgi?id=14267
- Not enough review to go into 2.17.
- This is an honest to goodness real problem, but we need more serious review.

"Removal of sysdeps/x86_64/fpu/s_sincos.S causes regressions"
http://sourceware.org/bugzilla/show_bug.cgi?id=14412
- No patches.
- Real problem though.

"power: memset is broken on powerpc 405"
http://sourceware.org/bugzilla/show_bug.cgi?id=14595
- Patch on a branch.
- No review or testing yet.

Cheers,
Carlos.