Differential D27453

[PowerPC] Floating-point exception trap followup
ClosedPublic
Actions

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

Tags

Referenced Files

	Unknown Object (File)
	Sun, Nov 23, 9:23 PM

	Unknown Object (File)
	Fri, Nov 21, 3:04 AM

	Unknown Object (File)
	Fri, Nov 21, 3:04 AM

	Unknown Object (File)
	Fri, Nov 21, 3:03 AM

	Unknown Object (File)
	Fri, Nov 21, 3:02 AM

	Unknown Object (File)
	Fri, Nov 21, 2:59 AM

	Unknown Object (File)
	Tue, Nov 18, 1:37 AM

	Unknown Object (File)
	Wed, Nov 12, 7:41 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...