Re: [PATCH 2/2] drm/nouveau: DRM_NOUVEAU_SET_ZCULL_CTXSW_BUFFER

[Danilo Krummrich <dakr@kernel.org>]

On Wed, Mar 12, 2025 at 05:36:15PM -0400, Mel Henning wrote:
> diff --git a/include/uapi/drm/nouveau_drm.h b/include/uapi/drm/nouveau_drm.h

Same here, please split the uAPI change in a separate commit.

> index 33361784eb4e..e9638f4dd7e6 100644
> --- a/include/uapi/drm/nouveau_drm.h
> +++ b/include/uapi/drm/nouveau_drm.h
> @@ -448,6 +448,20 @@ struct drm_nouveau_get_zcull_info {
>  	__u32 ctxsw_align;
>  };
>  
> +struct drm_nouveau_set_zcull_ctxsw_buffer {
> +	/**
> +	 * @ptr: The virtual address for the buffer, or null to bind nothing
> +	 */
> +	__u64 addr;

What is this buffer? Is this a GEM object backed buffer? How is it mapped?

> +
> +	/**
> +	 * @channel: the channel to set the buffer on
> +	 */
> +	__u32 channel;
> +
> +	__u32 pad;
> +};
> +
>  #define DRM_NOUVEAU_GETPARAM           0x00
>  #define DRM_NOUVEAU_SETPARAM           0x01 /* deprecated */
>  #define DRM_NOUVEAU_CHANNEL_ALLOC      0x02
> @@ -462,6 +476,7 @@ struct drm_nouveau_get_zcull_info {
>  #define DRM_NOUVEAU_VM_BIND            0x11
>  #define DRM_NOUVEAU_EXEC               0x12
>  #define DRM_NOUVEAU_GET_ZCULL_INFO     0x13
> +#define DRM_NOUVEAU_SET_ZCULL_CTXSW_BUFFER 0x14
>  #define DRM_NOUVEAU_GEM_NEW            0x40
>  #define DRM_NOUVEAU_GEM_PUSHBUF        0x41
>  #define DRM_NOUVEAU_GEM_CPU_PREP       0x42
> @@ -532,6 +547,7 @@ struct drm_nouveau_svm_bind {
>  #define DRM_IOCTL_NOUVEAU_EXEC               DRM_IOWR(DRM_COMMAND_BASE + DRM_NOUVEAU_EXEC, struct drm_nouveau_exec)
>  
>  #define DRM_IOCTL_NOUVEAU_GET_ZCULL_INFO     DRM_IOR (DRM_COMMAND_BASE + DRM_NOUVEAU_GET_ZCULL_INFO, struct drm_nouveau_get_zcull_info)
> +#define DRM_IOCTL_NOUVEAU_SET_ZCULL_CTXSW_BUFFER  DRM_IOW (DRM_COMMAND_BASE + DRM_NOUVEAU_SET_ZCULL_CTXSW_BUFFER, struct drm_nouveau_set_zcull_ctxsw_buffer)
>  #if defined(__cplusplus)
>  }
>  #endif
> -- 
> 2.48.1
>