From: Ben Skeggs <bske...@redhat.com>

[ Upstream commit 7adc77aa0e11f25b0e762859219c70852cd8d56f ]

Method init is typically ordered by class in the FW image as ThreeD,
TwoD, Compute.

Due to a bug in parsing the FW into our internal format, we've been
accidentally sending Twod + Compute methods to the ThreeD class, as
well as Compute methods to the TwoD class - oops.

Signed-off-by: Ben Skeggs <bske...@redhat.com>
Signed-off-by: Sasha Levin <sas...@kernel.org>
---
 .../gpu/drm/nouveau/nvkm/engine/gr/gk20a.c    | 21 ++++++++++---------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/gr/gk20a.c 
b/drivers/gpu/drm/nouveau/nvkm/engine/gr/gk20a.c
index de8b806b88fd9..7618b2eb4fdfd 100644
--- a/drivers/gpu/drm/nouveau/nvkm/engine/gr/gk20a.c
+++ b/drivers/gpu/drm/nouveau/nvkm/engine/gr/gk20a.c
@@ -143,23 +143,24 @@ gk20a_gr_av_to_method(struct gf100_gr *gr, const char 
*fw_name,
 
        nent = (fuc.size / sizeof(struct gk20a_fw_av));
 
-       pack = vzalloc((sizeof(*pack) * max_classes) +
-                      (sizeof(*init) * (nent + 1)));
+       pack = vzalloc((sizeof(*pack) * (max_classes + 1)) +
+                      (sizeof(*init) * (nent + max_classes + 1)));
        if (!pack) {
                ret = -ENOMEM;
                goto end;
        }
 
-       init = (void *)(pack + max_classes);
+       init = (void *)(pack + max_classes + 1);
 
-       for (i = 0; i < nent; i++) {
-               struct gf100_gr_init *ent = &init[i];
+       for (i = 0; i < nent; i++, init++) {
                struct gk20a_fw_av *av = &((struct gk20a_fw_av *)fuc.data)[i];
                u32 class = av->addr & 0xffff;
                u32 addr = (av->addr & 0xffff0000) >> 14;
 
                if (prevclass != class) {
-                       pack[classidx].init = ent;
+                       if (prevclass) /* Add terminator to the method list. */
+                               init++;
+                       pack[classidx].init = init;
                        pack[classidx].type = class;
                        prevclass = class;
                        if (++classidx >= max_classes) {
@@ -169,10 +170,10 @@ gk20a_gr_av_to_method(struct gf100_gr *gr, const char 
*fw_name,
                        }
                }
 
-               ent->addr = addr;
-               ent->data = av->data;
-               ent->count = 1;
-               ent->pitch = 1;
+               init->addr = addr;
+               init->data = av->data;
+               init->count = 1;
+               init->pitch = 1;
        }
 
        *ppack = pack;
-- 
2.20.1

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to