Mercurial > sdl-ios-xcode
annotate src/video/directfb/SDL_DirectFB_dyn.h @ 5053:b5b42be9333c
Fixed bug #1026
Vittorio Giovara 2010-07-16 19:09:28 PDT
i was reading SDL_renderer_gles and i noticed that every time we there
is some gl call the gl state is modified with a couple of
glEnableClientState()/glDisableClientState.
While this is completely fine for desktops systems, this is a major
performace kill on mobile devices, right where opengles is
implemented.
Normal practice in this case is to update the glstate once, keep it
always the same and disable/enable other states only in very special
occasions.
On the web there's plenty of documentation (on the top of my head
http://developer.apple.com/iphone/library/documentation/3DDrawing/Conceptual/OpenGLES_ProgrammingGuide/Performance/Performance.html#//apple_ref/doc/uid/TP40008793-CH105-SW5
) and i personally tried this.
I modified my code and got a 10 fps boost, then modified SDL_render_gles and
shifted from 40 fps to 50 fps alone -- considering that i started from ~30fps i
got an 80% performance increase with this technique.
I have attached a dif of my changes, hope that it will be included in
mainstream.
author | Sam Lantinga <slouken@libsdl.org> |
---|---|
date | Wed, 19 Jan 2011 23:56:16 -0800 |
parents | f7b03b6838cb |
children | 164f20ba08eb |
rev | line source |
---|---|
3000
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
1 /* |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
2 SDL - Simple DirectMedia Layer |
3697 | 3 Copyright (C) 1997-2010 Sam Lantinga |
3000
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
4 |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
5 This library is free software; you can redistribute it and/or |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
6 modify it under the terms of the GNU Lesser General Public |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
7 License as published by the Free Software Foundation; either |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
8 version 2.1 of the License, or (at your option) any later version. |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
9 |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
10 This library is distributed in the hope that it will be useful, |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
11 but WITHOUT ANY WARRANTY; without even the implied warranty of |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
12 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
13 Lesser General Public License for more details. |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
14 |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
15 You should have received a copy of the GNU Lesser General Public |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
16 License along with this library; if not, write to the Free Software |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
17 Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
18 |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
19 Sam Lantinga |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
20 slouken@libsdl.org |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
21 */ |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
22 |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
23 #ifndef _SDL_DirectFB_dyn_h |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
24 #define _SDL_DirectFB_dyn_h |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
25 |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
26 #define DFB_SYMS \ |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
27 DFB_SYM(DFBResult, DirectFBError, (const char *msg, DFBResult result), (msg, result), return) \ |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
28 DFB_SYM(DFBResult, DirectFBErrorFatal, (const char *msg, DFBResult result), (msg, result), return) \ |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
29 DFB_SYM(const char *, DirectFBErrorString, (DFBResult result), (result), return) \ |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
30 DFB_SYM(const char *, DirectFBUsageString, ( void ), (), return) \ |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
31 DFB_SYM(DFBResult, DirectFBInit, (int *argc, char *(*argv[]) ), (argc, argv), return) \ |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
32 DFB_SYM(DFBResult, DirectFBSetOption, (const char *name, const char *value), (name, value), return) \ |
3023
d72a0dd80e8b
DirectFB cleanups & simple window manager
Couriersud <couriersud@arcor.de>
parents:
3000
diff
changeset
|
33 DFB_SYM(DFBResult, DirectFBCreate, (IDirectFB **interface), (interface), return) \ |
d72a0dd80e8b
DirectFB cleanups & simple window manager
Couriersud <couriersud@arcor.de>
parents:
3000
diff
changeset
|
34 DFB_SYM(const char *, DirectFBCheckVersion, (unsigned int required_major, unsigned int required_minor, unsigned int required_micro), \ |
d72a0dd80e8b
DirectFB cleanups & simple window manager
Couriersud <couriersud@arcor.de>
parents:
3000
diff
changeset
|
35 (required_major, required_minor, required_micro), return) |
3000
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
36 // #define SDL_VIDEO_DRIVER_DIRECTFB_DYNAMIC "/usr/lib/libdirectfb-1.2.so.0" |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
37 |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
38 int SDL_DirectFB_LoadLibrary(void); |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
39 void SDL_DirectFB_UnLoadLibrary(void); |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
40 |
61081db2385a
Whoops, missing files from Couriersud's update
Sam Lantinga <slouken@libsdl.org>
parents:
diff
changeset
|
41 #endif |