Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

os/bluestore: misc fixes #10327

Merged
merged 5 commits into from Jul 20, 2016
Merged

os/bluestore: misc fixes #10327

merged 5 commits into from Jul 20, 2016

Conversation

xiexingguo
Copy link
Member

No description provided.

@xiexingguo
os/bluestore: make csum_type checking more efficient
1c07f2e 
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@xiexingguo
os/bluestore: en/decode csum_type/csum_chunk_order more efficiently
4a09620 
These two fields are of type uint8_t, so varint encoding/decoding
is unnecessary.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@xiexingguo
os/bluestore: assert available of stat doesn't underflow
9397819 
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@xiexingguo
os/bluestore: trim cache on collection_list() too
012f8f9 
Currently we trim cache on all other read operations
other than this one.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@xiexingguo
os/bluestore: fix error handling of posix_fadvise() syscall
82ccf7e 
According to Linux man page:
On success, zero is returned.  On error, an error number is returned.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@xiexingguo
Copy link
Member Author

Local tested:

2016-07-18 19:15:27.737725 7f9d2a040680  1 bluestore(store_test_temp_dir) fsck finish with 0 errors
[       OK ] ObjectStore/StoreTest.TooManyBlobsTest/2 (23860 ms)
[----------] 59 tests from ObjectStore/StoreTest (2600058 ms total)

[----------] Global test environment tear-down
[==========] 59 tests from 1 test case ran. (2600058 ms total)
[  PASSED  ] 59 tests.

@markhpc markhpc merged commit 12257c2 into ceph:master Jul 20, 2016
@xiexingguo xiexingguo deleted the xxg-wip-bluestore-2016-07-18 branch July 20, 2016 22:25
@xiexingguo xiexingguo changed the title os/bluestore: misc fixes/cleanups os/bluestore: misc fixes Aug 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bluestore performance wip-mark-testing
Projects
None yet
Milestone
No milestone
2 participants
@xiexingguo @markhpc