linux-dev/drivers/gpu/drm/nouveau/nvkm/subdev/mmu, branch master

drm/nouveau/mmu: drop unexpected word "the" in the comments

2022-06-21T20:02:52Z

there is an unexpected word "the" in the comments that need to be dropped file: drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmm.c line: 1051 * have the the deepest nesting of page tables. changed to * have the deepest nesting of page tables. Signed-off-by: Jiang Jian Signed-off-by: Lyude Paul Link: https://patchwork.freedesktop.org/patch/msgid/20220621133920.8112-1-jiangjian@cdjrlc.com

drm/nouveau/mmu: fix typo in comment

2022-05-23T19:55:16Z

Spelling mistake (triple letters) in comment. Detected with the help of Coccinelle. Signed-off-by: Julia Lawall Reviewed-by: Lyude Paul [also fix double space] Signed-off-by: Lyude Paul Link: https://patchwork.freedesktop.org/patch/msgid/20220521111145.81697-57-Julia.Lawall@inria.fr

drm/nouveau/mmu/gp100: remove unused variable

2021-10-12T17:50:44Z

Fixes a compilation issue introduced because I forgot to test with WERROR enabled. Cc: Stephen Rothwell Cc: DRI Cc: nouveau@lists.freedesktop.org Fixes: 404046cf4805 ("drm/nouveau/mmu/gp100-: drop unneeded assignment in the if condition.") Acked-by: Adam Jackson Reviewed-by: Lyude Paul Signed-off-by: Karol Herbst Link: https://patchwork.freedesktop.org/patch/msgid/20211012133334.1737918-1-kherbst@redhat.com

drm/nouveau/mmu/gp100-: drop unneeded assignment in the if condition.

2021-10-11T11:06:49Z

In order to keep the code style consistency of the whole file, the 'inst' assignments should be deleted. The clang_analyzer complains as follows: drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmgp100.c:499:8: warning: Although the value storedto 'inst' is used in the enclosing expression, the value is never actually read from 'inst'. Karol: Removed unneeded brackets. Reported-by: Zeal Robot Signed-off-by: Luo penghao Reviewed-by: Karol Herbst Signed-off-by: Karol Herbst Link: https://patchwork.freedesktop.org/patch/msgid/20210821021447.27097-1-luo.penghao@zte.com.cn

drm/nouveau/mmu: drop unneeded assignment in the nvkm_uvmm_mthd_page()

2021-10-11T11:06:49Z

In order to keep the code style consistency of the whole file, the 'ret' assignments should be deleted. The clang_analyzer complains as follows: drivers/gpu/drm/nouveau/nvkm/subdev/mmu/uvmm.c:317:8:warning: Although the value storedto 'ret' is used in the enclosing expression, the value is never actually read from 'ret'. Reported-by: Zeal Robot Signed-off-by: Luo penghao Reviewed-by: Karol Herbst Signed-off-by: Karol Herbst Link: https://patchwork.freedesktop.org/patch/msgid/20210821021106.27010-1-luo.penghao@zte.com.cn

drm/nouveau: rip out nvkm_client.super

2021-08-18T09:00:15Z

No longer required now that userspace can't touch anything that might need it, and should fix DRM MM operations racing with each other, and the random hangs/crashes that come with that. Signed-off-by: Ben Skeggs Reviewed-by: Lyude Paul

nouveau/svm: implement atomic SVM access

2021-07-01T18:06:04Z

Some NVIDIA GPUs do not support direct atomic access to system memory via PCIe. Instead this must be emulated by granting the GPU exclusive access to the memory. This is achieved by replacing CPU page table entries with special swap entries that fault on userspace access. The driver then grants the GPU permission to update the page undergoing atomic access via the GPU page tables. When CPU access to the page is required a CPU fault is raised which calls into the device driver via MMU notifiers to revoke the atomic access. The original page table entries are then restored allowing CPU access to proceed. Link: https://lkml.kernel.org/r/20210616105937.23201-11-apopple@nvidia.com Signed-off-by: Alistair Popple Reviewed-by: Ben Skeggs Cc: Christoph Hellwig Cc: Hugh Dickins Cc: Jason Gunthorpe Cc: John Hubbard Cc: "Matthew Wilcox (Oracle)" Cc: Peter Xu Cc: Ralph Campbell Cc: Shakeel Butt Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds

drm/nouveau/mmu: index engref by subdev type

2021-02-11T01:49:54Z

None of the chipsets we use this on have instanced engines, so this is fine. Signed-off-by: Ben Skeggs Reviewed-by: Lyude Paul

drm/nouveau/mmu: switch to instanced constructor

2021-02-11T01:49:54Z

Signed-off-by: Ben Skeggs Reviewed-by: Lyude Paul

drm/nouveau/subdev: store full subdev name in struct

2021-02-11T00:14:26Z

Much easier to store this to avoid having to reconstruct a string for a specific subdev, taking into account whether it's instanced or not. Signed-off-by: Ben Skeggs Reviewed-by: Lyude Paul