os/bluestore: a few more cleanups #11780

merged 4 commits into from Nov 5, 2016


None yet

3 participants


No description provided.

@xiexingguo xiexingguo added the bluestore label Nov 4, 2016
@liewegas liewegas was assigned by xiexingguo Nov 4, 2016
@xiexingguo xiexingguo changed the title from os/bluestore: two more cleanups to os/bluestore: a few more cleanups Nov 4, 2016
@@ -6693,8 +6693,8 @@ void BlueStore::_kv_sync_thread()
KeyValueDB::Transaction synct = db->get_transaction();
// increase {nid,blobid}_max? note that this covers both the
- // case wehre we are approaching the max and the case we passed
- // it. in either case, we increase the max in the earlier txn
+ // case we are approaching the max and the case we passed it.
ifed01 Nov 4, 2016 Contributor

should be: where we are approaching...?

xiexingguo Nov 5, 2016 Contributor

@ifed01 Thanks, Igor.

@liewegas liewegas added this to the kraken milestone Nov 4, 2016
xiexingguo added some commits Nov 4, 2016
@xiexingguo xiexingguo os/bluestore: narrow boundary check for fiemap
By including off-by-one.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@xiexingguo xiexingguo os/bluestore: add sanity check against collection existence
To keep pace with other methods.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@xiexingguo xiexingguo os/bluestore: fix typo in comments
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@xiexingguo xiexingguo os/bluestore: fix output message when passing the end of list range
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@liewegas liewegas merged commit b009f5a into ceph:master Nov 5, 2016

2 checks passed

Signed-off-by all commits in this PR are signed
default Build finished.
@xiexingguo xiexingguo deleted the xiexingguo:xxg-wip-bluestore-1104 branch Nov 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment