On Tue, Nov 21, 2017 at 1:07 PM, Mikko Perttunen <cyndis@xxxxxxxx> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
