| f1900e4d | 19-Jul-2024 |
Jim Harris <jim.harris@samsung.com> |
thread: convert iobuf nodes to 1-sized arrays
This is just a step towards supporting a bigger array, 1 element per NUMA node.
Signed-off-by: Jim Harris <jim.harris@samsung.com> Change-Id: If235aa91
thread: convert iobuf nodes to 1-sized arrays
This is just a step towards supporting a bigger array, 1 element per NUMA node.
Signed-off-by: Jim Harris <jim.harris@samsung.com> Change-Id: If235aa9120965921cda6291043169a5c55ceb144 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24520 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com> Reviewed-by: Ben Walker <ben@nvidia.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
|
| cc23f71e | 19-Dec-2023 |
Alexey Marchuk <alexeymar@nvidia.com> |
accel: Move aux_iovs and bounce to aux struct
iovs and bounce occupy 192 bytes, they are used by several _submit_ functions which don't support iovs. These functions do not support sequence which is
accel: Move aux_iovs and bounce to aux struct
iovs and bounce occupy 192 bytes, they are used by several _submit_ functions which don't support iovs. These functions do not support sequence which is the preferred way to use accel operations. Functions which support accel sequence use aux iovs and/or bounce buffers in rare cases when accel domain of staging buffer are used. Let's consider these cases as "slow" path and try to improve other use cases. These structures create a gap between hot data in the base spdk_accel_task and module-specific context located after the base structure. To make it more cache-friendly, move aux iovs and bounce buffers to a new aux structure, keep a pool of aux structure in io_cahnnel and use them where needed
Signed-off-by: Alexey Marchuk <alexeymar@nvidia.com> Change-Id: Ifec404513b7bb53edbdf22faf03fb9929b5fdd46 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21121 Reviewed-by: Ben Walker <ben@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> Community-CI: Mellanox Build Bot
show more ...
|
| 57c911bc | 29-Jun-2023 |
Konrad Sztyber <konrad.sztyber@intel.com> |
accel: remove erroneous assertion in accel_process_sequence()
The task can be NULL when accel_process_sequence() is called from the context of spdk_accel_sequence_continue() when all tasks are compl
accel: remove erroneous assertion in accel_process_sequence()
The task can be NULL when accel_process_sequence() is called from the context of spdk_accel_sequence_continue() when all tasks are completed. It hasn't happened before, bacuse the test would always call spdk_accel_sequence_continue() synchronously from within execute_sequence().
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I6fb66d3e05566499e8bae38a3efbd4ef83f8dc09 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18868 Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Reviewed-by: Jacek Kalwas <jacek.kalwas@intel.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| 3e0e0779 | 28-Mar-2023 |
Konrad Sztyber <konrad.sztyber@intel.com> |
accel: copy memory domain context when merging tasks
When changing src/dst buffers, we copied memory domain pointers, but we didn't copy memory domain context, which is obviously incorrect. It was
accel: copy memory domain context when merging tasks
When changing src/dst buffers, we copied memory domain pointers, but we didn't copy memory domain context, which is obviously incorrect. It was probably missed, because we never append a copy with non-NULL memory domain. Added a unit test case to verify this behavior.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: Ic174e0e72c33d3f437f0faddd3405638049f0c74 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17425 Reviewed-by: Jim Harris <james.r.harris@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 ...
|