<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 14, 2017 at 11:34 AM, Rafael Antognolli <span dir="ltr"><<a href="mailto:rafael.antognolli@intel.com" target="_blank">rafael.antognolli@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Patch is<br>
Reviewed-by: Rafael Antognolli <<a href="mailto:rafael.antognolli@intel.com">rafael.antognolli@intel.com</a>><br><div><div class="h5"></div></div></blockquote><div><br></div><div>Thanks!  Pushed.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
On Fri, Mar 24, 2017 at 04:45:01PM -0700, Jason Ekstrand wrote:<br>
> A gem handle of 0 can be used to check for whether or not 48-bit<br>
> addressing is available.  This keeps aubdump from failing on you if<br>
> you try to do the check.<br>
> ---<br>
>  tools/aubdump.c | 5 ++---<br>
>  1 file changed, 2 insertions(+), 3 deletions(-)<br>
><br>
> diff --git a/tools/aubdump.c b/tools/aubdump.c<br>
> index 8a89b8c..3aca7eb 100644<br>
> --- a/tools/aubdump.c<br>
> +++ b/tools/aubdump.c<br>
> @@ -131,7 +131,6 @@ get_bo(uint32_t handle)<br>
><br>
>       fail_if(handle >= MAX_BO_COUNT, "bo handle too large\n");<br>
>       bo = &bos[handle];<br>
> -     fail_if(bo->size == 0, "invalid bo handle (%d) in execbuf\n", handle);<br>
><br>
>       return bo;<br>
>  }<br>
> @@ -442,7 +441,7 @@ dump_execbuffer2(int fd, struct drm_i915_gem_execbuffer2 *execbuffer2)<br>
>                       offset = align_u32(offset + bo->size + 4095, 4096);<br>
>               }<br>
><br>
> -             if (bo->map == NULL)<br>
> +             if (bo->map == NULL && bo->size > 0)<br>
>                       bo->map = gem_mmap(fd, obj->handle, 0, bo->size);<br>
>               fail_if(bo->map == MAP_FAILED, "intel_aubdump: bo mmap failed\n");<br>
>       }<br>
> @@ -583,7 +582,7 @@ maybe_init(void)<br>
>       }<br>
>       fclose(config);<br>
><br>
> -     bos = malloc(MAX_BO_COUNT * sizeof(bos[0]));<br>
> +     bos = calloc(MAX_BO_COUNT, sizeof(bos[0]));<br>
>       fail_if(bos == NULL, "intel_aubdump: out of memory\n");<br>
>  }<br>
><br>
> --<br>
> 2.5.0.400.gff86faf<br>
><br>
</div></div>> ______________________________<wbr>_________________<br>
> Intel-gfx mailing list<br>
> <a href="mailto:Intel-gfx@lists.freedesktop.org">Intel-gfx@lists.freedesktop.<wbr>org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/intel-gfx" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/intel-gfx</a><br>
</blockquote></div><br></div></div>