Commit a1d8700d authored by Danilo Krummrich's avatar Danilo Krummrich
Browse files

drm/nouveau: omit to create schedulers using the legacy uAPI 



Omit to create scheduler instances when using the legacy uAPI. When
using the legacy NOUVEAU_GEM_PUSHBUF ioctl no scheduler instance is
required, hence omit creating scheduler instances in
nouveau_abi16_ioctl_channel_alloc().

Tested-by: default avatarTimur Tabi <ttabi@nvidia.com>
Reviewed-by: default avatarDave Airlie <airlied@redhat.com>
Signed-off-by: default avatarDanilo Krummrich <dakr@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240202000606.3526-2-dakr@redhat.com
parent 9a0c32d6

drivers/gpu/drm/nouveau/nouveau_abi16.c

+10 −4
Original line number Diff line number Diff line
@@ -339,10 +339,16 @@ nouveau_abi16_ioctl_channel_alloc(ABI16_IOCTL_ARGS) 
	if (ret)

		goto done;



	/* If we're not using the VM_BIND uAPI, we don't need a scheduler.

	 *

	 * The client lock is already acquired by nouveau_abi16_get().

	 */

	if (nouveau_cli_uvmm(cli)) {

		ret = nouveau_sched_create(&chan->sched, drm, drm->sched_wq,

					   chan->chan->dma.ib_max);

		if (ret)

			goto done;

	}



	init->channel = chan->chan->chid;