Memory Card problems

[KrossX]

The 7zip file contains the pcsx2.exe file. Use it to replace (or rename and put besides) the one in your PCSX2 folder.

[Azphel]

The 7zip file contains the pcsx2.exe file. Use it to replace (or rename and put besides) the one in your PCSX2 folder.

Ah I see. Thank you

[Azphel]

The 7zip file contains the pcsx2.exe file. Use it to replace (or rename and put besides) the one in your PCSX2 folder.

Replacing the .exe with that fixed the issue. Though now my log is flooding it self with "Buggy SIO command" I assume I need the SIO new patch you posted but I'm unsure of where to place that as well.

[KrossX]

Nope, that log is good actually. For testing purposes that is.

[rama]

Is there something to test out here? I have a bit of time and could update trunk or smth.

[KrossX]

Test this one. I tested BIOS (including PSX transfers) and Persona 3.

A God Hand test would be more than welcomed.

#EDIT:

@Azphel: Test this one please, it won't have the log flooding.

[KrossX]

Explanation...

Code:

u16 size1 = (sioreg >>  8) & 0x1FF;
u16 size2 = (sioreg >> 18) & 0x1FF;

sio.bufSize = size2; //max(size1, size2);

According to the... freesdk (or was it a loader?), there are two copies of the command length in that sioreg:

Code:

regdata = (((port & 1) + 2) & 3) | 0x70;
regdata |= mcman_cmdtable[(cmd << 1) + 1] << 18;
regdata |= mcman_cmdtable[(cmd << 1) + 1] << 8;

mcman_sio2packet.regdata[mcman_sio2packet.in_dma.count] = regdata;

Size1 is sio2.cmdlength, which has proven to be unreliable. On mismatch however (Persona 3), size2 is still the proper length. Previous patch with log used the max() since size1 was smaller in those cases. New build just uses the size2 copy.

I wonder though, does size1 being unreliable affect sio2?

[Azphel]

Test this one. I tested BIOS (including PSX transfers) and Persona 3.

A God Hand test would be more than welcomed.

#EDIT:

@Azphel: Test this one please, it won't have the log flooding.

Tested. Log is no-longer flooding.

Thanks!

[MerolaC]

Test this one. I tested BIOS (including PSX transfers) and Persona 3.

A God Hand test would be more than welcomed.

#EDIT:

@Azphel: Test this one please, it won't have the log flooding.

Hey, KrossX
Just tested MGS3 as you requested here.
http://forums.pcsx2.net/Thread-Metal-Gea...card-issue

Game works with this exe.

Thank you.

[KrossX]

Thanks for testing.

@avih: Can you commit the wololo patch? Seems to be working fine so far.