replace strtok with strtok_r

The strtok function is not reentrant. To address this issue in a
multithreaded environment, the strtok_r function should be used,
which is the reentrant version of strto

replace strtok with strtok_r

The strtok function is not reentrant. To address this issue in a
multithreaded environment, the strtok_r function should be used,
which is the reentrant version of strtok.

Signed-off-by: XuQi <1530865185@qq.com>
Change-Id: I35c07c7cf4e20bacb7b1e7c7adaedfcd1a81f86e
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25492
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Changpeng Liu <changpeliu@tencent.com>
Reviewed-by: Jim Harris <jim.harris@nvidia.com>
Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com>
Community-CI: Mellanox Build Bot

show more ...

test_dma: Update condition to corrupt mkey

Use `>=` instead of `>`, otherwise we corrupt mkey later
than expected.
Increment counter before the check if the mkey corruption
is needed.

Suggested-by:

test_dma: Update condition to corrupt mkey

Use `>=` instead of `>`, otherwise we corrupt mkey later
than expected.
Increment counter before the check if the mkey corruption
is needed.

Suggested-by: Jim Harris <jim.harris@samsung.com>
Signed-off-by: Alexey Marchuk <alexeymar@nvidia.com>
Change-Id: I5b948cd567a03498c44faaf81989e0b4b2341933
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24714
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

test/dma: Add verify mode

Use md5 to compare original and new buffer content.
Update crypto tests to use verify payload, add new
tests for small number of qp size and mkey pool

Signed-off-by: Alexe

test/dma: Add verify mode

Use md5 to compare original and new buffer content.
Update crypto tests to use verify payload, add new
tests for small number of qp size and mkey pool

Signed-off-by: Alexey Marchuk <alexeymar@nvidia.com>
Change-Id: I406c0c563f8ccf0d8e868cf9e8369eb1570ee4f9
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23134
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Reviewed-by: Ben Walker <ben@nvidia.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>

show more ...

test/dma: Remove task::is_random

That is enough to use a global variable since
all tasks are either random or not

Suggested-by: Jim Harris <jim.harris@samsung.com>
Signed-off-by: Alexey Marchuk <al

test/dma: Remove task::is_random

That is enough to use a global variable since
all tasks are either random or not

Suggested-by: Jim Harris <jim.harris@samsung.com>
Signed-off-by: Alexey Marchuk <alexeymar@nvidia.com>
Change-Id: If857f62a920266c0f6e56bb834e5b68224f928e8
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24816
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Reviewed-by: Ben Walker <ben@nvidia.com>
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

test/dma: Add option to set iovcnt

Data payload is split into `iovcnt` elements to
trigger handling of the fragmented payload in
accel_mlx5 module

Signed-off-by: Alexey Marchuk <alexeymar@nvidia.co

test/dma: Add option to set iovcnt

Data payload is split into `iovcnt` elements to
trigger handling of the fragmented payload in
accel_mlx5 module

Signed-off-by: Alexey Marchuk <alexeymar@nvidia.com>
Change-Id: I984b4c11041dfa100e47f5f0b436dc938bdb8fce
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23129
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Ben Walker <ben@nvidia.com>

show more ...

test/dma: Add param to corrupt mkey

A new parameters returns invalid mkey each Nth
translation operation. As result, accel_mlx5 qp
goes to error state and triggers recovery.

Signed-off-by: Alexey M

test/dma: Add param to corrupt mkey

A new parameters returns invalid mkey each Nth
translation operation. As result, accel_mlx5 qp
goes to error state and triggers recovery.

Signed-off-by: Alexey Marchuk <alexeymar@nvidia.com>
Change-Id: I7f96effab7c1892839ee85c257ed2f11ffb478a4
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23127
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Reviewed-by: Ben Walker <ben@nvidia.com>

show more ...

test/dma: Fix ibv_reg_mr usage

Translate_cb might be called for part of
the payload, but we must register MR for the
entrire buffer

Signed-off-by: Alexey Marchuk <alexeymar@nvidia.com>
Change-Id: I

test/dma: Fix ibv_reg_mr usage

Translate_cb might be called for part of
the payload, but we must register MR for the
entrire buffer

Signed-off-by: Alexey Marchuk <alexeymar@nvidia.com>
Change-Id: I2876ad6aa19df380acf37ecab87d8ce1b8352546
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23100
Reviewed-by: Ben Walker <ben@nvidia.com>
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

apps: set NULL as RPC address in some apps

Set opts.rpc_addr to NULL in SPDK applications that usually do not
benefit from starting RPC interface. These include programs in app,
examples and test di

apps: set NULL as RPC address in some apps

Set opts.rpc_addr to NULL in SPDK applications that usually do not
benefit from starting RPC interface. These include programs in app,
examples and test directories.

Some of the tests now require that we explicitly set RPC address,
because some apps no longer use the default SPDK address.

Change-Id: I417a3a756581f44c711257e6d2408d933b40386a
Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22634
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Community-CI: Mellanox Build Bot

show more ...

test/dma: call spdk_thread_exit()

We should always exit threads that we create.

Signed-off-by: Jim Harris <james.r.harris@intel.com>
Change-Id: Ife982798bc23a5da5706b9baca80e102a4541bab
Reviewed-on

test/dma: call spdk_thread_exit()

We should always exit threads that we create.

Signed-off-by: Jim Harris <james.r.harris@intel.com>
Change-Id: Ife982798bc23a5da5706b9baca80e102a4541bab
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15501
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>

show more ...