Project

General

Profile

Bug #1737

ceph-fuse crash in xlist::remove

Added by Josh Durgin over 5 years ago. Updated 5 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
11/18/2011
Due date:
% Done:

0%

Source:
Q/A
Tags:
Backport:
Regression:
Severity:
Reviewed:
Affected Versions:
ceph-qa-suite:
Release:
Component(FS):
Needs Doc:
No

Description

From teuthology:~/log/client.0.log.gz:

./include/xlist.h: In function 'void xlist<T>::remove(xlist<T>::item*) [with T = Cap*]', in thread '7effb67db700'
./include/xlist.h: 130: FAILED assert((bool)_front == (bool)_size)
 ceph version 0.38-199-gdedf2c4 (commit:dedf2c4a066876bdab9a0b0154196194cefc1340)
 1: (Client::remove_cap(Cap*)+0x87d) [0x4747bd]
 2: (Client::remove_all_caps(Inode*)+0x3b) [0x47496b]
 3: (Client::put_inode(Inode*, int)+0x17b) [0x47b6fb]
 4: (Client::handle_cap_flush_ack(Inode*, int, Cap*, MClientCaps*)+0x678) [0x49bf28]
 5: (Client::handle_caps(MClientCaps*)+0xb28) [0x4b4768]
 6: (Client::ms_dispatch(Message*)+0x613) [0x4b8b43]
 7: (SimpleMessenger::dispatch_entry()+0x99a) [0x52faea]
 8: (SimpleMessenger::DispatchThread::entry()+0x2c) [0x45e1cc]
 9: (Thread::_entry_func(void*)+0x12) [0x52a632]
 10: (()+0x7971) [0x7effbac55971]
 11: (clone()+0x6d) [0x7effb94e492d]

Associated revisions

Revision d78235be (diff)
Added by Sage Weil over 4 years ago

client: fix readdir locking

Several of the readdir-related methods were not taking client_lock.

Fixes: #1737
Backport: argonaut
Signed-off-by: Sage Weil <>

Revision 5c5ca4b8 (diff)
Added by Sage Weil over 4 years ago

move cfuse + dbench from marginal to regression

Fixed #1737, yay!

History

#1 Updated by Sage Weil over 5 years ago

  • translation missing: en.field_position set to 2

#2 Updated by Sage Weil over 5 years ago

  • translation missing: en.field_position deleted (2)
  • translation missing: en.field_position set to 5

#3 Updated by Sage Weil over 5 years ago

  • Assignee set to Sage Weil

#4 Updated by Sage Weil over 5 years ago

  • translation missing: en.field_position deleted (6)
  • translation missing: en.field_position set to 5

#5 Updated by Sage Weil over 5 years ago

  • Priority changed from Normal to High

#6 Updated by Sage Weil about 5 years ago

  • Target version changed from v0.39 to v0.40

#7 Updated by Sage Weil about 5 years ago

  • Assignee deleted (Sage Weil)

#8 Updated by Josh Durgin about 5 years ago

This happened again from a different path in teuthology:~/log/client.0.log.gz:

./include/xlist.h: In function 'void xlist<T>::remove(xlist<T>::item*) [with T = Cap*]', in thread '7f4e41bf9780'
./include/xlist.h: 130: FAILED assert((bool)_front == (bool)_size)
 ceph version 0.39-50-gbcd26fc (commit:bcd26fca71b38a20ce469447c7867940a3d000ed)
 1: (Client::remove_cap(Cap*)+0x87d) [0x4739bd]
 2: (Client::remove_all_caps(Inode*)+0x3b) [0x473b6b]
 3: (Client::put_inode(Inode*, int)+0x17b) [0x47b4eb]
 4: (Client::unlink(Dentry*, bool)+0x1d0) [0x47ddf0]
 5: (Client::trim_dentry(Dentry*)+0x84) [0x47f984]
 6: (Client::trim_cache()+0x3ed) [0x48006d]
 7: (Client::unmount()+0x35e) [0x4b0fae]
 8: (main()+0x749) [0x45d329]
 9: (__libc_start_main()+0xfe) [0x7f4e3fd64d8e]
 10: /tmp/cephtest/binary/usr/local/bin/ceph-fuse() [0x45c3e9]

#9 Updated by Sage Weil about 5 years ago

  • translation missing: en.field_position deleted (41)
  • translation missing: en.field_position set to 63

#10 Updated by Josh Durgin about 5 years ago

Another occurence today in teuthology:~teuthworker/archive/nightly_coverage_2011-12-20-b/4585/teuthology.log

#11 Updated by Sage Weil about 5 years ago

  • Priority changed from High to Normal

#12 Updated by Sage Weil about 5 years ago

  • Target version deleted (v0.40)
  • translation missing: en.field_position deleted (91)
  • translation missing: en.field_position set to 143

#13 Updated by Josh Durgin about 5 years ago

Happened again in teuthology:~teuthworker/archive/nightly_coverage_2012-01-09-b/6741/teuthology.log.

#14 Updated by Sage Weil almost 5 years ago

  • Status changed from New to Verified
  • Source set to Q/A

ubuntu@teuthology:/a/nightly_coverage_2012-03-12-b/727

#15 Updated by Sage Weil almost 5 years ago

ubuntu@teuthology:/a/nightly_coverage_2012-04-12-b/717

- chef: null
- ceph: null
- ceph-fuse: null
- workunit:
all:
- suites/dbench.sh

#16 Updated by Sage Weil almost 5 years ago

  • Target version set to v0.48

#17 Updated by Sage Weil almost 5 years ago

  • Target version deleted (v0.48)

#18 Updated by Sage Weil over 4 years ago

see ubuntu@teuthology:/a/teuthology-2012-06-17_19:00:03-regression-master-testing-gcov/1303 for a failure with logs!

#19 Updated by Sage Weil over 4 years ago

  • Status changed from Verified to Resolved

commit:d78235be1b5c20440f3a55abb3f4b980aa3f3596

#20 Updated by John Spray 5 months ago

  • Project changed from Ceph to fs
  • Category deleted (11)

Bulk updating project=ceph category=ceph-fuse issues to move to fs project so that we can remove the ceph-fuse category from the ceph project

Also available in: Atom PDF