[Mesa-dev] [PATCH v1] i965/batch/debug: Allow log be dumped before assert

Lionel Landwerlin lionel.g.landwerlin at intel.com
Mon Nov 5 13:12:38 UTC 2018


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

On 05/11/2018 14:02, Sergii Romantsov wrote:
> Message that may show the culprit of assert now will
> be dumped before that for debug purposes.
>
> CC: Kenneth Graunke <kenneth at whitecape.org>
> CC: Lionel G Landwerlin <lionel.g.landwerlin at intel.com>
> Signed-off-by: Sergii Romantsov <sergii.romantsov at globallogic.com>
> ---
>   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 8b769ea..353fcba 100644
> --- a/src/mesa/drivers/dri/i965/intel_batchbuffer.c
> +++ b/src/mesa/drivers/dri/i965/intel_batchbuffer.c
> @@ -725,10 +725,10 @@ execbuffer(int fd,
>   
>         /* Update brw_bo::gtt_offset */
>         if (batch->validation_list[i].offset != bo->gtt_offset) {
> -         assert(!(bo->kflags & EXEC_OBJECT_PINNED));
>            DBG("BO %d migrated: 0x%" PRIx64 " -> 0x%llx\n",
>                bo->gem_handle, bo->gtt_offset,
>                batch->validation_list[i].offset);
> +         assert(!(bo->kflags & EXEC_OBJECT_PINNED));
>            bo->gtt_offset = batch->validation_list[i].offset;
>         }
>      }




More information about the mesa-dev mailing list