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 ...

fio/nvme, fio/bdev: use specific NUMA ID when possible

Currently if any namespace or bdev reports NUMA_ID_ANY, we allocate
the buffer using NUMA_ID_ANY. This basically keeps original behavior,
unles

fio/nvme, fio/bdev: use specific NUMA ID when possible

Currently if any namespace or bdev reports NUMA_ID_ANY, we allocate
the buffer using NUMA_ID_ANY. This basically keeps original behavior,
unless all namespaces and bdevs reports a NUMA ID.

But it is OK to just use a specific NUMA ID as long as none of the
other namespaces or bdevs in that job have a conflicting NUMA ID. If
the other namespaces or bdevs report NUMA_ID_ANY we can just ignore
them.

Suggested-by: Konrad Sztyber <konrad.sztyber@intel.com>
Signed-off-by: Jim Harris <jim.harris@samsung.com>
Change-Id: I18743a5e26d1098804b7783c592b89bfe9286423
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24815
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Ben Walker <ben@nvidia.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>

show more ...

fio/bdev: use numa_id when allocating io buffers

fio does the io buffer allocations on a per-thread basis, so if
a thread/job is associated with multiple bdevs, and those
bdevs don't share the same

fio/bdev: use numa_id when allocating io buffers

fio does the io buffer allocations on a per-thread basis, so if
a thread/job is associated with multiple bdevs, and those
bdevs don't share the same numa_id, we will just revert to
SPDK_ENV_NUMA_ID_ANY.

Also, if any of the bdevs report SPDK_ENV_NUMA_ID_ANY, revert
to SPDK_ENV_NUMA_ID_ANY for that case too.

Signed-off-by: Jim Harris <jim.harris@samsung.com>
Change-Id: I92caf0dadc15af3196ae9e0f5ae410ad4cee28f3
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24155
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Ben Walker <ben@nvidia.com>
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

app/fio/bdev: remove use of spdk_subsystem_init_from_json_config

This function is deprecated, use spdk_subsystem_load_config
instead.

Signed-off-by: Jim Harris <jim.harris@samsung.com>
Change-Id: I

app/fio/bdev: remove use of spdk_subsystem_init_from_json_config

This function is deprecated, use spdk_subsystem_load_config
instead.

Signed-off-by: Jim Harris <jim.harris@samsung.com>
Change-Id: I0eb189550706b6f76df5263b3ad9a62f5fe11be9
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24663
Community-CI: Mellanox Build Bot
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

env: add opts_size to spdk_env_opts

This will help with ABI compatibility as new fields get added
to this structure.

User is responsible for setting opts_size. Major version has been
bumped, so use

env: add opts_size to spdk_env_opts

This will help with ABI compatibility as new fields get added
to this structure.

User is responsible for setting opts_size. Major version has been
bumped, so users will need to rebuild against latest code, if they
do not set opts_size we will just treat that as if the structure
ends right before opts_size. We will also print an ERRLOG to warn
user to set opts_size appropriately.

As part of this patch, update all in-tree callers of
spdk_env_opts_init() to set the opts_size.

Signed-off-by: Jim Harris <jim.harris@samsung.com>
Change-Id: If2611eccc8b3c43e1dc1654028d000a8a12a544e
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23940
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz@tzawadzki.com>

show more ...

test: ensure unique address for FIO plugin

FIO plugin (fio/spdk_bdev) always starts RPC server on
SPDK_DEFAULT_RPC_ADDR address. This is unnecessary, because in most
cases there will be no need for

test: ensure unique address for FIO plugin

FIO plugin (fio/spdk_bdev) always starts RPC server on
SPDK_DEFAULT_RPC_ADDR address. This is unnecessary, because in most
cases there will be no need for RPC interface to be active for
communication. We can change plugin's implementation similarly to what
we have in bdevperf: RPC server is going to be started only when
'--spdk_rpc_listen_addr' is specified.


Change-Id: I99227626c71ae5615cbdd5090fcdd4c42f193eaa
Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22682
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Reviewed-by: Boris Glimcher <Boris.Glimcher@emc.com>

show more ...

fio: Promote bdev fio plugin to app directory

We are going to start installing this in the near future.

Change-Id: I98ddf5fadf7505dfae2689ef4df54e7e0bf10f8c
Signed-off-by: Ben Walker <ben@nvidia.co

fio: Promote bdev fio plugin to app directory

We are going to start installing this in the near future.

Change-Id: I98ddf5fadf7505dfae2689ef4df54e7e0bf10f8c
Signed-off-by: Ben Walker <ben@nvidia.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21389
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot

show more ...