Bug 96647 - [SNB,CTS] es3-cts.gtf.gl3tests.npot_textures.npot_tex_image regression
Summary: [SNB,CTS] es3-cts.gtf.gl3tests.npot_textures.npot_tex_image regression
Status: RESOLVED FIXED
Alias: None
Product: Mesa
Classification: Unclassified
Component: Drivers/DRI/i965 (show other bugs)
Version: git
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: Jason Ekstrand
QA Contact: Intel 3D Bugs Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 98335
  Show dependency treegraph
 
Reported: 2016-06-23 02:58 UTC by Mark Janes
Modified: 2016-10-24 23:18 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Description Mark Janes 2016-06-23 02:58:09 UTC
Bisected to:
6ba88bce64b343761aabe3a6c7ee285c6020a959
Author:     Jason Ekstrand <jason@jlekstrand.net>
i965: Drop the maximum 3D texture size to 512 on Sandy Bridge

The RenderTargetViewExtent field of RENDER_SURFACE_STATE is supposed to be
set to the depth of a 3-D texture when rendering.  Unfortunatley, that
field is only 9 bits on Sandy Bridge and prior so we can't actually bind
a 3-D texturing for rendering if it has depth > 512.  On Ivy Bridge, this
field was bumpped to 11 bits so we can go all the way up to 2048.  On Iron
Lake and prior, we don't support layered rendering and we use OffsetX/Y
hacks to render to particular layers so 2048 is ok there too.

Signed-off-by: Jason Ekstrand <jason@jlekstrand.net>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Cc: "11.1 11.2 12.0" <mesa-stable@lists.freedesktop.org>

/tmp/build_root/m64/bin/cts/glcts --deqp-case=ES3-CTS.gtf.GL3Tests.npot_textures.npot_tex_image
dEQP Core GL-CTS-2.0 (0x0052484b) starting..
  target implementation = 'intel-gbm'

Test case 'ES3-CTS.gtf.GL3Tests.npot_textures.npot_tex_image'..
ERROR:npot_tex_image: GL_INVALID_VALUE, internalformat: GL_LUMINANCE, level: 2, target: -

  Fail (Fail)

DONE!

Test run totals:
  Passed:        0/1 (0.00%)
  Failed:        1/1 (100.00%)
  Not supported: 0/1 (0.00%)
  Warnings:      0/1 (0.00%)
Comment 1 Jason Ekstrand 2016-06-23 03:23:40 UTC
This bug was more-or-less expected.  Really, it's a CTS bug because the CTS assumes that you can create 3-D textures larger than 512 without checking.  More specifically, it tries to call glTexImage with level 2 and a size of 11x131x1.  That said, I'm going to leave this open as a reminder that we should really try and bump the max 3-D texture size back up on Sandy Bridge when we have the chance.
Comment 2 Mark Janes 2016-10-19 22:32:10 UTC
Can this bug be fixed, or should it be closed?
Comment 3 Kenneth Graunke 2016-10-24 23:18:49 UTC
Fixed by:

commit 24be6306609179efddfb7e5cc6ec5d6a335c9b88
Author: Jason Ekstrand <jason.ekstrand@intel.com>
Date:   Mon Sep 12 08:18:53 2016 -0700

    Revert "i965: Drop the maximum 3D texture size to 512 on Sandy Bridge"


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.