geemili
/
drm
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

radeon: Don't mess up page flipping when a file descriptor is closed. 

There can still be other contexts that may use page flipping later on, so don't
just unilaterally 'clean it up', which could lead to the wrong page being
displayed, e.g. when running 3D apps with a GLX compositing manager such as
compiz using page flipping.
main
Michel Dänzer 2007-04-29 12:17:57 +02:00
parent feb6803778
commit ca1cd3257c
2 changed files with 19 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
shared-core/radeon_drv.h
View File

@ -245,7 +245,6 @@ typedef struct drm_radeon_private {
int do_boxes; int do_boxes;
int page_flipping; int page_flipping;
int current_page;
u32 color_fmt; u32 color_fmt;
unsigned int front_offset; unsigned int front_offset;

46
shared-core/radeon_state.c
View File

@ -773,7 +773,7 @@ static void radeon_clear_box(drm_radeon_private_t * dev_priv,
RADEON_GMC_SRC_DATATYPE_COLOR | RADEON_GMC_SRC_DATATYPE_COLOR |
RADEON_ROP3_P | RADEON_GMC_CLR_CMP_CNTL_DIS); RADEON_ROP3_P | RADEON_GMC_CLR_CMP_CNTL_DIS);
if (dev_priv->page_flipping && dev_priv->current_page == 1) { if (dev_priv->sarea_priv->pfCurrentPage == 1) {
OUT_RING(dev_priv->front_pitch_offset); OUT_RING(dev_priv->front_pitch_offset);
} else { } else {
OUT_RING(dev_priv->back_pitch_offset); OUT_RING(dev_priv->back_pitch_offset);
@ -861,7 +861,7 @@ static void radeon_cp_dispatch_clear(drm_device_t * dev,
dev_priv->stats.clears++; dev_priv->stats.clears++;
if (dev_priv->page_flipping && dev_priv->current_page == 1) { if (dev_priv->sarea_priv->pfCurrentPage == 1) {
unsigned int tmp = flags; unsigned int tmp = flags;
flags &= ~(RADEON_FRONT | RADEON_BACK); flags &= ~(RADEON_FRONT | RADEON_BACK);
@ -1382,7 +1382,7 @@ static void radeon_cp_dispatch_swap(drm_device_t * dev)
/* Make this work even if front & back are flipped: /* Make this work even if front & back are flipped:
*/ */
OUT_RING(CP_PACKET0(RADEON_SRC_PITCH_OFFSET, 1)); OUT_RING(CP_PACKET0(RADEON_SRC_PITCH_OFFSET, 1));
if (dev_priv->current_page == 0) { if (dev_priv->sarea_priv->pfCurrentPage == 0) {
OUT_RING(dev_priv->back_pitch_offset); OUT_RING(dev_priv->back_pitch_offset);
OUT_RING(dev_priv->front_pitch_offset); OUT_RING(dev_priv->front_pitch_offset);
} else { } else {
@ -1416,12 +1416,12 @@ static void radeon_cp_dispatch_flip(drm_device_t * dev)
{ {
drm_radeon_private_t *dev_priv = dev->dev_private; drm_radeon_private_t *dev_priv = dev->dev_private;
drm_sarea_t *sarea = (drm_sarea_t *) dev_priv->sarea->handle; drm_sarea_t *sarea = (drm_sarea_t *) dev_priv->sarea->handle;
int offset = (dev_priv->current_page == 1) int offset = (dev_priv->sarea_priv->pfCurrentPage == 1)
? dev_priv->front_offset : dev_priv->back_offset; ? dev_priv->front_offset : dev_priv->back_offset;
RING_LOCALS; RING_LOCALS;
DRM_DEBUG("%s: page=%d pfCurrentPage=%d\n", DRM_DEBUG("%s: pfCurrentPage=%d\n",
__FUNCTION__, __FUNCTION__,
dev_priv->current_page, dev_priv->sarea_priv->pfCurrentPage); dev_priv->sarea_priv->pfCurrentPage);
/* Do some trivial performance monitoring... /* Do some trivial performance monitoring...
*/ */
@ -1449,8 +1449,8 @@ static void radeon_cp_dispatch_flip(drm_device_t * dev)
* performing the swapbuffer ioctl. * performing the swapbuffer ioctl.
*/ */
dev_priv->sarea_priv->last_frame++; dev_priv->sarea_priv->last_frame++;
dev_priv->sarea_priv->pfCurrentPage = dev_priv->current_page = dev_priv->sarea_priv->pfCurrentPage =
1 - dev_priv->current_page; 1 - dev_priv->sarea_priv->pfCurrentPage;
BEGIN_RING(2); BEGIN_RING(2);
@ -2162,24 +2162,10 @@ static int radeon_do_init_pageflip(drm_device_t * dev)
ADVANCE_RING(); ADVANCE_RING();
dev_priv->page_flipping = 1; dev_priv->page_flipping = 1;
dev_priv->current_page = 0;
dev_priv->sarea_priv->pfCurrentPage = dev_priv->current_page;
return 0; if (dev_priv->sarea_priv->pfCurrentPage != 1)
} dev_priv->sarea_priv->pfCurrentPage = 0;
/* Called whenever a client dies, from drm_release.
* NOTE: Lock isn't necessarily held when this is called!
*/
static int radeon_do_cleanup_pageflip(drm_device_t * dev)
{
drm_radeon_private_t *dev_priv = dev->dev_private;
DRM_DEBUG("\n");
if (dev_priv->current_page != 0)
radeon_cp_dispatch_flip(dev);
dev_priv->page_flipping = 0;
return 0; return 0;
} }
@ -3225,9 +3211,7 @@ void radeon_driver_preclose(drm_device_t * dev, DRMFILE filp)
{ {
if (dev->dev_private) { if (dev->dev_private) {
drm_radeon_private_t *dev_priv = dev->dev_private; drm_radeon_private_t *dev_priv = dev->dev_private;
if (dev_priv->page_flipping) { dev_priv->page_flipping = 0;
radeon_do_cleanup_pageflip(dev);
}
radeon_mem_release(filp, dev_priv->gart_heap); radeon_mem_release(filp, dev_priv->gart_heap);
radeon_mem_release(filp, dev_priv->fb_heap); radeon_mem_release(filp, dev_priv->fb_heap);
radeon_surfaces_release(filp, dev_priv); radeon_surfaces_release(filp, dev_priv);
@ -3236,6 +3220,14 @@ void radeon_driver_preclose(drm_device_t * dev, DRMFILE filp)
void radeon_driver_lastclose(drm_device_t * dev) void radeon_driver_lastclose(drm_device_t * dev)
{ {
if (dev->dev_private) {
drm_radeon_private_t *dev_priv = dev->dev_private;
if (dev_priv->sarea_priv &&
dev_priv->sarea_priv->pfCurrentPage != 0)
radeon_cp_dispatch_flip(dev);
}
radeon_do_release(dev); radeon_do_release(dev);
} }