721695e121
Fixes github issue #29. Because of how we handle the blobid and pagenum in blobstore, it was possible to have blobstore inadvertently open the wrong blob if open is provided a blobid where the lower 32 bits match an existing blob but the upper 32 are clear. Patch does the following: - removes assert() that caught this on MD load and replace with an error given that this condition can be induced via the API - cleanup of pagenum and blobid conversion/handling to make it clearer how they're related and converted - new UTs that would have failed w/o the new check in place Change-Id: I2b49b237922b3b8cfc4df296f5bc20195e41dc41 Signed-off-by: Paul Luse <paul.e.luse@intel.com> Reviewed-on: https://review.gerrithub.io/380872 Tested-by: SPDK Automated Test System <sys_sgsw@intel.com> Reviewed-by: Daniel Verkamp <daniel.verkamp@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> |
||
|---|---|---|
| .. | ||
| app | ||
| blobfs/rocksdb | ||
| cpp_headers | ||
| iscsi_tgt | ||
| lib | ||
| nvmf | ||
| unit | ||
| vhost | ||
| Makefile | ||
| spdk_cunit.h | ||