From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Chris Wilson <chris@chris-wilson.co.uk>, intel-gfx@lists.freedesktop.org
Subject: Re: ✗ Fi.CI.BAT: failure for drm/i915: DSI and DPLL stuff for VLV/CHV mostly
Date: Fri, 1 Apr 2016 22:47:13 +0300 [thread overview]
Message-ID: <20160401194713.GZ4329@intel.com> (raw)
In-Reply-To: <20160401193922.GF5981@nuc-i3427.alporthouse.com>
On Fri, Apr 01, 2016 at 08:39:22PM +0100, Chris Wilson wrote:
> On Fri, Apr 01, 2016 at 10:12:09PM +0300, Ville Syrjälä wrote:
> > On Tue, Mar 15, 2016 at 03:27:30PM -0000, Patchwork wrote:
> > > == Series Details ==
> > >
> > > Series: drm/i915: DSI and DPLL stuff for VLV/CHV mostly
> > > URL : https://patchwork.freedesktop.org/series/4472/
> > > State : failure
> > >
> > > == Summary ==
> > >
> > > Series 4472v1 drm/i915: DSI and DPLL stuff for VLV/CHV mostly
> > > http://patchwork.freedesktop.org/api/1.0/series/4472/revisions/1/mbox/
> > >
> > > Test gem_ringfill:
> > > Subgroup basic-default-s3:
> > > pass -> DMESG-WARN (bsw-nuc-2)
> >
> > lockdep splat
> > https://bugs.freedesktop.org/show_bug.cgi?id=94350
> >
> > already fixed it seems.
>
> Absolutely sure it is the same one? The GEM busy suspend tests are meant
> to be provoking a very similar lockdep splat!
>
> Can you paste it here?
I thought I had, but apparently not.
[ 517.449665] ======================================================
[ 517.449667] [ INFO: possible circular locking dependency detected ]
[ 517.449671] 4.5.0-gfxbench+ #1 Not tainted
[ 517.449672] -------------------------------------------------------
[ 517.449675] rtcwake/7020 is trying to acquire lock:
[ 517.449692] (s_active#43){++++.+}, at: [<ffffffff81250740>] kernfs_remove_by_name_ns+0x40/0xa0
[ 517.449693]
but task is already holding lock:
[ 517.449703] (cpu_hotplug.lock){+.+.+.}, at: [<ffffffff81078f5d>] cpu_hotplug_begin+0x6d/0xc0
[ 517.449704]
which lock already depends on the new lock.
[ 517.449705]
the existing dependency chain (in reverse order) is:
[ 517.449711]
-> #3 (cpu_hotplug.lock){+.+.+.}:
[ 517.449717] [<ffffffff810cd7e1>] lock_acquire+0xb1/0x200
[ 517.449724] [<ffffffff817c1412>] mutex_lock_nested+0x62/0x3b0
[ 517.449729] [<ffffffff81078c21>] get_online_cpus+0x61/0x80
[ 517.449736] [<ffffffff81118d0b>] stop_machine+0x1b/0xe0
[ 517.449742] [<ffffffffa015975d>] 0xffffffffa015975d
[ 517.449746] [<ffffffffa015ddc6>] 0xffffffffa015ddc6
[ 517.449749] [<ffffffffa015f5af>] 0xffffffffa015f5af
[ 517.449752] [<ffffffffa0167031>] 0xffffffffa0167031
[ 517.449755] [<ffffffffa01672e8>] 0xffffffffa01672e8
[ 517.449758] [<ffffffffa017b03e>] 0xffffffffa017b03e
[ 517.449761] [<ffffffffa0178052>] 0xffffffffa0178052
[ 517.449764] [<ffffffffa0167db3>] 0xffffffffa0167db3
[ 517.449767] [<ffffffffa01ec5c1>] 0xffffffffa01ec5c1
[ 517.449773] [<ffffffff815199b4>] drm_dev_register+0xa4/0xb0
[ 517.449779] [<ffffffff8151bbce>] drm_get_pci_dev+0xce/0x1e0
[ 517.449782] [<ffffffffa01272ff>] 0xffffffffa01272ff
[ 517.449789] [<ffffffff81446657>] pci_device_probe+0x87/0xf0
[ 517.449794] [<ffffffff8153d729>] driver_probe_device+0x229/0x450
[ 517.449798] [<ffffffff8153d9d3>] __driver_attach+0x83/0x90
[ 517.449804] [<ffffffff8153b401>] bus_for_each_dev+0x61/0xa0
[ 517.449808] [<ffffffff8153d019>] driver_attach+0x19/0x20
[ 517.449812] [<ffffffff8153caff>] bus_add_driver+0x1ef/0x290
[ 517.449816] [<ffffffff8153e6fb>] driver_register+0x5b/0xe0
[ 517.449822] [<ffffffff8144558b>] __pci_register_driver+0x5b/0x60
[ 517.449827] [<ffffffff8151bdb6>] drm_pci_init+0xd6/0x100
[ 517.449830] [<ffffffffa0260094>] 0xffffffffa0260094
[ 517.449837] [<ffffffff810003de>] do_one_initcall+0xae/0x1d0
[ 517.449844] [<ffffffff8115b415>] do_init_module+0x5b/0x1c6
[ 517.449850] [<ffffffff81107d3a>] load_module+0x1c0a/0x24b0
[ 517.449853] [<ffffffff811087ce>] SyS_finit_module+0x7e/0xa0
[ 517.449859] [<ffffffff817c5cdb>] entry_SYSCALL_64_fastpath+0x16/0x73
[ 517.449864]
-> #2 (&dev->struct_mutex){+.+.+.}:
[ 517.449869] [<ffffffff810cd7e1>] lock_acquire+0xb1/0x200
[ 517.449875] [<ffffffff81514ec7>] drm_gem_mmap+0x1c7/0x270
[ 517.449882] [<ffffffff81199354>] mmap_region+0x334/0x580
[ 517.449886] [<ffffffff81199904>] do_mmap+0x364/0x410
[ 517.449891] [<ffffffff8117d8ed>] vm_mmap_pgoff+0x6d/0xa0
[ 517.449895] [<ffffffff81197a34>] SyS_mmap_pgoff+0x184/0x220
[ 517.449900] [<ffffffff8100a28d>] SyS_mmap+0x1d/0x20
[ 517.449905] [<ffffffff817c5cdb>] entry_SYSCALL_64_fastpath+0x16/0x73
[ 517.449910]
-> #1 (&mm->mmap_sem){++++++}:
[ 517.449914] [<ffffffff810cd7e1>] lock_acquire+0xb1/0x200
[ 517.449919] [<ffffffff8118e575>] __might_fault+0x75/0xa0
[ 517.449923] [<ffffffff8125114a>] kernfs_fop_write+0x8a/0x180
[ 517.449928] [<ffffffff811d3ce3>] __vfs_write+0x23/0xe0
[ 517.449933] [<ffffffff811d4a44>] vfs_write+0xa4/0x190
[ 517.449937] [<ffffffff811d58e4>] SyS_write+0x44/0xb0
[ 517.449941] [<ffffffff817c5cdb>] entry_SYSCALL_64_fastpath+0x16/0x73
[ 517.449947]
-> #0 (s_active#43){++++.+}:
[ 517.449951] [<ffffffff810ccdc9>] __lock_acquire+0x1fc9/0x20f0
[ 517.449954] [<ffffffff810cd7e1>] lock_acquire+0xb1/0x200
[ 517.449958] [<ffffffff8124f770>] __kernfs_remove+0x210/0x2f0
[ 517.449962] [<ffffffff81250740>] kernfs_remove_by_name_ns+0x40/0xa0
[ 517.449968] [<ffffffff81252aed>] sysfs_unmerge_group+0x3d/0x60
[ 517.449974] [<ffffffff81545524>] dpm_sysfs_remove+0x34/0x60
[ 517.449980] [<ffffffff81538f94>] device_del+0x44/0x250
[ 517.449983] [<ffffffff815391b9>] device_unregister+0x19/0x60
[ 517.449989] [<ffffffff815438e1>] cpu_cache_sysfs_exit+0x51/0xb0
[ 517.449993] [<ffffffff81543eb8>] cacheinfo_cpu_callback+0x38/0x70
[ 517.449999] [<ffffffff8109ba09>] notifier_call_chain+0x39/0xa0
[ 517.450003] [<ffffffff8109ba79>] __raw_notifier_call_chain+0x9/0x10
[ 517.450007] [<ffffffff81078e3e>] cpu_notify+0x1e/0x40
[ 517.450011] [<ffffffff81078ed9>] cpu_notify_nofail+0x9/0x20
[ 517.450014] [<ffffffff81079223>] _cpu_down+0x233/0x340
[ 517.450020] [<ffffffff81079763>] disable_nonboot_cpus+0xb3/0x3b0
[ 517.450027] [<ffffffff810d3f74>] suspend_devices_and_enter+0x5b4/0xc50
[ 517.450032] [<ffffffff810d4b41>] pm_suspend+0x531/0x9f0
[ 517.450036] [<ffffffff810d2c57>] state_store+0x77/0xe0
[ 517.450043] [<ffffffff8140109f>] kobj_attr_store+0xf/0x20
[ 517.450047] [<ffffffff81251e40>] sysfs_kf_write+0x40/0x50
[ 517.450051] [<ffffffff812511fc>] kernfs_fop_write+0x13c/0x180
[ 517.450055] [<ffffffff811d3ce3>] __vfs_write+0x23/0xe0
[ 517.450060] [<ffffffff811d4a44>] vfs_write+0xa4/0x190
[ 517.450063] [<ffffffff811d58e4>] SyS_write+0x44/0xb0
[ 517.450068] [<ffffffff817c5cdb>] entry_SYSCALL_64_fastpath+0x16/0x73
[ 517.450070]
other info that might help us debug this:
[ 517.450078] Chain exists of:
s_active#43 --> &dev->struct_mutex --> cpu_hotplug.lock
[ 517.450079] Possible unsafe locking scenario:
[ 517.450080] CPU0 CPU1
[ 517.450081] ---- ----
[ 517.450084] lock(cpu_hotplug.lock);
[ 517.450087] lock(&dev->struct_mutex);
[ 517.450089] lock(cpu_hotplug.lock);
[ 517.450093] lock(s_active#43);
[ 517.450094]
*** DEADLOCK ***
[ 517.450098] 8 locks held by rtcwake/7020:
[ 517.450108] #0: (sb_writers#6){.+.+.+}, at: [<ffffffff811d8074>] __sb_start_write+0xd4/0xf0
[ 517.450116] #1: (&of->mutex){+.+.+.}, at: [<ffffffff81251121>] kernfs_fop_write+0x61/0x180
[ 517.450125] #2: (s_active#221){.+.+.+}, at: [<ffffffff81251129>] kernfs_fop_write+0x69/0x180
[ 517.450133] #3: (pm_mutex){+.+...}, at: [<ffffffff810d4686>] pm_suspend+0x76/0x9f0
[ 517.450143] #4: (acpi_scan_lock){+.+.+.}, at: [<ffffffff814798d9>] acpi_scan_lock_acquire+0x12/0x14
[ 517.450151] #5: (cpu_add_remove_lock){+.+.+.}, at: [<ffffffff810796d4>] disable_nonboot_cpus+0x24/0x3b0
[ 517.450159] #6: (cpu_hotplug.dep_map){++++++}, at: [<ffffffff81078ef0>] cpu_hotplug_begin+0x0/0xc0
[ 517.450166] #7: (cpu_hotplug.lock){+.+.+.}, at: [<ffffffff81078f5d>] cpu_hotplug_begin+0x6d/0xc0
[ 517.450167]
stack backtrace:
[ 517.450173] CPU: 0 PID: 7020 Comm: rtcwake Not tainted 4.5.0-gfxbench+ #1
[ 517.450174] Hardware name: /NUC5CPYB, BIOS PYBSWCEL.86A.0040.2015.0814.1353 08/14/2015
[ 517.450181] 0000000000000000 ffff88016638f820 ffffffff813fef15 ffffffff825f70a0
[ 517.450187] ffffffff825a0220 ffff88016638f860 ffffffff810c941c ffff88016638f8c0
[ 517.450192] ffff88017a8b8940 ffff88017a8b8000 0000000000000008 ffff88017a8b8968
[ 517.450194] Call Trace:
[ 517.450198] [<ffffffff813fef15>] dump_stack+0x67/0x92
[ 517.450202] [<ffffffff810c941c>] print_circular_bug+0x1fc/0x310
[ 517.450206] [<ffffffff810ccdc9>] __lock_acquire+0x1fc9/0x20f0
[ 517.450210] [<ffffffff810cd7e1>] lock_acquire+0xb1/0x200
[ 517.450215] [<ffffffff81250740>] ? kernfs_remove_by_name_ns+0x40/0xa0
[ 517.450218] [<ffffffff8124f770>] __kernfs_remove+0x210/0x2f0
[ 517.450222] [<ffffffff81250740>] ? kernfs_remove_by_name_ns+0x40/0xa0
[ 517.450225] [<ffffffff8124f988>] ? kernfs_find_ns+0x78/0x130
[ 517.450228] [<ffffffff81250740>] kernfs_remove_by_name_ns+0x40/0xa0
[ 517.450232] [<ffffffff81252aed>] sysfs_unmerge_group+0x3d/0x60
[ 517.450236] [<ffffffff81545524>] dpm_sysfs_remove+0x34/0x60
[ 517.450239] [<ffffffff81538f94>] device_del+0x44/0x250
[ 517.450242] [<ffffffff815391b9>] device_unregister+0x19/0x60
[ 517.450246] [<ffffffff815438e1>] cpu_cache_sysfs_exit+0x51/0xb0
[ 517.450249] [<ffffffff81543eb8>] cacheinfo_cpu_callback+0x38/0x70
[ 517.450253] [<ffffffff8109ba09>] notifier_call_chain+0x39/0xa0
[ 517.450256] [<ffffffff8109ba79>] __raw_notifier_call_chain+0x9/0x10
[ 517.450259] [<ffffffff81078e3e>] cpu_notify+0x1e/0x40
[ 517.450262] [<ffffffff81078ed9>] cpu_notify_nofail+0x9/0x20
[ 517.450265] [<ffffffff81079223>] _cpu_down+0x233/0x340
[ 517.450271] [<ffffffff810e5310>] ? __call_rcu.constprop.61+0x2f0/0x2f0
[ 517.450274] [<ffffffff810e5370>] ? call_rcu_bh+0x20/0x20
[ 517.450280] [<ffffffff810e0e00>] ? trace_raw_output_rcu_utilization+0x60/0x60
[ 517.450285] [<ffffffff810e0e00>] ? trace_raw_output_rcu_utilization+0x60/0x60
[ 517.450289] [<ffffffff81079763>] disable_nonboot_cpus+0xb3/0x3b0
[ 517.450293] [<ffffffff810d3f74>] suspend_devices_and_enter+0x5b4/0xc50
[ 517.450298] [<ffffffff810c80a9>] ? __lock_is_held+0x49/0x70
[ 517.450302] [<ffffffff810d4b41>] pm_suspend+0x531/0x9f0
[ 517.450306] [<ffffffff810d2c57>] state_store+0x77/0xe0
[ 517.450309] [<ffffffff8140109f>] kobj_attr_store+0xf/0x20
[ 517.450313] [<ffffffff81251e40>] sysfs_kf_write+0x40/0x50
[ 517.450316] [<ffffffff812511fc>] kernfs_fop_write+0x13c/0x180
[ 517.450320] [<ffffffff811d3ce3>] __vfs_write+0x23/0xe0
[ 517.450325] [<ffffffff810c6aa2>] ? percpu_down_read+0x52/0x90
[ 517.450328] [<ffffffff811d8074>] ? __sb_start_write+0xd4/0xf0
[ 517.450331] [<ffffffff811d8074>] ? __sb_start_write+0xd4/0xf0
[ 517.450334] [<ffffffff811d4a44>] vfs_write+0xa4/0x190
[ 517.450340] [<ffffffff811f324a>] ? __fget_light+0x6a/0x90
[ 517.450343] [<ffffffff811d58e4>] SyS_write+0x44/0xb0
[ 517.450347] [<ffffffff817c5cdb>] entry_SYSCALL_64_fastpath+0x16/0x73
--
Ville Syrjälä
Intel OTC
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2016-04-01 19:47 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-15 14:39 [PATCH 00/16] drm/i915: DSI and DPLL stuff for VLV/CHV mostly ville.syrjala
2016-03-15 14:39 ` [PATCH 01/16] drm/i915: Throw out BUGs from DPLL/PCH functions ville.syrjala
2016-03-16 9:02 ` Jani Nikula
2016-03-15 14:39 ` [PATCH 02/16] drm/i915: Make {vlv, chv}_{disable, update}_pll() more similar ville.syrjala
2016-03-30 13:31 ` Jani Nikula
2016-04-01 19:59 ` Ville Syrjälä
2016-03-15 14:39 ` [PATCH 03/16] drm/i915: Implement WaPixelRepeatModeFixForC0:chv ville.syrjala
2016-03-16 9:27 ` Jani Nikula
2016-03-16 13:07 ` Ville Syrjälä
2016-03-15 14:39 ` [PATCH 04/16] drm/i915: Add a local pipe variable to vlv_enable_pll() ville.syrjala
2016-03-16 9:03 ` Jani Nikula
2016-03-15 14:39 ` [PATCH 05/16] drm/i915: assert_panel_unlocked() in chv_enable_pll() ville.syrjala
2016-03-16 9:04 ` Jani Nikula
2016-03-15 14:39 ` [PATCH 06/16] drm/i915: Remove the "three times for luck" trick from vlv_enable_pll() ville.syrjala
2016-03-16 9:05 ` Jani Nikula
2016-04-01 19:58 ` Ville Syrjälä
2016-03-15 14:40 ` [PATCH 07/16] drm/i915: Setup DPLL/DPLLMD for DSI too on VLV/CHV ville.syrjala
2016-03-15 14:40 ` [PATCH 08/16] drm/i915: Don't read out port_clock on CHV when DPLL is disabled ville.syrjala
2016-03-16 9:06 ` Jani Nikula
2016-03-15 14:40 ` [PATCH 09/16] drm/i915: Change lfsr_converts[] to u16 ville.syrjala
2016-03-16 8:42 ` Jani Nikula
2016-03-15 14:40 ` [PATCH 10/16] drm/i915: Power down the DSI PLL before reconfiguring it ville.syrjala
2016-03-16 8:45 ` Jani Nikula
2016-03-15 14:40 ` [PATCH 11/16] drm/i915: Compute DSI PLL parameters during .compute_config() ville.syrjala
2016-03-16 8:56 ` Jani Nikula
2016-03-16 12:59 ` Ville Syrjälä
2016-03-15 14:40 ` [PATCH 12/16] drm/i915: Fix CHV DSI PLL refclk during state readout ville.syrjala
2016-03-16 8:58 ` Jani Nikula
2016-03-15 14:40 ` [PATCH 13/16] drm/i915: Eliminate {vlv, bxt}_configure_dsi_pll() ville.syrjala
2016-03-16 8:59 ` Jani Nikula
2016-03-15 14:40 ` [PATCH 14/16] drm/i915: Dump pfit PGM_RATIOS as hex ville.syrjala
2016-03-16 9:00 ` Jani Nikula
2016-03-15 14:40 ` [PATCH 15/16] drm/i915: Hook up pfit for DSI ville.syrjala
2016-03-30 13:35 ` Jani Nikula
2016-03-15 14:40 ` [PATCH 16/16] drm/i915: Reject 'Center' scaling mode for eDP/DSI on GMCH platforms ville.syrjala
2016-03-30 13:36 ` Jani Nikula
2016-03-15 15:27 ` ✗ Fi.CI.BAT: failure for drm/i915: DSI and DPLL stuff for VLV/CHV mostly Patchwork
2016-04-01 19:12 ` Ville Syrjälä
2016-04-01 19:39 ` Chris Wilson
2016-04-01 19:47 ` Ville Syrjälä [this message]
2016-04-01 19:53 ` Chris Wilson
2016-04-12 18:39 ` [PATCH 00/16] " Ville Syrjälä
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160401194713.GZ4329@intel.com \
--to=ville.syrjala@linux.intel.com \
--cc=chris@chris-wilson.co.uk \
--cc=intel-gfx@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.