Previous Thread
Next Thread
Print Thread
Page 1 of 5 1 2 3 4 5
#30631 06/22/07 12:29 AM
Joined: Mar 2001
Posts: 16,892
Likes: 51
R
Very Senior Member
OP Offline
Very Senior Member
R
Joined: Mar 2001
Posts: 16,892
Likes: 51
OMGWTFBBQ, yes, this includes the CPS-3 driver.

At the usual spot.

* Added -gl_alwayspow2texture switch which can fix OpenGL corruption on some cards/drivers (Sven Gothel)
* Fixed alt-enter fullscreen toggle in OpenGL mode (Sven Gothel)
* MinGW compile warning fixes/debugger stubs (Firewave)
* New unified makefile for baseline and SDLMAME. Note: SUBARCH is now TARGETOS! (Aaron Giles)

Smoke tested on Fedora 6 (x86) and Mac OS X 10.4.9 (Intel). Win32 is known not to build, this will be fixed later today. Play CPS3 on baseline until then ;-)

Last edited by R. Belmont; 06/22/07 12:31 AM.
Joined: Dec 2006
Posts: 148
Senior Member
Offline
Senior Member
Joined: Dec 2006
Posts: 148
Belmont,

Seriously, my jaw just hit the ground. I literally almost fell out of my chair. The CPS-3 performance in this build is incredible.

Last edited by Bob Boyle; 06/22/07 02:27 AM.
Joined: Feb 2005
Posts: 17
N
Member
Offline
Member
N
Joined: Feb 2005
Posts: 17
Whats your system? Whats the performance like?

Joined: Nov 2006
Posts: 239
S
Senior Member
Offline
Senior Member
S
Joined: Nov 2006
Posts: 239
Hi RB,

first of .. great, thank you.

Now the constructive criticism ;-)

1) clear_worker_param
memset is slower than the inlined 'clear_worker_param'
function. for small amounts of memory chunks, memset and memcpy
are much slower than a CPU implementation ..

2) videoperformancetest
you still don't love it .. sniff.
I thought, I could give you a good reason to keep it in,
since it is the only way to provide real video rendering performance numbers.

However, I am merging now, and will update my git repository.

Thank you.

-Sven



Joined: Mar 2001
Posts: 16,892
Likes: 51
R
Very Senior Member
OP Offline
Very Senior Member
R
Joined: Mar 2001
Posts: 16,892
Likes: 51
clear_worker_parm is 100% acceptable, I just need a diff against u2 that adds it.

videoperformancetest I still don't love, it's true. Part of it's that the output format isn't very readable.

Joined: Sep 2000
Posts: 234
W
Senior Member
Offline
Senior Member
W
Joined: Sep 2000
Posts: 234
I'm having trouble compiling which has never happened before. The makefile looks different. Is this correct?

ifndef TARGETOS
ifeq ($(OSD),windows)
TARGETOS = macosx
else
TARGETOS = macosx
endif
endif

because I'm getting this error:

Compiling src/osd/sdl/drawsdl.c...
src/osd/sdl/osinline.h: In function 'drawsdl_rgb888_draw_line':
src/osd/sdl/osinline.h:42: error: unknown register name '%edx' in 'asm'
src/osd/sdl/osinline.h:42: error: unknown register name '%edx' in 'asm'
make: *** [obj/sdl/mameg5/osd/sdl/drawsdl.o] Error 1


=will=
Joined: Nov 2006
Posts: 239
S
Senior Member
Offline
Senior Member
S
Joined: Nov 2006
Posts: 239
For your convenience, I have created a diff,
which 'fixes' the above 'issues'.

Be also aware, that one of your memset(&wp, ..)
is in the wrong spot, i.e. performed on a wp regardless of
'if OPENGL', even though that wp is only available in the OPENGL
code path.

Maybe you can reconsider the 'sdlvideofps' options.
(I changes the ugly option name from videoperformancetest ;-)
I strongly would appreciate this, with the clear_worker_param.

http://www.jausoft.ca/Files/GLMame/sdlmame0116u2_sdl_vs_sgothel_sdl.diff

Joined: Nov 2006
Posts: 239
S
Senior Member
Offline
Senior Member
S
Joined: Nov 2006
Posts: 239
Originally Posted By R. Belmont
clear_worker_parm is 100% acceptable, I just need a diff against u2 that adds it.


Ok.

Originally Posted By R. Belmont

videoperformancetest I still don't love, it's true. Part of it's that the output format isn't very readable.


Make a suggestion, just email me a 'demo' line, and it will be.

-Sven


Joined: Mar 2001
Posts: 16,892
Likes: 51
R
Very Senior Member
OP Offline
Very Senior Member
R
Joined: Mar 2001
Posts: 16,892
Likes: 51
Will, you're on PowerPC, right? I forgot I needed to change something for that from when Aaron sent me the "dry run" version. I'll fix it the same time I fix the Windows stuff.

For a quick fix, open src/osd/sdl/osinline.h in a text editor and add a line that says #define PPC_ASM at the top. That should get you going.

Sven: Thanks for the diff, I'll take a look. "sdlvideofps" is definitely less ugly smile

Last edited by R. Belmont; 06/22/07 04:25 AM.
Joined: May 2000
Posts: 275
Senior Member
Offline
Senior Member
Joined: May 2000
Posts: 275
@RB: This release seems to drop the CFLAGS += $(OPT_FLAGS) which I need for Fedora RPMs. I guess it got lost in the makefile changes.

Page 1 of 5 1 2 3 4 5

Moderated by  R. Belmont 

Link Copied to Clipboard
Who's Online Now
2 members (Rik, 1 invisible), 23 guests, and 2 robots.
Key: Admin, Global Mod, Mod
ShoutChat
Comment Guidelines: Do post respectful and insightful comments. Don't flame, hate, spam.
Forum Statistics
Forums9
Topics9,071
Posts118,945
Members5,014
Most Online890
Jan 17th, 2020
Our Sponsor
These forums are sponsored by Superior Solitaire, an ad-free card game collection for macOS and iOS. Download it today!

Superior Solitaire
Forum hosted by www.retrogamesformac.com