Bug 112127

Summary:	[CI][BAT]igt@i915_selftest@live_client - dmesg-warn - INFO: trying to register non-static key, the code is fine but needs lockdep annotation.
Product:	DRI	Reporter:	Lakshmi <lakshminarayana.vudum>
Component:	DRM/Intel	Assignee:	Intel GFX Bugs mailing list <intel-gfx-bugs>
Status:	RESOLVED FIXED	QA Contact:	Intel GFX Bugs mailing list <intel-gfx-bugs>
Severity:	not set
Priority:	not set	CC:	intel-gfx-bugs
Version:	DRI git
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:	PNV	i915 features:	GEM/Other

Description Lakshmi 2019-10-25 05:53:32 UTC

https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7175/fi-pnv-d510/igt@i915_selftest@live_client.html

<6> [522.712263] [drm] Initialized overlay support.
<7> [522.712649] [drm:intel_engines_driver_register [i915]] renamed rcs'0 to rcs0
<3> [522.712913] INFO: trying to register non-static key.
<3> [522.712943] the code is fine but needs lockdep annotation.
<3> [522.712965] turning off the locking correctness validator.
<4> [522.712989] CPU: 3 PID: 27 Comm: kworker/3:0 Tainted: G     U            5.4.0-rc4-CI-CI_DRM_7175+ #1
<4> [522.713017] Hardware name:  /D510MO, BIOS MOPNV10J.86A.0311.2010.0802.2346 08/02/2010
<4> [522.713018] Workqueue: events __intel_wakeref_put_work [i915]
<4> [522.713018] Call Trace:
<4> [522.713018]  dump_stack+0x67/0x9b
<4> [522.713018]  register_lock_class+0x5d0/0x5e0
<4> [522.713018]  ? add_lock_to_list.isra.27+0x9c/0x120
<4> [522.713018]  __lock_acquire+0x75/0x15d0
<4> [522.713018]  lock_acquire+0xa7/0x1c0
<4> [522.713018]  ? i915_pmu_gt_parked+0x1e/0x80 [i915]
<4> [522.713018]  _raw_spin_lock_irq+0x30/0x40
<4> [522.713018]  ? i915_pmu_gt_parked+0x1e/0x80 [i915]
<4> [522.713018]  i915_pmu_gt_parked+0x1e/0x80 [i915]
<4> [522.713018]  __gt_park+0x4e/0xe0 [i915]
<4> [522.713018]  ____intel_wakeref_put_last+0x1c/0x70 [i915]
<4> [522.713018]  process_one_work+0x26a/0x620
<4> [522.713018]  worker_thread+0x37/0x380
<4> [522.713018]  ? process_one_work+0x620/0x620
<4> [522.713018]  kthread+0x119/0x130
<4> [522.713018]  ? kthread_park+0x80/0x80
<4> [522.713018]  ret_from_fork+0x24/0x50
<6> [522.726545] [drm] Initialized i915 1.6.0 20191021 for 0000:00:02.0 on minor 0
<7> [522.731998] [drm:intel_power_well_disable [i915]] disabling always-on
<7> [522.732299] i915 device info: pciid=0xa001 rev=0x02 platform=PINEVIEW (subplatform=0x0) gen=3
<7> [522.732360] i915 device info: is_mobile: no
<7> [522.732396] i915 device info: is_lp: no
<7> [522.732432] i915 device info: require_force_probe: no
<7> [522.732473] i915 device info: has_64bit_reloc: no
<7> [522.732514] i915 device info: gpu_reset_clobbers_display: yes
<7> [522.732559] i915 device info: has_reset_engine: no
<7> [522.732598] i915 device info: has_fpga_dbg: no
<7> [522.732635] i915 device info: has_global_mocs: no
<7> [522.732675] i915 device info: has_gt_uc: no
<7> [522.732712] i915 device info: has_l3_dpf: no
<7> [522.732750] i915 device info: has_llc: no
<7> [522.732787] i915 device info: has_logical_ring_contexts: no
<7> [522.732830] i915 device info: has_logical_ring_elsq: no
<7> [522.732874] i915 device info: has_logical_ring_preemption: no
<7> [522.732920] i915 device info: has_pooled_eu: no
<7> [522.732960] i915 device info: has_rc6: no
<7> [522.733280] i915 device info: has_rc6p: no
<7> [522.733322] i915 device info: has_rps: no
<7> [522.733359] i915 device info: has_runtime_pm: no
<7> [522.733401] i915 device info: has_snoop: yes
<7> [522.733437] i915 device info: has_coherent_ggtt: yes
<7> [522.733476] i915 device info: unfenced_needs_alignment: no
<7> [522.733518] i915 device info: hws_needs_physical: no
<7> [522.733557] i915 device info: cursor_needs_physical: no
<7> [522.733597] i915 device info: has_csr: no
<7> [522.733631] i915 device info: has_ddi: no
<7> [522.733672] i915 device info: has_dp_mst: no
<7> [522.733710] i915 device info: has_dsb: no
<7> [522.733744] i915 device info: has_fbc: no
<7> [522.733778] i915 device info: has_gmch: yes
<7> [522.733813] i915 device info: has_hotplug: yes
<7> [522.733851] i915 device info: has_ipc: no
<7> [522.733894] i915 device info: has_modular_fia: no
<7> [522.733938] i915 device info: has_overlay: yes
<7> [522.734029] i915 device info: has_psr: no
<7> [522.734068] i915 device info: overlay_needs_physical: no
<7> [522.734112] i915 device info: supports_tv: no
<7> [522.734152] i915 device info: slice total: 0, mask=0000
<7> [522.734201] i915 device info: subslice total: 0
<7> [522.734400] i915 device info: EU total: 0
<7> [522.734441] i915 device info: EU per subslice: 0
<7> [522.734481] i915 device info: has slice power gating: no
<7> [522.734524] i915 device info: has subslice power gating: no
<7> [522.734570] i915 device info: has EU power gating: no
<7> [522.734613] i915 device info: CS timestamp frequency: 0 kHz
<6> [522.734656] [drm] DRM_I915_DEBUG enabled
<6> [522.734690] [drm] DRM_I915_DEBUG_GEM enabled
<6> [522.734727] [drm] DRM_I915_DEBUG_RUNTIME_PM enabled
<6> [522.734777] i915: Performing live selftests with st_random_seed=0xc8e3ad97 st_timeout=1000
<6> [522.734836] i915: Running client
<7> [522.734998] [drm:intel_power_well_enable [i915]] enabling always-on
<4> [522.853459] i915: probe of 0000:00:02.0 failed with error -25
<6> [523.174418] [IGT] i915_selftest: exiting, ret=0

Comment 1 CI Bug Log 2019-10-25 05:54:22 UTC

The CI Bug Log issue associated to this bug has been updated.

### New filters associated

* PNV: igt@i915_selftest@live_client - dmesg-warn - INFO: trying to register non-static key, the code is fine but needs lockdep annotation.
  - https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7175/fi-pnv-d510/igt@i915_selftest@live_client.html

Comment 2 Chris Wilson 2019-10-25 16:56:38 UTC

Oh, register raced with the ongoing GT work!

https://patchwork.freedesktop.org/series/68576/

Comment 3 Chris Wilson 2019-10-25 21:58:04 UTC

commit c442292a661bec3a32cf2a351c53c5f07da20e21 (HEAD -> drm-intel-next-queued, drm-intel/drm-intel-next-queued)
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Fri Oct 25 17:54:42 2019 +0100

    drm/i915/pmu: Initialise the spinlock before registering
    
    As the GT may be running in parallel with the module initialisation
    code, we may enter i915_pmu_gt_parked() as we are executing
    i915_pmu_register(). We have to init the spinlock before we mark
    pmu.event_init so that it is available for use by i915_pmu_gt_parked()
    (which may run as soon as event_init is set).
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=112127
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
    Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
    Reviewed-by: Matthew Auld <matthew.auld@intel.com>
    Link: https://patchwork.freedesktop.org/patch/msgid/20191025165442.23356-1-chris@chris-wilson.co.uk

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.