Bug 103353

Summary: aubinator prints more fields of an instruction than exist in batch
Product: Mesa Reporter: Nanley Chery <nanleychery>
Component: Drivers/DRI/i965Assignee: Intel 3D Bugs Mailing List <intel-3d-bugs>
Status: RESOLVED FIXED QA Contact: Intel 3D Bugs Mailing List <intel-3d-bugs>
Severity: normal    
Priority: medium    
Version: git   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:

Description Nanley Chery 2017-10-18 21:58:17 UTC 
Test case:
* On a CNL, take an aubdump of a workload that emits a draw call (can be provided if needed).
* Use aubinator (or INTEL_DEBUG=bat) to inspect the 3DPRIMITIVE command

Expected result:
* The instruction is specified to be 7 (5 + 2) dwords long according to the xml.
* All but the last 3 optional fields of the instruction would be shown as they  are dwords 8-10 and the ExtendedParametersPresent bit is cleared.

Actual result:
* All fields of the instruction are shown.
Comment 1 Lionel Landwerlin 2018-12-12 12:24:59 UTC 
I think I fixed this in :

commit 2841af6238b7648b388ba806f1ad181332b6eef0
Author: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Date:   Tue Apr 3 11:01:56 2018 +0100

    intel: gen-decoder: don't decode fields beyond a dword length

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.