diff options
| author | Rob Clark <[email protected]> | 2022-11-30 10:57:48 -0800 |
|---|---|---|
| committer | Javier Martinez Canillas <[email protected]> | 2022-12-04 18:56:53 +0100 |
| commit | 09bf649a74573cb596e211418a4f8008f265c5a9 (patch) | |
| tree | 5ba072d14eb9fa8c6e4a55b0d1e0f475454895ed /lib/test-string_helpers.c | |
| parent | 24013314be6ee4ee456114a671e9fa3461323de8 (diff) | |
drm/shmem-helper: Avoid vm_open error paths
vm_open() is not allowed to fail. Fortunately we are guaranteed that
the pages are already pinned, thanks to the initial mmap which is now
being cloned into a forked process, and only need to increment the
refcnt. So just increment it directly. Previously if a signal was
delivered at the wrong time to the forking process, the
mutex_lock_interruptible() could fail resulting in the pages_use_count
not being incremented.
Fixes: 2194a63a818d ("drm: Add library for shmem backed GEM objects")
Cc: [email protected]
Signed-off-by: Rob Clark <[email protected]>
Reviewed-by: Daniel Vetter <[email protected]>
Signed-off-by: Javier Martinez Canillas <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
Diffstat (limited to 'lib/test-string_helpers.c')
0 files changed, 0 insertions, 0 deletions