Opened 11 years ago
Closed 10 years ago
#9962 closed bug (fixed)
pow returns nan instead of inf for large results
Reported by: | markh | Owned by: | jessicah |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | System/libroot.so | Version: | R1/Development |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Platform: | All |
Description
When calculating large results with pow, it returns nan instead of inf. This causes one check in postgresql to fail.
For example: pow(-1004.3, 1e200) returns nan on Haiku, while inf is expected.
I created a small test program that shows the behaviour and attached it to this ticket.
Ingo told me that the problem is probably in glibc, but I'm not sure which component to select from the list.
Attachments (2)
Change History (11)
by , 11 years ago
comment:1 by , 11 years ago
Component: | - General → System/libroot.so |
---|---|
Owner: | changed from | to
comment:2 by , 11 years ago
axeld: Do you have time to look at this? I think this is the last issue I need to get fixed before I can attempt to get my changes in the Postgresql tree. If they accept the changes, postgresql could be built from the official source.
comment:3 by , 10 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
by , 10 years ago
Attachment: | 0001-libroot.so-update-glibc-s-e_pow.S-on-x86.-Fixes-9962.patch added |
---|
comment:4 by , 10 years ago
patch: | 0 → 1 |
---|
comment:5 by , 10 years ago
We should probably update a few other files (for powf and powl), x86_64 is also likely affected. Giving the exact source revision in the commit message would be nice (or simply glibc 2.19 if that's the source tag)
comment:6 by , 10 years ago
Could we just update the entire math section of glibc? I know we use an old glibc for gcc2 reasons, so don't know if that's feasible or not.
comment:7 by , 10 years ago
The reason we use an old glibc is not gcc2, but binary compatibility. The version that was used in BeOS unfortunately leaked some symbols that are not part of the official API, so we must stay compatible with those. So, updating the glibc must be done only with great care. We used to have a port of the abi-compliance-checker, which is the tool used by the glibc project to test for such changes. It should be possible to run it (even from another OS, I think the tool works at source level) to check such changes.
There is also a risk of the changes to the math part requiring updates to other parts of the lib, and so on.
IIRC, however, the x86_64 version of glibc actually uses assembler files from a newer glibc version, as our old one didn't have any x86_64 support. Looking at the commit logs for that may provide some hints on what's possible, but the x86_64 version didn't have to retain the binary compatibility.
comment:8 by , 10 years ago
As mentioned on the haiku-development mailing list, pow now returns inf as expected and the Postgresql "make check" tests now run without errors. I will do some more testing in the next few days to get a better idea if it works correctly.
Small example of pow returning nan instead of the expected inf