[Intel-xe] [PATCH v2 06/31] drm/xe: Ensure LR engines are not persistent

[Thomas Hellström]

On 5/2/23 02:17, Matthew Brost wrote:
> With our ref counting scheme LR engines only close properly if not
> persistent, ensure that LR engines are non-persistent.
>
> Signed-off-by: Matthew Brost <matthew.brost at intel.com>

With Rodrigos "Long-Running" comment addressed,

Reviewed-by: Thomas Hellström <thomas.hellstrom at linux.intel.com>


> ---
>   drivers/gpu/drm/xe/xe_engine.c | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_engine.c b/drivers/gpu/drm/xe/xe_engine.c
> index d1e84d7adbd4..91600b1e8249 100644
> --- a/drivers/gpu/drm/xe/xe_engine.c
> +++ b/drivers/gpu/drm/xe/xe_engine.c
> @@ -596,7 +596,9 @@ int xe_engine_create_ioctl(struct drm_device *dev, void *data,
>   			return -ENOENT;
>   
>   		e = xe_engine_create(xe, vm, logical_mask,
> -				     args->width, hwe, ENGINE_FLAG_PERSISTENT);
> +				     args->width, hwe,
> +				     xe_vm_no_dma_fences(vm) ? 0 :
> +				     ENGINE_FLAG_PERSISTENT);
>   		xe_vm_put(vm);
>   		if (IS_ERR(e))
>   			return PTR_ERR(e);