Bug #1737: ceph-fuse crash in xlist::remove - CephFS - Ceph

Bug #1737

ceph-fuse crash in xlist::remove

Added by Josh Durgin almost 12 years ago. Updated about 7 years ago.

Status:

Resolved

Priority:

Normal

Assignee:

Category:

Target version:

% Done:

Source:

Q/A

Tags:

Backport:

Regression:

Severity:

Reviewed:

Affected Versions:

ceph-qa-suite:

Component(FS):

Labels (FS):

Pull request ID:

Crash signature (v1):

Crash signature (v2):

Description

From teuthology:~teuthworker/archive/nightly_coverage_2011-11-18-2/2645/remote/ubuntu@sepia13.ceph.dreamhost.com/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 about 11 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 <sage@inktank.com>

Revision 5c5ca4b8 (diff)
Added by Sage Weil about 11 years ago

move cfuse + dbench from marginal to regression

Fixed #1737, yay!

History

#1 Updated by Sage Weil almost 12 years ago

translation missing: en.field_position set to 2

#2 Updated by Sage Weil almost 12 years ago

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

#3 Updated by Sage Weil almost 12 years ago

Assignee set to Sage Weil

#4 Updated by Sage Weil almost 12 years ago

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

#5 Updated by Sage Weil almost 12 years ago

Priority changed from Normal to High

#6 Updated by Sage Weil almost 12 years ago

Target version changed from v0.39 to v0.40

#7 Updated by Sage Weil almost 12 years ago

Assignee deleted (~~Sage Weil~~)

#8 Updated by Josh Durgin almost 12 years ago

This happened again from a different path in teuthology:~teuthworker/archive/nightly_coverage_2011-12-07-a/3843/remote/ubuntu@sepia47.ceph.dreamhost.com/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]