 No.03506
Tafoid Administrator
Jan 6, 2009, 11:22
edited on: Jan 6, 2009, 12:57
|
There was discussion of this being a possible SDL specific issue. Not sure if there was ever a resolution, but it might be worth bringing it up again there: http://www.bannister.org/forums/ubbthreads.php?ubb=showflat&Number=44136 As stated, in a normal baseline build - the problem doesn't happen. I'll wait for 64-bit Windows and SDL users to test and report. |
|---|---|
 No.03507
Osso Moderator
Jan 6, 2009, 12:55
|
Doesn't repoduce on MameUI64 - Vista64. |
 No.03515
Fujix Administrator
Jan 7, 2009, 15:09
|
No repro for me with vmame64 on vista 64. |
|
No.03516
Tafoid Administrator
Jan 7, 2009, 15:18
|
Ok - we need some input from SDLMAME users - is anyone having this issue? I'm guessing at this point it's an isolated (non MAME) issue, but you never know. |
 No.03523
mahlemiut Developer
Jan 7, 2009, 21:45
|
64-bit SDLMAME (based from MESS SVN, since that's what I have handy) shows C5 as bad, and some repeated clicking sound occurs during this too. Occasionally, C5 is given as OK, but it then resets at the cross hatch pattern. Have yet to try out 32-bit SDLMAME. |
|
No.03527
Haze Senior Tester
Jan 7, 2009, 23:51
edited on: Jan 7, 2009, 23:53
|
parodius doesn't use eeprom, so it's not an f2+f3 issue ROM_REGION( 0x80000, "konami", 0 ) /* 053260 samples */ ROM_LOAD( "955d04.bin", 0x00000, 0x80000, CRC(e671491a) SHA1(79e71cb5212eb7d14d3479b0734ea0270473a66d) ) is rom c3, the sample rom. it can read the sound rom with static READ8_HANDLER( parodius_sound_r ) { return k053260_0_r(space,2 + offset); } (why the +2?, looks odd..) which calls READ8_HANDLER( k053260_0_r ) { return k053260_read( space, offset, 0 ); } which reads the rom with case 0x2e: /* read rom */ if ( ic->mode & 1 ) { unsigned int offs = ic->channels[0].start + ( ic->channels[0].pos >> BASE_SHIFT ) + ( ic->channels[0].bank << 16 ); ic->channels[0].pos += ( 1 << 16 ); if ( offs > ic->rom_size ) { logerror("%s: K53260: Attempting to read past rom size in rom Read Mode (offs = %06x, size = %06x).\n", cpuexec_describe_context(space->machine),offs,ic->rom_size ); return 0; } return ic->rom[offs]; rom is unsigned char *rom; is unsigned char actually 8-bit on 64-bit ? (dumb question maybe) still.. should probably be using UINT8 etc.. I'm definitely not keen on that code. somebody with 64-bit and the issue will have to trace / log through the code anyway.. it's clearly failing to read the sound rom correctly. |
|
No.03535
Tafoid Administrator
Jan 8, 2009, 17:48
|
With the confirmation and some explanation even though I cannot reproduce this - I'll confirm it. |
|
No.03547
R. Belmont Developer
Jan 10, 2009, 04:52
|
Problem was the banking of the ROM reading went wrong when 'unsigned long' was > 32 bits. Replaced all instances of 'unsigned long' with UINT32 and Parodius plays fine on x64 Linux. |
|
No.03548
Haze Senior Tester
Jan 10, 2009, 13:21
edited on: Jan 10, 2009, 13:22
|
right.. as a general rule the mame types (UINT8, UINT16, UINT32 etc. should be used whenever possible..) (which is why I really didn't like the look of that sound code..) |