#
2172c432 |
| 04-Sep-2020 |
Tomasz Zawadzki <tomasz.zawadzki@intel.com> |
log: simplify SPDK_LOG_REGISTER_COMPONENT
This patch removes the string from register component. Removed are all instances in libs or hardcoded in apps.
Starting with this patch literal passed to r
log: simplify SPDK_LOG_REGISTER_COMPONENT
This patch removes the string from register component. Removed are all instances in libs or hardcoded in apps.
Starting with this patch literal passed to register, serves as name for the flag.
All instances of SPDK_LOG_* were replaced with just * in lowercase. No actual name change for flags occur in this patch.
Affected are SPDK_LOG_REGISTER_COMPONENT() and SPDK_*LOG() macros.
Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Change-Id: I002b232fde57ecf9c6777726b181fc0341f1bb17 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/4495 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Mellanox Build Bot Reviewed-by: Anil Veerabhadrappa <anil.veerabhadrappa@broadcom.com> Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Community-CI: Broadcom CI
show more ...
|
#
5363eb3c |
| 06-Sep-2020 |
Sochin Jiang <jiangxiaoqing.sochin@bytedance.com> |
lib/blob: fix an uint32_t overflow in blobstore init
uint32_t supports at most 2TB at most, we need to handle the larger blobstores, fix this overflow problem.
Signed-off-by: Sochin Jiang <jiangxia
lib/blob: fix an uint32_t overflow in blobstore init
uint32_t supports at most 2TB at most, we need to handle the larger blobstores, fix this overflow problem.
Signed-off-by: Sochin Jiang <jiangxiaoqing.sochin@bytedance.com> Change-Id: I27950eb759e9cb9ad48fa4aa8dd1976b4e852832 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/4075 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
show more ...
|
#
bd16f574 |
| 28-Aug-2020 |
Jim Harris <james.r.harris@intel.com> |
blob: switch to bit_pool for tracking used_clusters
We still need to be able to explicitly set specific bits in the cluster array during initialization and loading (especially recovery), so we use a
blob: switch to bit_pool for tracking used_clusters
We still need to be able to explicitly set specific bits in the cluster array during initialization and loading (especially recovery), so we use a bit_array during load, and then convert it to a bit_pool just before calling the user's cmopletion callback.
This gives a roughly 300% improvement over baseline on a benchmark which does continuous resize operations. The benefit is primarily from saving the lowest free bit rather than having to always start at bit 0. We may be able to further improve this by saving extents in the bit pool as well, although after this patch, the benchmark shows other hot spots different from the bit search.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: Idb1d75d8348bc50560b1f42d49dbe4d79d024619
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3975 Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Paul Luse <paul.e.luse@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
#
04eb8e05 |
| 28-Aug-2020 |
Jim Harris <james.r.harris@intel.com> |
blob: allocate ctx in bs_alloc
The ctx allocation was duplicated after both bs_alloc calling sites, so this reduces the code a bit. This change also enables some future changes involving the used_c
blob: allocate ctx in bs_alloc
The ctx allocation was duplicated after both bs_alloc calling sites, so this reduces the code a bit. This change also enables some future changes involving the used_clusters bit array.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I4ea98f079dbe385654e9cb9c0c58a1926a990c9e Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3973 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Paul Luse <paul.e.luse@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
#
39caf843 |
| 28-Aug-2020 |
Jim Harris <james.r.harris@intel.com> |
blob: consolidate init, load and dump ctx objects
This will allow for some additional simplifications in future patches.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: Ie421ad35f8c
blob: consolidate init, load and dump ctx objects
This will allow for some additional simplifications in future patches.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: Ie421ad35f8c0efbb775fbe6bf85799af515264ef Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3972 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
#
0a19f43e |
| 28-Aug-2020 |
Jim Harris <james.r.harris@intel.com> |
blob: move spdk_bs_load_ctx earlier in the file
This prepares for some future patches.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: If63c83f76e839b796c58200ddb0ca2137fbc4288 Revi
blob: move spdk_bs_load_ctx earlier in the file
This prepares for some future patches.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: If63c83f76e839b796c58200ddb0ca2137fbc4288 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3971 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
#
835fbef7 |
| 27-Aug-2020 |
Jim Harris <james.r.harris@intel.com> |
blob: simplify bs_allocate_cluster
Have it both find and set the lowest available cluster bit index.
This will temporarily hurt the performance for cluster allocation, since it will always search s
blob: simplify bs_allocate_cluster
Have it both find and set the lowest available cluster bit index.
This will temporarily hurt the performance for cluster allocation, since it will always search starting at bit index 0. But upcoming changes in this patch set will fix that again by using a new spdk_bit_pool object that will do allocations much more efficiently than the current implementation here.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: Iad199c9166b82cb9a31597a080f5a28823849e60 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3970 Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
#
ded02d09 |
| 02-Sep-2020 |
Jim Harris <james.r.harris@intel.com> |
blob: batch used_cluster_mutex operations
Currently if we create a blob of 128 clusters, we lock/unlock the used_cluster_mutex 128 times - once for each cluster. Same when those clusters are releas
blob: batch used_cluster_mutex operations
Currently if we create a blob of 128 clusters, we lock/unlock the used_cluster_mutex 128 times - once for each cluster. Same when those clusters are released when the blob is deleted. Batching these lock/unlock operations is very easy and gives a noticeable efficiency improvement.
My local benchmark (1GiB ramdisk, 4KB cluster size, 128 clusters/blob) creates enough blobs to fill the blobstore and then continuously deletes and recreates them. Performance increases 20% on that benchmark with this patch.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: Ic503accf1ca1ab1af7254b4067771d956f52014d Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/4069 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
#
bc039425 |
| 04-Sep-2020 |
Tomasz Zawadzki <tomasz.zawadzki@intel.com> |
lib/blob: explicitly pass ctx where possible
Some of the functions were only referenced directly. There is no need to use void* or pass any bserrno, in some cases.
Let's be explicit.
Signed-off-by
lib/blob: explicitly pass ctx where possible
Some of the functions were only referenced directly. There is no need to use void* or pass any bserrno, in some cases.
Let's be explicit.
Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Change-Id: Ib26dda7068965838f38dad856ea1e456fd87a655 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/4061 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>
show more ...
|
#
c8c73af9 |
| 03-Sep-2020 |
Tomasz Zawadzki <tomasz.zawadzki@intel.com> |
lib/blob: add clearing extents to persist path
This looks like a major omission on persist path. Especially visible for cases where blobstore was not reloaded between blob creations/deletion.
Added
lib/blob: add clearing extents to persist path
This looks like a major omission on persist path. Especially visible for cases where blobstore was not reloaded between blob creations/deletion.
Added writing out zeroes to md_pages that contained truncated extents (resized down). After zeroes are writen out, md_pages for those extents are released. In case of blob deletion, extents are resized down to 0 so all extent pages are released.
Fixes #1590
Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Change-Id: I9a2a1190e3f1f3b5d1bb806191c1fe4d27df7780 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/4051 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>
show more ...
|
#
3fe748f7 |
| 27-Aug-2020 |
Jim Harris <james.r.harris@intel.com> |
blob: move bs_release_cluster
We will use it earlier in this file in a future patch.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I554f2073185d466bd0b4e98bdeec721f763c1b44 Review
blob: move bs_release_cluster
We will use it earlier in this file in a future patch.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I554f2073185d466bd0b4e98bdeec721f763c1b44 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3969 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
#
8ba7450e |
| 27-Aug-2020 |
Jim Harris <james.r.harris@intel.com> |
blob: simplify check for available cluster count
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I56a9ca114b84ded4ac23a9eaacadaa6df8f031cc Reviewed-on: https://review.spdk.io/gerrit/
blob: simplify check for available cluster count
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I56a9ca114b84ded4ac23a9eaacadaa6df8f031cc Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3968 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
#
f5b949d5 |
| 27-Aug-2020 |
Jim Harris <james.r.harris@intel.com> |
blob: simplify free cluster counting in inflate path
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I5716290476d4abdf8646a36206f07acb5b2dd4ec Reviewed-on: https://review.spdk.io/ger
blob: simplify free cluster counting in inflate path
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I5716290476d4abdf8646a36206f07acb5b2dd4ec Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3967 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
#
2d87587f |
| 27-Aug-2020 |
Jim Harris <james.r.harris@intel.com> |
blob: claim clusters inline during initialization
When claiming clusters as part of blobstore initialization or recovery, just call spdk_bit_array_set directly rather than going through the bs_claim
blob: claim clusters inline during initialization
When claiming clusters as part of blobstore initialization or recovery, just call spdk_bit_array_set directly rather than going through the bs_claim_cluster function. We will be modifying how runtime cluster allocation works so need to separate the two use cases. This code is very small so inlining it has minimal code impact.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: Iaaa1c817e57b4a2eea62eb4683407364bac1fcc0 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3966 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
#
b7d508f5 |
| 27-Aug-2020 |
Jim Harris <james.r.harris@intel.com> |
blob: use spdk_bit_array_load/store_mask functions
These functions were added during FTL development and are more efficient than the roll-your-own implementations blobstore had previously.
Signed-o
blob: use spdk_bit_array_load/store_mask functions
These functions were added during FTL development and are more efficient than the roll-your-own implementations blobstore had previously.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: Ie09e5c305e6e171af0258e805f2aac3b88822b5e Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3965 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
#
846ca675 |
| 19-Aug-2020 |
Tomasz Zawadzki <tomasz.zawadzki@intel.com> |
lib/blob: remove requirement for md_page alignment
The md_page alignment is not really required for md_page buffers. Allocating 4k aligned buffers all the time, causes memory to be heavily fragmente
lib/blob: remove requirement for md_page alignment
The md_page alignment is not really required for md_page buffers. Allocating 4k aligned buffers all the time, causes memory to be heavily fragmented. Due to DPDK keeping track of the allocation in the same DMA region as the allocation themselves. Removing this alignment requirement will help DPDK when searching for the right part of memory in the heap.
Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reported-by: Mike Cui Change-Id: If2f4ca2be38d432d5740f6145b5e0ff46237806b Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3853 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
show more ...
|
#
ed7848f2 |
| 19-Aug-2020 |
Jim Harris <james.r.harris@intel.com> |
blob: handle overlapping open case
We only create one spdk_blob object for a given blob, and just increase the ref_count if it is opened multiple times. bs_open_blob would do the lookup for existin
blob: handle overlapping open case
We only create one spdk_blob object for a given blob, and just increase the ref_count if it is opened multiple times. bs_open_blob would do the lookup for existing opened blobs.
But if the blob is opened again, before the previous open operation has completed, we would end up with two spdk_blob objects for the same blob.
Solution is to do another lookup when the open operation completes. If we find the blob, free the one we just finished opening and return the existing one instead.
Also added unit test that failed on the existing code but passes now with this patch.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Reported-by: Mike Cui Change-Id: I00c3a913b413deddf06f0b63f7a669efb2b5658f
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3855 Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
Revision tags: v20.07, v20.07-rc1 |
|
#
db3d1201 |
| 13-Jul-2020 |
Sochin Jiang <jiangxiaoqing.sochin@bytedance.com> |
lib/blob: fix a data corruption bug
There is a fatal bug that could easily cause data corruption when using thin-provisioned blobs. In blob_request_submit_rw_iov(), we first get lba by calling blob_
lib/blob: fix a data corruption bug
There is a fatal bug that could easily cause data corruption when using thin-provisioned blobs. In blob_request_submit_rw_iov(), we first get lba by calling blob_calculate_lba_and_lba_count(), blob_calculate_lba_and_lba_count() calculates different lbas according to the return of bs_io_unit_is_allocated(). Later, we call bs_io_unit_is_allocated() again to judge whether the specific cluster is allocated, the problem is it may have be allocated here while not be allocated when calling blob_calculate_lba_and_lba_count() before. To ensure the correctness of lba, we can do lba recalculation when bs_io_unit_is_allocated() returns true, or make blob_calculate_lba_and_lba_count() return the result of bs_io_unit_is_allocated(), use the second solution in this patch.
By configuring more than one cpu core, md thread will run in a separate SPDK thread, this data corruption scenario could be easily reproduced by running fio verify in VMs using thin-provisioned Lvols as block devices.
Signed-off-by: Sochin Jiang <jiangxiaoqing.sochin@bytedance.com> Change-Id: I099865ff291ea42d5d49b693cc53f64b60881684 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3318 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
#
30ee8137 |
| 04-Jun-2020 |
Ben Walker <benjamin.walker@intel.com> |
blob: Add a bitmask for quickly checking which blobs are open
This can speed up the check for whether a blob is already open significantly.
Signed-off-by: Ben Walker <benjamin.walker@intel.com> Cha
blob: Add a bitmask for quickly checking which blobs are open
This can speed up the check for whether a blob is already open significantly.
Signed-off-by: Ben Walker <benjamin.walker@intel.com> Change-Id: If32b0b1f168fcdb58e61df6281d7b7520725a195 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/2781 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Paul Luse <paul.e.luse@intel.com>
show more ...
|
#
7137bde1 |
| 01-Jul-2020 |
Sochin Jiang <jiangxiaoqing.sochin@bytedance.com> |
lib/blob: fix a potential wrong release of md page
blob_insert_cluster_msg() will release the already claimed md page(intended for extent page) if the corresponding extent page is found to be alloca
lib/blob: fix a potential wrong release of md page
blob_insert_cluster_msg() will release the already claimed md page(intended for extent page) if the corresponding extent page is found to be allocated. But later blob_insert_extent() may fails, and this cause blob_insert_cluster_cpl() to release the same md page again, this could be wrong if this specific md page is clamied by others, thus cause data corruption. So, put it to zero after released in blob_insert_cluster_msg().
Signed-off-by: Sochin Jiang <jiangxiaoqing.sochin@bytedance.com> Change-Id: I46eba79b24b1950318002dcb27cb51b01ca566ec Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3152 Community-CI: Mellanox Build Bot Reviewed-by: Ben Walker <benjamin.walker@intel.com> 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 ...
|
#
abeae984 |
| 09-Jun-2020 |
Yan Wang <wangyan122@huawei.com> |
lib/blob: check return value after malloc() and strdup()
Signed-off-by: Yan Wang <wangyan122@huawei.com> Change-Id: Ieaaecfd12ddf7826c2e4ce6dc5f47411e7e077da Reviewed-on: https://review.spdk.io/gerr
lib/blob: check return value after malloc() and strdup()
Signed-off-by: Yan Wang <wangyan122@huawei.com> Change-Id: Ieaaecfd12ddf7826c2e4ce6dc5f47411e7e077da Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/2907 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Paul Luse <paul.e.luse@intel.com>
show more ...
|
Revision tags: v20.04.1, v20.01.2 |
|
#
964463e4 |
| 15-May-2020 |
Seth Howell <seth.howell@intel.com> |
lib/blob: remove spdk_ prefix from bs_batch_read_bs_dev
This function was removed from the public API last release when the map file was introduced, but I didn't clean up the name at that time.
Sig
lib/blob: remove spdk_ prefix from bs_batch_read_bs_dev
This function was removed from the public API last release when the map file was introduced, but I didn't clean up the name at that time.
Signed-off-by: Seth Howell <seth.howell@intel.com> Change-Id: I3101723b504531ce2c51dba2feb063511dd32684 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/2443 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> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com>
show more ...
|
#
b5d68d59 |
| 14-May-2020 |
Seth Howell <seth.howell@intel.com> |
lib/blob: remove _spdk prefix from all functions.
Signed-off-by: Seth Howell <seth.howell@intel.com> Change-Id: Idb33816e5b66266987845172c27c87667ac0a596 Reviewed-on: https://review.spdk.io/gerrit/c
lib/blob: remove _spdk prefix from all functions.
Signed-off-by: Seth Howell <seth.howell@intel.com> Change-Id: Idb33816e5b66266987845172c27c87667ac0a596 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/2437 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com>
show more ...
|
#
e230d49a |
| 14-May-2020 |
Sochin Jiang <jiangxiaoqing.sochin@bytedance.com> |
blob: fix a wrong memset parameter
A wrong parameter is passed to memset when loading blob->active.clusters, this leads to an unpredictable wrong lba value using thin provision LVOLs while submittin
blob: fix a wrong memset parameter
A wrong parameter is passed to memset when loading blob->active.clusters, this leads to an unpredictable wrong lba value using thin provision LVOLs while submitting IO requests, thus causes EIO error using QEMU vms.
Signed-off-by: Sochin Jiang <jiangxiaoqing.sochin@bytedance.com> Change-Id: Iecea80cfa58f7a025603430d666fd9cd4d3fea8b Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/2431 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Xiaodong Liu <xiaodong.liu@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
#
3456377b |
| 10-May-2020 |
Seth Howell <seth.howell@intel.com> |
lib: accel, bdev, blob, env_dpdk remove spdk_ prefix.
Hitting only the static functions from the above libraries with the spdk_ prefix.
Signed-off-by: Seth Howell <seth.howell@intel.com> Change-Id:
lib: accel, bdev, blob, env_dpdk remove spdk_ prefix.
Hitting only the static functions from the above libraries with the spdk_ prefix.
Signed-off-by: Seth Howell <seth.howell@intel.com> Change-Id: Ic6df38dfbeb53f0b1c30d350921f7216acba3170 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/2362 Community-CI: Mellanox Build Bot Community-CI: Broadcom CI Reviewed-by: Paul Luse <paul.e.luse@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|