From 0269339256dee0a8fcd2d6aa1180780039f22fab Mon Sep 17 00:00:00 2001 From: Deepak Nibade Date: Mon, 12 Oct 2015 18:09:43 +0530 Subject: gpu: nvgpu: restart timer instead of cancel In gk20a_fifo_handle_sched_error(), we currently cancel the timeout on all the channels But this could cause us to miss one of stuck channel hence, instead of cancelling, restart the timeout of channel on which it is already active Bug 200133289 Change-Id: I40e7e0e5394911fc110ab6fde39592b885dfaf7d Signed-off-by: Deepak Nibade Reviewed-on: http://git-master/r/816133 Reviewed-by: Ishan Mittal Tested-by: Ishan Mittal --- drivers/gpu/nvgpu/gk20a/channel_gk20a.c | 17 +++++++++++++++-- 1 file changed, 15 insertions(+), 2 deletions(-) (limited to 'drivers/gpu/nvgpu/gk20a/channel_gk20a.c') diff --git a/drivers/gpu/nvgpu/gk20a/channel_gk20a.c b/drivers/gpu/nvgpu/gk20a/channel_gk20a.c index 6f0d7375..34b62ac4 100644 --- a/drivers/gpu/nvgpu/gk20a/channel_gk20a.c +++ b/drivers/gpu/nvgpu/gk20a/channel_gk20a.c @@ -1574,7 +1574,7 @@ static void gk20a_channel_timeout_stop(struct channel_gk20a *ch) mutex_unlock(&ch->timeout.lock); } -void gk20a_channel_timeout_stop_all_channels(struct gk20a *g) +void gk20a_channel_timeout_restart_all_channels(struct gk20a *g) { u32 chid; struct fifo_gk20a *f = &g->fifo; @@ -1583,7 +1583,20 @@ void gk20a_channel_timeout_stop_all_channels(struct gk20a *g) struct channel_gk20a *ch = &f->channel[chid]; if (gk20a_channel_get(ch)) { - gk20a_channel_timeout_stop(ch); + mutex_lock(&ch->timeout.lock); + if (!ch->timeout.initialized) { + mutex_unlock(&ch->timeout.lock); + gk20a_channel_put(ch); + continue; + } + mutex_unlock(&ch->timeout.lock); + + cancel_delayed_work_sync(&ch->timeout.wq); + if (!ch->has_timedout) + schedule_delayed_work(&ch->timeout.wq, + msecs_to_jiffies( + gk20a_get_channel_watchdog_timeout(ch))); + gk20a_channel_put(ch); } } -- cgit v1.2.2