Re: [PATCH] drm/i915/mst: Check uapi enable not intel one during mst atomic check

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2019-11-15 at 22:25 +0200, Ville Syrjälä wrote:
> On Fri, Nov 15, 2019 at 12:04:30PM -0800, José Roberto de Souza wrote:
> > When the connector has VCPI allocated and is being moved to another
> > pipe it causes drm_dp_atomic_release_vcpi_slots() and
> > drm_dp_atomic_find_vcpi_slots() to be called in the same atomic check
> > causing the error bellow.
> > This happens because at this point Intel's hw.enable(and all other
> > flags in the same struct) is not set but checking to on the uapi one
> > it have the expected value.
> > 
> > [  580.804430] ------------[ cut here ]------------
> > [  580.804436] WARNING: CPU: 0 PID: 1221 at
> > drivers/gpu/drm/drm_dp_mst_topology.c:4094
> > drm_dp_atomic_find_vcpi_slots+0x157/0x180
> > [  580.804439] Modules linked in: cdc_ether r8152 i915 prime_numbers
> > snd_hda_codec_hdmi snd_hda_intel snd_intel_dspcfg snd_hda_codec snd_hwdep
> > asix snd_hda_core x86_pkg_temp_thermal usbnet mei_hdcp coretemp mii mei_me
> > crct10dif_pclmul snd_pcm crc32_pclmul mei ghash_clmulni_intel i2c_i801
> > [last unloaded: prime_numbers]
> > [  580.804462] CPU: 0 PID: 1221 Comm: kworker/0:0 Tainted:
> > G        W         5.4.0-rc7-zeh+ #1226
> > [  580.804465] Hardware name: Intel Corporation Tiger Lake Client
> > Platform/TigerLake U DDR4 SODIMM RVP, BIOS
> > TGLSFWI1.D00.2321.A09.1909250226 09/25/2019
> > [  580.804470] Workqueue: events output_poll_execute
> > [  580.804476] RIP: 0010:drm_dp_atomic_find_vcpi_slots+0x157/0x180
> > [  580.804481] Code: 6a ff ff ff 49 89 6d 08 4c 89 6b 10 4c 89 63 18 49 89
> > 6e 08 e9 55 ff ff ff 41 89 c7 5b 5d 44 89 f8 41 5c 41 5d 41 5e 41 5f c3
> > <0f> 0b 48 c7 c7 08 73 11 82 48 89 ee 41 bf ea ff ff ff e8 b2 e3 02
> > [  580.804484] RSP: 0018:ffffc900009b7ab8 EFLAGS: 00010246
> > [  580.804488] RAX: ffff88848c04ef50 RBX: ffff88848c04ef40 RCX:
> > 0000000000000214
> > [  580.804492] RDX: ffff88848c04f5e0 RSI: ffff888486eb2c68 RDI:
> > ffff88848e518800
> > [  580.804495] RBP: ffff88849d339000 R08: 00000000bc4e1092 R09:
> > 0000000000000000
> > [  580.804498] R10: 0000000000000000 R11: 0000000000000000 R12:
> > ffff88848c04e728
> > [  580.804501] R13: 0000000000000214 R14: ffff88848c04e720 R15:
> > ffff888486eb2c68
> > [  580.804504] FS:  0000000000000000(0000) GS:ffff8884a0000000(0000)
> > knlGS:0000000000000000
> > [  580.804507] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> > [  580.804510] CR2: 00007ff6bf1ba680 CR3: 0000000005210003 CR4:
> > 0000000000760ef0
> > [  580.804512] PKRU: 55555554
> > [  580.804515] Call Trace:
> > [  580.804574]  intel_dp_mst_compute_config+0x193/0x2b0 [i915]
> > [  580.804636]  intel_atomic_check+0x10cc/0x20b0 [i915]
> > [  580.804644]  ? drm_atomic_print_old_state+0xf1/0x130
> > [  580.804655]  drm_atomic_check_only+0x56a/0x810
> > [  580.804663]  drm_atomic_commit+0xe/0x50
> > [  580.804668]  drm_client_modeset_commit_atomic+0x18b/0x220
> > [  580.804680]  drm_client_modeset_commit_force+0x4d/0x180
> > [  580.804685]  drm_fb_helper_restore_fbdev_mode_unlocked+0x46/0xa0
> > [  580.804689]  drm_fb_helper_set_par+0x27/0x50
> > [  580.804692]  drm_fb_helper_hotplug_event.part.0+0xa7/0xc0
> > [  580.804696]  drm_kms_helper_hotplug_event+0x21/0x30
> > [  580.804699]  output_poll_execute+0x1a4/0x1c0
> > [  580.804706]  process_one_work+0x25b/0x5b0
> > [  580.804713]  worker_thread+0x4b/0x3b0
> > [  580.804720]  kthread+0x100/0x140
> > [  580.804723]  ? process_one_work+0x5b0/0x5b0
> > [  580.804725]  ? kthread_park+0x80/0x80
> > [  580.804730]  ret_from_fork+0x24/0x50
> > [  580.804740] irq event stamp: 40988
> > [  580.804743] hardirqs last  enabled at (40987): [<ffffffff81128567>]
> > console_unlock+0x437/0x590
> > [  580.804746] hardirqs last disabled at (40988): [<ffffffff81001cfa>]
> > trace_hardirqs_off_thunk+0x1a/0x20
> > [  580.804749] softirqs last  enabled at (40972): [<ffffffff81c00389>]
> > __do_softirq+0x389/0x47f
> > [  580.804752] softirqs last disabled at (40959): [<ffffffff810b6f19>]
> > irq_exit+0xa9/0xc0
> > [  580.804754] ---[ end trace 80052e0c60463c67 ]---
> > [  580.804758] [drm:drm_dp_atomic_find_vcpi_slots] *ERROR* cannot allocate
> > and release VCPI on [MST PORT:000000007880692e] in the same state
> > [  580.811370] [drm:intel_dp_hpd_pulse [i915]] got esi2 02 00 00
> > [  580.817239] [drm:intel_dp_hpd_pulse [i915]] got esi 02 00 00
> > [  580.817313] ------------[ cut here ]------------
> > [  580.817318] WARNING: CPU: 0 PID: 1221 at
> > drivers/gpu/drm/drm_dp_mst_topology.c:4094
> > drm_dp_atomic_find_vcpi_slots+0x157/0x180
> > [  580.817321] Modules linked in: cdc_ether r8152 i915 prime_numbers
> > snd_hda_codec_hdmi snd_hda_intel snd_intel_dspcfg snd_hda_codec snd_hwdep
> > asix snd_hda_core x86_pkg_temp_thermal
> > [  580.817412] [drm:intel_dp_hpd_pulse [i915]] got hpd irq on
> > [ENCODER:306:DDI E] - short
> > [  580.817413]  usbnet mei_hdcp coretemp mii mei_me crct10dif_pclmul
> > snd_pcm crc32_pclmul
> > [  580.817490] [drm:intel_dp_hpd_pulse [i915]]  is_mst
> > [  580.817491]  mei ghash_clmulni_intel i2c_i801 [last unloaded:
> > prime_numbers]
> > [  580.817498] CPU: 0 PID: 1221 Comm: kworker/0:0 Tainted:
> > G        W         5.4.0-rc7-zeh+ #1226
> > [  580.817503] Hardware name: Intel Corporation Tiger Lake Client
> > Platform/TigerLake U DDR4 SODIMM RVP, BIOS
> > TGLSFWI1.D00.2321.A09.1909250226 09/25/2019
> > [  580.817506] Workqueue: events output_poll_execute
> > [  580.817511] RIP: 0010:drm_dp_atomic_find_vcpi_slots+0x157/0x180
> > [  580.817514] Code: 6a ff ff ff 49 89 6d 08 4c 89 6b 10 4c 89 63 18 49 89
> > 6e 08 e9 55 ff ff ff 41 89 c7 5b 5d 44 89 f8 41 5c 41 5d 41 5e 41 5f c3
> > <0f> 0b 48 c7 c7 08 73 11 82 48 89 ee 41 bf ea ff ff ff e8 b2 e3 02
> > [  580.817516] RSP: 0018:ffffc900009b7ab8 EFLAGS: 00010246
> > [  580.817519] RAX: ffff88848c04ef50 RBX: ffff88848c04ef40 RCX:
> > 000000000000018f
> > [  580.817521] RDX: ffff88848c04f5e0 RSI: ffff888486eb2c68 RDI:
> > ffff88848e518800
> > [  580.817523] RBP: ffff88849d339000 R08: 00000000bc4e1092 R09:
> > 0000000000000000
> > [  580.817525] R10: 0000000000000000 R11: 0000000000000000 R12:
> > ffff88848c04e728
> > [  580.817528] R13: 000000000000018f R14: ffff88848c04e720 R15:
> > ffff888486eb2c68
> > [  580.817532] FS:  0000000000000000(0000) GS:ffff8884a0000000(0000)
> > knlGS:0000000000000000
> > [  580.817534] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> > [  580.817535] CR2: 00007ff6bf1ba680 CR3: 0000000005210003 CR4:
> > 0000000000760ef0
> > [  580.817537] PKRU: 55555554
> > [  580.817538] Call Trace:
> > [  580.817620]  intel_dp_mst_compute_config+0x193/0x2b0 [i915]
> > [  580.817690]  intel_atomic_check+0x10cc/0x20b0 [i915]
> > [  580.817697]  ? drm_atomic_print_old_state+0xf1/0x130
> > [  580.817711]  drm_atomic_check_only+0x56a/0x810
> > [  580.817721]  drm_atomic_commit+0xe/0x50
> > [  580.817726]  drm_client_modeset_commit_atomic+0x18b/0x220
> > [  580.817744]  drm_client_modeset_commit_force+0x4d/0x180
> > [  580.817751]  drm_fb_helper_restore_fbdev_mode_unlocked+0x46/0xa0
> > [  580.817756]  drm_fb_helper_set_par+0x27/0x50
> > [  580.817762]  drm_fb_helper_hotplug_event.part.0+0xa7/0xc0
> > [  580.817767]  drm_kms_helper_hotplug_event+0x21/0x30
> > [  580.817771]  output_poll_execute+0x1a4/0x1c0
> > [  580.817780]  process_one_work+0x25b/0x5b0
> > [  580.817791]  worker_thread+0x4b/0x3b0
> > [  580.817800]  kthread+0x100/0x140
> > [  580.817804]  ? process_one_work+0x5b0/0x5b0
> > [  580.817807]  ? kthread_park+0x80/0x80
> > [  580.817813]  ret_from_fork+0x24/0x50
> > [  580.817832] irq event stamp: 41028
> > [  580.817838] hardirqs last  enabled at (41027): [<ffffffff81128567>]
> > console_unlock+0x437/0x590
> > [  580.817841] hardirqs last disabled at (41028): [<ffffffff81001cfa>]
> > trace_hardirqs_off_thunk+0x1a/0x20
> > [  580.817846] softirqs last  enabled at (41022): [<ffffffff81c00389>]
> > __do_softirq+0x389/0x47f
> > [  580.817851] softirqs last disabled at (41013): [<ffffffff810b6f19>]
> > irq_exit+0xa9/0xc0
> > [  580.817854] ---[ end trace 80052e0c60463c68 ]---
> > [  580.817858] [drm:drm_dp_atomic_find_vcpi_slots] *ERROR* cannot allocate
> > and release VCPI on [MST PORT:000000007880692e] in the same state
> > [  580.830767] [drm:intel_dp_mst_compute_config [i915]] failed finding
> > vcpi slots:-22
> > [  580.830821] [drm:intel_atomic_check [i915]] Encoder config failure: -22
> > 
> > Cc: Lyude Paul <lyude@xxxxxxxxxx>
> > Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
> > Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> > Cc: Lucas De Marchi <lucas.demarchi@xxxxxxxxx>
> > Signed-off-by: José Roberto de Souza <jose.souza@xxxxxxxxx>
> > ---
> >  drivers/gpu/drm/i915/display/intel_dp_mst.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c
> > b/drivers/gpu/drm/i915/display/intel_dp_mst.c
> > index 9fc606c6dbee..f8a350359346 100644
> > --- a/drivers/gpu/drm/i915/display/intel_dp_mst.c
> > +++ b/drivers/gpu/drm/i915/display/intel_dp_mst.c
> > @@ -191,7 +191,7 @@ intel_dp_mst_atomic_check(struct drm_connector
> > *connector,
> >  
> >  		if (!crtc_state ||
> >  		    !drm_atomic_crtc_needs_modeset(&crtc_state->uapi) ||
> > -		    crtc_state->hw.enable)
> > +		    crtc_state->uapi.enable)
> 
> Yeah, I already mentioned this during the review but I guess it slipped
> through. We may have to spend more brain cells on this stuff for
> bigjoiner but let's at least try not to break everything else in the
> meantime.

Bigjoiner?
> 
> Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> 
> >  			return 0;
> >  	}
> >  
> > -- 
> > 2.24.0
-- 
Cheers,
	Lyude Paul

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux