Page MenuHomeFreeBSD
Differential D4155

Always specify MIPS ABI flags.
AbandonedPublic
Actions

Authored by imp on Nov 14 2015, 9:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 24, 1:17 AM
Unknown Object (File)
Fri, Nov 15, 3:44 PM
Unknown Object (File)
Wed, Nov 6, 11:00 AM
Unknown Object (File)
Wed, Nov 6, 7:00 AM
Unknown Object (File)
Fri, Nov 1, 6:32 PM
Unknown Object (File)
Fri, Nov 1, 6:32 PM
Unknown Object (File)
Fri, Nov 1, 6:31 PM
Unknown Object (File)
Fri, Nov 1, 6:31 PM
View All Files
Subscribers
bdrewery
emaste
ngie

Details

Reviewers
adrian
bdrewery
Commits
rS291053: Fix mips CPUTYPE so that we can pass it through to gcc.
Summary

Always over-specify mips. Makes life easier when the cross compiler
isn't configured quite correctly and outputs the wrong thing by
default. Introduce _ABI_CFLAGS to hold the over specification. Use
_ABI_CFLAGS in COPTFLAGS in the kernel build, and add those flags
to the various FOO_CFLAGS definitions.

XXX haven't made a pass through files to pick up the crazy, loony
XXX stuff that goes on there.

Test Plan

make TARGET_ARCH=mipsel CROSS_COMPILER_PREFIX=/usr/local/bin/mips64-portbld-freebsd11.0- XCC=/usr/local/bin/mips64-portbld-freebsd11.0-gcc CROSS_BINUTILS_PREFIX=mips64-freebsd- X_COMPILER_TYPE=gcc buildkernel KERNCONF=RT305X
make TARGET_ARCH=mipsel CROSS_COMPILER_PREFIX=/usr/local/bin/mips64-portbld-freebsd11.0- XCC=/usr/local/bin/mips64-portbld-freebsd11.0-gcc CROSS_BINUTILS_PREFIX=mips64-freebsd- X_COMPILER_TYPE=gcc buildworld

Note: the gcc 5.2.0 compiler that I'm testing with barfs with set but unused warnings everywhere. These
are fatal, so the buildworld doesn't complete.

The kernel fails at the moment because it uses floating point stores with the mips32 ISA, which apparently is undefined (or I'm misunderstanding the swc1 opcode warnings). This is swtch.S

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1136
Build 1140: arc lint + arc unit

Event Timeline

imp updated this revision to Diff 10178.Nov 14 2015, 9:26 PM
imp retitled this revision from to Always specify MIPS ABI flags.
imp updated this object.
imp edited the test plan for this revision. (Show Details)
imp added a reviewer: adrian.
Herald added subscribers: emaste, bdrewery. · View Herald TranscriptNov 14 2015, 9:26 PM
imp added inline comments.Nov 14 2015, 9:29 PM
share/mk/bsd.cpu.mk
125

This change is unrelated, but also something Adrian has been asking for.
It harmonizes the mips CPUTYPE with the rest of the system and gives
a list of helpful choices based on what we support or nearly support today.

imp retitled this revision from Always specify MIPS ABI flags to Always specify MIPS ABI flags. Harmonizes MIPS CPUTYPE processing.Nov 14 2015, 9:32 PM
imp edited the test plan for this revision. (Show Details)
ngie accepted this revision.Nov 15 2015, 4:41 AM
ngie added a reviewer: ngie.
ngie added a subscriber: ngie.

LGTM!

This revision is now accepted and ready to land.Nov 15 2015, 4:41 AM
ngie requested changes to this revision.Nov 15 2015, 4:43 AM
ngie edited edge metadata.

Sorry. Missed a spot.

sys/conf/kern.pre.mk
165

This is a bit inconsistent with the other CFLAGS. In particular, DTRACE_CFLAGS does the same thing FBT_CFLAGS does, but only DTRACE_CFLAGS gets the COPTFLAGS treatment?

This revision now requires changes to proceed.Nov 15 2015, 4:43 AM
imp updated this revision to Diff 10206.Nov 15 2015, 6:14 PM
imp edited edge metadata.
imp edited the test plan for this revision. (Show Details)

Cleanup of repeated CFLAGS stuff ngie found the tip of.

Updating D4155: Always specify MIPS ABI flags.

Harmonizes MIPS CPUTYPE processing

ngie accepted this revision.Nov 15 2015, 6:18 PM
ngie edited edge metadata.

A lot simpler. Like it a lot :).

This revision is now accepted and ready to land.Nov 15 2015, 6:18 PM
imp marked an inline comment as done.Nov 15 2015, 6:19 PM
imp added inline comments.
sys/conf/kern.pre.mk
165

You're right. I didn't notice the CDDL_CFLAGS there.

166

It's actually more insidious than that. See the updated change.

imp marked 3 inline comments as done.Nov 15 2015, 6:20 PM
imp added inline comments.
sys/conf/kern.pre.mk
165

You're right. I didn't notice the CDDL_CFLAGS there.

imp marked an inline comment as done.Nov 15 2015, 6:20 PM

Ticked off things I'd done.

imp updated this revision to Diff 10249.Nov 16 2015, 10:46 PM
imp edited edge metadata.

Regen after lifting one of the parts out of this patch
that's really just a bug fix.

Updating D4155: Always specify MIPS ABI flags.

Harmonizes MIPS CPUTYPE processing

This revision now requires review to proceed.Nov 16 2015, 10:46 PM
imp added inline comments.Nov 16 2015, 10:49 PM
sys/conf/kern.pre.mk
166

Ignore the above comments: I've peeled out this sub-match and committed separately since it's just a bug fix.

bdrewery requested changes to this revision.Nov 16 2015, 11:04 PM
bdrewery added a reviewer: bdrewery.
bdrewery added inline comments.
share/mk/bsd.cpu.mk
258–260

Seems redundant with line 17 and 18.

This revision now requires changes to proceed.Nov 16 2015, 11:04 PM
bdrewery accepted this revision.Nov 16 2015, 11:06 PM
bdrewery edited edge metadata.

Nevermind, I see it's all an else branch.

It seems fine to me. I did not review the specifics of the flags for ABI/impact correctness.

This revision is now accepted and ready to land.Nov 16 2015, 11:06 PM
emaste added a comment.Nov 16 2015, 11:19 PM

LGTM

share/mk/bsd.cpu.mk
338

ACFLAGS is for when cc is used as an assembler?

imp added inline comments.Nov 17 2015, 5:14 AM
share/mk/bsd.cpu.mk
338

Yes. It is used to compile all .S files in the tree. We don't call the assembler directly in the tree (or shouldn't IMHO). If we don't add _ABI_CFLAGS to ACFLAGS, then we produce the default output, not the desired output.

Closed by commit rS291053: Fix mips CPUTYPE so that we can pass it through to gcc. (authored by imp). · Explain WhyNov 19 2015, 3:11 AM
This revision was automatically updated to reflect the committed changes.
imp mentioned this in rS291053: Fix mips CPUTYPE so that we can pass it through to gcc..
imp reopened this revision.Nov 19 2015, 3:21 AM
imp retitled this revision from Always specify MIPS ABI flags. Harmonizes MIPS CPUTYPE processing to Always specify MIPS ABI flags..
imp edited edge metadata.
This revision is now accepted and ready to land.Nov 19 2015, 3:21 AM
imp updated this revision to Diff 10323.Nov 19 2015, 3:23 AM
imp edited edge metadata.

Split out the Harmonzing MIPS CPUTYPE processing bits
leaving just the ABI flags.

Updating D4155: Always specify MIPS ABI flags.

Harmonizes MIPS CPUTYPE processing

This revision now requires review to proceed.Nov 19 2015, 3:23 AM
ngie resigned from this revision.Mar 8 2017, 7:04 AM
imp abandoned this revision.Jul 9 2021, 7:16 AM

Revision Contents
Changeset List

PathSize
share/
mk/
66 lines
sys/
conf/
10 lines

Diff 10178

View Options

share/mk/bsd.cpu.mk

Loading...
View Options

sys/conf/kern.pre.mk

Loading...