forums.ps2dev.org

Rev1Dev · Joined: 13 Apr 2008 Posts: 22

Ah...
The patch added the bleh.cpp in there but I (or it) hadn't modified the makefile.
I also noticed $(LOGGEROBJ) wasn't setup in my copy of the makefile either... is this something you added?
Well, that fixed that it looks like. (THANK YOU once again Ragnarok... I wish I wasn't so new to this stuff... alot of these fixes I'm having trouble getting around... and creating new functions I'm not very good at... Sorta learning this as I'm going... anyhoo... thank you = D )

Okay... so I'm going to post this to see if it looks identical to where you're at:

ragnarok2040 · Joined: 09 Aug 2006 Posts: 230

Yep, looks like you're at the same point I am. You're missing an object file or two I added back into the process but other than that, it looks good. I'm not sure what you mean by malloc and sound core initialization? I added some more checks for logger and screenshot functions that won't be needed in the Makefile and source code which will be in the new patch, :D. When the source gets to a compile-able state, I'll probably start looking at individual functions to determine which ones are needed or not and start commenting them out as well.

On another note:
I took a look at the docs folder, and it mentions in the readme.txt file that snes9x can run on a 486 (133 mhz) machine but that's only for the bare minimum of games. They recommend at least a 600 mhz processor for the ability to run most games at the basic settings. The ps2's EE is clocked at 294 mhz, and while mips processors do tend to run code faster than x86 cpus, I'm sure it's not enough. I would consider Snes-Station to actually be near the far end of the spectrum on performance on the ps2, in my opinion.

Having said that, commenting out a lot of the unneeded stuff should help improve performance. I think that Hiryu said he was running a thread on the IOP to do the sound in Snes-Station way back when or maybe it was that he was planning to do that... Well, that's for the far future :D.

Rev1Dev · Joined: 13 Apr 2008 Posts: 22

Yeah... I see what you mean.

In comparison, however, I think once it's in a state where optimization can occur, it ought to be alright.

Some system specs:

ragnarok2040 · Joined: 09 Aug 2006 Posts: 230

The IOP is a co-processor for the EE, that's designed to offload the I/O functions from the EE. It's basically the same cpu that the PS1 uses, but I think at a higher clockrate. I forgot what it was though. The various device drivers each run their own thread which can interface with client code from the EE. You can look at the clientside code in the ps2sdk source's ee/rpc and serverside code in the iop folder. I think the simplest example to look at is iop/dev9/poweroff and ee/rpc/poweroff. Offloading some of the emulation onto the IOP would help a bit, but that's for later :D.

The apu code looks like it defines some internal functions for sound emulation that spc700.cpp uses.

I've gotten down to 38 errors. Some of these are from functions not implemented in ps2sdk's libc yet. For the _impure_ptr errors, http://www.embedded.com/story/OEG20011220S0058 down at Reentrancy.

I also grabbed a prototype function for _splitpath from http://forums.hostrocket.com/archive/index.php/t-15876.html. It's not useable right now for the ps2, but it can be modified. device:dir/file or device:/dir/file or device:\dir\file (one of those cdvd modules) isn't too far different from C:\dir\file.ext.

Here's the latest patch. Make sure to apply it to the clean snes9x-1.51 source.

Rev1Dev · Joined: 13 Apr 2008 Posts: 22

Good Job Ragnarok... you're going to end up cracking this one = )
Man I can't wait until I know what I'm doing -=laughing=-

Well, I wasn't able to d/l the new patch; do you think you would mind re-uploading?

Here's an example of what I've been doing (just took and added it onto bleh.cpp).
For the second one it's now set to FALSE (*correction).

ragnarok2040 · Joined: 09 Aug 2006 Posts: 230

Hrmm, that's strange, I can't download it from my link, either, but I can get it from the main page. Go to http://homebrew.thewaffleiron.net/ragnarok2040 and download it from that page. It might be some sort of hotlink protection.

Actually, not all of the undefined references are undefined. Some of them are defined, but aren't linked in, so they need to be commented out when they're called, instead of defined. For other undefined functions, it depends on if you need them or want them or not. The function definitions in bleh.cpp are ones that porting.html said were needed for a port, so I provided them. Also, a bool8 function should return either true or false (or an unsigned char, since that's the type). I only returned NULL on functions that returned a pointer, such as char *function() which will cause either a NULL exception error if the pointer is used or tell the function that called it that NULL was returned.

In the new patch, I added snapshot.cpp back into the build because it's needed for savestates (snapshots) so S9xSPCDump and S9xUnfreezeGame are defined again. I also added #ifdef/#undef guards to the function calls that are defined in objects that aren't linked in, so their calls won't be included into the final object at compile-time. For examples of what I did, search for the CHEAT_SUPPORT symbol or MOVIE_SUPPORT symbol inside of the source code using an ide like CodeBlocks. You could also open up the patch inside of Wordpad to see what I've done.

If you want a good C programming book, you should look for Practical C Programming by O'Reilly and for C++, you should get C++: The Complete Reference by Osborne. And for pointers, http://www.gamedev.net/reference/articles/article1697.asp, which I found useful. There's also plenty of sites that have information about programming in C, as well.

Snes9x is coded mostly in C, but compiled with the C++ compiler because of some features they wanted that weren't available with just gcc, but because of that it has the same quirks a C++ program would have. I think that answered everything from your last two posts, :D.

Extra note:
Can someone try compiling snes9x-1.51 with my patch using their ps2dev toolchain on Linux or Cygwin? That'll tell me if some of the errors are from a bug in MinGW not compiling libstdc++ correctly (weird bug where gcc tries to use a windows path for some reason but doesn't properly '\\' so I get c:msyslocalps2deveebinar.exe type errors).

Rev1Dev · Joined: 13 Apr 2008 Posts: 22

I'm trying to apply the patch, but keep getting this:

ragnarok2040 · Joined: 09 Aug 2006 Posts: 230

Hrmm, working fine for me. I'm using the source from http://files.ipherswipsite.com/snes9x/snes9x-1.51-src.tar.bz2 but other than that it looks like it's not finding the snes9x source files correctly. The snes9x folder you're in should have all the .cpp files in it.

Rev1Dev · Joined: 13 Apr 2008 Posts: 22

Nevermind...
Sorry about the first post. Here's the correct output I received, this time with the patch applied correctly:

ragnarok2040 · Joined: 09 Aug 2006 Posts: 230

Weird, _makepath and _splitpath are defined in bleh.cpp. All references to "cheats.h" are commented out via the preprocessor at compile-time so you shouldn't even have undefined references to S9xAddCheat and etc. You sure you aren't using the old patch?

Rev1Dev · Joined: 13 Apr 2008 Posts: 22

Okay... you were right. Wrong patch X D

Okay this is where it's really at -=smiling=-

ragnarok2040 · Joined: 09 Aug 2006 Posts: 230

All of that can be shortened down to this list of 13 functions: ctime, localtime, setlocale, setvbuf, finitef, strtoull, strtoll, strftime, strxfrm, strcoll, __assert, atoi, and _impure_ptr.

It's at the point, now, where real work can start, :D.

Rev1Dev · Joined: 13 Apr 2008 Posts: 22

Are these (some of) the files I want to be looking at right?

ragnarok2040 · Joined: 09 Aug 2006 Posts: 230

They're parts of the standard c++ library (libstdc++) and the standard c library (libc or newlib). All we have to do is write a frontend or rewrite the internal parts to not use the specific libstdc++ functions that require those undefined newlib definitions or just implement them in ps2sdk's libc (preferrable method as it helps the community).

Rev1Dev · Joined: 13 Apr 2008 Posts: 22

Nice. = ) So there's the added bonus that those adapted libraries will be available if ever needed for other projects in the future... I guess it'll also make it easier when compiling this from source for future updates, while making the SDK itself a more fully featured adaption of existing C/C++ standard.
On another note, I was wondering if condensing the script to only include the features necessary to the PS2 version would require any major rewrites to the foundation code in helping to optimize it?

I was looking at the article you posted in regards to reentrancy and was searching for some examples that used it to gain a better understanding of how to rewrite into a format more suitable for the SDK.
You've been working pretty fast at this so I'm certain you've either begun work towards or have even already completed some implementation of it.
The learning curve still seems pretty steep from this side of the fence, but I just wanted to say what a big help you've been to me in particular in being patient with a newcomer and all the questions that come along with taking on anything new as well as sharing so much valuable information to help me along. Very Much appreciated! = )

ragnarok2040 · Joined: 09 Aug 2006 Posts: 230

Sorry I was away a bit, heh. I was busy playing with Linux on my ps2, :D. I'm nowhere good enough to start implementing standards compliant functions into newlib/libstdc , so I decided to see if I could find a workaround. I took a look at the snes9x port on Wii/GC to see if I could come up with any ideas, as well. It's based on 1.43, and there are huge differences between 1.51 and 1.43. It looks like it was based on the xSnes9x port. There's no controls.cpp, and reader.cpp was replaced with a custom gamecube-based file loader, from the looks of it. Taking out both of those from the project removed all of the libstdc errors I was getting.

The following functions are ones that I commented out:

And, the fReader calls in memmap.c need to be replaced with a custom file loader for ps2. Lines in memmap.c: 3426 3441 3462 3482 3557 3572 3593 3613, seem to be the only place it's used. I'll work more on it.

Update, I replaced reader.c/.h with just an empty fReader class with a constructor that takes a FILE* reference, and it compiled :D.

There seems to be something weird afoot. Every symbol that's undefined in libstdc++ is supported inside of ps2sdk's stdlib.c/.h. I recompiled my toolchain, fixing my c:ps2deveeeebinnm.exe error, and that didn't fix the problem. I'm out of clues :S.

ragnarok2040 · Joined: 09 Aug 2006 Posts: 230

Sorry about posting back to back, but I reverted back to 1.43 and it was relatively easy to get it to compile. It only took me around 2 1/2 hours to create a new makefile, add the function stubs for the port, and comment out miscellaneous movie functions. Just wanted to post to give a heads up.

dlanor · Joined: 28 Oct 2004 Posts: 269 Location: Stockholm, Sweden

@ragnarok2040: Please check your PM box.

Best regards: dlanor

ragnarok2040 · Joined: 09 Aug 2006 Posts: 230

The GSTEXTURE structure was throwing me off a bit. The Mem pointer is u32*, and the texture data I was giving it was being cast to 32-bit numbers by default when compiling. By casting the pointer to u16* I was able to fix this

to

this.

:D.

bootsector_ · Joined: 25 Feb 2008 Posts: 15

I knew you would show up and take this personally!

Keep up the great job!

syproject · Joined: 28 Mar 2008 Posts: 5

nice work ragnarok2040 i know many people will ask the same qestion.... when will a beta be released?

keep up the good work bud

ragnarok2040 · Joined: 09 Aug 2006 Posts: 230

I've already released a couple of builds so people could try it out. Alpha2 is the latest version. It supports one controller mapped to a standard snes controller, just push L2 to get back to the file browser. There's no sound support, or hi-res support or transparency support, and booting new roms is a little broken. It's also kind of fast because the sound core isn't enabled, heh. With it enabled, it's pretty slow.

If you want to try it out, I uploaded the source and the latest alpha2 builds to:
http://homebrew.thewaffleiron.net/ragnarok2040/

They're named packed_snes9xPAL.zip and packed_snes9xNTSC.zip.

syproject · Joined: 28 Mar 2008 Posts: 5

nice work ragnarok2040 when you starting on the n64 heheh j/k

will have a play about tonight

goldman · Joined: 25 Aug 2008 Posts: 1

bumped
this thread shouldn't fade away

dirsors · Posted: Sun Aug 31, 2008 11:27 am Post subject:

Eagerly wating for a stable version!

ragnarok2040 · Joined: 09 Aug 2006 Posts: 230

Sorry, real life got in the way, heh. My old computer broke down in July, then it took about a month to build another one, then I was supposed to move in August, which didn't really happen until early September. On top of that, I had been learning MIPS assembly, which I need to still complete. Other than that, I'm about to rewrite the ports a bit so I can separate the code a little more cleanly from the browser, redo the startup code a bit so it starts faster, various other changes so it'll be easier to work on the core "porting" of Snes9x as my old code keeps getting in the way, heh.