intel: Fix a case when mapping large texture fails

This patch handles a case when mapping a large texture fails
in drm_intel_gem_bo_map_gtt(). These changes avoid assertion
failure later in the driver as reported in following bugs:

https://bugs.freedesktop.org/show_bug.cgi?id=44970
https://bugs.freedesktop.org/show_bug.cgi?id=46303

Testing: No regressions in piglit quick.tests

Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
This commit is contained in:
Anuj Phogat 2012-02-27 11:09:29 -08:00
parent 2097af9728
commit cdcfd5d1d6
2 changed files with 23 additions and 10 deletions

View file

@ -702,15 +702,20 @@ intel_miptree_map_gtt(struct intel_context *intel,
y /= bh;
base = intel_region_map(intel, mt->region, map->mode);
/* Note that in the case of cube maps, the caller must have passed the slice
* number referencing the face.
*/
intel_miptree_get_image_offset(mt, level, 0, slice, &image_x, &image_y);
x += image_x;
y += image_y;
map->stride = mt->region->pitch * mt->cpp;
map->ptr = base + y * map->stride + x * mt->cpp;
if (base == NULL)
map->ptr = NULL;
else {
/* Note that in the case of cube maps, the caller must have passed the
* slice number referencing the face.
*/
intel_miptree_get_image_offset(mt, level, 0, slice, &image_x, &image_y);
x += image_x;
y += image_y;
map->stride = mt->region->pitch * mt->cpp;
map->ptr = base + y * map->stride + x * mt->cpp;
}
DBG("%s: %d,%d %dx%d from mt %p (%s) %d,%d = %p/%d\n", __FUNCTION__,
map->x, map->y, map->w, map->h,
@ -1067,6 +1072,11 @@ intel_miptree_map(struct intel_context *intel,
*out_ptr = map->ptr;
*out_stride = map->stride;
if (map->ptr == NULL) {
mt->level[level].slice[slice].map = NULL;
free(map);
}
}
void

View file

@ -124,7 +124,7 @@ intel_region_map(struct intel_context *intel, struct intel_region *region,
*/
_DBG("%s %p\n", __FUNCTION__, region);
if (!region->map_refcount++) {
if (!region->map_refcount) {
intel_flush(&intel->ctx);
if (region->tiling != I915_TILING_NONE)
@ -133,7 +133,10 @@ intel_region_map(struct intel_context *intel, struct intel_region *region,
drm_intel_bo_map(region->bo, true);
region->map = region->bo->virtual;
++intel->num_mapped_regions;
if (region->map) {
intel->num_mapped_regions++;
region->map_refcount++;
}
}
return region->map;