dma-buf/heaps: replace kmap_atomic with kmap_local_page
authorPintu Kumar <quic_pintu@quicinc.com>
Tue, 1 Oct 2024 17:50:57 +0000 (23:20 +0530)
committerSumit Semwal <sumit.semwal@linaro.org>
Mon, 14 Oct 2024 14:46:03 +0000 (20:16 +0530)
Use of kmap_atomic/kunmap_atomic is deprecated, use
kmap_local_page/kunmap_local instead.

This is reported by checkpatch.
Also fix repeated word issue.

WARNING: Deprecated use of 'kmap_atomic', prefer 'kmap_local_page' instead
+                       void *vaddr = kmap_atomic(page);

WARNING: Deprecated use of 'kunmap_atomic', prefer 'kunmap_local' instead
+                       kunmap_atomic(vaddr);

WARNING: Possible repeated word: 'by'
+                        * has been killed by by SIGKILL

total: 0 errors, 3 warnings, 405 lines checked

Signed-off-by: Pintu Kumar <quic_pintu@quicinc.com>
Reviewed-by: T.J. Mercier <tjmercier@google.com>
Signed-off-by: Sumit Semwal <sumit.semwal@linaro.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20241001175057.27172-1-quic_pintu@quicinc.com
drivers/dma-buf/heaps/cma_heap.c

index 0e6e198..9512d05 100644 (file)
@@ -309,13 +309,13 @@ static struct dma_buf *cma_heap_allocate(struct dma_heap *heap,
                struct page *page = cma_pages;
 
                while (nr_clear_pages > 0) {
-                       void *vaddr = kmap_atomic(page);
+                       void *vaddr = kmap_local_page(page);
 
                        memset(vaddr, 0, PAGE_SIZE);
-                       kunmap_atomic(vaddr);
+                       kunmap_local(vaddr);
                        /*
                         * Avoid wasting time zeroing memory if the process
-                        * has been killed by by SIGKILL
+                        * has been killed by SIGKILL.
                         */
                        if (fatal_signal_pending(current))
                                goto free_cma;