comparison src/audio/esd/SDL_esdaudio.c @ 3162:dc1eb82ffdaa

Von: Thomas Zimmermann Betreff: [SDL] [PATCH] Make static variables const Datum: Tue, 19 May 2009 19:45:37 +0200 Hi, this is a set of simple changes which make some of SDL's internal static arrays constant. The purpose is to shrink the number of write-able static bytes and thus increase the number of memory pages shared between SDL applications. The patch set is against trunk@4513. Each of the attached patch files is specific to a sub-system. The set is completed by a second mail, because of the list's 40 KiB limit. The files readelf-r4513.txt and readelf-const-patch.txt where made by calling 'readelf -S libSDL.so'. They show the difference in ELF sections without and with the patch. Some numbers measured on my x86-64: Before [13] .rodata PROGBITS 00000000000eaaa0 000eaaa0 0000000000008170 0000000000000000 A 0 0 32 [19] .data.rel.ro PROGBITS 00000000003045e0 001045e0 00000000000023d0 0000000000000000 WA 0 0 32 [23] .data PROGBITS 00000000003076e0 001076e0 0000000000004988 0000000000000000 WA 0 0 32 After [13] .rodata PROGBITS 00000000000eaaa0 000eaaa0 0000000000009a50 0000000000000000 A 0 0 32 [19] .data.rel.ro PROGBITS 0000000000306040 00106040 0000000000002608 0000000000000000 WA 0 0 32 [23] .data PROGBITS 0000000000309360 00109360 0000000000002e88 0000000000000000 WA 0 0 32 The size of the write-able data section decreased considerably. Some entries became const-after-relocation, while most of its content went straight into the read-only data section. Best regards, Thomas
author Sam Lantinga <slouken@libsdl.org>
date Wed, 03 Jun 2009 04:37:27 +0000
parents b21348d47cab
children 15eea7a1fa97
comparison
equal deleted inserted replaced
3161:494559cc723b 3162:dc1eb82ffdaa
58 #define SDL_ESD_SYM(x) { #x, (void **) (char *) &SDL_NAME(x) } 58 #define SDL_ESD_SYM(x) { #x, (void **) (char *) &SDL_NAME(x) }
59 static struct 59 static struct
60 { 60 {
61 const char *name; 61 const char *name;
62 void **func; 62 void **func;
63 } esd_functions[] = { 63 } const esd_functions[] = {
64 SDL_ESD_SYM(esd_open_sound), 64 SDL_ESD_SYM(esd_open_sound),
65 SDL_ESD_SYM(esd_close), SDL_ESD_SYM(esd_play_stream),}; 65 SDL_ESD_SYM(esd_close), SDL_ESD_SYM(esd_play_stream),
66 };
67
66 #undef SDL_ESD_SYM 68 #undef SDL_ESD_SYM
67 69
68 static void 70 static void
69 UnloadESDLibrary() 71 UnloadESDLibrary()
70 { 72 {