drm/nouveau/gr/gm107-: fix touching non-existent ppcs in attrib cb setup

Also removes an XXX; according to nvgpu headers the field is called
NV_PGRAPH_GPCS_SWDX_TC_BETA_CB_SIZE_DIV3, so, apparently not some
magic we need to figure out :)

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>

1 files changed, 3 insertions, 1 deletions

diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgm107.c b/drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgm107.c
index 053ff2f850b9..6d3c5011e18c 100644
--- a/drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgm107.c
+++ b/drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgm107.c
@@ -920,13 +920,15 @@ gm107_grctx_generate_attrib(struct gf100_grctx *info)
                         const u32 bs = attrib * gr->ppc_tpc_nr[gpc][ppc];
                         const u32 u = 0x418ea0 + (n * 0x04);
                         const u32 o = PPC_UNIT(gpc, ppc, 0);
+                        if (!(gr->ppc_mask[gpc] & (1 << ppc)))
+                                continue;
                         mmio_wr32(info, o + 0xc0, bs);
                         mmio_wr32(info, o + 0xf4, bo);
                         bo += grctx->attrib_nr_max * gr->ppc_tpc_nr[gpc][ppc];
                         mmio_wr32(info, o + 0xe4, as);
                         mmio_wr32(info, o + 0xf8, ao);
                         ao += grctx->alpha_nr_max * gr->ppc_tpc_nr[gpc][ppc];
-                        mmio_wr32(info, u, ((bs / 3 /*XXX*/) << 16) | bs);
+                        mmio_wr32(info, u, ((bs / 3) << 16) | bs);
                 }
         }
 }