Cleanups and enable MMU test

openrisc / or1k-tests

Unified OpenRISC 1000 test suite

8 stars 11 forks source link

Cleanups and enable MMU test #6

Closed stffrdhrn closed 4 years ago

stffrdhrn commented 4 years ago

Various whitespace and duplicate code cleanups. Fix the Makefile so it actually tracks dependencies.

The MMU test seems completely broken

Make it check tlb sets and ways from the CPU not hard coded
Fix compiler warnings
Fix logic for bss/stack being in different places it assumed everything was in the start of memory

olofk commented 4 years ago

I took a quick look, and other than a couple of commented lines I couldn't find anything controversial so I trust you got this right

stffrdhrn commented 4 years ago

Note, I started working on this cleanup a while back and stopped. Now I need to test traps on mor1kx as I suspect a bug is causing the debugger to fall. It may be fixed by this pr but I need a test bench to be sure.

https://github.com/openrisc/mor1kx/pull/108

stffrdhrn commented 4 years ago

ok I messed up merging this. Fixed it manually