[Mesa-dev] [PATCH 2/3] i965: Let batchbuffers be placed anywhere in the 48-bit address space.

[Lionel Landwerlin]

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>

On 18/04/18 14:22, Kenneth Graunke wrote:
> We were trying to mark batch buffers with EXEC_OBJECT_CAPTURE, and
> accidentally stomped EXEC_OBJECT_SUPPORTS_48B_ADDRESS in the process.
>
> There's no reason to restrict batch buffers to the lower 4GB.
> ---
>   src/mesa/drivers/dri/i965/intel_batchbuffer.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/intel_batchbuffer.c b/src/mesa/drivers/dri/i965/intel_batchbuffer.c
> index a29159e41ba..d745c2a3113 100644
> --- a/src/mesa/drivers/dri/i965/intel_batchbuffer.c
> +++ b/src/mesa/drivers/dri/i965/intel_batchbuffer.c
> @@ -191,7 +191,7 @@ recreate_growing_buffer(struct brw_context *brw,
>      struct brw_bufmgr *bufmgr = screen->bufmgr;
>   
>      grow->bo = brw_bo_alloc(bufmgr, name, size);
> -   grow->bo->kflags = can_do_exec_capture(screen) ? EXEC_OBJECT_CAPTURE : 0;
> +   grow->bo->kflags |= can_do_exec_capture(screen) ? EXEC_OBJECT_CAPTURE : 0;
>      grow->partial_bo = NULL;
>      grow->partial_bo_map = NULL;
>      grow->partial_bytes = 0;