From 2788943d3871cd84ada8cfe1dc56ce4f5a1ae6ab Mon Sep 17 00:00:00 2001 From: Konsta Holtta Date: Fri, 18 May 2018 16:13:35 +0300 Subject: gpu: nvgpu: remove broken force_pramin feature The forced PRAMIN reads and writes for sysmem buffers haven't worked in a while since the PRAMIN access code was refactored to work with vidmem-only sgt allocs. This feature was only ever meant for testing and debugging PRAMIN access and early dGPU support, but that is stable enough now so just delete the broken feature instead of fixing it. Change-Id: Ib31dae4550f3b6fea3c426a2e4ad126864bf85d2 Signed-off-by: Konsta Holtta Reviewed-on: https://git-master.nvidia.com/r/1723725 Reviewed-by: mobile promotions Tested-by: mobile promotions --- drivers/gpu/nvgpu/common/linux/channel.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) (limited to 'drivers/gpu/nvgpu/common/linux/channel.c') diff --git a/drivers/gpu/nvgpu/common/linux/channel.c b/drivers/gpu/nvgpu/common/linux/channel.c index e2a10634..727b5067 100644 --- a/drivers/gpu/nvgpu/common/linux/channel.c +++ b/drivers/gpu/nvgpu/common/linux/channel.c @@ -692,10 +692,6 @@ static int gk20a_submit_append_gpfifo(struct channel_gk20a *c, * This path (from userspace to sysmem) is special in order to * avoid two copies unnecessarily (from user to pipe, then from * pipe to gpu sysmem buffer). - * - * As a special case, the pipe buffer exists if PRAMIN writes - * are forced, although the buffers may not be in vidmem in - * that case. */ if (end > gpfifo_size) { /* wrap-around */ @@ -723,8 +719,7 @@ static int gk20a_submit_append_gpfifo(struct channel_gk20a *c, 0, num_entries); goto out; } else if (user_gpfifo) { - /* from userspace to vidmem or sysmem when pramin forced, use - * the common copy path below */ + /* from userspace to vidmem, use the common copy path below */ err = copy_from_user(c->gpfifo.pipe, user_gpfifo, len); if (err) return err; -- cgit v1.2.2