Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: [matt-nb5-mips64] src/sys/arch/mips/mips



Cliff Neighbors wrote:

> Module Name:  src
> Committed By: cliff
> Date:         Mon Nov  9 10:00:02 UTC 2009
> 
> Modified Files:
> 
>       src/sys/arch/mips/mips [matt-nb5-mips64]: db_interface.c
> 
> Log Message:
> 
> arch/mips/mips/db_interface.c
> - add MIPS64_SHOW32() and MIPS64_SHOW64()  macros
>   to show MIPS64 specific cp0 regs

Is the use of ".set mips64" safe in MIPS64_SHOW32 without the ".set
push"/".set pop" dance?

> - add EIRR, EIMR, EBASE, CONFIG[0,1,7] to show regs
>   >>> these switched by runtime vairable  is_rmi_xls
>         is there a better variable to use ???
>         cpu_arch == 64 ??

Querying cpu_id in db_interface.c is definitely wrong - maybe we should
add a new field to cputab in mips_machdep.c to say which CP0 regs are
available on each CPU.  You should KNF the "is_rmi_xls=0" as well.

> - add mach mtcr, mfcr MD ddb commande
>   >>> these are not runtime switched based on cpu arch,
>       they are just 'there'
>       if you use them on the wrong CPU, it's your problem!

Do these even link for a non-RMI config?  You're unconditionally calling
rmi functions from "MI" MIPS code.


In general, I'm a tad concerned about the chip-specific code that is
getting added here.

Cheers,
Simon.


Home | Main Index | Thread Index | Old Index