[Mesa-dev] [PATCH 1/3] Revert "aubdump: Bump the GTT size to 256MB"

[Jordan Justen]

intel-gfx?

Series Reviewed-by: Jordan Justen <jordan.l.justen at intel.com>

On 2016-11-23 20:27:00, Jason Ekstrand wrote:
> This reverts commit c6191a41f6c858cbdd4280e9ffcc91170e87ebeb.  It turns out
> that having a GTT that large causes the simulator massive headaches.
> ---
>  tools/aubdump.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/tools/aubdump.c b/tools/aubdump.c
> index 8904255..b1c46e2 100644
> --- a/tools/aubdump.c
> +++ b/tools/aubdump.c
> @@ -176,8 +176,8 @@ gtt_entry_size(void)
>  static uint32_t
>  gtt_size(void)
>  {
> -       /* Enough for 256MB assuming 4kB pages. */
> -       const unsigned entries = 0x10000;
> +       /* Enough for 64MB assuming 4kB pages. */
> +       const unsigned entries = 0x4000;
>         return entries * gtt_entry_size();
>  }
>  
> @@ -197,7 +197,7 @@ write_header(void)
>         dword_out(0); /* timestamp */
>         dword_out(0); /* comment len */
>  
> -       /* Set up the GTT. The max we can handle is 256M */
> +       /* Set up the GTT. The max we can handle is 64M */
>         dword_out(CMD_AUB_TRACE_HEADER_BLOCK | ((gen >= 8 ? 6 : 5) - 2));
>         dword_out(AUB_TRACE_MEMTYPE_GTT_ENTRY |
>                   AUB_TRACE_TYPE_NOTYPE | AUB_TRACE_OP_DATA_WRITE);
> -- 
> 2.5.0.400.gff86faf
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev