Commit a08db1f0 authored by D Scott Phillips's avatar D Scott Phillips Committed by Ben Widawsky

tools/intel_aubdump: Set addr_bits before write_header

write_header() uses addr_bits, so do the initialization earlier.
Also set the gen to a non-zero value in case of unknown device,
for use by a later patch.
Signed-off-by: D Scott Phillips's avatarScott D Phillips <scott.d.phillips@intel.com>
Reviewed-by: Jordan Justen's avatarJordan Justen <jordan.l.justen@intel.com>
Signed-off-by: Ben Widawsky's avatarBen Widawsky <ben@bwidawsk.net>
parent 539a86ec
......@@ -417,6 +417,15 @@ dump_execbuffer2(int fd, struct drm_i915_gem_execbuffer2 *execbuffer2)
}
if (gen == 0) {
gen = intel_gen(device);
/* If we don't know the device gen, then it probably is a
* newer device. Set gen to some arbitrarily high number.
*/
if (gen == 0)
gen = 9999;
addr_bits = gen >= 8 ? 48 : 32;
write_header();
if (verbose)
......@@ -425,11 +434,6 @@ dump_execbuffer2(int fd, struct drm_i915_gem_execbuffer2 *execbuffer2)
filename, device, gen);
}
/* If we don't know the device gen, then it probably is a
* newer device which uses 48-bit addresses.
*/
addr_bits = (gen >= 8 || gen == 0) ? 48 : 32;
if (verbose)
printf("Dumping execbuffer2:\n");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment