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
>
