[dpdk-dev] [PATCH] test: reduce duration for timer racecond testcase

2018-09-22 Thread Jananee Parthasarathy
Reduced test duration for timer_racecond unit test. This wil help to receive quicker test results. Signed-off-by: Jananee Parthasarathy --- test/test/test_timer_racecond.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/test/test_timer_racecond.c b/test/test/test_timer_r

Re: [dpdk-dev] [PATCH] test: reduce duration for timer racecond testcase

2018-09-22 Thread Parthasarathy, JananeeX M
CC Robert as per MAINTAINERS list. >-Original Message- >From: Parthasarathy, JananeeX M >Sent: Saturday, September 22, 2018 2:27 PM >To: dev@dpdk.org >Cc: Pattan, Reshma ; Parthasarathy, JananeeX M > >Subject: [PATCH] test: reduce duration for timer racecond testcase > >Reduced test durati

[dpdk-dev] [PATCH] test: reduce duration for timer autotest

2018-09-22 Thread Jananee Parthasarathy
Reduced the test duration of timer_autotest. The number of iterations was very large to test the timer functionality. Although the method to identify random timers can be different, many iterations would be needed for stress or performance test only. Signed-off-by: Jananee Parthasarathy --- test

[dpdk-dev] [PATCH 1/2] compress/qat: fix for zero checksum on decompression

2018-09-22 Thread Fiona Trahe
Checksum was always 0 on QAT decompression due to incorrect use of union variable. Fixes: 6a7ea14819e9 ("compress/qat: add xform processing") Cc: sta...@dpdk.org Signed-off-by: Fiona Trahe --- drivers/compress/qat/qat_comp.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git

[dpdk-dev] [PATCH 2/2] compress/qat: remove unnecessary assignment

2018-09-22 Thread Fiona Trahe
Same variable was assigned twice, remove one. Fixes: 6a7ea14819e9 ("compress/qat: add xform processing") Cc: sta...@dpdk.org Signed-off-by: Fiona Trahe --- drivers/compress/qat/qat_comp.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/compress/qat/qat_comp.c b/drivers/compress/qat/q

[dpdk-dev] [PATCH] test/crypto: fix number of queue pairs

2018-09-22 Thread Fiona Trahe
Some of the tests use a QAT-specific value (2) for maximum nr of queue pairs to create valid/invalid test cases. This has accidentally worked ok as default max_qps for all PMDs is larger. It is incorrect however and would fail if a device had a max lower than the QAT value. Instead use the value re

[dpdk-dev] [PATCH v4 1/5] vhost: unify struct VhostUserMsg usage

2018-09-22 Thread Nikolay Nikolaev
Do not use the typedef version of struct VhostUserMsg. Also unify the related parameter name. Signed-off-by: Nikolay Nikolaev --- lib/librte_vhost/vhost_user.c | 41 + 1 file changed, 21 insertions(+), 20 deletions(-) diff --git a/lib/librte_vhost/vhost

[dpdk-dev] [PATCH v4 0/5] vhost: vhost_user.c code cleanup

2018-09-22 Thread Nikolay Nikolaev
vhost: vhost_user.c code cleanup This patchesries introduce a set of code redesigns in vhost_user.c. The goal is to unify and simplify vhost-user message handling. The patches do not intend to introduce any functional changes. v4 changes: - use struct VhostUserMsg as the coding style guide sugg

[dpdk-dev] [PATCH v4 3/5] vhost: handle unsupported message types in functions

2018-09-22 Thread Nikolay Nikolaev
Add new functions to handle the unsupported vhost message types: - vhost_user_set_vring_err - vhost_user_set_log_fd Signed-off-by: Nikolay Nikolaev --- lib/librte_vhost/vhost_user.c | 22 +- 1 file changed, 17 insertions(+), 5 deletions(-) diff --git a/lib/librte_vhost/v

[dpdk-dev] [PATCH v4 2/5] vhost: make message handling functions prepare the reply

2018-09-22 Thread Nikolay Nikolaev
As VhostUserMsg structure is reused to generate the reply, move the relevant fields update into the respective message handling functions. Signed-off-by: Nikolay Nikolaev --- lib/librte_vhost/vhost_user.c | 24 1 file changed, 16 insertions(+), 8 deletions(-) diff --g

[dpdk-dev] [PATCH v4 4/5] vhost: unify message handling function signature

2018-09-22 Thread Nikolay Nikolaev
Each vhost-user message handling function will return an int result which is described in the new enum vh_result: error, OK and reply. All functions will now have two arguments, virtio_net double pointer and VhostUserMsg pointer. Signed-off-by: Nikolay Nikolaev --- lib/librte_vhost/vhost_user.c

[dpdk-dev] [PATCH v4 5/5] vhost: message handling implemented as a callback array

2018-09-22 Thread Nikolay Nikolaev
Introduce vhost_message_handlers, which maps the message request type to the message handler. Then replace the switch construct with a map and call. Failing vhost_user_set_features is fatal and all processing should stop immediately and propagate the error to the upper layers. Change the code acco