| e93f0f94 | 24-Oct-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
bdev/malloc: Support accel sequence when DIF is enabled
malloc bdev module did not support accel sequence if DIF was enabled. Remove the limitation by utilizing spdk_bdev_io_hide_metadata().
If DIF
bdev/malloc: Support accel sequence when DIF is enabled
malloc bdev module did not support accel sequence if DIF was enabled. Remove the limitation by utilizing spdk_bdev_io_hide_metadata().
If DIF insert/strip is done by accel sequence, it is not possible to verify DIF of I/O buffer. Hence, verify DIF of malloc buffer instead.
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: Idb088b6fc3cc045028c33649c84218c708fd601b Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25326 Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com> Reviewed-by: Jim Harris <jim.harris@nvidia.com> Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| c86e5b18 | 11-Nov-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
bdev/malloc: Extract internal of verify_pi() for code reuse
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: Ib4f3f9e038d824ac5a8e4c65a2cff1a3e365a432 Reviewed-on: https://review.s
bdev/malloc: Extract internal of verify_pi() for code reuse
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: Ib4f3f9e038d824ac5a8e4c65a2cff1a3e365a432 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25418 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <jim.harris@nvidia.com> Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.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 ...
|
| c016e6ff | 17-Jul-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
bdev/malloc: Make DIF PI format configurable at bdev creation
Make bdev's PI format configurable via the bdev_malloc_create RPC. The PI format check is done by spdk_dif_ctx_init() as same as other p
bdev/malloc: Make DIF PI format configurable at bdev creation
Make bdev's PI format configurable via the bdev_malloc_create RPC. The PI format check is done by spdk_dif_ctx_init() as same as other parameters.
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: I28afca948916e36987d82e86f1a343203b83127d Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24108 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@samsung.com>
show more ...
|
| b1a1e099 | 16-Jul-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
bdev/malloc Add missing DIF parameters into write_json_config()
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: I2dbb47e249ba261ea7c51cd07e28ff4e02a5636a Reviewed-on: https://revi
bdev/malloc Add missing DIF parameters into write_json_config()
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: I2dbb47e249ba261ea7c51cd07e28ff4e02a5636a Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24206 Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.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 ...
|
| b62aadf2 | 14-Feb-2024 |
Konrad Sztyber <konrad.sztyber@intel.com> |
bdev/malloc: fix reads with separate metadata buffers
If the bdev was configured to use a separate buffers for metadata and the user submitted a read with buffers=NULL, the metadata pointer would no
bdev/malloc: fix reads with separate metadata buffers
If the bdev was configured to use a separate buffers for metadata and the user submitted a read with buffers=NULL, the metadata pointer would not get set.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: If19d5450edba7caf144a5795913212d8e0af682d Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21858 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Reviewed-by: Karl Bonde Torp <k.torp@samsung.com>
show more ...
|
| 9bec3f58 | 20-Nov-2023 |
Changpeng Liu <changpeng.liu@intel.com> |
bdev/malloc: don't do verify when separate metadata buffer is null
The `bdev_gpt` module will examine Malloc device without passing separate metadata buffer, then the Malloc driver will report error
bdev/malloc: don't do verify when separate metadata buffer is null
The `bdev_gpt` module will examine Malloc device without passing separate metadata buffer, then the Malloc driver will report error logs for Malloc device created with separate metadata, here we skip doing verify if the input metadata buffer is NULL.
Fix #3191.
Change-Id: Ia6a160dca238534a7b534f05e0646123c7870d1e Signed-off-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/20632 Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Karol Latecki <karol.latecki@intel.com> Reviewed-by: Jim Harris <jim.harris@samsung.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| 97ce07c2 | 09-Mar-2023 |
Konrad Sztyber <konrad.sztyber@intel.com> |
bdev/malloc: report accel sequence support
This actually allows malloc bdev to chain multiple accel operations together. And, since the last operation will always be a copy, accel should remove tha
bdev/malloc: report accel sequence support
This actually allows malloc bdev to chain multiple accel operations together. And, since the last operation will always be a copy, accel should remove that copy by modifying previous operation's dst/src.
On my system, it improved bdevperf performance (single core, qd=4, bs=128k, bdev_crypto on top of bdev_malloc, crypto_sw):
randread: 5668M/s -> 8201M/s randwrite: 5148M/s -> 7856M/s
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I5b9173fa70a42ee56f56c496a34037d46d2f420f Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17202 Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <james.r.harris@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| 92760190 | 09-Mar-2023 |
Konrad Sztyber <konrad.sztyber@intel.com> |
bdev/malloc: report memory domain support
Because the copying is handled by accel, which will do push/pull when necessary, we can report support for each registered memory domain.
Also, since verif
bdev/malloc: report memory domain support
Because the copying is handled by accel, which will do push/pull when necessary, we can report support for each registered memory domain.
Also, since verifying PI information would require doing a push/pull, we don't report support for memory domains if bdev has DIF enabled.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: Id80f82aaac68e9dec2a6cae81d96a460105161d6 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17201 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
show more ...
|
| 06fd87e4 | 09-Mar-2023 |
Konrad Sztyber <konrad.sztyber@intel.com> |
bdev/malloc: use appends for write requests
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: Ief6c873a5f65274a25b67bc3f2811d8f3e4a33b3 Reviewed-on: https://review.spdk.io/gerrit/c
bdev/malloc: use appends for write requests
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: Ief6c873a5f65274a25b67bc3f2811d8f3e4a33b3 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17200 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
show more ...
|