req: nestest.asm

zagadka · Post by **zagadka** » Sat Nov 17, 2007 12:37 am

hi everyone

does anyone here know where to find the nestest.asm source file?

i found the readme, but i'm failing some tests and the error codes don't help me much. the readme says

Code: Select all

For a more detailed reason
for the failure, you should check out the .ASM file included with this
document.

but i haven't been able to find the source code anywhere

atari2600a · Post by **atari2600a** » Sat Nov 17, 2007 1:02 am

perhaps using a 6502 dissasembler may help?

http://www.ameth.org/~veilleux/dcc6502.html

zagadka · Post by **zagadka** » Sat Nov 17, 2007 1:24 am

your link is dead for me at the moment.. i'll try it again later

Code: Select all

Firefox can't establish a connection to the server at www.ameth.org.

zagadka · Post by **zagadka** » Sat Nov 17, 2007 5:32 am

ok, fixed the flag tests, i didn't realise bit 5 of the status register was meant to be 1 at all times!

Code: Select all

  // 1 at all times
 cpu->status |= MASK_BIT5;

zagadka · Post by **zagadka** » Sat Nov 17, 2007 6:51 am

ok, fixed the remaining bugs, i pass all of nestest.asm now

thanks a lot to whoever wrote it!

the actual bugs were

1. overflow for sbc/adc was wrong

this is what i had originally

Code: Select all

static void cpu_updateOverflow(CPU cpu, Byte a, Byte b, Byte c) {
  assert(cpu != NULL);

              // positive                    // positive                     // negative
  if ( ((a & MASK_BIT7) == MASK_BIT7) && ((b & MASK_BIT7) == MASK_BIT7) && ((c & MASK_BIT7) == 0) ) {

    cpu_setOverflow(cpu, TRUE);

               // negative                 // negative                 // positive
 } else if ( ((a & MASK_BIT7) == 0) && ((b & MASK_BIT7) == 0) && ((c & MASK_BIT7) == MASK_BIT7) ) {

    cpu_setOverflow(cpu, TRUE);

  } else {
    cpu_setOverflow(cpu, FALSE);
  }
}

it still looks right to me, but apparently it was wrong. i did a rewrite based on the overflow stuff here

2. jmp indirect wasn't wrapping around the low byte property

3. all the indirect,x functions weren't wrapping around the low byte properly

(i made the indirect y functions wrap too for now)

Post by **tepples** » Sat Nov 17, 2007 8:11 am

atari2600a wrote:perhaps using a 6502 dissasembler may help?

http://www.ameth.org/~veilleux/dcc6502.html

"For a more detailed reason for the failure, you should check out the .ASM file" implies that the reasons are in code comments. Disassembly does not recover code comments.

blargg · Post by **blargg** » Sat Nov 17, 2007 10:19 am

zagadka wrote:ok, fixed the flag tests, i didn't realise bit 5 of the status register was meant to be 1 at all times!

Status bits 4 and 5 doesn't even exist. Lots of docs say otherwise, which is the start of confusion! The 6502 remembers only 6 status bits. When pushing them on the stack as a byte, the two extra bits are set to fixed values based on the cause of the push. Bit 5 is always set, and bit 4 is set for BRK and PHP, clear for an interrupt (IRQ/NMI).

atari2600a · Post by **atari2600a** » Sat Nov 17, 2007 1:00 pm

tepples wrote:
atari2600a wrote:perhaps using a 6502 dissasembler may help?

http://www.ameth.org/~veilleux/dcc6502.html
"For a more detailed reason for the failure, you should check out the .ASM file" implies that the reasons are in code comments. Disassembly does not recover code comments.

Yeah, but if the original ASM files are unavailable, at least having a disassembly can be of a little help...

Zepper · Post by **Zepper** » Sat Nov 17, 2007 2:41 pm

Actually, I have a built-in disassembler that can dump a piece of code into a text file, making my analysis much easier, regardless the original source of the program.

kevtris · Post by **kevtris** » Sun Nov 18, 2007 12:19 am

blargg wrote:
zagadka wrote:ok, fixed the flag tests, i didn't realise bit 5 of the status register was meant to be 1 at all times!
Status bits 4 and 5 doesn't even exist. Lots of docs say otherwise, which is the start of confusion! The 6502 remembers only 6 status bits. When pushing them on the stack as a byte, the two extra bits are set to fixed values based on the cause of the push. Bit 5 is always set, and bit 4 is set for BRK and PHP, clear for an interrupt (IRQ/NMI).

Yes, but reading it (i.e. PHP PLA) results in bit 5 always being set. Checking out the 6502 chip schematic, it is shown pulled up! the BRK flag comes from the interrupt logic like you'd expect.

As for the test ROM, I am the dork that wrote it a long time ago. Sorry there's no source for it but I don't have a clean version of the source that I released. Disassembling it (or running the code under a debugger (you DO have a simple debugger in your emu, right?

) should result in the code you seek.

The code is REAL simple, I just do the tests one after another and write a number to a ZP location if something fails. I tried to make it so that the earlier tests have to pass before those functions get used later on in other tests, so that SOMETHING would work at first and it wouldn't just fail right off the bat. Though if your CPU is REAL broken it still might not work.

zagadka · Post by **zagadka** » Sun Nov 18, 2007 1:05 am

i pass all your tests now

in other news i've decided to call my emulator 1337nes

atari2600a · Post by **atari2600a** » Sun Nov 18, 2007 1:30 am

lol, kind of reminds me of a friend's 8086-compatible IRC client, 1337IRC!