diff options
author | David Nieto <dmartineznie@nvidia.com> | 2017-09-22 13:47:12 -0400 |
---|---|---|
committer | mobile promotions <svcmobile_promotions@nvidia.com> | 2017-09-25 07:48:12 -0400 |
commit | 82ef5f7b3b51bff51b18f525ee14033a4b6befd4 (patch) | |
tree | 5505aeb7bbd30b95bbc3f440e37f48bd823bd403 /drivers/gpu/nvgpu/common | |
parent | 350bb74859eb6eb0d0ba7c8e6792a0b4e48849b4 (diff) |
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 <dmartineznie@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1566545
Reviewed-by: Alex Waterman <alexw@nvidia.com>
Reviewed-by: Seshendra Gadagottu <sgadagottu@nvidia.com>
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: svccoveritychecker <svccoveritychecker@nvidia.com>
Reviewed-by: svc-mobile-coverity <svc-mobile-coverity@nvidia.com>
Reviewed-by: Terje Bergstrom <tbergstrom@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Vijayakumar Subbu <vsubbu@nvidia.com>
Diffstat (limited to 'drivers/gpu/nvgpu/common')
-rw-r--r-- | drivers/gpu/nvgpu/common/pmu/pmu.c | 2 |
1 files changed, 1 insertions, 1 deletions
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) | |||
409 | 409 | ||
410 | while (true) { | 410 | while (true) { |
411 | 411 | ||
412 | NVGPU_COND_WAIT(&pg_init->wq, | 412 | NVGPU_COND_WAIT_INTERRUPTIBLE(&pg_init->wq, |
413 | (pg_init->state_change == true), 0); | 413 | (pg_init->state_change == true), 0); |
414 | 414 | ||
415 | pmu->pg_init.state_change = false; | 415 | pmu->pg_init.state_change = false; |