On (01/12/13 20:27), Dave Jones wrote:
> BUG: sleeping function called from invalid context at mm/slub.c:925
> in_atomic(): 1, irqs_disabled(): 0, pid: 566, name: Xorg
> INFO: lockdep is turned off.
> Pid: 566, comm: Xorg Not tainted 3.8.0-rc3+ #49
> Call Trace:
> [<ffffffff81087ad1>] __might_sleep+0x141/0x200
> [<ffffffff8119eddb>] kmem_cache_alloc_trace+0x4b/0x2a0
> [<ffffffffa0083fb3>] ttm_bo_move_accel_cleanup+0x1d3/0x330 [ttm]
> [<ffffffffa00c5418>] radeon_move_blit.isra.4+0xf8/0x160 [radeon]
> [<ffffffffa00c5db0>] radeon_bo_move+0xb0/0x1f0 [radeon]
> [<ffffffffa00820ed>] ttm_bo_handle_move_mem+0x27d/0x5d0 [ttm]
> [<ffffffff8108dd51>] ? get_parent_ip+0x11/0x50
> [<ffffffffa0083307>] ttm_bo_move_buffer+0x127/0x140 [ttm]
> [<ffffffffa008717d>] ? ttm_eu_list_ref_sub+0x3d/0x60 [ttm]
> [<ffffffffa00833c2>] ttm_bo_validate+0xa2/0x120 [ttm]
> [<ffffffffa00c6f45>] radeon_bo_list_validate+0x75/0x90 [radeon]
> [<ffffffffa00dca22>] radeon_cs_ioctl+0x582/0x950 [radeon]
> [<ffffffffa002e573>] drm_ioctl+0x4d3/0x580 [drm]
> [<ffffffffa00dc4a0>] ? radeon_cs_finish_pages+0xf0/0xf0 [radeon]
> [<ffffffff811c49f9>] do_vfs_ioctl+0x99/0x5a0
> [<ffffffff812cd3d7>] ? file_has_perm+0x97/0xb0
> [<ffffffff81103715>] ? rcu_eqs_exit+0x65/0xb0
> [<ffffffff811c4f91>] sys_ioctl+0x91/0xb0
> [<ffffffff81695ad4>] tracesys+0xdd/0xe2
>
I see lots of these [mostly from page fault], the following one (quick and dirty) works for me.
----
Forbid ttm_buffer_object_transfer() sleeping in ISR context, do not pass
__GFP_WAIT flag to kzalloc():
BUG: sleeping function called from invalid context at mm/slub.c:925
in_atomic(): 1, irqs_disabled(): 0, pid: 2179, name: X
3 locks held by X/2179:
#0: (&mm->mmap_sem){++++++}, at: [<ffffffff815fac53>] __do_page_fault+0xf3/0x590
#1: (&rdev->pm.mclk_lock){++++.+}, at: [<ffffffff81428e40>] radeon_ttm_fault+0x40/0x70
#2: (&(&bdev->fence_lock)->rlock){+.+.+.}, at: [<ffffffff813f3227>] ttm_bo_move_accel_cleanup+0x57/0x330
Pid: 2179, comm: X Not tainted 3.8.0-rc3-dbg-01259-gbdac2d4-dirty #1500
Call Trace:
[<ffffffff8107adea>] __might_sleep+0x14a/0x200
[<ffffffff811649c3>] kmem_cache_alloc_trace+0x153/0x240
[<ffffffff813f33a3>] ttm_bo_move_accel_cleanup+0x1d3/0x330
[<ffffffff814292e8>] radeon_move_blit.isra.6+0xf8/0x160
[<ffffffff81429a90>] radeon_bo_move+0xb0/0x1f0
[<ffffffff813f14b5>] ttm_bo_handle_move_mem+0x275/0x5c0
[<ffffffff813f26f7>] ttm_bo_move_buffer+0x127/0x140
[<ffffffff813f27b2>] ttm_bo_validate+0xa2/0x130
[<ffffffff8142b0ed>] radeon_bo_fault_reserve_notify+0xad/0xe0
[<ffffffff813f4600>] ttm_bo_vm_fault+0x60/0x370
[<ffffffff81428e51>] radeon_ttm_fault+0x51/0x70
[<ffffffff8114150a>] __do_fault+0x6a/0x540
[<ffffffff81144771>] handle_pte_fault+0x91/0x810
[<ffffffff815fac53>] ? __do_page_fault+0xf3/0x590
[<ffffffff811464c1>] handle_mm_fault+0x231/0x330
[<ffffffff815facbc>] __do_page_fault+0x15c/0x590
[<ffffffff81184cd9>] ? do_vfs_ioctl+0x99/0x580
[<ffffffff815ff7c9>] ? sysret_check+0x22/0x5d
[<ffffffff815fb11b>] do_page_fault+0x2b/0x50
[<ffffffff815f79a8>] page_fault+0x28/0x30
Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@xxxxxxxxx>
---
drivers/gpu/drm/ttm/ttm_bo_util.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/ttm/ttm_bo_util.c b/drivers/gpu/drm/ttm/ttm_bo_util.c
index d73d6e3..92e0dea 100644
--- a/drivers/gpu/drm/ttm/ttm_bo_util.c
+++ b/drivers/gpu/drm/ttm/ttm_bo_util.c
@@ -422,7 +422,7 @@ static int ttm_buffer_object_transfer(struct ttm_buffer_object *bo,
struct ttm_bo_device *bdev = bo->bdev;
struct ttm_bo_driver *driver = bdev->driver;
- fbo = kzalloc(sizeof(*fbo), GFP_KERNEL);
+ fbo = kzalloc(sizeof(*fbo), __GFP_IO | __GFP_FS);
if (!fbo)
return -ENOMEM;
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel
