Attachment 23815 Details for Bug 20265 – don't set up fence registers if the kernel can do it

[patch] don't set up fence registers if the kernel can do it

i830-no-user-fencing.patch (text/plain), 1.04 KB, created by Jesse Barnes on 2009-03-12 15:44:01 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jesse Barnes

Created: 2009-03-12 15:44:01 UTC

Size: 1.04 KB

patch

obsolete

>diff --git a/src/i830_memory.c b/src/i830_memory.c
>index 96e1763..36dab11 100644
>--- a/src/i830_memory.c
>+++ b/src/i830_memory.c
>@@ -253,7 +253,8 @@ i830_bind_memory(ScrnInfoPtr pScrn, i830_memory *mem)
> 	mem->bound = TRUE;
>     }
> 
>-    if (mem->tiling != TILE_NONE && !pI830->use_drm_mode) {
>+    if (mem->tiling != TILE_NONE && !pI830->use_drm_mode &&
>+	!pI830->kernel_exec_fencing) {
> 	mem->fence_nr = i830_set_tiling(pScrn, mem->offset, mem->pitch,
> 					mem->allocated_size, mem->tiling);
>     }
>@@ -532,12 +533,8 @@ i830_allocator_init(ScrnInfoPtr pScrn, unsigned long offset, unsigned long size)
> 		int ret;
> 
> 		sp.param = I915_SETPARAM_NUM_USED_FENCES;
>-		if (pI830->use_drm_mode)
>-		    sp.value = 0; /* kernel gets them all */
>-		else if (pI830->directRenderingType == DRI_XF86DRI)
>-		    sp.value = 3; /* front/back/depth */
>-		else
>-		    sp.value = 2; /* just front for DRI2 (both old & new though) */
>+		sp.value = 0; /* kernel gets them all */
>+
> 		ret = drmCommandWrite(pI830->drmSubFD, DRM_I915_SETPARAM, &sp,
> 				      sizeof(sp));
> 		if (ret == 0)

Actions: View | Splinter Review

Attachments on bug 20265: 23202 | 23203 | 23204 | 23815

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.