[dpdk-test-report] |WARNING| pw103676 [PATCH] [v2, 1/1] gpu/cuda: introduce CUDA driver

[dpdk-test-report] |WARNING| pw103676 [PATCH] [v2, 1/1] gpu/cuda: introduce CUDA driver

[dpdklab]

[-- Attachment #1: Type: text/plain, Size: 947 bytes --]

Test-Label: iol-testing
Test-Status: WARNING
http://dpdk.org/patch/103676

_apply patch failure_

Submitter: Elena Agostini <eagostini@nvidia.com>
Date: Thursday, November 04 2021 02:01:28 
Applied on: CommitID:f88b0b892204d071a46a629b00424a8d47055ba4
Apply patch set 103676 failed:

Checking patch doc/guides/gpus/cuda.rst...
Checking patch doc/guides/gpus/index.rst...
error: doc/guides/gpus/index.rst: does not exist in index
Checking patch drivers/gpu/cuda/cuda.c...
Checking patch drivers/gpu/cuda/meson.build...
Checking patch drivers/gpu/cuda/version.map...
Checking patch drivers/gpu/meson.build...
error: drivers/gpu/meson.build: does not exist in index
Applied patch doc/guides/gpus/cuda.rst cleanly.
Applied patch drivers/gpu/cuda/cuda.c cleanly.
Applied patch drivers/gpu/cuda/meson.build cleanly.
Applied patch drivers/gpu/cuda/version.map cleanly.

https://lab.dpdk.org/results/dashboard/patchsets/19980/

UNH-IOL DPDK Community Lab

[dpdk-test-report] |WARNING| pw103676 [PATCH v2 1/1] gpu/cuda: introduce CUDA driver

[checkpatch]

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

_coding style issues_


WARNING:TYPO_SPELLING: 'straighforward' may be misspelled - perhaps 'straightforward'?
#165: FILE: doc/guides/gpus/cuda.rst:20:
+is quite straighforward and doesn't create any compatibility problem.

WARNING:TYPO_SPELLING: 'enviroment' may be misspelled - perhaps 'environment'?
#176: FILE: doc/guides/gpus/cuda.rst:31:
+If the CUDA driver enviroment has been already initialized, the ``cuInit(0)``

ERROR:GLOBAL_INITIALISERS: do not initialise globals to NULL
#338: FILE: drivers/gpu/cuda/cuda.c:87:
+struct mem_entry *mem_alloc_list_head = NULL;

ERROR:GLOBAL_INITIALISERS: do not initialise globals to NULL
#339: FILE: drivers/gpu/cuda/cuda.c:88:
+struct mem_entry *mem_alloc_list_tail = NULL;

ERROR:GLOBAL_INITIALISERS: do not initialise globals to 0
#340: FILE: drivers/gpu/cuda/cuda.c:89:
+uint32_t mem_alloc_list_last_elem = 0;

WARNING:LONG_LINE: line length of 103 exceeds 100 columns
#361: FILE: drivers/gpu/cuda/cuda.c:110:
+		mem_alloc_list_head = rte_zmalloc(NULL, sizeof(struct mem_entry), RTE_CACHE_LINE_SIZE);

WARNING:LONG_LINE: line length of 120 exceeds 100 columns
#371: FILE: drivers/gpu/cuda/cuda.c:120:
+		struct mem_entry *mem_alloc_list_cur = rte_zmalloc(NULL, sizeof(struct mem_entry), RTE_CACHE_LINE_SIZE);

WARNING:LONG_LINE: line length of 110 exceeds 100 columns
#496: FILE: drivers/gpu/cuda/cuda.c:245:
+		dev->mpshared->dev_private = rte_zmalloc(NULL, sizeof(struct cuda_info), RTE_CACHE_LINE_SIZE);

WARNING:LONG_LINE: line length of 105 exceeds 100 columns
#587: FILE: drivers/gpu/cuda/cuda.c:336:
+	res = cuPointerSetAttribute(&flag, CU_POINTER_ATTRIBUTE_SYNC_MEMOPS, mem_alloc_list_tail->ptr_d);

WARNING:LONG_LINE: line length of 138 exceeds 100 columns
#589: FILE: drivers/gpu/cuda/cuda.c:338:
+		rte_gpu_log(ERR, "Could not set SYNC MEMOP attribute for GPU memory at %llx , err %d
", mem_alloc_list_tail->ptr_d, res);

WARNING:LONG_LINE: line length of 147 exceeds 100 columns
#655: FILE: drivers/gpu/cuda/cuda.c:404:
+	res = cuMemHostRegister(mem_alloc_list_tail->ptr_h, mem_alloc_list_tail->size, CU_MEMHOSTREGISTER_PORTABLE | CU_MEMHOSTREGISTER_DEVICEMAP);

WARNING:LONG_LINE: line length of 113 exceeds 100 columns
#659: FILE: drivers/gpu/cuda/cuda.c:408:
+						err_string, mem_alloc_list_tail->ptr_h, mem_alloc_list_tail->size

WARNING:LONG_LINE: line length of 132 exceeds 100 columns
#666: FILE: drivers/gpu/cuda/cuda.c:415:
+									CU_DEVICE_ATTRIBUTE_CAN_USE_HOST_POINTER_FOR_REGISTERED_MEM,

WARNING:LONG_LINE: line length of 130 exceeds 100 columns
#667: FILE: drivers/gpu/cuda/cuda.c:416:
+									((struct cuda_info *)(dev->mpshared->dev_private))->cu_dev

WARNING:LONG_LINE: line length of 110 exceeds 100 columns
#679: FILE: drivers/gpu/cuda/cuda.c:428:
+		res = cuMemHostGetDevicePointer(&(mem_alloc_list_tail->ptr_d), mem_alloc_list_tail->ptr_h, 0);

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

WARNING:LONG_LINE: line length of 105 exceeds 100 columns
#696: FILE: drivers/gpu/cuda/cuda.c:445:
+	res = cuPointerSetAttribute(&flag, CU_POINTER_ATTRIBUTE_SYNC_MEMOPS, mem_alloc_list_tail->ptr_d);

WARNING:LONG_LINE: line length of 138 exceeds 100 columns
#698: FILE: drivers/gpu/cuda/cuda.c:447:
+		rte_gpu_log(ERR, "Could not set SYNC MEMOP attribute for GPU memory at %llx , err %d
", mem_alloc_list_tail->ptr_d, res);

WARNING:LONG_LINE: line length of 102 exceeds 100 columns
#781: FILE: drivers/gpu/cuda/cuda.c:530:
+			rte_gpu_log(ERR, "cuMemHostUnregister current failed with %s.
", err_string);

WARNING:UNNECESSARY_ELSE: else is not generally useful after a break or return
#787: FILE: drivers/gpu/cuda/cuda.c:536:
+		return mem_list_del_item(hk);
+	} else {

WARNING:LONG_LINE: line length of 108 exceeds 100 columns
#873: FILE: drivers/gpu/cuda/cuda.c:622:
+	res = cuDeviceGetAttribute(&(processor_count), CU_DEVICE_ATTRIBUTE_MULTIPROCESSOR_COUNT, cu_dev_id);

WARNING:LONG_LINE: line length of 102 exceeds 100 columns
#898: FILE: drivers/gpu/cuda/cuda.c:647:
+	dev->mpshared->dev_private = rte_zmalloc(NULL, sizeof(struct cuda_info), RTE_CACHE_LINE_SIZE);

total: 3 errors, 19 warnings, 849 lines checked
Warning in drivers/gpu/cuda/cuda.c:
Using %l format, prefer %PRI*64 if type is [u]int64_t