This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: PATCH: Fix the MIPS ISA ELF setting (Re: RedHat 7.1/mips update)



> 1. The command line is not compatible with the previous version. That is
> "-m4100 -mips2" is not allowed any more.

Ok. This is me trying to keep the gcc and binutils options close
together. I'm moving the mips compiler to the more standard
-march/-mtune and deprecating everything else. This should make it
easier for the kernel folks since they'll only have to worry about one
way of doing things.

Part of the problem I think is because of the idea around -mipsX. 
-mipsX does not specify anything other than a generic ISA to conform to.
It most assuredly doesn't specify 32 v. 64 bit like most people try to
use it as. That's what the -mgp32 and -mgp64 options are for.
Historically this has been confusing and the -mipsX options are now
there for compatibility with the SGI compiler only.

> 2. It won't generate a working kernel no matter what you do.
> 

This is because of the 64-bit v. 32-bit problem? Have you tried
compiling with -march=vr4100 and -mgp32? That _may_ work, anyhow...

> I don't believe the binutils should break the existing Linux kernel
> source. Chris's solution is ok by itself. But I don't think it is the
> right solution for Linux.

What is the right solution? I feel that allowing code like:

.set arch foo
<insns>
.set arch 0

Along with the possibility of compiling/assembling for 32/64 bit mode
via -mgp32/gp64 is sufficient. 

-eric

-- 
Fridays are not "pants optional"


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]