[intel] Remove useless intel_region_idle.
authorEric Anholt <eric@anholt.net>
Fri, 14 Dec 2007 21:28:39 +0000 (13:28 -0800)
committerEric Anholt <eric@anholt.net>
Fri, 14 Dec 2007 22:40:03 +0000 (14:40 -0800)
The idling it was trying to ensure was covered by the
intel_miptree_image_map()->intel_region_map() that immediately followed it.

src/mesa/drivers/dri/intel/intel_regions.c
src/mesa/drivers/dri/intel/intel_regions.h
src/mesa/drivers/dri/intel/intel_tex_image.c
src/mesa/drivers/dri/intel/intel_tex_subimage.c

index a47b2880907ee3494291c0ecd775d940a86e95c1..56ca6d2fd274079cc5268981f8d4430b69745135 100644 (file)
 
 #define FILE_DEBUG_FLAG DEBUG_REGION
 
-void
-intel_region_idle(struct intel_context *intel, struct intel_region *region)
-{
-   DBG("%s\n", __FUNCTION__);
-   /* XXX: Using this function is likely bogus -- it ought to only have been
-    * used before a map, anyway, but leave this cheap implementation of it
-    * for now.
-    */
-   if (region && region->buffer) {
-      /* Mapping it for read will ensure that any acceleration to the region
-       * would have landed already.
-       */
-      dri_bo_map(region->buffer, GL_TRUE);
-      dri_bo_unmap(region->buffer);
-   }
-}
-
 /* XXX: Thread safety?
  */
 GLubyte *
index 1975d729e473b4d15dee4c0ddbb3b75e1e878a73..b6a3b5a7397f7c390954dec46699ef60f4fa656e 100644 (file)
@@ -73,9 +73,6 @@ void intel_region_release(struct intel_region **ib);
 
 void intel_recreate_static_regions(struct intel_context *intel);
 
-void intel_region_idle(struct intel_context *intel,
-                      struct intel_region *ib);
-
 /* Map/unmap regions.  This is refcounted also: 
  */
 GLubyte *intel_region_map(struct intel_context *intel,
index fd800a7bb6839b00868e1329c1628e25341647f6..c2af74095c9f66bb78fd269cfc17a92fdc79b02a 100644 (file)
@@ -457,10 +457,6 @@ intelTexImage(GLcontext * ctx,
    if (!pixels)
       return;
 
-
-   if (intelImage->mt)
-      intel_region_idle(intel, intelImage->mt->region);
-
    LOCK_HARDWARE(intel);
 
    if (intelImage->mt) {
index 32de2cfb48ed07274715469a9a1d7b842f13a096..5410df203f468412611f2ef9345363701529f3a6 100644 (file)
@@ -64,9 +64,6 @@ intelTexSubimage(GLcontext * ctx,
    if (!pixels)
       return;
 
-   if (intelImage->mt)
-      intel_region_idle(intel, intelImage->mt->region);
-
    LOCK_HARDWARE(intel);
 
    /* Map buffer if necessary.  Need to lock to prevent other contexts