Differential D27453

[PowerPC] Floating-point exception trap followup
ClosedPublic
Actions

Authored by bdragon on Dec 3 2020, 1:44 AM.

Tags

Referenced Files

	F163105802: D27453.diff
	Mon, Jul 20, 3:19 AM

	F163086131: D27453.diff
	Sun, Jul 19, 11:10 PM

	Unknown Object (File)
	Sat, Jul 18, 8:59 AM

	Unknown Object (File)
	Fri, Jul 17, 12:25 PM

	Unknown Object (File)
	Wed, Jul 15, 3:12 AM

	Unknown Object (File)
	Thu, Jun 25, 5:43 AM

	Unknown Object (File)
	Sun, Jun 21, 8:47 PM

	Unknown Object (File)
	Jun 4 2026, 9:07 PM

Subscribers

Details

Reviewers

jhibbits
alfredo

Commits

rS368604: [PowerPC] Floating-point exception trap followup

Summary

Fix incorrect operation on 32-bit caused by incorrectly-sized storage for a temporary FPSCR.
Fix several whitespace problems.
Don't unconditionally try to enable VSX during cleanup_fpscr().

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

bdragon created this revision.Dec 3 2020, 1:44 AM

Herald added subscribers: luporl, imp. · View Herald TranscriptDec 3 2020, 1:44 AM

bdragon requested review of this revision.Dec 3 2020, 1:44 AM

jhibbits added inline comments.Dec 3 2020, 2:20 AM

sys/powerpc/powerpc/fpu.c
220–221	I don't think PSL_VSX is needed here at all.

alfredo added inline comments.Dec 3 2020, 11:09 AM

sys/powerpc/powerpc/fpu.c
220–221	Yeah, FPSCR has VXSOFT bit that is being cleared, but documentation doesn't mention PSL_VSX bit should be enabled when manipulating FPSCR. PSL_FP is mandatory

Addressing review comments.

bdragon marked 2 inline comments as done.Dec 12 2020, 11:47 PM

alfredo accepted this revision.Dec 12 2020, 11:49 PM

This revision is now accepted and ready to land.Dec 12 2020, 11:49 PM

Closed by commit rS368604: [PowerPC] Floating-point exception trap followup (authored by bdragon). · Explain WhyDec 13 2020, 3:59 AM

This revision was automatically updated to reflect the committed changes.

bdragon added a commit: rS368604: [PowerPC] Floating-point exception trap followup.

Revision Contents
Changeset List

Path

Size

sys/

powerpc/

include/

8 lines

powerpc/

11 lines

10 lines

Diff 80653

sys/powerpc/include/cpufunc.h

Loading...

sys/powerpc/powerpc/exec_machdep.c

Loading...

sys/powerpc/powerpc/fpu.c

Loading...