[Nouveau] GP10B regression

[Ilia Mirkin]

On Tue, Nov 21, 2017 at 1:07 PM, Mikko Perttunen <cyndis at kapsi.fi> wrote:
> Thanks to Thierry for finding this - applying
>
> index e14643615698..00eeaaffeae5 100644
> --- a/drivers/gpu/drm/nouveau/nvkm/engine/device/base.c
> +++ b/drivers/gpu/drm/nouveau/nvkm/engine/device/base.c
> @@ -2369,7 +2369,7 @@ nv13b_chipset = {
>         .imem = gk20a_instmem_new,
>         .ltc = gp100_ltc_new,
>         .mc = gp10b_mc_new,
> -       .mmu = gf100_mmu_new,
> +       .mmu = gp10b_mmu_new,
>         .secboot = gp10b_secboot_new,
>         .pmu = gm20b_pmu_new,
>         .timer = gk20a_timer_new,

This seems like an obvious typo. Feel free to send a proper patch with
a Fixes: tag referencing either the commit that flips things over or
6359c98224 (drm/nouveau/mmu/gp10b: fork from gf100)

> diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmgp10b.c
> b/drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmgp10b.c
> index 3dcc6bddb32f..470a4fadc165 100644
> --- a/drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmgp10b.c
> +++ b/drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmgp10b.c
> @@ -33,7 +33,7 @@ gp10b_vmm = {
>                 { 38, &gp100_vmm_desc_16[3], NVKM_VMM_PAGE_Sxxx },
>                 { 29, &gp100_vmm_desc_16[2], NVKM_VMM_PAGE_Sxxx },
>                 { 21, &gp100_vmm_desc_16[1], NVKM_VMM_PAGE_SxHC },
> -               { 16, &gp100_vmm_desc_16[0], NVKM_VMM_PAGE_SxHC },
> +/*             { 16, &gp100_vmm_desc_16[0], NVKM_VMM_PAGE_SxHC },*/
>                 { 12, &gp100_vmm_desc_12[0], NVKM_VMM_PAGE_SxHx },
>                 {}
>         }
>
> on top of next-20171121 works at least for a simple test.

You're on your own for this one :) I think Ben's on vacation until
next week too.

gp100_vmm_desc_16[0] is

 +       { LPT, 5,  8, 0x0100, &gp100_vmm_desc_lpt },

_16 is in reference to 64KB large pages (1 << 16).
LPT is a sub-allocated page table? Not sure on the details.
SxHC = sparse, host memory, compression enabled

Good luck.

  -ilia