[Scummvm-devel] PS2 backend - moving from 0.10.0 to 0.11.0

[sunmax at libero.it]

Hi there, noble scummvm-devel souls!

;-)

I got your latest PS2 drop (0.10.0-alpha) compiling and running,
it wasn't too bad.

I still need to have in place:

- mpeg2dec (aka libmpeg2)

- tremor

Robert, could you send me their ps2-friendly source you modified?

I can probably figure it out by myself in a couple of hours,
but since it has already been done I don't see why reinventing
the wheel.

Regarding 'mpeg2': why are we using mpeg2dec instead of the HW
accelerated libmpeg from Eugene Plotnikov in ps2sdk?

Regarding 'tremor': it is the fixed-math version of Xiph's vorbis, so
is it supposed to replace it or do we still need to compile vorbis too?
Did you use the code from the PSP version as a base?

The only headache I had was with 'hddPreparePoweroff'...

hddPreparePoweroff() has been deprecated in July 2007 and replaced by
poweroffInit() which does more than it promises. We get likely caught
in the unsafe loop:

while(((res = SifBindRpc(&cd0, PWROFF_IRX, 0)) >= 0) && (cd0.server == NULL))
    nopdelay();

This loop was not in the hddPreparePoweroff() in the previous PS2
tools.

Robert, could you please confirm the toolchains version you were using?

I will investigate this further.

Why did you call the PS2 0.10.0 code an alpha? I have been
playing the DIG (from CD ;-) + a couple of other LucasArts
classics for a while and it's doing fine.

Is there anything you are aware of that used to crash
and need to be fixed? I would like to reproduce it then
before the final users discover that... ;=)

Regarding all the IRX that PS2 scummvm loads at startup,
what's their licence status? Are we allowed to redistribute
them when we do a binary tarball for the PS2?

Max, regarding moving on to 0.11.0, now that I have it running,
what do I actually need to do to upgrade it to 0.11.0?

I saw that we have a few more engines (Elvira*! Cool, I loved
that game in my Amiga days). Are they going to work out of
the box, or they will require me to change somehing in the PS2
backend?

Could you tell me more about the "Completely reworked internal
access to files". Is it implemention well hidden or did it
imply API changes?

What about the new features, things like TiMidity++ support?
Is that a show-stopper? Do we need to have support
to call it a 0.11.0 backend?

Could you please give me a list of the must-have that all
0.11.0 backends need to comply to? Thanks!

Looking forward to your feedback!

I will keep you updated.

Ciao,
 -max