Commit e6ba7648 authored by Chris Wilson's avatar Chris Wilson
Browse files

drm/i915: Remove i915->kernel_context 



Allocate only an internal intel_context for the kernel_context, forgoing
a global GEM context for internal use as we only require a separate
address space (for our own protection).

Now having weaned GT from requiring ce->gem_context, we can stop
referencing it entirely. This also means we no longer have to create random
and unnecessary GEM contexts for internal use.

GEM contexts are now entirely for tracking GEM clients, and intel_context
the execution environment on the GPU.

Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Cc: Andi Shyti <andi.shyti@intel.com>
Acked-by: default avatarAndi Shyti <andi.shyti@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20191221160324.1073045-1-chris@chris-wilson.co.uk
parent ed7dd73c

drivers/gpu/drm/i915/gem/i915_gem_context.c

+43 −54
Original line number Diff line number Diff line
@@ -209,6 +209,35 @@ context_get_vm_rcu(struct i915_gem_context *ctx) 
	} while (1);

}



static void intel_context_set_gem(struct intel_context *ce,

				  struct i915_gem_context *ctx)

{

	GEM_BUG_ON(ce->gem_context);

	ce->gem_context = ctx;



	if (!test_bit(CONTEXT_ALLOC_BIT, &ce->flags))

		ce->ring = __intel_context_ring_size(SZ_16K);



	if (rcu_access_pointer(ctx->vm)) {

		struct i915_address_space *vm;



		rcu_read_lock();

		vm = context_get_vm_rcu(ctx); /* hmm */

		rcu_read_unlock();



		i915_vm_put(ce->vm);

		ce->vm = vm;

	}



	GEM_BUG_ON(ce->timeline);

	if (ctx->timeline)

		ce->timeline = intel_timeline_get(ctx->timeline);



	if (ctx->sched.priority >= I915_PRIORITY_NORMAL &&

	    intel_engine_has_semaphores(ce->engine))

		__set_bit(CONTEXT_USE_SEMAPHORES, &ce->flags);

}



static void __free_engines(struct i915_gem_engines *e, unsigned int count)

{

	while (count--) {
@@ -251,12 +280,14 @@ static struct i915_gem_engines *default_engines(struct i915_gem_context *ctx) 
		GEM_BUG_ON(engine->legacy_idx >= I915_NUM_ENGINES);

		GEM_BUG_ON(e->engines[engine->legacy_idx]);



		ce = intel_context_create(ctx, engine);

		ce = intel_context_create(engine);

		if (IS_ERR(ce)) {

			__free_engines(e, e->num_engines + 1);

			return ERR_CAST(ce);

		}



		intel_context_set_gem(ce, ctx);



		e->engines[engine->legacy_idx] = ce;

		e->num_engines = max(e->num_engines, engine->legacy_idx);

	}
@@ -706,37 +737,6 @@ i915_gem_create_context(struct drm_i915_private *i915, unsigned int flags) 
	return ctx;

}



static void

destroy_kernel_context(struct i915_gem_context **ctxp)

{

	struct i915_gem_context *ctx;



	/* Keep the context ref so that we can free it immediately ourselves */

	ctx = i915_gem_context_get(fetch_and_zero(ctxp));

	GEM_BUG_ON(!i915_gem_context_is_kernel(ctx));



	context_close(ctx);

	i915_gem_context_free(ctx);

}



struct i915_gem_context *

i915_gem_context_create_kernel(struct drm_i915_private *i915, int prio)

{

	struct i915_gem_context *ctx;



	ctx = i915_gem_create_context(i915, 0);

	if (IS_ERR(ctx))

		return ctx;



	i915_gem_context_clear_bannable(ctx);

	i915_gem_context_set_persistence(ctx);

	ctx->sched.priority = I915_USER_PRIORITY(prio);



	GEM_BUG_ON(!i915_gem_context_is_kernel(ctx));



	return ctx;

}



static void init_contexts(struct i915_gem_contexts *gc)

{

	spin_lock_init(&gc->lock);
@@ -746,32 +746,16 @@ static void init_contexts(struct i915_gem_contexts *gc) 
	init_llist_head(&gc->free_list);

}



int i915_gem_init_contexts(struct drm_i915_private *i915)

void i915_gem_init__contexts(struct drm_i915_private *i915)

{

	struct i915_gem_context *ctx;



	/* Reassure ourselves we are only called once */

	GEM_BUG_ON(i915->kernel_context);



	init_contexts(&i915->gem.contexts);



	/* lowest priority; idle task */

	ctx = i915_gem_context_create_kernel(i915, I915_PRIORITY_MIN);

	if (IS_ERR(ctx)) {

		DRM_ERROR("Failed to create default global context\n");

		return PTR_ERR(ctx);

	}

	i915->kernel_context = ctx;



	DRM_DEBUG_DRIVER("%s context support initialized\n",

			 DRIVER_CAPS(i915)->has_logical_contexts ?

			 "logical" : "fake");

	return 0;

}



void i915_gem_driver_release__contexts(struct drm_i915_private *i915)

{

	destroy_kernel_context(&i915->kernel_context);

	flush_work(&i915->gem.contexts.free_work);

}
@@ -840,7 +824,6 @@ int i915_gem_context_open(struct drm_i915_private *i915, 
	if (err < 0)

		goto err_ctx;



	GEM_BUG_ON(i915_gem_context_is_kernel(ctx));

	GEM_BUG_ON(err > 0);



	return 0;
@@ -1531,12 +1514,14 @@ set_engines__load_balance(struct i915_user_extension __user *base, void *data) 
		}

	}



	ce = intel_execlists_create_virtual(set->ctx, siblings, n);

	ce = intel_execlists_create_virtual(siblings, n);

	if (IS_ERR(ce)) {

		err = PTR_ERR(ce);

		goto out_siblings;

	}



	intel_context_set_gem(ce, set->ctx);



	if (cmpxchg(&set->engines->engines[idx], NULL, ce)) {

		intel_context_put(ce);

		err = -EEXIST;
@@ -1706,12 +1691,14 @@ set_engines(struct i915_gem_context *ctx, 
			return -ENOENT;

		}



		ce = intel_context_create(ctx, engine);

		ce = intel_context_create(engine);

		if (IS_ERR(ce)) {

			__free_engines(set.engines, n);

			return PTR_ERR(ce);

		}



		intel_context_set_gem(ce, ctx);



		set.engines->engines[n] = ce;

	}

	set.engines->num_engines = num_engines;
@@ -2016,13 +2003,15 @@ static int clone_engines(struct i915_gem_context *dst, 
		 */

		if (intel_engine_is_virtual(engine))

			clone->engines[n] =

				intel_execlists_clone_virtual(dst, engine);

				intel_execlists_clone_virtual(engine);

		else

			clone->engines[n] = intel_context_create(dst, engine);

			clone->engines[n] = intel_context_create(engine);

		if (IS_ERR_OR_NULL(clone->engines[n])) {

			__free_engines(clone, n);

			goto err_unlock;

		}



		intel_context_set_gem(clone->engines[n], dst);

	}

	clone->num_engines = n;

drivers/gpu/drm/i915/gem/i915_gem_context.h

+1 −9
Original line number Diff line number Diff line
@@ -109,13 +109,8 @@ i915_gem_context_clear_user_engines(struct i915_gem_context *ctx) 
	clear_bit(CONTEXT_USER_ENGINES, &ctx->flags);

}



static inline bool i915_gem_context_is_kernel(struct i915_gem_context *ctx)

{

	return !ctx->file_priv;

}



/* i915_gem_context.c */

int __must_check i915_gem_init_contexts(struct drm_i915_private *i915);

void i915_gem_init__contexts(struct drm_i915_private *i915);

void i915_gem_driver_release__contexts(struct drm_i915_private *i915);



int i915_gem_context_open(struct drm_i915_private *i915,
@@ -140,9 +135,6 @@ int i915_gem_context_setparam_ioctl(struct drm_device *dev, void *data, 
int i915_gem_context_reset_stats_ioctl(struct drm_device *dev, void *data,

				       struct drm_file *file);



struct i915_gem_context *

i915_gem_context_create_kernel(struct drm_i915_private *i915, int prio);



static inline struct i915_gem_context *

i915_gem_context_get(struct i915_gem_context *ctx)

{

drivers/gpu/drm/i915/gem/i915_gem_userptr.c

+1 −5
Original line number Diff line number Diff line
@@ -779,15 +779,11 @@ i915_gem_userptr_ioctl(struct drm_device *dev, 
		return -EFAULT;



	if (args->flags & I915_USERPTR_READ_ONLY) {

		struct i915_address_space *vm;



		/*

		 * On almost all of the older hw, we cannot tell the GPU that

		 * a page is readonly.

		 */

		vm = rcu_dereference_protected(dev_priv->kernel_context->vm,

					       true); /* static vm */

		if (!vm || !vm->has_read_only)

		if (!dev_priv->gt.vm->has_read_only)

			return -ENODEV;

	}

drivers/gpu/drm/i915/gem/selftests/i915_gem_context.c

+2 −3
Original line number Diff line number Diff line
@@ -337,7 +337,7 @@ static int live_parallel_switch(void *arg) 
			if (!data[m].ce[0])

				continue;



			ce = intel_context_create(ctx, data[m].ce[0]->engine);

			ce = intel_context_create(data[m].ce[0]->engine);

			if (IS_ERR(ce))

				goto out;
@@ -1264,8 +1264,7 @@ __igt_ctx_sseu(struct drm_i915_private *i915, 
			hweight32(engine->sseu.slice_mask),

			hweight32(pg_sseu.slice_mask));



		ce = intel_context_create(engine->kernel_context->gem_context,

					  engine);

		ce = intel_context_create(engine);

		if (IS_ERR(ce)) {

			ret = PTR_ERR(ce);

			goto out_put;

drivers/gpu/drm/i915/gem/selftests/mock_context.c

+10 −1
Original line number Diff line number Diff line
@@ -96,7 +96,16 @@ err_ctx: 
struct i915_gem_context *

kernel_context(struct drm_i915_private *i915)

{

	return i915_gem_context_create_kernel(i915, I915_PRIORITY_NORMAL);

	struct i915_gem_context *ctx;



	ctx = i915_gem_create_context(i915, 0);

	if (IS_ERR(ctx))

		return ctx;



	i915_gem_context_clear_bannable(ctx);

	i915_gem_context_set_persistence(ctx);



	return ctx;

}



void kernel_context_close(struct i915_gem_context *ctx)

CRA Git | Maintained and supported by SUSTech CRA and CCSE