test: Stop grep'ing through config.h, *.mk, etc. for build config

All the parameters are already available in the environment through
build_config.sh.

Change-Id: Icd6f536c1a31d6bb39f289f84cb0ac7cd8

test: Stop grep'ing through config.h, *.mk, etc. for build config

All the parameters are already available in the environment through
build_config.sh.

Change-Id: Icd6f536c1a31d6bb39f289f84cb0ac7cd87ca57d
Signed-off-by: Michal Berger <michal.berger@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24770
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Jim Harris <jim.harris@samsung.com>

show more ...

app/spdk_tgt: link with fsdev libs

Change-Id: Ieee0aa83e0ab722e4f1f188cdb26fbc74c731a35
Signed-off-by: Anton Nayshtut <anayshtut@nvidia.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/

app/spdk_tgt: link with fsdev libs

Change-Id: Ieee0aa83e0ab722e4f1f188cdb26fbc74c731a35
Signed-off-by: Anton Nayshtut <anayshtut@nvidia.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24083
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Ben Walker <ben@nvidia.com>

show more ...

general: fix misspells and typos

Signed-off-by: Kamil Godzwon <kamilx.godzwon@intel.com>
Change-Id: Iab206ef526eb7032c6681a3145450010c91705a4
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+

general: fix misspells and typos

Signed-off-by: Kamil Godzwon <kamilx.godzwon@intel.com>
Change-Id: Iab206ef526eb7032c6681a3145450010c91705a4
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24120
Community-CI: Mellanox Build Bot
Reviewed-by: Karol Latecki <karol.latecki@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>

show more ...

test/json_config/json_config.sh: order-agnostic comparison

It seems that notify_get_types RPC does not guarantee the order. This is
probably due to the dynamic loader's logic the SOs are, in turn, l

test/json_config/json_config.sh: order-agnostic comparison

It seems that notify_get_types RPC does not guarantee the order. This is
probably due to the dynamic loader's logic the SOs are, in turn, loaded
in a nondeterministic order.

This patch replaces the array comparison in the tgt_check_notification_types()
by an order-agnostic one.

Change-Id: I10d6008bfd188f6843495b62305396304007a215
Signed-off-by: Anton Nayshtut <anayshtut@nvidia.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24104
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 ...

python/rpc: remove deprecated retry_count param

Remove deprecated retry_count parameter from
bdev_nvme_set_options rpc function

Change-Id: I1b415cfcfd7b3226d6331960ad42c500ba77f0aa
Signed-off-by: E

python/rpc: remove deprecated retry_count param

Remove deprecated retry_count parameter from
bdev_nvme_set_options rpc function

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

show more ...

test/json_config: skip undefined clear_*_subsystem methods

Many subsystems don't need to clear their configurations in the
json_config test, so simply skip those that don't define a
clear_*_subsyste

test/json_config: skip undefined clear_*_subsystem methods

Many subsystems don't need to clear their configurations in the
json_config test, so simply skip those that don't define a
clear_*_subsystem method.

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

show more ...

event/app: enable JSON config with delay

Delete arbitrary limitation disallowing users from starting SPDK
applications with JSON provided config file and subsystem
initialization delay activated (--

event/app: enable JSON config with delay

Delete arbitrary limitation disallowing users from starting SPDK
applications with JSON provided config file and subsystem
initialization delay activated (--wait-for-rpc).

Add a new script with functional test of the delay + JSON
initialization.

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

show more ...

test/json_config: commonize functions

Commonize some of the functions to avoid code duplication.

Change-Id: I4ec1af2b803f9776e78f6f697dcba9739795d79b
Signed-off-by: Krzysztof Karas <krzysztof.karas

test/json_config: commonize functions

Commonize some of the functions to avoid code duplication.

Change-Id: I4ec1af2b803f9776e78f6f697dcba9739795d79b
Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21259
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>

show more ...

module/raid: configurable process window size

Make the process window size configurable. In order to make it possible,
add new RPC bdev_raid_set_options.

Change-Id: I68f0ecda1b2057c58dfdce7369afa52

module/raid: configurable process window size

Make the process window size configurable. In order to make it possible,
add new RPC bdev_raid_set_options.

Change-Id: I68f0ecda1b2057c58dfdce7369afa520e659e1aa
Signed-off-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21477
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Tested-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>

show more ...

iobuf: always dump config

Previously, the config was only dumped, if the iobuf_set_options RPC has
been received. It used to work that way, because when iobuf was
initially introduced, these option

iobuf: always dump config

Previously, the config was only dumped, if the iobuf_set_options RPC has
been received. It used to work that way, because when iobuf was
initially introduced, these options could also be configured via
spdk_bdev_set_opts, for backwards compatibility. However, this behavior
was removed in v23.05, so the config can be dumped indiscriminately now.

Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
Change-Id: Id2fb4a47ff6da2eb770b1339e9afe6fc18c4f64d
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19681
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Jim Harris <jim.harris@gmail.com>
Reviewed-by: Krzysztof Karas <krzysztof.karas@intel.com>

show more ...

bdev_aio: Add FreeBSD support

The FreeBSD kernel aio is built around kevent() system call.
Use it for polling, like it is done using io_getevents() on Linux side.

Change-Id: I558236081b8d6870f9a4a3

bdev_aio: Add FreeBSD support

The FreeBSD kernel aio is built around kevent() system call.
Use it for polling, like it is done using io_getevents() on Linux side.

Change-Id: I558236081b8d6870f9a4a3a937df60710045bce9
Signed-off-by: Fedor Uporov <fuporov.vstack@gmail.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18498
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>

show more ...

test/json_config: Sort notifications

On some systems it's been noticed that the lvol notifications were
being recorded out of the expected order. Null and void similar
discrepancy by sorting all the

test/json_config: Sort notifications

On some systems it's been noticed that the lvol notifications were
being recorded out of the expected order. Null and void similar
discrepancy by sorting all the events.

Change-Id: I31a8695d07d20b4a10d847c118107f5e4d65aabe
Signed-off-by: Michal Berger <michal.berger@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18526
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>

show more ...

accel: add accel_set_options

It'll allow for setting accel-specific options. For now, it makes the
size of iobuf caches configurable.

Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
Chang

accel: add accel_set_options

It'll allow for setting accel-specific options. For now, it makes the
size of iobuf caches configurable.

Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
Change-Id: Iaf505cc5e98dc6411453d9964250a4ba22267d79
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17188
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

test/json_config: detach virtio controllers before shutdown

If the controllers are left attached when the application is terminated,
the virtio library might receive notifications about memory being

test/json_config: detach virtio controllers before shutdown

If the controllers are left attached when the application is terminated,
the virtio library might receive notifications about memory being
unregistered from the mem_map. The apps in json_config test are
preallocating 1GB of memory, which should stop DPDK from releasing the
hugepages back to the system, but if they allocate more, that extra
pages can be released, which would generate the mem_map notifications .

This actually happened in #2951 - spdk_tgt was built with ocf support,
which preallocates an extra ~300MB mempools, exceeding 1GB, which caused
mem_map notifications to be sent when iscsi mempools were destroyed
triggering an assertion in virtio.

Another option to fix this would be to increase the size of the
preallocated memory, but detaching the controllers achieves the same
thing without requiring more resources.

Fixes #2951.

Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I743af39acfee68caba55ec4fccd593a696ab688b
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17265
Reviewed-by: Karol Latecki <karol.latecki@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>

show more ...

bdev/pmem: Removing bdev pmem - updated tests

Removed tests associated with bdev pmem

Change-Id: Ibfb699eb5626242a7c1930be13eb1f5ded8d6392
Signed-off-by: Marcin Spiewak <marcin.spiewak@intel.com>
R

bdev/pmem: Removing bdev pmem - updated tests

Removed tests associated with bdev pmem

Change-Id: Ibfb699eb5626242a7c1930be13eb1f5ded8d6392
Signed-off-by: Marcin Spiewak <marcin.spiewak@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17003
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>

show more ...

test/ublk: add basic tests for verifying ublk

Signed-off-by: Yifan Bian <yifan.bian@intel.com>
Co-authored-by: Xiaodong Liu <xiaodong.liu@intel.com>
Change-Id: I3416b904e61b673ab622e5ff70099c97d0f77

test/ublk: add basic tests for verifying ublk

Signed-off-by: Yifan Bian <yifan.bian@intel.com>
Co-authored-by: Xiaodong Liu <xiaodong.liu@intel.com>
Change-Id: I3416b904e61b673ab622e5ff70099c97d0f7765c
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16081
Reviewed-by: Michal Berger <michal.berger@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

virtio_blk: add dump opts

Currently we do not have a way to dump opts
for virtio_blk transports. This patch introduces
necessary changes to let us save and load those
via JOSN config.

Change-Id: I7

virtio_blk: add dump opts

Currently we do not have a way to dump opts
for virtio_blk transports. This patch introduces
necessary changes to let us save and load those
via JOSN config.

Change-Id: I7ee4f31062f3d4a264f322e66a67ba3d075f1d75
Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15248
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>

show more ...

bdev/crypto: Use accel framework

All DPDK related code is removed, handling of
RESET command was sligthly updated.
Handling of -ENOMEM was updated for cases when
accel API returns -ENOMEM

Crypto te

bdev/crypto: Use accel framework

All DPDK related code is removed, handling of
RESET command was sligthly updated.
Handling of -ENOMEM was updated for cases when
accel API returns -ENOMEM

Crypto tests in blockdev.sh were extended with more
crypto_bdevs to verify NOMEM cases - that failed
with original vbdev_crypto implementation

Signed-off-by: Alexey Marchuk <alexeymar@nvidia.com>
Change-Id: If1feba2449bee852c6c4daca4b3406414db6fded
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/14860
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Paul Luse <paul.e.luse@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>

show more ...