ligang/vpp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

memif: fix coverity warnings

Browse Source 
Type: fix

Signed-off-by: Marvin Liu <yong.liu@intel.com>
Change-Id: Ie3f390be16df81f6824344034377f9a6f4fa9f92
This commit is contained in:
Marvin Liu
2023-05-05 23:03:46 +08:00
committed by Damjan Marion
parent 00c37199d7
commit 754f79c36a
3 changed files with 7 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/plugins/memif/device.c
View File

@ -611,7 +611,7 @@ no_free_slots:
if (PREDICT_TRUE (!fallback)) if (PREDICT_TRUE (!fallback))
{ {
vlib_dma_batch_set_cookie (vm, b, vlib_dma_batch_set_cookie (vm, b,
(mif_id << 16) | (mq - mif->tx_queues)); ((u64) mif_id << 16) | (mq - mif->tx_queues));
vlib_dma_batch_submit (vm, b); vlib_dma_batch_submit (vm, b);
dma_info->finished = 0; dma_info->finished = 0;

11
src/plugins/memif/memif_test.c
View File

@ -407,12 +407,11 @@ api_memif_create_v2 (vat_main_t *vam)
mp->ring_size = clib_host_to_net_u32 (ring_size); mp->ring_size = clib_host_to_net_u32 (ring_size);
mp->buffer_size = clib_host_to_net_u16 (buffer_size & 0xffff); mp->buffer_size = clib_host_to_net_u16 (buffer_size & 0xffff);
mp->socket_id = clib_host_to_net_u32 (socket_id); mp->socket_id = clib_host_to_net_u32 (socket_id);
if (secret != 0)
{ char *p = (char *) &mp->secret;
char *p = (char *) &mp->secret; p += vl_api_vec_to_api_string (secret, (vl_api_string_t *) p);
p += vl_api_vec_to_api_string (secret, (vl_api_string_t *) p); vec_free (secret);
vec_free (secret);
}
memcpy (mp->hw_addr, hw_addr, 6); memcpy (mp->hw_addr, hw_addr, 6);
mp->rx_queues = rx_queues; mp->rx_queues = rx_queues;
mp->tx_queues = tx_queues; mp->tx_queues = tx_queues;

2
src/plugins/memif/node.c
View File

@ -1286,7 +1286,7 @@ memif_device_input_inline_dma (vlib_main_t *vm, vlib_node_runtime_t *node,
mq->dma_info_tail = 0; mq->dma_info_tail = 0;
done: done:
vlib_dma_batch_set_cookie (vm, db, (mif_id << 16) | qid); vlib_dma_batch_set_cookie (vm, db, ((u64) mif_id << 16) | qid);
vlib_dma_batch_submit (vm, db); vlib_dma_batch_submit (vm, db);
vec_reset_length (ptd->copy_ops); vec_reset_length (ptd->copy_ops);