Page MenuHomeFreeBSD
Differential D22526

[PowerPC] Fix floating point environment manipulation
ClosedPublic
Actions

Authored by bdragon on Nov 24 2019, 12:56 AM.
Tags
Referenced Files
F168382094: D22526.id.diff
Thu, Aug 27, 9:45 PM
F168304359: D22526.id64789.diff
Thu, Aug 27, 1:14 PM
F168294978: D22526.id65518.diff
Thu, Aug 27, 11:37 AM
F168287856: D22526.id65517.diff
Thu, Aug 27, 10:31 AM
Unknown Object (File)
Wed, Aug 26, 5:56 PM
Unknown Object (File)
Wed, Aug 26, 6:23 AM
Unknown Object (File)
Tue, Aug 25, 11:38 AM
Unknown Object (File)
Tue, Aug 25, 11:33 AM
View All Files
Subscribers
alfredo
andrew
emaste
imp
luporl

Details

Reviewers
jhibbits
Commits
rGd905133e26d0: [PowerPC] Fix SPE floating point environment manipulation
rS355656: [PowerPC] Fix SPE floating point environment manipulation
Summary
  • Fix endianness problems in functions accessing and manipulating the floating point environment via fenv.h. (jhibbits wrote the original bits)
  • Fix bugs regarding synchronization requirements after writing SPEFSCR.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

bdragon created this revision.Nov 24 2019, 12:56 AM
Herald added a subscriber: imp. · View Herald TranscriptNov 24 2019, 12:56 AM
jhibbits added a comment.Nov 24 2019, 1:00 AM

You found the bugs with the kyua tests, did you retest post-changes?

lib/msun/powerpc/fenv.h
94–95 ↗(On Diff #64789)

I think these need line wrapped.

bdragon updated this revision to Diff 65517.Dec 12 2019, 12:31 AM

Linewrap fenv.h macros.

Herald added a reviewer: manu. · View Herald TranscriptDec 12 2019, 12:31 AM
Herald added subscribers: alfredo, luporl, emaste, andrew. · View Herald Transcript
bdragon added a comment.Dec 12 2019, 12:33 AM

ugh, think that was the wrong diff. Moment.

bdragon updated this revision to Diff 65518.Dec 12 2019, 12:35 AM

Actually upload the *correct* diff this time

bdragon removed a reviewer: manu.Dec 12 2019, 12:36 AM
bdragon marked an inline comment as done.
jhibbits accepted this revision.Dec 12 2019, 4:47 PM
This revision is now accepted and ready to land.Dec 12 2019, 4:47 PM
Closed by commit rS355656: [PowerPC] Fix SPE floating point environment manipulation (authored by bdragon). · Explain WhyDec 12 2019, 5:12 PM
This revision was automatically updated to reflect the committed changes.
bdragon added a commit: rS355656: [PowerPC] Fix SPE floating point environment manipulation.
bdragon added a commit: rGd905133e26d0: [PowerPC] Fix SPE floating point environment manipulation.Mar 8 2021, 7:08 PM

Revision Contents
Changeset List

PathSize
sys/
arm/
arm/
7 lines
arm64/
arm64/
7 lines
conf/
3 lines
i386/
i386/
7 lines
kern/
2 lines
powerpc/
powerpc/
56 lines
7 lines
riscv/
riscv/
7 lines
sparc64/
sparc64/
7 lines
sys/
4 lines

Diff 65517

View Options

sys/arm/arm/elf_machdep.c

Loading...
View Options

sys/arm64/arm64/elf_machdep.c

Loading...
View Options

sys/conf/kmod.mk

Loading...
View Options

sys/i386/i386/elf_machdep.c

Loading...
View Options

sys/kern/link_elf.c

Loading...
View Options

sys/powerpc/powerpc/elf32_machdep.c

Loading...
View Options

sys/powerpc/powerpc/elf64_machdep.c

Loading...
View Options

sys/riscv/riscv/elf_machdep.c

Loading...
View Options

sys/sparc64/sparc64/elf_machdep.c

Loading...
View Options

sys/sys/linker.h

Loading...