majianpeng a1dc193733 ceph: fix sleeping function called from invalid context. ... [ 1121.231883] BUG: sleeping function called from invalid context at kernel/rwsem.c:20 [ 1121.231935] in_atomic(): 1, irqs_disabled(): 0, pid: 9831, name: mv [ 1121.231971] 1 lock held by mv/9831: [ 1121.231973] #0: (&(&ci->i_ceph_lock)->rlock){+.+...},at:[<ffffffffa02bbd38>] ceph_getxattr+0x58/0x1d0 [ceph] [ 1121.231998] CPU: 3 PID: 9831 Comm: mv Not tainted 3.10.0-rc6+ #215 [ 1121.232000] Hardware name: To Be Filled By O.E.M. To Be Filled By O.E.M./To be filled by O.E.M., BIOS 080015 11/09/2011 [ 1121.232027] ffff88006d355a80 ffff880092f69ce0 ffffffff8168348c ffff880092f69cf8 [ 1121.232045] ffffffff81070435 ffff88006d355a20 ffff880092f69d20 ffffffff816899ba [ 1121.232052] 0000000300000004 ffff8800b76911d0 ffff88006d355a20 ffff880092f69d68 [ 1121.232056] Call Trace: [ 1121.232062] [<ffffffff8168348c>] dump_stack+0x19/0x1b [ 1121.232067] [<ffffffff81070435>] __might_sleep+0xe5/0x110 [ 1121.232071] [<ffffffff816899ba>] down_read+0x2a/0x98 [ 1121.232080] [<ffffffffa02baf70>] ceph_vxattrcb_layout+0x60/0xf0 [ceph] [ 1121.232088] [<ffffffffa02bbd7f>] ceph_getxattr+0x9f/0x1d0 [ceph] [ 1121.232093] [<ffffffff81188d28>] vfs_getxattr+0xa8/0xd0 [ 1121.232097] [<ffffffff8118900b>] getxattr+0xab/0x1c0 [ 1121.232100] [<ffffffff811704f2>] ? final_putname+0x22/0x50 [ 1121.232104] [<ffffffff81155f80>] ? kmem_cache_free+0xb0/0x260 [ 1121.232107] [<ffffffff811704f2>] ? final_putname+0x22/0x50 [ 1121.232110] [<ffffffff8109e63d>] ? trace_hardirqs_on+0xd/0x10 [ 1121.232114] [<ffffffff816957a7>] ? sysret_check+0x1b/0x56 [ 1121.232120] [<ffffffff81189c9c>] SyS_fgetxattr+0x6c/0xc0 [ 1121.232125] [<ffffffff81695782>] system_call_fastpath+0x16/0x1b [ 1121.232129] BUG: scheduling while atomic: mv/9831/0x10000002 [ 1121.232154] 1 lock held by mv/9831: [ 1121.232156] #0: (&(&ci->i_ceph_lock)->rlock){+.+...}, at: [<ffffffffa02bbd38>] ceph_getxattr+0x58/0x1d0 [ceph] I think move the ci->i_ceph_lock down is safe because we can't free ceph_inode_info at there. CC: stable@vger.kernel.org # 3.8+ Signed-off-by: Jianpeng Ma <majianpeng@gmail.com> Reviewed-by: Sage Weil <sage@inktank.com>		2013-07-03 15:32:51 -07:00
..
addr.c	ceph: fix race between page writeback and truncate	2013-07-03 15:32:47 -07:00
caps.c	ceph: move inode to proper flushing list when auth MDS changes	2013-07-03 15:32:50 -07:00
ceph_frag.c	ceph: factor out libceph from Ceph file system	2010-10-20 15:37:28 -07:00
debugfs.c	libceph: delay debugfs initialization until we learn global_id	2012-08-20 10:03:15 -07:00
dir.c	ceph: use i_release_count to indicate dir's completeness	2013-05-01 21:17:07 -07:00
export.c	fs: encode_fh: return FILEID_INVALID if invalid fid_type	2013-02-26 02:46:10 -05:00
file.c	aio: don't include aio.h in sched.h	2013-05-07 20:16:25 -07:00
inode.c	ceph: fix symlink inode operations	2013-05-01 21:18:50 -07:00
ioctl.c	libceph: rename ceph_calc_object_layout()	2013-05-01 21:16:17 -07:00
ioctl.h	ceph: fully initialize new layout	2012-05-16 14:28:27 -05:00
Kconfig	fs/ceph: remove depends on CONFIG_EXPERIMENTAL	2013-01-11 11:39:04 -08:00
locks.c	ceph: fix up comment for ceph_count_locks() as to which lock to hold	2013-07-01 09:52:01 -07:00
Makefile	ceph: Makefile: Remove unnessary code	2011-01-12 15:15:13 -08:00
mds_client.c	ceph: clear migrate seq when MDS restarts	2013-07-03 15:32:49 -07:00
mds_client.h	Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/sage/ceph-client	2013-02-28 17:43:09 -08:00
mdsmap.c	ceph: tidy ceph_mdsmap_decode() a little	2013-07-01 09:52:02 -07:00
snap.c	ceph: use ceph_create_snap_context()	2013-05-01 21:20:09 -07:00
strings.c	libceph: update ceph_mds_state_name() and ceph_mds_op_name()	2013-02-18 12:20:34 -06:00
super.c	ceph: set up page array mempool with correct size	2013-05-01 21:17:50 -07:00
super.h	ceph: ceph_pagelist_append might sleep while atomic	2013-05-17 12:45:48 -05:00
xattr.c	ceph: fix sleeping function called from invalid context.	2013-07-03 15:32:51 -07:00