Commit d96bf437 authored by Ben Skeggs's avatar Ben Skeggs
Browse files

drm/nouveau/gem: remove (now) unneeded pre-validate fence sync 



Now that nouveau_bo.c can handle sync when it actually needs to, we can
remove this and avoid a double semaphore acquire when syncing in the
command submission path.

Signed-off-by: default avatarBen Skeggs <bskeggs@redhat.com>
parent cef9e99e

drivers/gpu/drm/nouveau/nouveau_gem.c

+0 −6
Original line number Diff line number Diff line
@@ -463,12 +463,6 @@ validate_list(struct nouveau_channel *chan, struct nouveau_cli *cli, 
	list_for_each_entry(nvbo, list, entry) {

		struct drm_nouveau_gem_pushbuf_bo *b = &pbbo[nvbo->pbbo_index];



		ret = validate_sync(chan, nvbo);

		if (unlikely(ret)) {

			NV_ERROR(cli, "fail pre-validate sync\n");

			return ret;

		}



		ret = nouveau_gem_set_domain(&nvbo->gem, b->read_domains,

					     b->write_domains,

					     b->valid_domains);