ff173863 | 20-Nov-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
ut/bdev: Remove duplication with many stups among unit test files
Move all duplicated stubs in bdev.c, mt/bdev.c, and part.c unit test files into the new file common_stubs.h in test/common/lib/bdev.
ut/bdev: Remove duplication with many stups among unit test files
Move all duplicated stubs in bdev.c, mt/bdev.c, and part.c unit test files into the new file common_stubs.h in test/common/lib/bdev.
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: Ic3d75821bf828e196fa576a18feae90d8bd2ffeb Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25455 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Reviewed-by: Jim Harris <jim.harris@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com> Community-CI: Mellanox Build Bot
show more ...
|
0836dccd | 30-Sep-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
bdev: Add spdk_dif_ctx and spdk_dif_error into spdk_bdev_io
If the generic bdev layer and the underlying bdev module use T10 DIF APIs of the accel framework for T10 DIF, DIF context and DIF error st
bdev: Add spdk_dif_ctx and spdk_dif_error into spdk_bdev_io
If the generic bdev layer and the underlying bdev module use T10 DIF APIs of the accel framework for T10 DIF, DIF context and DIF error structures must be persistent while executing the APIs.
As a preparation, embed spdk_dif_ctx structure and spdk_dif_error structure into spdk_bdev_io structure, and initialize both at the start of the I/O submission if the DIF type of the bdev is not disabled or no_metadata option is enabled.
These embedded data structure will be able to use for other purposes too.
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: I4e61222034bb33d4dd862692a878bd283b5d32d4 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23741 Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Reviewed-by: Jim Harris <jim.harris@nvidia.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Community-CI: Mellanox Build Bot
show more ...
|
fb1630bf | 21-Aug-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
bdev: Use data_block_size for upper layer buffer if hide_metadata is true
Add two helper functions, bdev_desc_get_block_size() and bdev_io_get_block_size().
Then, use these functions for buffers of
bdev: Use data_block_size for upper layer buffer if hide_metadata is true
Add two helper functions, bdev_desc_get_block_size() and bdev_io_get_block_size().
Then, use these functions for buffers of upper layer.
The following patches will do DIF insert/strip for read/write if no_metadata is true.
bdev_io_update_io_stat() keeps using bdev->blocklen because I/O to the underlying bdev module uses bdev->blocklen regardless of the no_metadata option.
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: I46cedb846a4362ba75742d4e543df466ef43f112 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24629 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <jim.harris@nvidia.com> Community-CI: Mellanox Build Bot Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
|
16e5e505 | 20-Nov-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
bdev: Add spdk_bdev_open_ext_v2() to support per-open options
We will add the DIF insert/strip feature into the generic bdev layer. We want to make the feature option per bdev open.
There exists sp
bdev: Add spdk_bdev_open_ext_v2() to support per-open options
We will add the DIF insert/strip feature into the generic bdev layer. We want to make the feature option per bdev open.
There exists spdk_bdev_open_async_opts. However it is only for spdk_bdev_open_async(). spdk_bdev_open_async() is not generally usable.
spdk_bdev_open_ext() does not receive any option structure via parameter.
It is not practical to change the existing spdk_bdev_open_ext().
Hence, add a new API spdk_bdev_open_ext_v2() with spdk_bdev_open_opts structure. We find many examples to use v2 in DPDK.
Add hide_metadata option as the first option of the spdk_bdev_open_opts structure.
Last 7 bytes in spdk_bdev_open_opts structure are unused and are not initialized by the caller. To zero out these clearly with minimal user effort, add spdk_bdev_open_opts_init() for initialization.
opts in spdk_bdev_desc is a hot data. Put it into the first cache line in spdk_bdev_desc.
Furthermore, add simple unit test for verification.
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: I38d93ffbb2becc59e57f9a7163defd5f8f201f07 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23771 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <jim.harris@nvidia.com> Community-CI: Mellanox Build Bot Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com>
show more ...
|
557f022f | 23-Jul-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
bdev: Change 1st parameter of bdev_bytes_to_blocks from bdev to desc
The following patches will add option to spdk_bdev_desc and it will affect block size. As a preparation, change the first paramet
bdev: Change 1st parameter of bdev_bytes_to_blocks from bdev to desc
The following patches will add option to spdk_bdev_desc and it will affect block size. As a preparation, change the first parameter of bdev_bytes_to_blocks from bdev pointer to desc pointer.
Even before this patch, the caller used spdk_bdev_desc_get_bdev() as a function parameter. Hence, this is a good clean up.
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: I4b84351bce13b109c17b5477e2711ed632208178 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24628 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.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 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
02935561 | 12-Nov-2024 |
Ed Reed <edreed@microsoft.com> |
bdev_ut: add manual examine bdev unit test case
This commit adds a unit test case for the manual Bdev examination workflow and validates that a Bdev can be re-registered under the same name when the
bdev_ut: add manual examine bdev unit test case
This commit adds a unit test case for the manual Bdev examination workflow and validates that a Bdev can be re-registered under the same name when the autoexamine Bdev option is disabled.
Change-Id: Ideefe1c60fe7efb4191caca88c3551674781f987 Signed-off-by: Ed Reed <edreed@microsoft.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25425 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
show more ...
|
c26697bf | 11-Oct-2024 |
Slawomir Ptak <slawomir.ptak@intel.com> |
bdev_ut: Comparison operator and tests fixes
Operator fix and fixes for num blocks in the I/O split tests, to match the expected and requested values.
Change-Id: I67c7211c5f8cae409d078baaefbeb8851e
bdev_ut: Comparison operator and tests fixes
Operator fix and fixes for num blocks in the I/O split tests, to match the expected and requested values.
Change-Id: I67c7211c5f8cae409d078baaefbeb8851ed71f48 Signed-off-by: Slawomir Ptak <slawomir.ptak@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25199 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: Alliswell <hisunzhenliang@outlook.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
show more ...
|
63e0c25d | 12-Sep-2024 |
Vasilii Ivanov <iwanovvvasilij@gmail.com> |
bdev: add reset_mode to bdev_get_iostat
In current implementation of resetting stats explicit bdev_reset_iostat call is needed. It is not optimal for usecases when stat is obtained periodically and
bdev: add reset_mode to bdev_get_iostat
In current implementation of resetting stats explicit bdev_reset_iostat call is needed. It is not optimal for usecases when stat is obtained periodically and then resetted, because some stats may be lost in between of get and reset. Introduce new option for bdev_get_iostat that allows to reset stat right after obtaining it.
Fixes #3525
Change-Id: I77a5ea9310718c25e678336e2450a04fb582c050 Signed-off-by: Vasilii Ivanov <iwanovvvasilij@gmail.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24900 Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@samsung.com> Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> 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 ...
|
7fee8002 | 12-Oct-2023 |
Ben Walker <ben@nvidia.com> |
bdev: Add a flag indicating whether a bounce buffer is in use
The flag is in a hot cache line, rather than checking a pointer that isn't.
Change-Id: If3669e65ebf178e4be5008f63c6d949ac81ed6d1 Signed
bdev: Add a flag indicating whether a bounce buffer is in use
The flag is in a hot cache line, rather than checking a pointer that isn't.
Change-Id: If3669e65ebf178e4be5008f63c6d949ac81ed6d1 Signed-off-by: Ben Walker <ben@nvidia.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21950 Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
153e3344 | 13-Sep-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
ut/bdev: Add bdev_abort test cases for all I/O split conditions
A long-term bug of spdk_bdev_abort() with I/O split was fixed very recently. To avoid future degradation, add test cases for all I/O s
ut/bdev: Add bdev_abort test cases for all I/O split conditions
A long-term bug of spdk_bdev_abort() with I/O split was fixed very recently. To avoid future degradation, add test cases for all I/O split conditions.
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Suggested-by: Kaushik Chug <kaushik.chug@ngxstorage.com> Change-Id: I86741d740355cf680fb00dae60074fba43729390 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24912 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.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: Konrad Sztyber <konrad.sztyber@intel.com>
show more ...
|
fee788f6 | 11-Jul-2024 |
Jim Harris <jim.harris@samsung.com> |
bdev: add numa.id to spdk_bdev
Note, we also need to add a "numa.id_valid" bit, so that we can differentiate between bdevs that explicitly set a numa.id of 0, vs. those that just calloc() the memory
bdev: add numa.id to spdk_bdev
Note, we also need to add a "numa.id_valid" bit, so that we can differentiate between bdevs that explicitly set a numa.id of 0, vs. those that just calloc() the memory and therefore just set 0 implicitly.
Signed-off-by: Jim Harris <jim.harris@samsung.com> Change-Id: I23c6f1bd0fb976de395cc22f46e2fb121b96c244 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24152 Reviewed-by: Ben Walker <ben@nvidia.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
dc996557 | 29-Sep-2023 |
Ben Walker <ben@nvidia.com> |
bdev: Remove module_link from spdk_bdev_io
This is no longer used.
Change-Id: Ic74b4a91941b0e3fcac921489ce2c62783be2881 Signed-off-by: Ben Walker <ben@nvidia.com> Reviewed-on: https://review.spdk.i
bdev: Remove module_link from spdk_bdev_io
This is no longer used.
Change-Id: Ic74b4a91941b0e3fcac921489ce2c62783be2881 Signed-off-by: Ben Walker <ben@nvidia.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24364 Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
34edd9f1 | 10-Jul-2024 |
Kamil Godzwon <kamilx.godzwon@intel.com> |
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 ...
|
cd7064c7 | 10-Jun-2024 |
Nathan Claudel <nclaudel@kalrayinc.com> |
bdev_ut: store initialization order of the examine claim test modules
The test `examine_claimed` relies on the examine order of bdev modules.
However, bdev modules are registered using GCC construc
bdev_ut: store initialization order of the examine claim test modules
The test `examine_claimed` relies on the examine order of bdev modules.
However, bdev modules are registered using GCC constructors, which don't provide any guarantee on the order of execution. As a consequence, the test can result in a segfault if the order is not the one expected by the test.
To fix this issue, implement the init functions of the `examine_claimed` modules so that their relative order in the modules list is known by the test. With this change, no change to the actual test is required.
This can be simply tested by altering the order of `SPDK_BDEV_MODULE_REGISTER` occurences in the test file.
Change-Id: I129ec3cb48f3053ae7cace459d4de73dacf88d2d Signed-off-by: Nathan Claudel <nathan.claudel@kalrayinc.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23635 Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reviewed-by: Jim Harris <jim.harris@samsung.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
dbf0a9a5 | 15-May-2024 |
Ben Walker <ben@nvidia.com> |
Revert "bdev: Remove module_link from spdk_bdev_io"
This reverts commit d1fd0d13acccbc0f0e9bfd5449d7c2882c644d82.
Change-Id: I5f82863f3948e2e17113575aa103ddbdc5cdbc14 Signed-off-by: Ben Walker <ben
Revert "bdev: Remove module_link from spdk_bdev_io"
This reverts commit d1fd0d13acccbc0f0e9bfd5449d7c2882c644d82.
Change-Id: I5f82863f3948e2e17113575aa103ddbdc5cdbc14 Signed-off-by: Ben Walker <ben@nvidia.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23190 Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
|
79e2a56f | 25-Apr-2024 |
Konrad Sztyber <konrad.sztyber@intel.com> |
accel: remove flags
They were unused, deprecated, and scheduled for removal in v24.05.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I48e7e59df56cf2368f23e925360644665948585a
accel: remove flags
They were unused, deprecated, and scheduled for removal in v24.05.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I48e7e59df56cf2368f23e925360644665948585a Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22950 Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.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 ...
|
d1fd0d13 | 29-Sep-2023 |
Ben Walker <ben@nvidia.com> |
bdev: Remove module_link from spdk_bdev_io
This is no longer used.
Change-Id: I11adf75d18669ae950eced57d52a97bc5b453de4 Signed-off-by: Ben Walker <ben@nvidia.com> Reviewed-on: https://review.spdk.i
bdev: Remove module_link from spdk_bdev_io
This is no longer used.
Change-Id: I11adf75d18669ae950eced57d52a97bc5b453de4 Signed-off-by: Ben Walker <ben@nvidia.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21938 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
show more ...
|
bbd27a75 | 17-Jan-2024 |
Slawomir Ptak <slawomir.ptak@intel.com> |
bdev: Add a per-IO dif check flags exclude mask
Added a way to mask the dif check flags bits on a per-IO basis.
Change-Id: Ib513841a4531ea7e4cee483d4002002fe6f0b8ff Signed-off-by: Slawomir Ptak <sl
bdev: Add a per-IO dif check flags exclude mask
Added a way to mask the dif check flags bits on a per-IO basis.
Change-Id: Ib513841a4531ea7e4cee483d4002002fe6f0b8ff Signed-off-by: Slawomir Ptak <slawomir.ptak@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21476 Reviewed-by: Jim Harris <jim.harris@samsung.com> Community-CI: Mellanox Build Bot Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
effe7a01 | 21-Dec-2023 |
Artur Paszkiewicz <artur.paszkiewicz@intel.com> |
bdev: make quiesce work also for read I/O
The quiesce API is based on range lock functionality, which ignores read I/O by design because it was meant for compare and write. However, quiesce should w
bdev: make quiesce work also for read I/O
The quiesce API is based on range lock functionality, which ignores read I/O by design because it was meant for compare and write. However, quiesce should work for all I/O - also reads.
Change-Id: I2d81173d1e41e80a6de0b5d69b2dd7c7f761b66c Signed-off-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21153 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@samsung.com>
show more ...
|
e7912a28 | 21-Nov-2023 |
Artur Paszkiewicz <artur.paszkiewicz@intel.com> |
bdev: fix callback not working if unquiesce called inside quiesce cb
A callback for unquiescing a range inside the quiesce callback of the same range would not be execued. The quiesce_ctx for a rang
bdev: fix callback not working if unquiesce called inside quiesce cb
A callback for unquiescing a range inside the quiesce callback of the same range would not be execued. The quiesce_ctx for a range is shared between quiesce and unquiesce operations and the unquiesce callback, set by the quiesce cb, was cleared immediately after quiesce cb finished.
Change-Id: I8cc34cb6394e533c536c0f4efb831d8bed8f7bce Signed-off-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/20699 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 ...
|
01ccef6e | 15-Nov-2023 |
Konrad Sztyber <konrad.sztyber@intel.com> |
bdev: allow bdevs to specify max size of IOs
There are cases when a bdev module needs to limit the maximum size of an IO to a certain value (e.g. large iobuf if it needs to allocate a bounce buffer)
bdev: allow bdevs to specify max size of IOs
There are cases when a bdev module needs to limit the maximum size of an IO to a certain value (e.g. large iobuf if it needs to allocate a bounce buffer).
Previously, this was covered by specifying optimal_io_boundary and setting the split_on_optimal_io_boundary flag. However, this can be inefficient, as it'll also split smaller requests that do not exceed the maximum size, but cross the boundary. For instance, with optimal_io_boundary=4, an IO with offset_blocks=3, num_blocks=2 will be split.
To avoid that, a new field, max_rw_size, is introduced to allow for specifying the maximum size of an IO without taking into account offset_blocks. It only affects READ and WRITE requests, as other IO types are already controlled by other values (e.g. max_unmap, max_copy, max_write_zeroes).
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: Idcafdeaf22a34f25aa599f03726caa9307949e1f Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/20584 Community-CI: Mellanox Build Bot 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 ...
|
6e17adcb | 06-Aug-2023 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
bdev: Factor out internal of spdk_bdev_open_ext() into a function
Factor out the internal of spdk_bdev_open_ext() into a function bdev_open_ext(). Rename a unit test case by bdev_open_ext_test() to
bdev: Factor out internal of spdk_bdev_open_ext() into a function
Factor out the internal of spdk_bdev_open_ext() into a function bdev_open_ext(). Rename a unit test case by bdev_open_ext_test() to avoid conflict(). bdev_open_ext() will be used to add spdk_bdev_open_timeout().
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: Iee9f5c83b1b3700feaa9ed16c77d55703731e61b Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19357 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
ea941cae | 28-Jul-2023 |
Konrad Sztyber <konrad.sztyber@intel.com> |
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 ...
|
ae431e31 | 28-Jul-2023 |
Konrad Sztyber <konrad.sztyber@intel.com> |
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 ...
|
95a367d6 | 02-Dec-2022 |
Artur Paszkiewicz <artur.paszkiewicz@intel.com> |
util/uuid: add API to test/set null uuid
Refactor the code to use these new functions.
Change-Id: I21ee7e9a96f30fbd60106add5e8b071e86bf93c9 Signed-off-by: Artur Paszkiewicz <artur.paszkiewicz@intel
util/uuid: add API to test/set null uuid
Refactor the code to use these new functions.
Change-Id: I21ee7e9a96f30fbd60106add5e8b071e86bf93c9 Signed-off-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16156 Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.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> Community-CI: Mellanox Build Bot
show more ...
|