[lvc-project] [PATCH v2 2/4] dma-mapping: benchmark: avoid needless	copy_to_user if benchmark fails
    Fedor Pchelkin 
    pchelkin at ispras.ru
       
    Sat May  4 14:47:02 MSK 2024
    
    
  
If do_map_benchmark() has failed, there is nothing useful to copy back
to userspace.
Suggested-by: Barry Song <21cnbao at gmail.com>
Signed-off-by: Fedor Pchelkin <pchelkin at ispras.ru>
---
 kernel/dma/map_benchmark.c | 3 +++
 1 file changed, 3 insertions(+)
diff --git a/kernel/dma/map_benchmark.c b/kernel/dma/map_benchmark.c
index 2478957cf9f8..a6edb1ef98c8 100644
--- a/kernel/dma/map_benchmark.c
+++ b/kernel/dma/map_benchmark.c
@@ -256,6 +256,9 @@ static long map_benchmark_ioctl(struct file *file, unsigned int cmd,
 		 * dma_mask changed by benchmark
 		 */
 		dma_set_mask(map->dev, old_dma_mask);
+
+		if (ret)
+			return ret;
 		break;
 	default:
 		return -EINVAL;
-- 
2.45.0
    
    
More information about the lvc-project
mailing list