[Mesa-dev] [PATCH] i965: Bump aperture tracking to u64

Kenneth Graunke kenneth at whitecape.org
Fri Sep 7 15:20:23 UTC 2018


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.work at gmail.com>
> Cc: Kenneth Graunke <kenneth at 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 <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/20180907/8ba6cfe1/attachment.sig>


More information about the mesa-dev mailing list