RepoMirrors/ceph
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2025-01-04 02:02:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

os/bluestore: clean up compression debug output

Browse Source 
The previous output was impossible to interpret properly.

Signed-off-by: Sage Weil <sage@redhat.com>
This commit is contained in:
Sage Weil 2016-10-06 12:03:05 -04:00
parent 97cb04a15b
commit f78c54359a
1 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/os/bluestore/BlueStore.cc
View File

@ -7549,10 +7549,10 @@ int BlueStore::_do_alloc_write(
compressed_bl.claim_append(t);
uint64_t rawlen = compressed_bl.length();
uint64_t newlen = P2ROUNDUP(rawlen, min_alloc_size);
uint64_t dstlen = final_length *
uint64_t want_len_raw = final_length *
g_conf->bluestore_compression_required_ratio;
dstlen = P2ROUNDUP(dstlen, min_alloc_size);
if (newlen <= dstlen && newlen < final_length) {
uint64_t want_len = P2ROUNDUP(want_len_raw, min_alloc_size);
if (newlen <= want_len && newlen < final_length) {
// Cool. We compressed at least as much as we were hoping to.
// pad out to min_alloc_size
compressed_bl.append_zero(newlen - rawlen);
@ -7572,14 +7572,17 @@ int BlueStore::_do_alloc_write(
compressed = true;
logger->inc(l_bluestore_compress_success_count);
} else {
dout(20) << __func__ << std::hex << " compressed 0x" << l->length()
<< " -> 0x" << rawlen << " with " << (int)chdr.type
<< ", which is more than required 0x" << dstlen
dout(20) << __func__ << std::hex << " 0x" << l->length()
<< " compressed to 0x" << rawlen << " -> 0x" << newlen
<< " with " << (int)chdr.type
<< ", which is more than required 0x" << want_len_raw
<< " -> 0x" << want_len
<< ", leaving uncompressed"
<< std::dec << dendl;
logger->inc(l_bluestore_compress_rejected_count);
}
logger->tinc(l_bluestore_compress_lat, ceph_clock_now(g_ceph_context) - start);
logger->tinc(l_bluestore_compress_lat,
ceph_clock_now(g_ceph_context) - start);
}
if (!compressed) {
b->dirty_blob().set_flag(bluestore_blob_t::FLAG_MUTABLE);