On Friday, September 7, 2018 6:34:45 AM PDT Chris Wilson wrote:
> As a prelude to handling large address spaces, first allow ourselves the
> luxury of handling the full 4G.
> 
> Reported-by: Andrey Simiklit <asimiklit.w...@gmail.com>
> Cc: Kenneth Graunke <kenn...@whitecape.org>
> ---
>  src/mesa/drivers/dri/i965/brw_context.h       | 2 +-
>  src/mesa/drivers/dri/i965/intel_batchbuffer.c | 9 +--------
>  src/mesa/drivers/dri/i965/intel_batchbuffer.h | 8 ++++++--
>  3 files changed, 8 insertions(+), 11 deletions(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_context.h 
> b/src/mesa/drivers/dri/i965/brw_context.h
> index d3b96953467..ca1fe8ef0ea 100644
> --- a/src/mesa/drivers/dri/i965/brw_context.h
> +++ b/src/mesa/drivers/dri/i965/brw_context.h
> @@ -509,7 +509,7 @@ struct intel_batchbuffer {
>     int exec_array_size;
>  
>     /** The amount of aperture space (in bytes) used by all exec_bos */
> -   int aperture_space;
> +   uint64_t aperture_space;

Yeah, this should have been a uint32_t at least. :(

Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to