diff options
author | Seema Khowala <seemaj@nvidia.com> | 2017-08-31 15:45:10 -0400 |
---|---|---|
committer | mobile promotions <svcmobile_promotions@nvidia.com> | 2017-09-18 21:19:07 -0400 |
commit | e229cb76ba348f8ab445a029adfc55a9b05d1161 (patch) | |
tree | cdceb46f68ae0dbbc7c7cc218b0de090f2e5dc4a | |
parent | 5d09c908b0679f060bc89ae70eef681a6783ebbc (diff) |
gpu: nvgpu: fix verbose return value
verbose return value is not taking all the channels into account.
Fix this by ORing verbose values for all channels.
Change-Id: Id77c74458067c72792422aa69be1626c3d164e1c
Signed-off-by: Seema Khowala <seemaj@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1549645
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
-rw-r--r-- | drivers/gpu/nvgpu/gk20a/fifo_gk20a.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/gpu/nvgpu/gk20a/fifo_gk20a.c b/drivers/gpu/nvgpu/gk20a/fifo_gk20a.c index 1815c15b..cb2d8813 100644 --- a/drivers/gpu/nvgpu/gk20a/fifo_gk20a.c +++ b/drivers/gpu/nvgpu/gk20a/fifo_gk20a.c | |||
@@ -1333,12 +1333,12 @@ bool gk20a_fifo_error_tsg(struct gk20a *g, | |||
1333 | struct tsg_gk20a *tsg) | 1333 | struct tsg_gk20a *tsg) |
1334 | { | 1334 | { |
1335 | struct channel_gk20a *ch = NULL; | 1335 | struct channel_gk20a *ch = NULL; |
1336 | bool verbose = true; | 1336 | bool verbose = false; |
1337 | 1337 | ||
1338 | down_read(&tsg->ch_list_lock); | 1338 | down_read(&tsg->ch_list_lock); |
1339 | list_for_each_entry(ch, &tsg->ch_list, ch_entry) { | 1339 | list_for_each_entry(ch, &tsg->ch_list, ch_entry) { |
1340 | if (gk20a_channel_get(ch)) { | 1340 | if (gk20a_channel_get(ch)) { |
1341 | verbose = gk20a_fifo_error_ch(g, ch); | 1341 | verbose |= gk20a_fifo_error_ch(g, ch); |
1342 | gk20a_channel_put(ch); | 1342 | gk20a_channel_put(ch); |
1343 | } | 1343 | } |
1344 | } | 1344 | } |