Project

General

Profile

Bug #17229

crushtool will using the out of bound space when testing with "--show-choose-tries"

Added by Wei-Chung Cheng about 6 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Normal
Category:
-
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(RADOS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

when we testing the crush rule which could not be mapped successfully under our `choose_total_tries`

it seems to use the out of bound space to save our retries times.

test-ec-crush have 18 osds (one osd on each host) and min_size: 17, max_size: 18, choose_total_tries: 50

then crushtool testes

#./crushtool -i test-ec-crush --test --show-choose-tries
*** Error in `./crushtool': free(): invalid next size (fast): 0x00007f3a807328d0 ***
*** Caught signal (Aborted) **
in thread 7f3a76829d00 thread_name:crushtool
ceph version 11.0.0-1772-g577f027 (577f027ba88d95b480721f17c8925cbc949c8d1f)
1: (()+0x129d62) [0x7f3a76971d62]
2: (()+0x10330) [0x7f3a75bde330]
3: (gsignal()+0x37) [0x7f3a7488fc37]
4: (abort()+0x148) [0x7f3a74893028]
5: (()+0x732a4) [0x7f3a748cc2a4]
6: (()+0x7f55e) [0x7f3a748d855e]
7: (std::_Rb_tree<int, std::pair<int const, int>, std::_Select1st<std::pair<int const, int> >, std::less<int>, std::allocator<std::pair<int const, int> > >::_M_erase(std::_Rb_tree_node<std::pair<int const, int> >*)+0x28) [0x7f3a7696a6e8]
8: (CrushTester::test()+0xbca) [0x7f3a7699380a]
9: (main()+0x1a4d) [0x7f3a7696350d]
10: (__libc_start_main()+0xf5) [0x7f3a7487af45]
11: (()+0x121327) [0x7f3a76969327]
2016-09-07 16:32:11.541499 7f3a76829d00 -1 *** Caught signal (Aborted) **
in thread 7f3a76829d00 thread_name:crushtool

History

#1 Updated by Wei-Chung Cheng about 6 years ago

  • Status changed from New to In Progress

#2 Updated by Kefu Chai about 6 years ago

  • Status changed from In Progress to Fix Under Review

#3 Updated by Kefu Chai about 6 years ago

  • Status changed from Fix Under Review to Resolved

#4 Updated by Greg Farnum over 5 years ago

  • Project changed from Ceph to RADOS
  • Category deleted (10)

Also available in: Atom PDF