test/unit/idxd: Initialize flags before use

Initialize flages before use, Clear a compile-time warning

Change-Id: I78c3a5054067503700c2905af9144f50f9442e3d
Signed-off-by: Yankun Li <845245370@qq.co

test/unit/idxd: Initialize flags before use

Initialize flages before use, Clear a compile-time warning

Change-Id: I78c3a5054067503700c2905af9144f50f9442e3d
Signed-off-by: Yankun Li <845245370@qq.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24716
Reviewed-by: GangCao <gang.cao@intel.com>
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>

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

idxd: Fixed minor issues in comments

Formatted function descriptions, added missing dots
at the end of sentences, removed some unnecessary comments.

Change-Id: Idb1073a9f403b41be3e4bc817acd968e49bd

idxd: Fixed minor issues in comments

Formatted function descriptions, added missing dots
at the end of sentences, removed some unnecessary comments.

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

show more ...

lib/idxd: DIF strip DSA implementation

Extend the IDXD API with DIF Strip operation support.

The DIF Strip operation copies memory from the source address to
the destination address, removing the D

lib/idxd: DIF strip DSA implementation

Extend the IDXD API with DIF Strip operation support.

The DIF Strip operation copies memory from the source address to
the destination address, removing the Data Integrity Field (DIF).
It optionally computes the DIF on the source data and compares the
computed DIF to the DIF contained in the source data.
DIF computation is performed on each block of source data.

If the operation is completed successfully, the final Reference Tag
and Application Tag is written to the completion record along with
a Success completion status. If an error is detected in the DIF in
the source data, the operation stops.

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

show more ...

test/unit: DIF insert DSA unit tests

Extend the test module with DIF Insert operation unit tests.

Change-Id: Ied46a90903f047f323b3e269e59a46ac4a8a6e21
Signed-off-by: Krzysztof Sprzaczkowski <krzysz

test/unit: DIF insert DSA unit tests

Extend the test module with DIF Insert operation unit tests.

Change-Id: Ied46a90903f047f323b3e269e59a46ac4a8a6e21
Signed-off-by: Krzysztof Sprzaczkowski <krzysztof.sprzaczkowski@intel.com>
Signed-off-by: Slawomir Ptak <slawomir.ptak@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/20279
Reviewed-by: Sebastian Brzezinka <sebastian.brzezinka@intel.com>
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

test/unit: DIF check DSA unit tests

Extend the test module with DIF Check operation unit tests.

Change-Id: I4005d978511339ff5db5292731567925b18dccf8
Signed-off-by: Krzysztof Sprzaczkowski <krzyszto

test/unit: DIF check DSA unit tests

Extend the test module with DIF Check operation unit tests.

Change-Id: I4005d978511339ff5db5292731567925b18dccf8
Signed-off-by: Krzysztof Sprzaczkowski <krzysztof.sprzaczkowski@intel.com>
Signed-off-by: Slawomir Ptak <slawomir.ptak@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/20278
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Reviewed-by: Sebastian Brzezinka <sebastian.brzezinka@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>

show more ...

lib/idxd: match batch size to max reported from device

Rather than limiting the batch size to DESC_PER_BATCH,
use the maximum batch size value presented from the device.
To accommodate increased the

lib/idxd: match batch size to max reported from device

Rather than limiting the batch size to DESC_PER_BATCH,
use the maximum batch size value presented from the device.
To accommodate increased the allocated descriptors for a batch.

Prior to this patch full batch was flushed on task submission,
meanwhile any built up batch was submitted in spdk_idxd_process_events().

Rate of flushing is now defined by IDXD_MIN_BATCH_FLUSH,
while still allowing for larger batches to be created within single
task submission.
If IDXD_MIN_BATCH_FLUSH is modified to very high value, batches
would still be submitted in spdk_idxd_process_events().

For now the IDXD_MIN_BATCH_FLUSH matches prior DESC_PER_BATCH limit,
making this parameter configurable might be explored.

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

show more ...

test/unit: use spdk_ut_run_tests()

Replaced direct calls to the CUnit's functions to run the tests with
spdk_ut_run_tests(). That way, each test will have the ability to run
a specific test case.

test/unit: use spdk_ut_run_tests()

Replaced direct calls to the CUnit's functions to run the tests with
spdk_ut_run_tests(). That way, each test will have the ability to run
a specific test case.

The blob.c unit test wasn't changed, because it runs all tests multiple
times with different parameter combinations, so it cannot be easily
converted. In the future, each such combination could be split into a
separate test suite, which would make it compatible with
spdk_ut_run_tests().

Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I4463f808f89844e9bf32b5b31eda197c5d729d1d
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19288
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

test/unit: move spdk_cunit.h to include/spdk_internal

It'll make it easier to include this file outside of unit tests.

Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I171ddb864

test/unit: move spdk_cunit.h to include/spdk_internal

It'll make it easier to include this file outside of unit tests.

Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I171ddb8649f67b5786f08647560e2907603d0574
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19284
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>

show more ...

update Intel copyright notices

per Intel policy to include file commit date using git cmd
below. The policy does not apply to non-Intel (C) notices.

git log --follow -C90% --format=%ad --date defa

update Intel copyright notices

per Intel policy to include file commit date using git cmd
below. The policy does not apply to non-Intel (C) notices.

git log --follow -C90% --format=%ad --date default <file> | tail -1

and then pull just the 4 digit year from the result.

Intel copyrights were not added to files where Intel either had
no contribution ot the contribution lacked substance (ie license
header updates, formatting changes, etc). Contribution date used
"--follow -C95%" to get the most accurate date.

Note that several files in this patch didn't end the license/(c)
block with a blank comment line so these were added as the vast
majority of files do have this last blank line. Simply there for
consistency.

Signed-off-by: paul luse <paul.e.luse@intel.com>
Change-Id: Id5b7ce4f658fe87132f14139ead58d6e285c04d4
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15192
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>
Community-CI: Mellanox Build Bot

show more ...