diff options
author | Will Dyson <[email protected]> | 2010-05-25 01:10:20 -0400 |
---|---|---|
committer | Alex Deucher <[email protected]> | 2010-05-26 18:17:28 -0400 |
commit | a9ee95651131e27d5acf3d10909b5b7e5c8d3e92 (patch) | |
tree | 726a794471697d3bdad2887dc7c88237105bdf50 /src/mesa/drivers/dri | |
parent | f49da110a81cc964100efa6a09e0fcdc7a35a935 (diff) |
Fix image_matches_texture_obj() MaxLevel check
When generating or uploading a new (higher) mipmap level for an image,
we can need to allocate a miptree for a level greater than
texObj->MaxLevel.
Signed-off-by: Maciej Cencora <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
Diffstat (limited to 'src/mesa/drivers/dri')
-rw-r--r-- | src/mesa/drivers/dri/radeon/radeon_texture.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/src/mesa/drivers/dri/radeon/radeon_texture.c b/src/mesa/drivers/dri/radeon/radeon_texture.c index 2b655fbd953..5cb33a4751d 100644 --- a/src/mesa/drivers/dri/radeon/radeon_texture.c +++ b/src/mesa/drivers/dri/radeon/radeon_texture.c @@ -588,7 +588,12 @@ static int image_matches_texture_obj(struct gl_texture_object *texObj, if (!baseImage) return 0; - if (level < texObj->BaseLevel || level > texObj->MaxLevel) + /* Check image level against object BaseLevel, but not MaxLevel. MaxLevel is not + * the highest level that can be assigned to the miptree. + */ + const unsigned maxLevel = texObj->BaseLevel + baseImage->MaxLog2; + if (level < texObj->BaseLevel || level > maxLevel + || level > RADEON_MIPTREE_MAX_TEXTURE_LEVELS) return 0; const unsigned levelDiff = level - texObj->BaseLevel; @@ -610,9 +615,7 @@ static void teximage_assign_miptree(radeonContextPtr rmesa, radeonTexObj *t = radeon_tex_obj(texObj); radeon_texture_image* image = get_radeon_texture_image(texImage); - /* Since miptree holds only images for levels <BaseLevel..MaxLevel> - * don't allocate the miptree if the teximage won't fit. - */ + /* check image for dimension and level compatibility with texture */ if (!image_matches_texture_obj(texObj, texImage, level)) return; |