New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LC write-enabled state is not maintained correctly (1.26.2.0) #395

Closed
peterferrie opened this Issue Mar 18, 2017 · 6 comments

Comments

Projects
None yet
2 participants
@peterferrie

peterferrie commented Mar 18, 2017

Booting Gemstone Warrior, it claims to need 64kb of RAM.
The game performs two reads of $C081 which should write-enable LC, and later performs four STA $C083s (which should cause LC to remain write-enabled) and then writes to LC. Only pre-write should be disabled by the STA - no effect on write-protection - but the LC becomes write-protected instead.

@tomcw

This comment has been minimized.

Show comment
Hide comment
@tomcw

tomcw Mar 18, 2017

Contributor

Thanks for the quick check & report about this.

Ironically I broke this AppleWin LC support when I added support for the a2audit.dsk (#392).

I'll feedback this back to Zellyn so that his $C08n LC test can catch this.

Contributor

tomcw commented Mar 18, 2017

Thanks for the quick check & report about this.

Ironically I broke this AppleWin LC support when I added support for the a2audit.dsk (#392).

I'll feedback this back to Zellyn so that his $C08n LC test can catch this.

@peterferrie

This comment has been minimized.

Show comment
Hide comment
@peterferrie

peterferrie Mar 18, 2017

peterferrie commented Mar 18, 2017

@tomcw

This comment has been minimized.

Show comment
Hide comment
@tomcw

tomcw Mar 19, 2017

Contributor

NB. Add regression test here.

Contributor

tomcw commented Mar 19, 2017

NB. Add regression test here.

@tomcw

This comment has been minimized.

Show comment
Hide comment
@tomcw

tomcw Mar 19, 2017

Contributor

Hi Peter,

I've built a new experiment (pre-release) 1.26.2.1 version here.

Gemstone Warrior now boots correctly. But let me know if you notice any other LC-related issues.

Thanks.

Contributor

tomcw commented Mar 19, 2017

Hi Peter,

I've built a new experiment (pre-release) 1.26.2.1 version here.

Gemstone Warrior now boots correctly. But let me know if you notice any other LC-related issues.

Thanks.

@peterferrie

This comment has been minimized.

Show comment
Hide comment
@peterferrie

peterferrie Mar 20, 2017

peterferrie commented Mar 20, 2017

@tomcw

This comment has been minimized.

Show comment
Hide comment
@tomcw

tomcw Mar 20, 2017

Contributor

Thanks for confirming.

Contributor

tomcw commented Mar 20, 2017

Thanks for confirming.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment