Re: [PATCH v12 00/14] In order to readout DP SDPs, refactors the handling of DP SDPs
On Fri, 15 May 2020, "Mun, Gwan-gyeong" wrote: > Hi Ville, > Thank you for notifying me that. I definitely missed the crash. > Sorry for that. > Danial and Jani, I' under debugging the crash case. > If you are availabe please do not merge current version. It has been merged, and that's the oops is the main reason it got noticed. If you can produce a fairly quick fix, great, but otherwise we'll need to revert enough commits to make this work again. BR, Jani. > > Br, > > G.G. > >> > On Fri, 2020-05-15 at 16:14 +0200, Daniel Vetter wrote: >> On Fri, May 15, 2020 at 04:13:18PM +0300, Jani Nikula wrote: >> > On Fri, 15 May 2020, Ville Syrjälä >> > wrote: >> > > On Thu, May 14, 2020 at 02:19:23PM +0300, Jani Nikula wrote: >> > > > On Thu, 14 May 2020, Gwan-gyeong Mun > > > > > wrote: >> > > > > In order to readout DP SDPs (Secondary Data Packet: DP HDR >> > > > > Metadata >> > > > > Infoframe SDP, DP VSC SDP), it refactors handling DP SDPs >> > > > > codes. >> > > > > It adds new compute routines for DP HDR Metadata Infoframe >> > > > > SDP >> > > > > and DP VSC SDP. >> > > > > And new writing routines of DP SDPs (Secondary Data Packet) >> > > > > that uses >> > > > > computed configs. >> > > > > New reading routines of DP SDPs are added for readout. >> > > > > It adds a logging function for DP VSC SDP. >> > > > > When receiving video it is very useful to be able to log DP >> > > > > VSC SDP. >> > > > > This greatly simplifies debugging. >> > > > > In order to use a common VSC SDP Colorimetry calculating code >> > > > > on PSR, >> > > > > it uses a new psr vsc sdp compute routine. >> > > > >> > > > Pushed the series to drm-intel-next-queued with Daniel's irc >> > > > ack for >> > > > merging the two non-i915 patches that route too. >> > > >> > > fi-hsw-4770 now oopses at boot: >> > >> > /o\ >> > >> > What did I miss? What part about the CI report did I overlook? >> >> Participating hosts (48 -> 45) >> -- >> >> Additional (1): fi-kbl-7560u >> Missing(4): fi-byt-clapper fi-byt-squawks fi-bsw-cyan fi-hsw- >> 4200u >> >> >> You kill machines at boot, CI won't tell you. >> >> This is (or at least was) because the network is shitty enough that >> we >> have more spurious failures because the ethernet went into the ether >> than >> because of people having killed the machine with their patches for >> real. >> Also it's hard to grab logs if the thing doesn't work at all, so cant >> give >> you any more data than the above. >> >> Yes this sucks :-/ >> >> Cheers, Daniel >> >> > BR, >> > Jani. >> > >> > >> > > <1>[3.736903] BUG: kernel NULL pointer dereference, address: >> > > >> > > <1>[3.736916] #PF: supervisor read access in kernel mode >> > > <1>[3.736916] #PF: error_code(0x) - not-present page >> > > <6>[3.736917] PGD 0 P4D 0 >> > > <4>[3.736919] Oops: [#1] PREEMPT SMP PTI >> > > <4>[3.736921] CPU: 0 PID: 363 Comm: systemd-udevd Not tainted >> > > 5.7.0-rc5-CI-CI_DRM_8485+ #1 >> > > <4>[3.736922] Hardware name: LENOVO 10AGS00601/SHARKBAY, BIOS >> > > FBKT34AUS 04/24/2013 >> > > <4>[3.736986] RIP: 0010:intel_psr_enabled+0x8/0x70 [i915] >> > > <4>[3.736988] Code: 18 48 c7 c6 40 09 79 a0 e8 e3 e2 04 e1 0f >> > > b6 44 24 03 e9 f4 fd ff ff 90 66 2e 0f 1f 84 00 00 00 00 00 41 54 >> > > 55 53 48 83 ec 08 <48> 8b 9f d8 fe ff ff f6 83 5e 0d 00 00 20 74 >> > > 09 80 bb 6c b6 00 00 >> > > <4>[3.737036] RSP: 0018:c947f8a0 EFLAGS: 00010286 >> > > <4>[3.737042] RAX: 0002 RBX: 8883ffd04000 >> > > RCX: 0001 >> > > <4>[3.737048] RDX: 0007 RSI: 8883ffd04000 >> > > RDI: 0128 >> > > <4>[3.737055] RBP: 888406afe200 R08: 000f >> > > R09: 0001 >> > > <4>[3.737061] R10: R11: >> > > R12: >> > > <4>[3.737068] R13: 8883f75d R14: 888406afe200 >> > > R15: 8883f75d0870 >> > > <4>[3.737075] FS: 7f71618f9680() >> > > GS:88840ec0() knlGS: >> > > <4>[3.737082] CS: 0010 DS: ES: CR0: >> > > 80050033 >> > > <4>[3.737088] CR2: CR3: 000402510002 >> > > CR4: 001606f0 >> > > <4>[3.737094] DR0: DR1: >> > > DR2: >> > > <4>[3.737101] DR3: DR6: fffe0ff0 >> > > DR7: 0400 >> > > <4>[3.737107] Call Trace: >> > > <4>[3.737175] intel_read_dp_sdp+0x1a4/0x380 [i915] >> > > <4>[3.737246] hsw_crt_get_config+0x12/0x40 [i915] >> > > <4>[3.737317] intel_modeset_setup_hw_state+0x3b3/0x16a0 >> > > [i915] >> > > ... >> > >> > -- >> > Jani Nikula, Intel Open Source Graphics Center -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org
Re: [PATCH v12 00/14] In order to readout DP SDPs, refactors the handling of DP SDPs
Hi Ville, Thank you for notifying me that. I definitely missed the crash. Sorry for that. Danial and Jani, I' under debugging the crash case. If you are availabe please do not merge current version. Br, G.G. > On Fri, 2020-05-15 at 16:14 +0200, Daniel Vetter wrote: > On Fri, May 15, 2020 at 04:13:18PM +0300, Jani Nikula wrote: > > On Fri, 15 May 2020, Ville Syrjälä > > wrote: > > > On Thu, May 14, 2020 at 02:19:23PM +0300, Jani Nikula wrote: > > > > On Thu, 14 May 2020, Gwan-gyeong Mun > > > > wrote: > > > > > In order to readout DP SDPs (Secondary Data Packet: DP HDR > > > > > Metadata > > > > > Infoframe SDP, DP VSC SDP), it refactors handling DP SDPs > > > > > codes. > > > > > It adds new compute routines for DP HDR Metadata Infoframe > > > > > SDP > > > > > and DP VSC SDP. > > > > > And new writing routines of DP SDPs (Secondary Data Packet) > > > > > that uses > > > > > computed configs. > > > > > New reading routines of DP SDPs are added for readout. > > > > > It adds a logging function for DP VSC SDP. > > > > > When receiving video it is very useful to be able to log DP > > > > > VSC SDP. > > > > > This greatly simplifies debugging. > > > > > In order to use a common VSC SDP Colorimetry calculating code > > > > > on PSR, > > > > > it uses a new psr vsc sdp compute routine. > > > > > > > > Pushed the series to drm-intel-next-queued with Daniel's irc > > > > ack for > > > > merging the two non-i915 patches that route too. > > > > > > fi-hsw-4770 now oopses at boot: > > > > /o\ > > > > What did I miss? What part about the CI report did I overlook? > > Participating hosts (48 -> 45) > -- > > Additional (1): fi-kbl-7560u > Missing(4): fi-byt-clapper fi-byt-squawks fi-bsw-cyan fi-hsw- > 4200u > > > You kill machines at boot, CI won't tell you. > > This is (or at least was) because the network is shitty enough that > we > have more spurious failures because the ethernet went into the ether > than > because of people having killed the machine with their patches for > real. > Also it's hard to grab logs if the thing doesn't work at all, so cant > give > you any more data than the above. > > Yes this sucks :-/ > > Cheers, Daniel > > > BR, > > Jani. > > > > > > > <1>[3.736903] BUG: kernel NULL pointer dereference, address: > > > > > > <1>[3.736916] #PF: supervisor read access in kernel mode > > > <1>[3.736916] #PF: error_code(0x) - not-present page > > > <6>[3.736917] PGD 0 P4D 0 > > > <4>[3.736919] Oops: [#1] PREEMPT SMP PTI > > > <4>[3.736921] CPU: 0 PID: 363 Comm: systemd-udevd Not tainted > > > 5.7.0-rc5-CI-CI_DRM_8485+ #1 > > > <4>[3.736922] Hardware name: LENOVO 10AGS00601/SHARKBAY, BIOS > > > FBKT34AUS 04/24/2013 > > > <4>[3.736986] RIP: 0010:intel_psr_enabled+0x8/0x70 [i915] > > > <4>[3.736988] Code: 18 48 c7 c6 40 09 79 a0 e8 e3 e2 04 e1 0f > > > b6 44 24 03 e9 f4 fd ff ff 90 66 2e 0f 1f 84 00 00 00 00 00 41 54 > > > 55 53 48 83 ec 08 <48> 8b 9f d8 fe ff ff f6 83 5e 0d 00 00 20 74 > > > 09 80 bb 6c b6 00 00 > > > <4>[3.737036] RSP: 0018:c947f8a0 EFLAGS: 00010286 > > > <4>[3.737042] RAX: 0002 RBX: 8883ffd04000 > > > RCX: 0001 > > > <4>[3.737048] RDX: 0007 RSI: 8883ffd04000 > > > RDI: 0128 > > > <4>[3.737055] RBP: 888406afe200 R08: 000f > > > R09: 0001 > > > <4>[3.737061] R10: R11: > > > R12: > > > <4>[3.737068] R13: 8883f75d R14: 888406afe200 > > > R15: 8883f75d0870 > > > <4>[3.737075] FS: 7f71618f9680() > > > GS:88840ec0() knlGS: > > > <4>[3.737082] CS: 0010 DS: ES: CR0: > > > 80050033 > > > <4>[3.737088] CR2: CR3: 000402510002 > > > CR4: 001606f0 > > > <4>[3.737094] DR0: DR1: > > > DR2: > > > <4>[3.737101] DR3: DR6: fffe0ff0 > > > DR7: 0400 > > > <4>[3.737107] Call Trace: > > > <4>[3.737175] intel_read_dp_sdp+0x1a4/0x380 [i915] > > > <4>[3.737246] hsw_crt_get_config+0x12/0x40 [i915] > > > <4>[3.737317] intel_modeset_setup_hw_state+0x3b3/0x16a0 > > > [i915] > > > ... > > > > -- > > Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v12 00/14] In order to readout DP SDPs, refactors the handling of DP SDPs
On Fri, May 15, 2020 at 04:13:18PM +0300, Jani Nikula wrote: > On Fri, 15 May 2020, Ville Syrjälä wrote: > > On Thu, May 14, 2020 at 02:19:23PM +0300, Jani Nikula wrote: > >> On Thu, 14 May 2020, Gwan-gyeong Mun wrote: > >> > In order to readout DP SDPs (Secondary Data Packet: DP HDR Metadata > >> > Infoframe SDP, DP VSC SDP), it refactors handling DP SDPs codes. > >> > It adds new compute routines for DP HDR Metadata Infoframe SDP > >> > and DP VSC SDP. > >> > And new writing routines of DP SDPs (Secondary Data Packet) that uses > >> > computed configs. > >> > New reading routines of DP SDPs are added for readout. > >> > It adds a logging function for DP VSC SDP. > >> > When receiving video it is very useful to be able to log DP VSC SDP. > >> > This greatly simplifies debugging. > >> > In order to use a common VSC SDP Colorimetry calculating code on PSR, > >> > it uses a new psr vsc sdp compute routine. > >> > >> Pushed the series to drm-intel-next-queued with Daniel's irc ack for > >> merging the two non-i915 patches that route too. > > > > fi-hsw-4770 now oopses at boot: > > /o\ > > What did I miss? What part about the CI report did I overlook? Participating hosts (48 -> 45) -- Additional (1): fi-kbl-7560u Missing(4): fi-byt-clapper fi-byt-squawks fi-bsw-cyan fi-hsw-4200u You kill machines at boot, CI won't tell you. This is (or at least was) because the network is shitty enough that we have more spurious failures because the ethernet went into the ether than because of people having killed the machine with their patches for real. Also it's hard to grab logs if the thing doesn't work at all, so cant give you any more data than the above. Yes this sucks :-/ Cheers, Daniel > > BR, > Jani. > > > > > > <1>[3.736903] BUG: kernel NULL pointer dereference, address: > > > > <1>[3.736916] #PF: supervisor read access in kernel mode > > <1>[3.736916] #PF: error_code(0x) - not-present page > > <6>[3.736917] PGD 0 P4D 0 > > <4>[3.736919] Oops: [#1] PREEMPT SMP PTI > > <4>[3.736921] CPU: 0 PID: 363 Comm: systemd-udevd Not tainted > > 5.7.0-rc5-CI-CI_DRM_8485+ #1 > > <4>[3.736922] Hardware name: LENOVO 10AGS00601/SHARKBAY, BIOS FBKT34AUS > > 04/24/2013 > > <4>[3.736986] RIP: 0010:intel_psr_enabled+0x8/0x70 [i915] > > <4>[3.736988] Code: 18 48 c7 c6 40 09 79 a0 e8 e3 e2 04 e1 0f b6 44 24 > > 03 e9 f4 fd ff ff 90 66 2e 0f 1f 84 00 00 00 00 00 41 54 55 53 48 83 ec 08 > > <48> 8b 9f d8 fe ff ff f6 83 5e 0d 00 00 20 74 09 80 bb 6c b6 00 00 > > <4>[3.737036] RSP: 0018:c947f8a0 EFLAGS: 00010286 > > <4>[3.737042] RAX: 0002 RBX: 8883ffd04000 RCX: > > 0001 > > <4>[3.737048] RDX: 0007 RSI: 8883ffd04000 RDI: > > 0128 > > <4>[3.737055] RBP: 888406afe200 R08: 000f R09: > > 0001 > > <4>[3.737061] R10: R11: R12: > > > > <4>[3.737068] R13: 8883f75d R14: 888406afe200 R15: > > 8883f75d0870 > > <4>[3.737075] FS: 7f71618f9680() GS:88840ec0() > > knlGS: > > <4>[3.737082] CS: 0010 DS: ES: CR0: 80050033 > > <4>[3.737088] CR2: CR3: 000402510002 CR4: > > 001606f0 > > <4>[3.737094] DR0: DR1: DR2: > > > > <4>[3.737101] DR3: DR6: fffe0ff0 DR7: > > 0400 > > <4>[3.737107] Call Trace: > > <4>[3.737175] intel_read_dp_sdp+0x1a4/0x380 [i915] > > <4>[3.737246] hsw_crt_get_config+0x12/0x40 [i915] > > <4>[3.737317] intel_modeset_setup_hw_state+0x3b3/0x16a0 [i915] > > ... > > -- > Jani Nikula, Intel Open Source Graphics Center -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v12 00/14] In order to readout DP SDPs, refactors the handling of DP SDPs
On Fri, 15 May 2020, Ville Syrjälä wrote: > On Thu, May 14, 2020 at 02:19:23PM +0300, Jani Nikula wrote: >> On Thu, 14 May 2020, Gwan-gyeong Mun wrote: >> > In order to readout DP SDPs (Secondary Data Packet: DP HDR Metadata >> > Infoframe SDP, DP VSC SDP), it refactors handling DP SDPs codes. >> > It adds new compute routines for DP HDR Metadata Infoframe SDP >> > and DP VSC SDP. >> > And new writing routines of DP SDPs (Secondary Data Packet) that uses >> > computed configs. >> > New reading routines of DP SDPs are added for readout. >> > It adds a logging function for DP VSC SDP. >> > When receiving video it is very useful to be able to log DP VSC SDP. >> > This greatly simplifies debugging. >> > In order to use a common VSC SDP Colorimetry calculating code on PSR, >> > it uses a new psr vsc sdp compute routine. >> >> Pushed the series to drm-intel-next-queued with Daniel's irc ack for >> merging the two non-i915 patches that route too. > > fi-hsw-4770 now oopses at boot: /o\ What did I miss? What part about the CI report did I overlook? BR, Jani. > > <1>[3.736903] BUG: kernel NULL pointer dereference, address: > > <1>[3.736916] #PF: supervisor read access in kernel mode > <1>[3.736916] #PF: error_code(0x) - not-present page > <6>[3.736917] PGD 0 P4D 0 > <4>[3.736919] Oops: [#1] PREEMPT SMP PTI > <4>[3.736921] CPU: 0 PID: 363 Comm: systemd-udevd Not tainted > 5.7.0-rc5-CI-CI_DRM_8485+ #1 > <4>[3.736922] Hardware name: LENOVO 10AGS00601/SHARKBAY, BIOS FBKT34AUS > 04/24/2013 > <4>[3.736986] RIP: 0010:intel_psr_enabled+0x8/0x70 [i915] > <4>[3.736988] Code: 18 48 c7 c6 40 09 79 a0 e8 e3 e2 04 e1 0f b6 44 24 03 > e9 f4 fd ff ff 90 66 2e 0f 1f 84 00 00 00 00 00 41 54 55 53 48 83 ec 08 <48> > 8b 9f d8 fe ff ff f6 83 5e 0d 00 00 20 74 09 80 bb 6c b6 00 00 > <4>[3.737036] RSP: 0018:c947f8a0 EFLAGS: 00010286 > <4>[3.737042] RAX: 0002 RBX: 8883ffd04000 RCX: > 0001 > <4>[3.737048] RDX: 0007 RSI: 8883ffd04000 RDI: > 0128 > <4>[3.737055] RBP: 888406afe200 R08: 000f R09: > 0001 > <4>[3.737061] R10: R11: R12: > > <4>[3.737068] R13: 8883f75d R14: 888406afe200 R15: > 8883f75d0870 > <4>[3.737075] FS: 7f71618f9680() GS:88840ec0() > knlGS: > <4>[3.737082] CS: 0010 DS: ES: CR0: 80050033 > <4>[3.737088] CR2: CR3: 000402510002 CR4: > 001606f0 > <4>[3.737094] DR0: DR1: DR2: > > <4>[3.737101] DR3: DR6: fffe0ff0 DR7: > 0400 > <4>[3.737107] Call Trace: > <4>[3.737175] intel_read_dp_sdp+0x1a4/0x380 [i915] > <4>[3.737246] hsw_crt_get_config+0x12/0x40 [i915] > <4>[3.737317] intel_modeset_setup_hw_state+0x3b3/0x16a0 [i915] > ... -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v12 00/14] In order to readout DP SDPs, refactors the handling of DP SDPs
On Thu, May 14, 2020 at 02:19:23PM +0300, Jani Nikula wrote: > On Thu, 14 May 2020, Gwan-gyeong Mun wrote: > > In order to readout DP SDPs (Secondary Data Packet: DP HDR Metadata > > Infoframe SDP, DP VSC SDP), it refactors handling DP SDPs codes. > > It adds new compute routines for DP HDR Metadata Infoframe SDP > > and DP VSC SDP. > > And new writing routines of DP SDPs (Secondary Data Packet) that uses > > computed configs. > > New reading routines of DP SDPs are added for readout. > > It adds a logging function for DP VSC SDP. > > When receiving video it is very useful to be able to log DP VSC SDP. > > This greatly simplifies debugging. > > In order to use a common VSC SDP Colorimetry calculating code on PSR, > > it uses a new psr vsc sdp compute routine. > > Pushed the series to drm-intel-next-queued with Daniel's irc ack for > merging the two non-i915 patches that route too. fi-hsw-4770 now oopses at boot: <1>[3.736903] BUG: kernel NULL pointer dereference, address: <1>[3.736916] #PF: supervisor read access in kernel mode <1>[3.736916] #PF: error_code(0x) - not-present page <6>[3.736917] PGD 0 P4D 0 <4>[3.736919] Oops: [#1] PREEMPT SMP PTI <4>[3.736921] CPU: 0 PID: 363 Comm: systemd-udevd Not tainted 5.7.0-rc5-CI-CI_DRM_8485+ #1 <4>[3.736922] Hardware name: LENOVO 10AGS00601/SHARKBAY, BIOS FBKT34AUS 04/24/2013 <4>[3.736986] RIP: 0010:intel_psr_enabled+0x8/0x70 [i915] <4>[3.736988] Code: 18 48 c7 c6 40 09 79 a0 e8 e3 e2 04 e1 0f b6 44 24 03 e9 f4 fd ff ff 90 66 2e 0f 1f 84 00 00 00 00 00 41 54 55 53 48 83 ec 08 <48> 8b 9f d8 fe ff ff f6 83 5e 0d 00 00 20 74 09 80 bb 6c b6 00 00 <4>[3.737036] RSP: 0018:c947f8a0 EFLAGS: 00010286 <4>[3.737042] RAX: 0002 RBX: 8883ffd04000 RCX: 0001 <4>[3.737048] RDX: 0007 RSI: 8883ffd04000 RDI: 0128 <4>[3.737055] RBP: 888406afe200 R08: 000f R09: 0001 <4>[3.737061] R10: R11: R12: <4>[3.737068] R13: 8883f75d R14: 888406afe200 R15: 8883f75d0870 <4>[3.737075] FS: 7f71618f9680() GS:88840ec0() knlGS: <4>[3.737082] CS: 0010 DS: ES: CR0: 80050033 <4>[3.737088] CR2: CR3: 000402510002 CR4: 001606f0 <4>[3.737094] DR0: DR1: DR2: <4>[3.737101] DR3: DR6: fffe0ff0 DR7: 0400 <4>[3.737107] Call Trace: <4>[3.737175] intel_read_dp_sdp+0x1a4/0x380 [i915] <4>[3.737246] hsw_crt_get_config+0x12/0x40 [i915] <4>[3.737317] intel_modeset_setup_hw_state+0x3b3/0x16a0 [i915] ... -- Ville Syrjälä Intel ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v12 00/14] In order to readout DP SDPs, refactors the handling of DP SDPs
On Thu, 14 May 2020, Gwan-gyeong Mun wrote: > In order to readout DP SDPs (Secondary Data Packet: DP HDR Metadata > Infoframe SDP, DP VSC SDP), it refactors handling DP SDPs codes. > It adds new compute routines for DP HDR Metadata Infoframe SDP > and DP VSC SDP. > And new writing routines of DP SDPs (Secondary Data Packet) that uses > computed configs. > New reading routines of DP SDPs are added for readout. > It adds a logging function for DP VSC SDP. > When receiving video it is very useful to be able to log DP VSC SDP. > This greatly simplifies debugging. > In order to use a common VSC SDP Colorimetry calculating code on PSR, > it uses a new psr vsc sdp compute routine. Pushed the series to drm-intel-next-queued with Daniel's irc ack for merging the two non-i915 patches that route too. Thanks for the patches and review! BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
