Mercurial > sdl-ios-xcode
annotate src/video/x11/SDL_x11yuv.c @ 434:ed58b98c0d9d
Commented double-free buggy code. Will examine more closely later.
| author | Ryan C. Gordon <icculus@icculus.org> |
|---|---|
| date | Mon, 12 Aug 2002 14:54:38 +0000 |
| parents | c1666427297c |
| children | 95aa4041e7f9 |
| rev | line source |
|---|---|
| 0 | 1 /* |
| 2 SDL - Simple DirectMedia Layer | |
|
297
f6ffac90895c
Updated copyright information for 2002
Sam Lantinga <slouken@libsdl.org>
parents:
292
diff
changeset
|
3 Copyright (C) 1997, 1998, 1999, 2000, 2001, 2002 Sam Lantinga |
| 0 | 4 |
| 5 This library is free software; you can redistribute it and/or | |
| 6 modify it under the terms of the GNU Library General Public | |
| 7 License as published by the Free Software Foundation; either | |
| 8 version 2 of the License, or (at your option) any later version. | |
| 9 | |
| 10 This library is distributed in the hope that it will be useful, | |
| 11 but WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 12 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | |
| 13 Library General Public License for more details. | |
| 14 | |
| 15 You should have received a copy of the GNU Library General Public | |
| 16 License along with this library; if not, write to the Free | |
| 17 Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA | |
| 18 | |
| 19 Sam Lantinga | |
|
252
e8157fcb3114
Updated the source with the correct e-mail address
Sam Lantinga <slouken@libsdl.org>
parents:
182
diff
changeset
|
20 slouken@libsdl.org |
| 0 | 21 */ |
| 22 | |
| 23 #ifdef SAVE_RCSID | |
| 24 static char rcsid = | |
| 25 "@(#) $Id$"; | |
| 26 #endif | |
| 27 | |
| 28 /* This is the XFree86 Xv extension implementation of YUV video overlays */ | |
| 29 | |
| 30 #ifdef XFREE86_XV | |
| 31 | |
| 32 #include <stdlib.h> | |
| 33 #include <string.h> | |
| 34 #include <X11/Xlib.h> | |
| 35 #include <sys/ipc.h> | |
| 36 #include <sys/shm.h> | |
| 37 #include <X11/extensions/XShm.h> | |
|
292
eadc0746dfaf
Added SDL_LockRect() and SDL_UnlockRect()
Sam Lantinga <slouken@libsdl.org>
parents:
252
diff
changeset
|
38 #include <XFree86/extensions/Xvlib.h> |
|
429
c1666427297c
Memory leak fixes by Bill May.
Ryan C. Gordon <icculus@icculus.org>
parents:
410
diff
changeset
|
39 #include <X11/Xlibint.h> |
| 0 | 40 |
| 41 #include "SDL_error.h" | |
| 42 #include "SDL_video.h" | |
| 43 #include "SDL_x11yuv_c.h" | |
| 44 #include "SDL_yuvfuncs.h" | |
| 45 | |
| 46 #define XFREE86_REFRESH_HACK | |
| 47 #ifdef XFREE86_REFRESH_HACK | |
| 48 #include "SDL_x11image_c.h" | |
| 49 #endif | |
| 50 | |
| 51 /* Workaround when pitch != width */ | |
| 52 #define PITCH_WORKAROUND | |
| 53 | |
|
182
d4ebd1bbea9a
Fix XVideo on GeForce by using last available adaptor
Sam Lantinga <slouken@libsdl.org>
parents:
0
diff
changeset
|
54 /* Fix for the NVidia GeForce 2 - use the last available adaptor */ |
|
341
6ca967a46bf1
*** empty log message ***
Sam Lantinga <slouken@libsdl.org>
parents:
297
diff
changeset
|
55 /*#define USE_LAST_ADAPTOR*/ /* Apparently the NVidia drivers are fixed */ |
|
182
d4ebd1bbea9a
Fix XVideo on GeForce by using last available adaptor
Sam Lantinga <slouken@libsdl.org>
parents:
0
diff
changeset
|
56 |
| 0 | 57 /* The functions used to manipulate software video overlays */ |
| 58 static struct private_yuvhwfuncs x11_yuvfuncs = { | |
| 59 X11_LockYUVOverlay, | |
| 60 X11_UnlockYUVOverlay, | |
| 61 X11_DisplayYUVOverlay, | |
| 62 X11_FreeYUVOverlay | |
| 63 }; | |
| 64 | |
| 65 struct private_yuvhwdata { | |
| 66 int port; | |
| 67 XShmSegmentInfo yuvshm; | |
|
292
eadc0746dfaf
Added SDL_LockRect() and SDL_UnlockRect()
Sam Lantinga <slouken@libsdl.org>
parents:
252
diff
changeset
|
68 SDL_NAME(XvImage) *image; |
| 0 | 69 }; |
| 70 | |
| 71 | |
| 72 SDL_Overlay *X11_CreateYUVOverlay(_THIS, int width, int height, Uint32 format, SDL_Surface *display) | |
| 73 { | |
| 74 SDL_Overlay *overlay; | |
| 75 struct private_yuvhwdata *hwdata; | |
| 76 int xv_port; | |
| 77 int i, j, k; | |
| 78 int adaptors; | |
|
292
eadc0746dfaf
Added SDL_LockRect() and SDL_UnlockRect()
Sam Lantinga <slouken@libsdl.org>
parents:
252
diff
changeset
|
79 SDL_NAME(XvAdaptorInfo) *ainfo; |
| 0 | 80 XShmSegmentInfo *yuvshm; |
| 81 | |
| 82 /* Look for the XVideo extension with a valid port for this format */ | |
| 83 xv_port = -1; | |
|
292
eadc0746dfaf
Added SDL_LockRect() and SDL_UnlockRect()
Sam Lantinga <slouken@libsdl.org>
parents:
252
diff
changeset
|
84 if ( (Success == SDL_NAME(XvQueryExtension)(GFX_Display, &j, &j, &j, &j, &j)) && |
|
eadc0746dfaf
Added SDL_LockRect() and SDL_UnlockRect()
Sam Lantinga <slouken@libsdl.org>
parents:
252
diff
changeset
|
85 (Success == SDL_NAME(XvQueryAdaptors)(GFX_Display, |
| 0 | 86 RootWindow(GFX_Display, SDL_Screen), |
| 87 &adaptors, &ainfo)) ) { | |
|
182
d4ebd1bbea9a
Fix XVideo on GeForce by using last available adaptor
Sam Lantinga <slouken@libsdl.org>
parents:
0
diff
changeset
|
88 #ifdef USE_LAST_ADAPTOR |
|
d4ebd1bbea9a
Fix XVideo on GeForce by using last available adaptor
Sam Lantinga <slouken@libsdl.org>
parents:
0
diff
changeset
|
89 for ( i=0; i < adaptors; ++i ) { |
|
d4ebd1bbea9a
Fix XVideo on GeForce by using last available adaptor
Sam Lantinga <slouken@libsdl.org>
parents:
0
diff
changeset
|
90 #else |
| 0 | 91 for ( i=0; (i < adaptors) && (xv_port == -1); ++i ) { |
|
182
d4ebd1bbea9a
Fix XVideo on GeForce by using last available adaptor
Sam Lantinga <slouken@libsdl.org>
parents:
0
diff
changeset
|
92 #endif /* USE_LAST_ADAPTOR */ |
| 0 | 93 /* Check to see if the visual can be used */ |
| 94 if ( BUGGY_XFREE86(<=, 4001) ) { | |
| 95 int visual_ok = 0; | |
| 96 for ( j=0; j<ainfo[i].num_formats; ++j ) { | |
| 97 if ( ainfo[i].formats[j].visual_id == | |
| 98 SDL_Visual->visualid ) { | |
| 99 visual_ok = 1; | |
| 100 break; | |
| 101 } | |
| 102 } | |
| 103 if ( ! visual_ok ) { | |
| 104 continue; | |
| 105 } | |
| 106 } | |
| 107 if ( (ainfo[i].type & XvInputMask) && | |
| 108 (ainfo[i].type & XvImageMask) ) { | |
| 109 int num_formats; | |
|
292
eadc0746dfaf
Added SDL_LockRect() and SDL_UnlockRect()
Sam Lantinga <slouken@libsdl.org>
parents:
252
diff
changeset
|
110 SDL_NAME(XvImageFormatValues) *formats; |
|
eadc0746dfaf
Added SDL_LockRect() and SDL_UnlockRect()
Sam Lantinga <slouken@libsdl.org>
parents:
252
diff
changeset
|
111 formats = SDL_NAME(XvListImageFormats)(GFX_Display, |
| 0 | 112 ainfo[i].base_id, &num_formats); |
|
182
d4ebd1bbea9a
Fix XVideo on GeForce by using last available adaptor
Sam Lantinga <slouken@libsdl.org>
parents:
0
diff
changeset
|
113 #ifdef USE_LAST_ADAPTOR |
|
d4ebd1bbea9a
Fix XVideo on GeForce by using last available adaptor
Sam Lantinga <slouken@libsdl.org>
parents:
0
diff
changeset
|
114 for ( j=0; j < num_formats; ++j ) { |
|
d4ebd1bbea9a
Fix XVideo on GeForce by using last available adaptor
Sam Lantinga <slouken@libsdl.org>
parents:
0
diff
changeset
|
115 #else |
| 0 | 116 for ( j=0; (j < num_formats) && (xv_port == -1); ++j ) { |
|
182
d4ebd1bbea9a
Fix XVideo on GeForce by using last available adaptor
Sam Lantinga <slouken@libsdl.org>
parents:
0
diff
changeset
|
117 #endif /* USE_LAST_ADAPTOR */ |
| 0 | 118 if ( (Uint32)formats[j].id == format ) { |
| 119 for ( k=0; k < ainfo[i].num_ports; ++k ) { | |
|
292
eadc0746dfaf
Added SDL_LockRect() and SDL_UnlockRect()
Sam Lantinga <slouken@libsdl.org>
parents:
252
diff
changeset
|
120 if ( Success == SDL_NAME(XvGrabPort)(GFX_Display, ainfo[i].base_id+k, CurrentTime) ) { |
| 0 | 121 xv_port = ainfo[i].base_id+k; |
| 122 break; | |
| 123 } | |
| 124 } | |
| 125 } | |
| 126 } | |
|
410
365f57b7c4ac
*** empty log message ***
Sam Lantinga <slouken@libsdl.org>
parents:
409
diff
changeset
|
127 if ( formats ) { |
|
365f57b7c4ac
*** empty log message ***
Sam Lantinga <slouken@libsdl.org>
parents:
409
diff
changeset
|
128 XFree(formats); |
|
365f57b7c4ac
*** empty log message ***
Sam Lantinga <slouken@libsdl.org>
parents:
409
diff
changeset
|
129 } |
| 0 | 130 } |
| 131 } | |
|
409
1c3cf7231cd4
Fixed memory leaks in YUV code
Sam Lantinga <slouken@libsdl.org>
parents:
341
diff
changeset
|
132 SDL_NAME(XvFreeAdaptorInfo)(ainfo); |
| 0 | 133 } |
|
429
c1666427297c
Memory leak fixes by Bill May.
Ryan C. Gordon <icculus@icculus.org>
parents:
410
diff
changeset
|
134 |
|
434
ed58b98c0d9d
Commented double-free buggy code. Will examine more closely later.
Ryan C. Gordon <icculus@icculus.org>
parents:
429
diff
changeset
|
135 #if 0 |
|
429
c1666427297c
Memory leak fixes by Bill May.
Ryan C. Gordon <icculus@icculus.org>
parents:
410
diff
changeset
|
136 /* |
|
c1666427297c
Memory leak fixes by Bill May.
Ryan C. Gordon <icculus@icculus.org>
parents:
410
diff
changeset
|
137 * !!! FIXME: |
|
c1666427297c
Memory leak fixes by Bill May.
Ryan C. Gordon <icculus@icculus.org>
parents:
410
diff
changeset
|
138 * "Here are some diffs for X11 and yuv. Note that the last part 2nd |
|
c1666427297c
Memory leak fixes by Bill May.
Ryan C. Gordon <icculus@icculus.org>
parents:
410
diff
changeset
|
139 * diff should probably be a new call to XvQueryAdaptorFree with ainfo |
|
c1666427297c
Memory leak fixes by Bill May.
Ryan C. Gordon <icculus@icculus.org>
parents:
410
diff
changeset
|
140 * and the number of adaptors, instead of the loop through like I did." |
|
434
ed58b98c0d9d
Commented double-free buggy code. Will examine more closely later.
Ryan C. Gordon <icculus@icculus.org>
parents:
429
diff
changeset
|
141 * |
|
ed58b98c0d9d
Commented double-free buggy code. Will examine more closely later.
Ryan C. Gordon <icculus@icculus.org>
parents:
429
diff
changeset
|
142 * ACHTUNG: This is broken! It looks like XvFreeAdaptorInfo does this |
|
ed58b98c0d9d
Commented double-free buggy code. Will examine more closely later.
Ryan C. Gordon <icculus@icculus.org>
parents:
429
diff
changeset
|
143 * for you, so we end up with a double-free. I need to look at this |
|
ed58b98c0d9d
Commented double-free buggy code. Will examine more closely later.
Ryan C. Gordon <icculus@icculus.org>
parents:
429
diff
changeset
|
144 * more closely... --ryan. |
|
429
c1666427297c
Memory leak fixes by Bill May.
Ryan C. Gordon <icculus@icculus.org>
parents:
410
diff
changeset
|
145 */ |
|
c1666427297c
Memory leak fixes by Bill May.
Ryan C. Gordon <icculus@icculus.org>
parents:
410
diff
changeset
|
146 for ( i=0; i < adaptors; ++i ) { |
|
c1666427297c
Memory leak fixes by Bill May.
Ryan C. Gordon <icculus@icculus.org>
parents:
410
diff
changeset
|
147 if (ainfo[i].name != NULL) Xfree(ainfo[i].name); |
|
c1666427297c
Memory leak fixes by Bill May.
Ryan C. Gordon <icculus@icculus.org>
parents:
410
diff
changeset
|
148 if (ainfo[i].formats != NULL) Xfree(ainfo[i].formats); |
|
c1666427297c
Memory leak fixes by Bill May.
Ryan C. Gordon <icculus@icculus.org>
parents:
410
diff
changeset
|
149 } |
|
c1666427297c
Memory leak fixes by Bill May.
Ryan C. Gordon <icculus@icculus.org>
parents:
410
diff
changeset
|
150 Xfree(ainfo); |
|
434
ed58b98c0d9d
Commented double-free buggy code. Will examine more closely later.
Ryan C. Gordon <icculus@icculus.org>
parents:
429
diff
changeset
|
151 #endif |
|
429
c1666427297c
Memory leak fixes by Bill May.
Ryan C. Gordon <icculus@icculus.org>
parents:
410
diff
changeset
|
152 |
| 0 | 153 if ( xv_port == -1 ) { |
| 154 SDL_SetError("No available video ports for requested format"); | |
| 155 return(NULL); | |
| 156 } | |
| 157 | |
| 158 /* Create the overlay structure */ | |
| 159 overlay = (SDL_Overlay *)malloc(sizeof *overlay); | |
| 160 if ( overlay == NULL ) { | |
|
292
eadc0746dfaf
Added SDL_LockRect() and SDL_UnlockRect()
Sam Lantinga <slouken@libsdl.org>
parents:
252
diff
changeset
|
161 SDL_NAME(XvUngrabPort)(GFX_Display, xv_port, CurrentTime); |
| 0 | 162 SDL_OutOfMemory(); |
| 163 return(NULL); | |
| 164 } | |
| 165 memset(overlay, 0, (sizeof *overlay)); | |
| 166 | |
| 167 /* Fill in the basic members */ | |
| 168 overlay->format = format; | |
| 169 overlay->w = width; | |
| 170 overlay->h = height; | |
| 171 | |
| 172 /* Set up the YUV surface function structure */ | |
| 173 overlay->hwfuncs = &x11_yuvfuncs; | |
| 174 overlay->hw_overlay = 1; | |
| 175 | |
| 176 /* Create the pixel data and lookup tables */ | |
| 177 hwdata = (struct private_yuvhwdata *)malloc(sizeof *hwdata); | |
| 178 overlay->hwdata = hwdata; | |
| 179 if ( hwdata == NULL ) { | |
|
292
eadc0746dfaf
Added SDL_LockRect() and SDL_UnlockRect()
Sam Lantinga <slouken@libsdl.org>
parents:
252
diff
changeset
|
180 SDL_NAME(XvUngrabPort)(GFX_Display, xv_port, CurrentTime); |
| 0 | 181 SDL_OutOfMemory(); |
| 182 SDL_FreeYUVOverlay(overlay); | |
| 183 return(NULL); | |
| 184 } | |
| 185 yuvshm = &hwdata->yuvshm; | |
| 186 memset(yuvshm, 0, sizeof(*yuvshm)); | |
| 187 hwdata->port = xv_port; | |
|
292
eadc0746dfaf
Added SDL_LockRect() and SDL_UnlockRect()
Sam Lantinga <slouken@libsdl.org>
parents:
252
diff
changeset
|
188 hwdata->image = SDL_NAME(XvShmCreateImage)(GFX_Display, xv_port, format, |
| 0 | 189 0, width, height, yuvshm); |
| 190 | |
| 191 #ifdef PITCH_WORKAROUND | |
| 192 if ( hwdata->image != NULL && hwdata->image->pitches[0] != width ) | |
| 193 { | |
| 194 /* Ajust overlay width according to pitch */ | |
| 195 switch (format) { | |
| 196 case SDL_YV12_OVERLAY: | |
| 197 case SDL_IYUV_OVERLAY: | |
| 198 width = hwdata->image->pitches[0]; | |
| 199 break; | |
| 200 case SDL_YUY2_OVERLAY: | |
| 201 case SDL_UYVY_OVERLAY: | |
| 202 case SDL_YVYU_OVERLAY: | |
| 203 width = hwdata->image->pitches[0] / 2; | |
| 204 break; | |
| 205 default: | |
| 206 /* We should never get here (caught above) */ | |
| 207 return(NULL); | |
| 208 } | |
| 209 | |
| 210 XFree(hwdata->image); | |
|
292
eadc0746dfaf
Added SDL_LockRect() and SDL_UnlockRect()
Sam Lantinga <slouken@libsdl.org>
parents:
252
diff
changeset
|
211 hwdata->image = SDL_NAME(XvShmCreateImage)(GFX_Display, xv_port, format, |
| 0 | 212 0, width, height, yuvshm); |
| 213 } | |
| 214 #endif | |
| 215 | |
| 216 if ( hwdata->image == NULL ) { | |
| 217 SDL_OutOfMemory(); | |
| 218 SDL_FreeYUVOverlay(overlay); | |
| 219 return(NULL); | |
| 220 } | |
| 221 yuvshm->shmid = shmget(IPC_PRIVATE, hwdata->image->data_size, | |
| 222 IPC_CREAT | 0777); | |
| 223 if ( yuvshm->shmid < 0 ) { | |
| 224 SDL_SetError("Unable to get %d bytes shared memory", | |
| 225 hwdata->image->data_size); | |
| 226 SDL_FreeYUVOverlay(overlay); | |
| 227 return(NULL); | |
| 228 } | |
| 229 yuvshm->shmaddr = (char *) shmat(yuvshm->shmid, 0, 0); | |
| 230 yuvshm->readOnly = False; | |
| 231 hwdata->image->data = yuvshm->shmaddr; | |
| 232 | |
| 233 XShmAttach(GFX_Display, yuvshm); | |
| 234 XSync(GFX_Display, False); | |
| 235 shmctl(yuvshm->shmid, IPC_RMID, 0); | |
| 236 | |
| 237 /* Find the pitch and offset values for the overlay */ | |
| 238 overlay->planes = hwdata->image->num_planes; | |
| 239 overlay->pitches = (Uint16 *)malloc(overlay->planes * sizeof(Uint16)); | |
| 240 overlay->pixels = (Uint8 **)malloc(overlay->planes * sizeof(Uint8 *)); | |
| 241 if ( !overlay->pitches || !overlay->pixels ) { | |
| 242 SDL_OutOfMemory(); | |
| 243 SDL_FreeYUVOverlay(overlay); | |
| 244 return(NULL); | |
| 245 } | |
| 246 for ( i=0; i<overlay->planes; ++i ) { | |
| 247 overlay->pitches[i] = hwdata->image->pitches[i]; | |
| 248 overlay->pixels[i] = (Uint8 *)hwdata->image->data + | |
| 249 hwdata->image->offsets[i]; | |
| 250 } | |
| 251 | |
| 252 #ifdef XFREE86_REFRESH_HACK | |
| 253 /* Work around an XFree86 X server bug (?) | |
| 254 We can't perform normal updates in windows that have video | |
| 255 being output to them. See SDL_x11image.c for more details. | |
| 256 */ | |
| 257 X11_DisableAutoRefresh(this); | |
| 258 #endif | |
| 259 | |
| 260 /* We're all done.. */ | |
| 261 return(overlay); | |
| 262 } | |
| 263 | |
| 264 int X11_LockYUVOverlay(_THIS, SDL_Overlay *overlay) | |
| 265 { | |
| 266 return(0); | |
| 267 } | |
| 268 | |
| 269 void X11_UnlockYUVOverlay(_THIS, SDL_Overlay *overlay) | |
| 270 { | |
| 271 return; | |
| 272 } | |
| 273 | |
| 274 int X11_DisplayYUVOverlay(_THIS, SDL_Overlay *overlay, SDL_Rect *dstrect) | |
| 275 { | |
| 276 struct private_yuvhwdata *hwdata; | |
| 277 | |
| 278 hwdata = overlay->hwdata; | |
|
292
eadc0746dfaf
Added SDL_LockRect() and SDL_UnlockRect()
Sam Lantinga <slouken@libsdl.org>
parents:
252
diff
changeset
|
279 SDL_NAME(XvShmPutImage)(GFX_Display, hwdata->port, SDL_Window, SDL_GC, |
| 0 | 280 hwdata->image, 0, 0, overlay->w, overlay->h, |
| 281 dstrect->x, dstrect->y, dstrect->w, dstrect->h, False); | |
| 282 XSync(GFX_Display, False); | |
| 283 return(0); | |
| 284 } | |
| 285 | |
| 286 void X11_FreeYUVOverlay(_THIS, SDL_Overlay *overlay) | |
| 287 { | |
| 288 struct private_yuvhwdata *hwdata; | |
| 289 | |
| 290 hwdata = overlay->hwdata; | |
| 291 if ( hwdata ) { | |
|
292
eadc0746dfaf
Added SDL_LockRect() and SDL_UnlockRect()
Sam Lantinga <slouken@libsdl.org>
parents:
252
diff
changeset
|
292 SDL_NAME(XvUngrabPort)(GFX_Display, hwdata->port, CurrentTime); |
| 0 | 293 if ( hwdata->yuvshm.shmaddr ) { |
| 294 XShmDetach(GFX_Display, &hwdata->yuvshm); | |
| 295 shmdt(hwdata->yuvshm.shmaddr); | |
| 296 } | |
| 297 if ( hwdata->image ) { | |
| 298 XFree(hwdata->image); | |
| 299 } | |
| 300 free(hwdata); | |
| 301 } | |
| 302 if ( overlay->pitches ) { | |
| 303 free(overlay->pitches); | |
| 304 overlay->pitches = NULL; | |
| 305 } | |
| 306 if ( overlay->pixels ) { | |
| 307 free(overlay->pixels); | |
| 308 overlay->pixels = NULL; | |
| 309 } | |
| 310 #ifdef XFREE86_REFRESH_HACK | |
| 311 X11_EnableAutoRefresh(this); | |
| 312 #endif | |
| 313 } | |
| 314 | |
| 315 #endif /* XFREE86_XV */ |