Hi Dave,
Below is a qxl lockdep spew that I got with Linus' tree a few days
ago. I actually suspect this is present in older kernels too as the
DRM merge hasn't landed yet. If you have any ideas, I'd appreciate
it. I tried to look at the locking a bit, but I'm not familiar with
the ww ticket locking.
josh
[ +8.261799] ======================================================
[ +0.000001] [ INFO: possible circular locking dependency detected ]
[ +0.000001] 4.1.0-0.rc0.git2.1.fc23.x86_64 #1 Not tainted
[ +0.000001] -------------------------------------------------------
[ +0.000001] gnome-shell/1012 is trying to acquire lock:
[ +0.000001] (reservation_ww_class_acquire){+.+.+.}, at:
[<ffffffffa010a635>] qxl_release_reserve_list+0x55/0x120 [qxl]
[ +0.000008]
but task is already holding lock:
[ +0.000001] (reservation_ww_class_mutex){+.+.+.}, at:
[<ffffffffa01017cb>] qxl_crtc_page_flip+0xbb/0x230 [qxl]
[ +0.000004]
which lock already depends on the new lock.
[ +0.000002]
the existing dependency chain (in reverse order) is:
[ +0.000001]
-> #1 (reservation_ww_class_mutex){+.+.+.}:
[ +0.000002] [<ffffffff811129f7>] lock_acquire+0xc7/0x2a0
[ +0.000005] [<ffffffff81888acf>] __ww_mutex_lock+0x7f/0x770
[ +0.000004] [<ffffffffa0090c35>]
ttm_eu_reserve_buffers+0x365/0x640 [ttm]
[ +0.000007] [<ffffffffa010a635>]
qxl_release_reserve_list+0x55/0x120 [qxl]
[ +0.000005] [<ffffffffa0108026>] qxl_draw_opaque_fb+0xe6/0x3c0 [qxl]
[ +0.000005] [<ffffffffa0103ffd>]
qxl_fb_imageblit_internal+0x4d/0x70 [qxl]
[ +0.000018] [<ffffffffa0104687>] qxl_fb_imageblit+0x1a7/0x1c0 [qxl]
[ +0.000006] [<ffffffff814aea6c>] soft_cursor+0x1ac/0x230
[ +0.000003] [<ffffffff814ae2fb>] bit_cursor+0x68b/0x6c0
[ +0.000002] [<ffffffff814aa2fd>] fbcon_cursor+0x10d/0x180
[ +0.000001] [<ffffffff8154808c>] hide_cursor+0x2c/0x90
[ +0.000004] [<ffffffff81549e78>] redraw_screen+0x188/0x250
[ +0.000003] [<ffffffff8154a883>] vc_do_resize+0x4c3/0x4f0
[ +0.000003] [<ffffffff8154a8cf>] vc_resize+0x1f/0x30
[ +0.000003] [<ffffffff814aca6c>] fbcon_init+0x3cc/0x610
[ +0.000002] [<ffffffff815482dc>] visual_init+0xbc/0x120
[ +0.000003] [<ffffffff8154acce>] do_bind_con_driver+0x17e/0x3c0
[ +0.000003] [<ffffffff8154b474>] do_take_over_console+0xb4/0x1b0
[ +0.000003] [<ffffffff814a7a23>] do_fbcon_takeover+0x63/0xd0
[ +0.000004] [<ffffffff814ad77d>] fbcon_event_notify+0x6dd/0x7e0
[ +0.000003] [<ffffffff810d9b6e>] notifier_call_chain+0x3e/0xb0
[ +0.000003] [<ffffffff810d9df1>]
__blocking_notifier_call_chain+0x51/0x70
[ +0.000003] [<ffffffff810d9e26>] blocking_notifier_call_chain+0x16/0x20
[ +0.000003] [<ffffffff814b381b>] fb_notifier_call_chain+0x1b/0x20
[ +0.000004] [<ffffffff814b5cc2>] register_framebuffer+0x222/0x370
[ +0.000003] [<ffffffffa00e6ae4>]
drm_fb_helper_initial_config+0x264/0x3c0 [drm_kms_helper]
[ +0.000008] [<ffffffffa0105214>] qxl_fbdev_init+0xd4/0x100 [qxl]
[ +0.000005] [<ffffffffa0103339>] qxl_modeset_init+0x229/0x260 [qxl]
[ +0.000013] [<ffffffffa0100d01>] qxl_driver_load+0x811/0x840 [qxl]
[ +0.000003] [<ffffffffa003dfc5>] drm_dev_register+0xb5/0x110 [drm]
[ +0.000008] [<ffffffffa004122d>] drm_get_pci_dev+0x8d/0x200 [drm]
[ +0.000006] [<ffffffffa01002fb>] qxl_pci_probe+0x1b/0x40 [qxl]
[ +0.000003] [<ffffffff81484425>] local_pci_probe+0x45/0xa0
[ +0.000002] [<ffffffff81485779>] pci_device_probe+0xf9/0x150
[ +0.000017] [<ffffffff81592699>] driver_probe_device+0x209/0x4b0
[ +0.000004] [<ffffffff81592a23>] __driver_attach+0xa3/0xb0
[ +0.000001] [<ffffffff815900a3>] bus_for_each_dev+0x73/0xc0
[ +0.000002] [<ffffffff81591f4e>] driver_attach+0x1e/0x20
[ +0.000002] [<ffffffff81591ae8>] bus_add_driver+0x188/0x260
[ +0.000002] [<ffffffff81593854>] driver_register+0x64/0xf0
[ +0.000002] [<ffffffff81483ca4>] __pci_register_driver+0x64/0x70
[ +0.000002] [<ffffffffa004149a>] drm_pci_init+0xfa/0x130 [drm]
[ +0.000007] [<ffffffffa00b1048>]
acpi_cpufreq_resume+0x28/0x70 [acpi_cpufreq]
[ +0.000004] [<ffffffff81002148>] do_one_initcall+0xd8/0x210
[ +0.000004] [<ffffffff81881e11>] do_init_module+0x61/0x1ce
[ +0.000003] [<ffffffff8115daf9>] load_module+0x2359/0x2710
[ +0.000003] [<ffffffff8115dff5>] SyS_init_module+0x145/0x190
[ +0.000002] [<ffffffff8188caee>] system_call_fastpath+0x12/0x76
[ +0.000003]
-> #0 (reservation_ww_class_acquire){+.+.+.}:
[ +0.000002] [<ffffffff81111bb9>] __lock_acquire+0x1ce9/0x2050
[ +0.000002] [<ffffffff811129f7>] lock_acquire+0xc7/0x2a0
[ +0.000002] [<ffffffffa009099c>]
ttm_eu_reserve_buffers+0xcc/0x640 [ttm]
[ +0.000004] [<ffffffffa010a635>]
qxl_release_reserve_list+0x55/0x120 [qxl]
[ +0.000003] [<ffffffffa0108484>] qxl_draw_dirty_fb+0x184/0x480 [qxl]
[ +0.000003] [<ffffffffa0101810>] qxl_crtc_page_flip+0x100/0x230 [qxl]
[ +0.000003] [<ffffffffa004af5b>]
drm_mode_page_flip_ioctl+0x1ab/0x360 [drm]
[ +0.000007] [<ffffffffa0039c9f>] drm_ioctl+0x1df/0x6a0 [drm]
[ +0.000005] [<ffffffff81297668>] do_vfs_ioctl+0x308/0x560
[ +0.000003] [<ffffffff81297941>] SyS_ioctl+0x81/0xa0
[ +0.000002] [<ffffffff8188caee>] system_call_fastpath+0x12/0x76
[ +0.000002]
other info that might help us debug this:
[ +0.000001] Possible unsafe locking scenario:
[ +0.000001] CPU0 CPU1
[ +0.000001] ---- ----
[ +0.000001] lock(reservation_ww_class_mutex);
[ +0.000003] lock(reservation_ww_class_acquire);
[ +0.000002] lock(reservation_ww_class_mutex);
[ +0.000002] lock(reservation_ww_class_acquire);
[ +0.000003]
*** DEADLOCK ***
[ +0.000003] 3 locks held by gnome-shell/1012:
[ +0.000001] #0: (crtc_ww_class_acquire){+.+.+.}, at:
[<ffffffffa00554fc>] drm_modeset_lock_crtc+0x4c/0x110 [drm]
[ +0.000038] #1: (crtc_ww_class_mutex){+.+.+.}, at:
[<ffffffffa005531d>] drm_modeset_lock+0x3d/0x110 [drm]
[ +0.000013] #2: (reservation_ww_class_mutex){+.+.+.}, at:
[<ffffffffa01017cb>] qxl_crtc_page_flip+0xbb/0x230 [qxl]
[ +0.000007]
stack backtrace:
[ +0.000003] CPU: 0 PID: 1012 Comm: gnome-shell Not tainted
4.1.0-0.rc0.git2.1.fc23.x86_64 #1
[ +0.000001] Hardware name: Bochs Bochs, BIOS Bochs 01/01/2011
[ +0.000000] 0000000000000000 00000000139e8b9c ffff88002c54f918
ffffffff81882fc7
[ +0.000000] 0000000000000000 ffffffff82c0f040 ffff88002c54f968
ffffffff8110e37d
[ +0.000000] 00000000001d9640 ffff88002c54f9d8 ffff88002e82dbf8
0000000000000003
[ +0.000000] Call Trace:
[ +0.000000] [<ffffffff81882fc7>] dump_stack+0x4c/0x65
[ +0.000000] [<ffffffff8110e37d>] print_circular_bug+0x1cd/0x230
[ +0.000000] [<ffffffff81111bb9>] __lock_acquire+0x1ce9/0x2050
[ +0.000000] [<ffffffff81889a23>] ? __mutex_unlock_slowpath+0xc3/0x1c0
[ +0.000000] [<ffffffff811129f7>] lock_acquire+0xc7/0x2a0
[ +0.000000] [<ffffffffa010a635>] ? qxl_release_reserve_list+0x55/0x120 [qxl]
[ +0.000000] [<ffffffff8110fa8d>] ? trace_hardirqs_on_caller+0x13d/0x1e0
[ +0.000000] [<ffffffffa009099c>] ttm_eu_reserve_buffers+0xcc/0x640 [ttm]
[ +0.000000] [<ffffffffa010a635>] ? qxl_release_reserve_list+0x55/0x120 [qxl]
[ +0.000000] [<ffffffffa010a635>] qxl_release_reserve_list+0x55/0x120 [qxl]
[ +0.000000] [<ffffffffa0108484>] qxl_draw_dirty_fb+0x184/0x480 [qxl]
[ +0.000000] [<ffffffff810ee618>] ? sched_clock_cpu+0x98/0xc0
[ +0.000000] [<ffffffffa0101810>] qxl_crtc_page_flip+0x100/0x230 [qxl]
[ +0.000000] [<ffffffffa004af5b>] drm_mode_page_flip_ioctl+0x1ab/0x360 [drm]
[ +0.000000] [<ffffffffa0039c9f>] drm_ioctl+0x1df/0x6a0 [drm]
[ +0.000000] [<ffffffff813b1935>] ? inode_has_perm.isra.48+0x55/0xa0
[ +0.000000] [<ffffffff81297668>] do_vfs_ioctl+0x308/0x560
[ +0.000000] [<ffffffff81297941>] SyS_ioctl+0x81/0xa0
[ +0.000000] [<ffffffff8188caee>] system_call_fastpath+0x12/0x76
