accel: Fix a bug that append_dif_generate_copy() did not set dif_ctx

Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Change-Id: Iabc69238d6ec5045c190fa5e942950d6d1027dd2
Reviewed-on: https:/

accel: Fix a bug that append_dif_generate_copy() did not set dif_ctx

Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Change-Id: Iabc69238d6ec5045c190fa5e942950d6d1027dd2
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25157
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Jim Harris <jim.harris@nvidia.com>
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Reviewed-by: Alliswell <hisunzhenliang@outlook.com>
Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com>

show more ...

accel: fix a unused variable warning

Change-Id: I01c796ae675cc795ad1636555cf8d09defc8a992
Signed-off-by: Changpeng Liu <changpeliu@tencent.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk

accel: fix a unused variable warning

Change-Id: I01c796ae675cc795ad1636555cf8d09defc8a992
Signed-off-by: Changpeng Liu <changpeliu@tencent.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24957
Reviewed-by: Yankun Li <845245370@qq.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Alliswell <hisunzhenliang@outlook.com>

show more ...

lib/accel: Allow to complete any task in the sequence

Current implementation assumes that only first task
can be completed but platform driver can optimize
the sequence and merge a task in the middl

lib/accel: Allow to complete any task in the sequence

Current implementation assumes that only first task
can be completed but platform driver can optimize
the sequence and merge a task in the middle

Signed-off-by: Alexey Marchuk <alexeymar@nvidia.com>
Change-Id: Iccf3f4b3ff92e6bd88a393d36332c5c926a7354a
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24691
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Ben Walker <ben@nvidia.com>
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

lib/accel: add spdk_accel_append_dix_generate/verify

Add new APIs spdk_accel_append_dix_generate/verify to accel library.

Change-Id: Ic8f66a486383428eae8f4a846a95cac45e09b2e1
Signed-off-by: Krzyszt

lib/accel: add spdk_accel_append_dix_generate/verify

Add new APIs spdk_accel_append_dix_generate/verify to accel library.

Change-Id: Ic8f66a486383428eae8f4a846a95cac45e09b2e1
Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24213
Community-CI: Mellanox Build Bot
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Reviewed-by: Slawomir Ptak <slawomir.ptak@intel.com>

show more ...

lib/accel: support lz4 in accel sw

The lz4 compression package performs very well and is
a good choice for compression that requires performance

Change-Id: Iefb5e04722c9be9d1a09ba3b4da364795a3a44e6

lib/accel: support lz4 in accel sw

The lz4 compression package performs very well and is
a good choice for compression that requires performance

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

show more ...

lib/accel: add the compression level support

Change-Id: I1c5d8953cee71865a764dde65a109e0715d54697
Signed-off-by: Yankun Li <845245370@qq.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+

lib/accel: add the compression level support

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

show more ...

lib/accel: Check if compress and decompress are same module

Change-Id: I0fc8c3dc8edf3da9352cb49341b9db3a43653521
Signed-off-by: Yankun Li <845245370@qq.com>
Reviewed-on: https://review.spdk.io/gerri

lib/accel: Check if compress and decompress are same module

Change-Id: I0fc8c3dc8edf3da9352cb49341b9db3a43653521
Signed-off-by: Yankun Li <845245370@qq.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24874
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Reviewed-by: GangCao <gang.cao@intel.com>

show more ...

lib/accel: support multiple [de]compression algorithms

Change-Id: I4b070ccdcdddb684a72a354245d4d14ed63e6bbf
Signed-off-by: Yankun Li <845245370@qq.com>
Reviewed-on: https://review.spdk.io/gerrit/c/s

lib/accel: support multiple [de]compression algorithms

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

show more ...

accel: dump driver config in write_config_json()

Reported-by: Jim Harris <jim.harris@samsung.com>
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
Change-Id: Iee7aa9fc6313f5bd594c8a1eb98540d

accel: dump driver config in write_config_json()

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

show more ...

accel: allow clearing previously assigned driver

It could be useful for some users to be able to clear the driver after
it was previously set (e.g. use a config w/ a driver set and then clear
it via

accel: allow clearing previously assigned driver

It could be useful for some users to be able to clear the driver after
it was previously set (e.g. use a config w/ a driver set and then clear
it via an RPC).

Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I172c468956243a6fa74f2e66a4124b50d50e5b13
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24469
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Michal Berger <michal.berger@intel.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Reviewed-by: Maciej Szulik <maciej.szulik@intel.com>
Community-CI: Mellanox Build Bot

show more ...

lib/accel: add DIX verify

Add DIX Verify operation to the library API and to accel_sw module.

Change-Id: I9eb0123a49e3b7b9218f94ed6dba26c92d42525f
Signed-off-by: Krzysztof Karas <krzysztof.karas@in

lib/accel: add DIX verify

Add DIX Verify operation to the library API and to accel_sw module.

Change-Id: I9eb0123a49e3b7b9218f94ed6dba26c92d42525f
Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24002
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot

show more ...

lib/accel: add DIX generate

Add DIX Generate operation to the library API and to accel_sw module.

Change-Id: Ic63a7e6e5c658e3ce1331faad51bbbb173471e2e
Signed-off-by: Krzysztof Karas <krzysztof.kara

lib/accel: add DIX generate

Add DIX Generate operation to the library API and to accel_sw module.

Change-Id: Ic63a7e6e5c658e3ce1331faad51bbbb173471e2e
Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23980
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot

show more ...

accel: Add append version for T10 DIF APIs

Add append versions of dif_verify, dif_verify_copy, dif_generate, and
dif_generate_copy to public APIs.

Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia

accel: Add append version for T10 DIF APIs

Add append versions of dif_verify, dif_verify_copy, dif_generate, and
dif_generate_copy to public APIs.

Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Change-Id: I98a117577e2b9445390299b6ad1e6e8423efbf18
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23739
Reviewed-by: Ben Walker <ben@nvidia.com>
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>

show more ...

isa-l_crypto: update submodule to 2.25

This patch updates ISA-L submodule to version 2.25.
Rather than disabling deprecation warnings, immediately
update to new APIs introduced in ISA-L 2.25.

ISA-L

isa-l_crypto: update submodule to 2.25

This patch updates ISA-L submodule to version 2.25.
Rather than disabling deprecation warnings, immediately
update to new APIs introduced in ISA-L 2.25.

ISA-L_crypto is replacing following APIs with one
that adds parameter checking and FIPS support:
XTS_AES_128_enc -> isal_aes_xts_enc_128
XTS_AES_128_dec -> isal_aes_xts_dec_128
XTS_AES_256_enc -> isal_aes_xts_enc_256
XTS_AES_256_dec -> isal_aes_xts_dec_256

Please see more details at:
https://github.com/intel/isa-l_crypto/wiki/New-API-introduced-from-v2.25

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

show more ...

accel/rpc: Remove newline at the end of error messages

Change-Id: Ie1bc37dbc4a6b42a38b725535a69253e3fb28249
Signed-off-by: Pierre Lestringant <plestringant@kalray.eu>
Reviewed-on: https://review.spd

accel/rpc: Remove newline at the end of error messages

Change-Id: Ie1bc37dbc4a6b42a38b725535a69253e3fb28249
Signed-off-by: Pierre Lestringant <plestringant@kalray.eu>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24242
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz@tzawadzki.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.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 ...

dif: Update spdk_dif_error structure to bump SO_VER forcefully

We will match enum spdk_dif_pi_format with the NVMe specification.

enum spdk_dif_pi_format was added a year ago. we need users to reco

dif: Update spdk_dif_error structure to bump SO_VER forcefully

We will match enum spdk_dif_pi_format with the NVMe specification.

enum spdk_dif_pi_format was added a year ago. we need users to recompile
their application if they use enum spdk_dif_pi_format.

However, enum spdk_dif_pi_format was not used in any data structure and
ABIDIFF tool could not detect the change.

The change breaks ABI. Hence, we should bump SO_VER of lib/util in any
other way to notify the change to users.

The cleanest way to do that without hacking check_so_deps.sh is to add
a reserved field to spdk_dif_error structure.

Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Change-Id: I7e12094bbe33f6c75780f09eb58d673d75752781
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24182
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Community-CI: Mellanox Build Bot
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 ...

accel: introduce tasks in sequence limit

Sequence cannot be allocated if number of available task objects cannot
satisfy required limit. This is to prevent potential dead lock when few
requests are

accel: introduce tasks in sequence limit

Sequence cannot be allocated if number of available task objects cannot
satisfy required limit. This is to prevent potential dead lock when few
requests are pending task resource and none can advance the processing.
This solution should work only if there is single async operation after
sequence obj obtained, so assume that is possible to happen with io
buffer allocation now. If there are more async operations then solution
should be improved.

Change-Id: I45165dd96539b00849dc67d5bd1bfea206ca5ed2
Signed-off-by: Jacek Kalwas <jacek.kalwas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24040
Community-CI: Mellanox Build Bot
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: adjust task per ch define name

ACCEL_ prefix was missing and MAX was misleading as it is more default
than max.

Change-Id: I2ee729fd2ca85eb3bb05c6529a93b66482b228e7
Signed-off-by: Jacek Kalw

accel: adjust task per ch define name

ACCEL_ prefix was missing and MAX was misleading as it is more default
than max.

Change-Id: I2ee729fd2ca85eb3bb05c6529a93b66482b228e7
Signed-off-by: Jacek Kalwas <jacek.kalwas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24164
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 ...

accel: add sequence outstanding stat

That is just to improve debug capabilities now.

Change-Id: I9755d0d370bb6f63fcddc5ae031312d7c6212353
Signed-off-by: Jacek Kalwas <jacek.kalwas@intel.com>
Review

accel: add sequence outstanding stat

That is just to improve debug capabilities now.

Change-Id: I9755d0d370bb6f63fcddc5ae031312d7c6212353
Signed-off-by: Jacek Kalwas <jacek.kalwas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24039
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Community-CI: Mellanox Build Bot

show more ...

accel: add utility to put task

There is similar utility (_get_task) already. It is a prework for
further changes for tracking stat under this function.

Change-Id: I8cee55a7937f6bd77e3b416a3e1830e77

accel: add utility to put task

There is similar utility (_get_task) already. It is a prework for
further changes for tracking stat under this function.

Change-Id: I8cee55a7937f6bd77e3b416a3e1830e779ebf6e8
Signed-off-by: Jacek Kalwas <jacek.kalwas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24038
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>

show more ...

accel: move get task utility

It is a prework for further change where put is being introduced so we
can have both close to each other.

Change-Id: Id070542980389b04a6e424ab4a5456857817be42
Signed-of

accel: move get task utility

It is a prework for further change where put is being introduced so we
can have both close to each other.

Change-Id: Id070542980389b04a6e424ab4a5456857817be42
Signed-off-by: Jacek Kalwas <jacek.kalwas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24048
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Community-CI: Mellanox Build Bot

show more ...

accel: improve accel sequence obj release

It might be beneficial to put sequence obj before executing callback
function as this gives oportunity to reuse the same sequence obj by the
user immediatel

accel: improve accel sequence obj release

It might be beneficial to put sequence obj before executing callback
function as this gives oportunity to reuse the same sequence obj by the
user immediately.

Change-Id: I33101aa9b47a46ff01f6af2bdbc25bc4c1a53ddc
Signed-off-by: Jacek Kalwas <jacek.kalwas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24037
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 ...

accel: add fn to get accel driver name

The main purpose of this function is to allow for validation when some
particular driver is expected to be set.

Change-Id: I4e84efbb1fbcd5096e28fecd7f36fadb8e

accel: add fn to get accel driver name

The main purpose of this function is to allow for validation when some
particular driver is expected to be set.

Change-Id: I4e84efbb1fbcd5096e28fecd7f36fadb8ea5f501
Signed-off-by: Jacek Kalwas <jacek.kalwas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23937
Community-CI: Mellanox Build Bot
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Reviewed-by: Kamuda Szymon <szymon.kamuda@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

lib/accel: API to get memory domains per opcode

There is an accel module API to return a memory domain
but no public wrapper for it. Next patch uses
the new API in vbdev_crypto

Signed-off-by: Alexe

lib/accel: API to get memory domains per opcode

There is an accel module API to return a memory domain
but no public wrapper for it. Next patch uses
the new API in vbdev_crypto

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

show more ...