|WARNING| pw104398 [PATCH v6 1/1] gpu/cuda: introduce CUDA driver

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Nov 16 13:37:22 CET 2021


Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/104398

_coding style issues_


ERROR:TRAILING_WHITESPACE: trailing whitespace
#162: FILE: doc/guides/gpus/cuda.rst:18:
+- Install `CUDA Toolkit <https://developer.nvidia.com/cuda-toolkit>`_ $

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#347: FILE: drivers/gpu/cuda/cuda.c:24:
+CUresult CUDAAPI (*sym_cuInit)(unsigned int flags) = NULL;

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#348: FILE: drivers/gpu/cuda/cuda.c:25:
+CUresult CUDAAPI (*sym_cuDriverGetVersion)(int *driverVersion) = NULL;

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#349: FILE: drivers/gpu/cuda/cuda.c:26:
+CUresult CUDAAPI (*sym_cuGetProcAddress)(const char *symbol,

ERROR:CODE_INDENT: code indent should use tabs where possible
#412: FILE: drivers/gpu/cuda/cuda.c:89:
+        {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#412: FILE: drivers/gpu/cuda/cuda.c:89:
+        {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#413: FILE: drivers/gpu/cuda/cuda.c:90:
+                RTE_PCI_DEVICE(NVIDIA_GPU_VENDOR_ID,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#413: FILE: drivers/gpu/cuda/cuda.c:90:
+                RTE_PCI_DEVICE(NVIDIA_GPU_VENDOR_ID,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#414: FILE: drivers/gpu/cuda/cuda.c:91:
+                                NVIDIA_GPU_A100_80GB_DEVICE_ID)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#414: FILE: drivers/gpu/cuda/cuda.c:91:
+                                NVIDIA_GPU_A100_80GB_DEVICE_ID)$

ERROR:CODE_INDENT: code indent should use tabs where possible
#415: FILE: drivers/gpu/cuda/cuda.c:92:
+        },$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#415: FILE: drivers/gpu/cuda/cuda.c:92:
+        },$

ERROR:CODE_INDENT: code indent should use tabs where possible
#416: FILE: drivers/gpu/cuda/cuda.c:93:
+        {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#416: FILE: drivers/gpu/cuda/cuda.c:93:
+        {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#417: FILE: drivers/gpu/cuda/cuda.c:94:
+                RTE_PCI_DEVICE(NVIDIA_GPU_VENDOR_ID,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#417: FILE: drivers/gpu/cuda/cuda.c:94:
+                RTE_PCI_DEVICE(NVIDIA_GPU_VENDOR_ID,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#418: FILE: drivers/gpu/cuda/cuda.c:95:
+                                NVIDIA_GPU_A30_24GB_DEVICE_ID)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#418: FILE: drivers/gpu/cuda/cuda.c:95:
+                                NVIDIA_GPU_A30_24GB_DEVICE_ID)$

ERROR:CODE_INDENT: code indent should use tabs where possible
#419: FILE: drivers/gpu/cuda/cuda.c:96:
+        },$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#419: FILE: drivers/gpu/cuda/cuda.c:96:
+        },$

ERROR:CODE_INDENT: code indent should use tabs where possible
#420: FILE: drivers/gpu/cuda/cuda.c:97:
+        {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#420: FILE: drivers/gpu/cuda/cuda.c:97:
+        {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#421: FILE: drivers/gpu/cuda/cuda.c:98:
+                RTE_PCI_DEVICE(NVIDIA_GPU_VENDOR_ID,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#421: FILE: drivers/gpu/cuda/cuda.c:98:
+                RTE_PCI_DEVICE(NVIDIA_GPU_VENDOR_ID,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#422: FILE: drivers/gpu/cuda/cuda.c:99:
+                                NVIDIA_GPU_A10_24GB_DEVICE_ID)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#422: FILE: drivers/gpu/cuda/cuda.c:99:
+                                NVIDIA_GPU_A10_24GB_DEVICE_ID)$

ERROR:CODE_INDENT: code indent should use tabs where possible
#423: FILE: drivers/gpu/cuda/cuda.c:100:
+        },$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#423: FILE: drivers/gpu/cuda/cuda.c:100:
+        },$

ERROR:CODE_INDENT: code indent should use tabs where possible
#424: FILE: drivers/gpu/cuda/cuda.c:101:
+        {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#424: FILE: drivers/gpu/cuda/cuda.c:101:
+        {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#425: FILE: drivers/gpu/cuda/cuda.c:102:
+                RTE_PCI_DEVICE(NVIDIA_GPU_VENDOR_ID,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#425: FILE: drivers/gpu/cuda/cuda.c:102:
+                RTE_PCI_DEVICE(NVIDIA_GPU_VENDOR_ID,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#426: FILE: drivers/gpu/cuda/cuda.c:103:
+                                NVIDIA_GPU_V100_32GB_DEVICE_ID)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#426: FILE: drivers/gpu/cuda/cuda.c:103:
+                                NVIDIA_GPU_V100_32GB_DEVICE_ID)$

ERROR:CODE_INDENT: code indent should use tabs where possible
#427: FILE: drivers/gpu/cuda/cuda.c:104:
+        },$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#427: FILE: drivers/gpu/cuda/cuda.c:104:
+        },$

ERROR:CODE_INDENT: code indent should use tabs where possible
#428: FILE: drivers/gpu/cuda/cuda.c:105:
+        {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#428: FILE: drivers/gpu/cuda/cuda.c:105:
+        {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#429: FILE: drivers/gpu/cuda/cuda.c:106:
+                RTE_PCI_DEVICE(NVIDIA_GPU_VENDOR_ID,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#429: FILE: drivers/gpu/cuda/cuda.c:106:
+                RTE_PCI_DEVICE(NVIDIA_GPU_VENDOR_ID,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#430: FILE: drivers/gpu/cuda/cuda.c:107:
+                                NVIDIA_GPU_V100_16GB_DEVICE_ID)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#430: FILE: drivers/gpu/cuda/cuda.c:107:
+                                NVIDIA_GPU_V100_16GB_DEVICE_ID)$

ERROR:CODE_INDENT: code indent should use tabs where possible
#431: FILE: drivers/gpu/cuda/cuda.c:108:
+        },$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#431: FILE: drivers/gpu/cuda/cuda.c:108:
+        },$

WARNING:LONG_LINE: line length of 103 exceeds 100 columns
#1026: FILE: drivers/gpu/cuda/cuda.c:703:
+		if ((uintptr_t) mem_alloc_list_tail->ptr_d != (uintptr_t) mem_alloc_list_tail->ptr_h) {

WARNING:BRACES: braces {} are not necessary for single statement blocks
#1410: FILE: drivers/gpu/cuda/cuda.c:1087:
+		if (private->gdr_flush_type != CU_FLUSH_GPU_DIRECT_RDMA_WRITES_OPTION_HOST) {
+			rte_cuda_log(ERR, "GPUDirect RDMA flush writes API is not supported");
+		}

total: 21 errors, 25 warnings, 1334 lines checked


More information about the test-report mailing list