diff options
| author | Christian König <ckoenig.leichtzumerken@gmail.com> | 2025-11-20 15:41:08 +0100 |
|---|---|---|
| committer | Christian König <christian.koenig@amd.com> | 2025-11-25 10:07:08 +0100 |
| commit | 17331d0f339fc6ec00a118e72ca28cb39d18e960 (patch) | |
| tree | af2c48f6fe67feeea55e527fa621d6aa213bba3e /drivers/dma-buf | |
| parent | 0a8bc1d03e3eefb5d160fd51add3a016f29292e8 (diff) | |
dma-buf/sw-sync: always taint the kernel when sw-sync is used
The SW-sync functionality should only be used for testing and debugging
since it is inherently unsave.
Signed-off-by: Christian König <christian.koenig@amd.com>
Acked-by: Sumit Semwal <sumit.semwal@linaro.org>
Link: https://lore.kernel.org/r/20251120150018.27385-3-christian.koenig@amd.com
Diffstat (limited to 'drivers/dma-buf')
| -rw-r--r-- | drivers/dma-buf/sw_sync.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/dma-buf/sw_sync.c b/drivers/dma-buf/sw_sync.c index 3c20f1d31cf5..6f09d13be6b6 100644 --- a/drivers/dma-buf/sw_sync.c +++ b/drivers/dma-buf/sw_sync.c @@ -8,6 +8,7 @@ #include <linux/file.h> #include <linux/fs.h> #include <linux/uaccess.h> +#include <linux/panic.h> #include <linux/slab.h> #include <linux/sync_file.h> @@ -349,6 +350,9 @@ static long sw_sync_ioctl_create_fence(struct sync_timeline *obj, struct sync_file *sync_file; struct sw_sync_create_fence_data data; + /* SW sync fence are inherently unsafe and can deadlock the kernel */ + add_taint(TAINT_SOFTLOCKUP, LOCKDEP_STILL_OK); + if (fd < 0) return fd; |