Skip to content

Peer review 2

  1. When i tried running it myself i got the Error "Exception: Tried to write in uninitialized memory at address: 41000000" however since i see that the code works through the passed Pipeline so the Error might be from me. Maybe because i Use SyncSim instead of SyncRim? But i though i should mention that atleast.
  2. The kernel code only uses $k0 and $k1 for exception handling and output operations, good👍
  3. Any exception/interrupt besides "SYSCALL", ends up in the infinite "kernel_loop. Could not test it but based "--generate overflow-- part that is commented out in proc1 and how you coded the `kernel_loop it should work👍
  4. Two lowest bits of the STATUS REGISTER is "user mode" and "interrupt enabled",👍
  5. Good and clear comments, Good job!👍

Good work everything looks great except that i could not run it but that could very well be an miss from my part and since pipeline passed and you can run it i would not worry that much about it