From 82ef5f7b3b51bff51b18f525ee14033a4b6befd4 Mon Sep 17 00:00:00 2001 From: David Nieto Date: Fri, 22 Sep 2017 10:47:12 -0700 Subject: gpu: nvgpu: pg init task hogging cpu The Pg init task hogs the kernel by having a wait condition with no timeout waiting for pg state change, but ps state may not post a change in a long time depending on runtime conditions, so we get soft-crashes warning spews in the kernel We solve this by making the condition wait interruptible bug 200346134 Change-Id: I8a3349031acc5065b767dc22eec6e5df113d3ad7 Signed-off-by: David Nieto Reviewed-on: https://git-master.nvidia.com/r/1566545 Reviewed-by: Alex Waterman Reviewed-by: Seshendra Gadagottu Reviewed-by: Automatic_Commit_Validation_User Reviewed-by: svccoveritychecker Reviewed-by: svc-mobile-coverity Reviewed-by: Terje Bergstrom GVS: Gerrit_Virtual_Submit Reviewed-by: Vijayakumar Subbu --- drivers/gpu/nvgpu/common/pmu/pmu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'drivers') diff --git a/drivers/gpu/nvgpu/common/pmu/pmu.c b/drivers/gpu/nvgpu/common/pmu/pmu.c index 89216ea2..c0ab27db 100644 --- a/drivers/gpu/nvgpu/common/pmu/pmu.c +++ b/drivers/gpu/nvgpu/common/pmu/pmu.c @@ -409,7 +409,7 @@ static int nvgpu_pg_init_task(void *arg) while (true) { - NVGPU_COND_WAIT(&pg_init->wq, + NVGPU_COND_WAIT_INTERRUPTIBLE(&pg_init->wq, (pg_init->state_change == true), 0); pmu->pg_init.state_change = false; -- cgit v1.2.2