Re: [PATCH 1/2] drm/nouveau: bound sync and op counts in EXEC and VM_BIND

From: Dave Airlie

Date: Tue Aug 11 2026 - 00:06:01 EST


On Tue, 11 Aug 2026 at 14:00, Junrui Luo <moonafterrain@xxxxxxxxxxx> wrote:
>
> On Mon, Aug 10, 2026 at 08:43:50AM +1000, David Airlie wrote:
> > Have you run a complete Vulkan CTS with NVK across this with no regressions?
> >
> > I'm weary of those limits being too low, but I think the only app that
> > might push them beyond CTS would be Forza Horizon 5 or 6.
>
> No, I have not. I don't currently have the hardware to run NVK, so I can't
> offer a CTS run or any other regression test for this.
>
> Is there a better guard that avoids a hard limit here, or does this need
> a test before it can go in?

Looking at xe it also has 1024 limit on number of syncs, but it
doesn't have any limit on number of binds,

*bind_ops = kvmalloc_objs(struct drm_xe_vm_bind_op,
args->num_binds,
GFP_KERNEL | __GFP_ACCOUNT |
__GFP_RETRY_MAYFAIL | __GFP_NOWARN);

Maybe nouveau could do the same.

Dave.
>
> Thanks,
> Junrui Luo