[Mesa-dev] [PATCH 5/5] docs: describe the INTEL_* envvars that do exist

[Chris Forbes]

Yikes, really sorry for the trouble.

-- Chris

On Thu, Nov 28, 2013 at 10:40 AM, Matt Turner <mattst88 at gmail.com> wrote:
> On Mon, Nov 25, 2013 at 12:20 AM, Chris Forbes <chrisf at ijw.co.nz> wrote:
>> I've pushed these now, including dropping the old names for those debug bits.
>>
>> I've left the stats stuff as-is for now, but suggested more useful
>> options in the docs.
>
> I'm really unhappy that and how 195994fe4cd ("drop old INTEL_DEBUG
> names for `perf` (fall) and `fs` (wm)") was committed. I don't see it
> ever being sent to the list much less Ken actually reviewing it.
>
> shader-db uses the old wm name, so I've now wasted a pile of time
> tracking down why none of my fragment shaders are being executed. If
> it had been sent to the list I could have told you this.
>
> I'm reverting this (195994fe4cd) out of principle.
>
>> On Mon, Nov 25, 2013 at 4:44 AM, Kenneth Graunke <kenneth at whitecape.org> wrote:
>>> On 11/23/2013 09:13 PM, Chris Forbes wrote:
>>>> Signed-off-by: Chris Forbes <chrisf at ijw.co.nz>
>>>> ---
>>>>  docs/envvars.html | 32 ++++++++++++++++++++++++++++++++
>>>>  1 file changed, 32 insertions(+)
>>>>
>>>> diff --git a/docs/envvars.html b/docs/envvars.html
>>>> index 81e74e6..d831826 100644
>>>> --- a/docs/envvars.html
>>>> +++ b/docs/envvars.html
>>>> @@ -121,6 +121,38 @@ See the <a href="xlibdriver.html">Xlib software driver page</a> for details.
>>>>  <h2>i945/i965 driver environment variables (non-Gallium)</h2>
>>>>
>>>>  <ul>
>>>> +<li>INTEL_NO_HW - if set to 1, prevents batches from being submitted to the hardware.
>>>> +   This is useful for debugging hangs, etc.</li>
>>>> +<li>INTEL_DEBUG - a comma-separated list of named flags, which do various things:
>>>> +<ul>
>>>> +   <li>tex - emit messages about textures.</li>
>>>> +   <li>state - emit messages about state flag tracking</li>
>>>> +   <li>blit - emit messages about blit operations</li>
>>>> +   <li>miptree - emit messages about miptrees</li>
>>>> +   <li>fall/perf - emit messages about performance issues</li>
>>>
>>> Not sure if the old names are worth documenting.  I guess if people find
>>> old text on wikis or something that says INTEL_DEBUG=fall, they'll know
>>> what it means.
>>>
>>> We might want to actually just drop the 'fall' name at this
>>> point...people seem to have moved over completely.
>>>
>>>> +   <li>perfmon - emit messages about AMD_performance_monitor</li>
>>>> +   <li>bat - emit batch information</li>
>>>> +   <li>pix - emit messages about pixel operations</li>
>>>> +   <li>buf - emit messages about buffer objects</li>
>>>> +   <li>reg - emit messages about regions</li>
>>>> +   <li>fbo - emit messages about framebuffers</li>
>>>> +   <li>fs/wm - dump shader assembly for fragment shaders</li>
>>>
>>> I've pretty much universally moved over to INTEL_DEBUG=fs too, but I
>>> don't know about others.
>>>
>>>> +   <li>gs - dump shader assembly for geometry shaders</li>
>>>> +   <li>sync - emit messages about synchronization</li>
>>>> +   <li>prim - emit messages about drawing primitives</li>
>>>> +   <li>vert - emit messages about vertex assembly</li>
>>>> +   <li>dri - emit messages about the DRI interface</li>
>>>> +   <li>sf - emit messages about the strips & fans unit (for old gens, includes the SF program)</li>
>>>> +   <li>stats - ?</li>
>>>
>>> This enables statistics counters for the vertex fetcher (on all
>>> generations), and for other units on Gen4-5.  That said, the counters
>>> aren't exposed other than reading registers, and on Gen6+ you can't even
>>> use intel_reg_read due to hardware contexts.
>>>
>>> Frankly, it seems pretty useless, and I think we ought to delete it.
>>>
>>>> +   <li>urb - emit messages about URB setup</li>
>>>> +   <li>vs - dump shader assembly for vertex shaders</li>
>>>> +   <li>clip - emit messages about the clip unit (for old gens, includes the CLIP program)</li>
>>>> +   <li>aub - dump batches into an AUB trace for use with simulation tools</li>
>>>> +   <li>shader_time - record how much GPU time is spent in each shader</li>
>>>> +   <li>no16 - suppress generation of 16-wide fragment shaders. useful for debugging broken shaders</li>
>>>> +   <li>blorp - emit messages about the blorp operations (blits & clears)</li>
>>>> +   <li>nodualobj - suppress generation of dual-object geometry shader code</li>
>>>> +</ul>
>>>>  </ul>
>>>
>>> Thanks for doing this, Chris.  For the series:
>>> Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev