[Mesa-dev] [PATCH v3] intel/decoders: fix end of batch limit

Kenneth Graunke kenneth at whitecape.org
Tue Oct 23 00:43:49 UTC 2018


On Monday, October 22, 2018 5:39:39 PM PDT Lionel Landwerlin wrote:
> Pointer arithmetic...
> 
> v2: s/4/sizeof(uint32_t)/ (Eric)
> 
> v3: Give bytes to print_batch() in error_decode (Lionel)
>     Make clear what values we're dealing with in error_decode (Lionel)
> 
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
> Reviewed-by: Eric Engestrom <eric.engestrom at intel.com> (v2)
> ---
>  src/intel/common/gen_batch_decoder.c         |  2 +-
>  src/intel/tools/aubinator_error_decode.c     | 15 ++++++++-------
>  src/intel/tools/aubinator_viewer_decoder.cpp |  2 +-
>  3 files changed, 10 insertions(+), 9 deletions(-)

Thanks for fixing this!  Looks like everything's properly in bytes now.

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20181022/da4ecc71/attachment-0001.sig>


More information about the mesa-dev mailing list