e58885f9 | 05-Dec-2022 |
Alexey Marchuk <alexeymar@nvidia.com> |
bdevperf: Limit queue depth for verify job with big IO
By design verify payload, uses a bit array to find an offset of IO request. The bit array's size is calculated as bdev_num_blocks / (io_size/bl
bdevperf: Limit queue depth for verify job with big IO
By design verify payload, uses a bit array to find an offset of IO request. The bit array's size is calculated as bdev_num_blocks / (io_size/block_size), if bdev is small, queue depth requested by the user might be bigger than the bit array size and in that case bdevperf won't find a free bit for IO request offset.
To fix that issue, limit queue_depth of such bdevs by "size_in_ios" value
Signed-off-by: Alexey Marchuk <alexeymar@nvidia.com> Change-Id: I3117f5af7ae3ea18219c25982f33db936dd24c0b Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15777 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
show more ...
|
893aaacc | 05-Dec-2022 |
Alexey Marchuk <alexeymar@nvidia.com> |
bdevperf: Check current_queue_depth when job starts draining
During debug of some issue, it was found that current_queue_depth might be 0 when job timer expires, in that case we won't trigger the jo
bdevperf: Check current_queue_depth when job starts draining
During debug of some issue, it was found that current_queue_depth might be 0 when job timer expires, in that case we won't trigger the job end flow and bdevperf may hang.
Signed-off-by: Alexey Marchuk <alexeymar@nvidia.com> Change-Id: Ia49ce6905d329f3ef40216c277bf095782ac9b2d Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15776 Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
045c781d | 06-Dec-2022 |
Richael Zhuang <richael.zhuang@arm.com> |
bdevperf: use SPDK_SEC_TO_USEC for unit conversion
Use SPDK_SEC_TO_USEC instead of '1000 * 1000' or '1000000' for unit conversion.
Change-Id: Ie9fb016b51ef3346ec640297941408f6c643b9b8 Signed-off-by
bdevperf: use SPDK_SEC_TO_USEC for unit conversion
Use SPDK_SEC_TO_USEC instead of '1000 * 1000' or '1000000' for unit conversion.
Change-Id: Ie9fb016b51ef3346ec640297941408f6c643b9b8 Signed-off-by: Richael Zhuang <richael.zhuang@arm.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15784 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
show more ...
|
081b190b | 17-Nov-2022 |
Jim Harris <james.r.harris@intel.com> |
bdevperf: call spdk_thread_exit on job threads
Note: we can only exit a job thread after all jobs are done executing. This is because some bdevs like Ceph send messages to a main thread, so we need
bdevperf: call spdk_thread_exit on job threads
Note: we can only exit a job thread after all jobs are done executing. This is because some bdevs like Ceph send messages to a main thread, so we need to make sure a main thread doesn't exit before another thread that is sending messages to it.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: Iaa51f13f3bc659f6eda7fcefff82c127e095ce29 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15516 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: wanghailiang <hailiangx.e.wang@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
|
dcb296a3 | 17-Nov-2022 |
Jim Harris <james.r.harris@intel.com> |
bdevperf: add bdevperf_job_free() function
Introduce a helper function to properly free everything associated with a job structure, and then use that in all of the various error paths to reduce the
bdevperf: add bdevperf_job_free() function
Introduce a helper function to properly free everything associated with a job structure, and then use that in all of the various error paths to reduce the number of changes required if/when new strings are added to the job structure.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I05280371326ce9582e8452a7753a7990072f25b7 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15506 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: GangCao <gang.cao@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
|