diff options
| author | Christian König <christian.koenig@amd.com> | 2021-06-01 19:47:50 +0200 |
|---|---|---|
| committer | Christian König <christian.koenig@amd.com> | 2021-06-08 11:47:51 +0200 |
| commit | f91142c62161aecb6570d1d0a01aefcb6130b12b (patch) | |
| tree | 2357efae00423f561935a120cdb3988af240250b /drivers/gpu/drm/amd/amdgpu/amdgpu_preempt_mgr.c | |
| parent | ceb515ba29ba6b798a579d7c62c5f71a766f7aa6 (diff) | |
| download | linux-f91142c62161aecb6570d1d0a01aefcb6130b12b.tar.gz linux-f91142c62161aecb6570d1d0a01aefcb6130b12b.tar.bz2 linux-f91142c62161aecb6570d1d0a01aefcb6130b12b.zip | |
drm/ttm: nuke VM_MIXEDMAP on BO mappings v3
We discussed if that is really the right approach for quite a while now, but
digging deeper into a bug report on arm turned out that this is actually
horrible broken right now.
The reason for this is that vmf_insert_mixed_prot() always tries to grab
a reference to the underlaying page on architectures without
ARCH_HAS_PTE_SPECIAL and as far as I can see also enabled GUP.
So nuke using VM_MIXEDMAP here and use VM_PFNMAP instead.
Also make sure to reject mappings without VM_SHARED.
v2: reject COW mappings, merge function with only caller
v3: adjust comment as suggested by Thomas
Signed-off-by: Christian König <christian.koenig@amd.com>
Bugs: https://gitlab.freedesktop.org/drm/amd/-/issues/1606#note_936174
Link: https://patchwork.freedesktop.org/patch/msgid/20210607135830.8574-1-christian.koenig@amd.com
Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_preempt_mgr.c')
0 files changed, 0 insertions, 0 deletions