Skip to content

Add tool to test DSP instructions#70

Open
JayFoxRox wants to merge 3 commits intomasterfrom
dsp_instruction_test
Open

Add tool to test DSP instructions#70
JayFoxRox wants to merge 3 commits intomasterfrom
dsp_instruction_test

Conversation

@JayFoxRox
Copy link
Copy Markdown
Owner

@JayFoxRox JayFoxRox commented Dec 26, 2018

This should make it easier to test DSP instruction behaviour.

Can be ran like this with debug bios:

A56=/home/user/a56/a56 XBOX=localhost:8731 ./dsp_instruction_test.py dsp_instruction_tests/*-test.json 

(Requires a56; example was with -redir tcp:8731::731 in XQEMU CLI)

For test syntax, check the examples. Further changes to input / output syntax are likely.


Most critical missing features / known issues:

  • Memory can not be set as input / output, no fatal error is generated
  • Some registers are blacklisted / ignored, no fatal error is generated
  • DSP assembler errors are silently ignored, no fatal error is generated
  • JSON and assembly parser issues are not reported properly
  • Output folder is same as input folder, so it's hard to generate 2 outputs for different machines
  • No tool to compare JSON output files
  • Code is ugly

@JayFoxRox JayFoxRox force-pushed the dsp_instruction_test branch from a628f2c to ce511a5 Compare January 6, 2019 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant