All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Mun, Gwan-gyeong" <gwan-gyeong.mun@intel.com>
To: "daniel@ffwll.ch" <daniel@ffwll.ch>,
	"jani.nikula@linux.intel.com" <jani.nikula@linux.intel.com>
Cc: "linux-fbdev@vger.kernel.org" <linux-fbdev@vger.kernel.org>,
	"daniel.vetter@ffwll.ch" <daniel.vetter@ffwll.ch>,
	"intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>,
	"dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>,
	"laurent.pinchart@ideasonboard.com"
	<laurent.pinchart@ideasonboard.com>
Subject: Re: [PATCH v12 00/14] In order to readout DP SDPs, refactors the handling of DP SDPs
Date: Fri, 15 May 2020 17:22:43 +0000	[thread overview]
Message-ID: <e2cf963c6db302d94195f4af643d8fbad3ece5dc.camel@intel.com> (raw)
In-Reply-To: <20200515141426.GI206103@phenom.ffwll.local>

SGkgVmlsbGUsDQpUaGFuayB5b3UgZm9yIG5vdGlmeWluZyBtZSB0aGF0LiBJIGRlZmluaXRlbHkg
bWlzc2VkIHRoZSBjcmFzaC4NClNvcnJ5IGZvciB0aGF0Lg0KRGFuaWFsIGFuZCBKYW5pLCBJJyB1
bmRlciBkZWJ1Z2dpbmcgdGhlIGNyYXNoIGNhc2UuDQpJZiB5b3UgYXJlIGF2YWlsYWJlIHBsZWFz
ZSBkbyBub3QgbWVyZ2UgY3VycmVudCB2ZXJzaW9uLg0KDQpCciwNCg0KRy5HLg0KDQo+IA0KT24g
RnJpLCAyMDIwLTA1LTE1IGF0IDE2OjE0ICswMjAwLCBEYW5pZWwgVmV0dGVyIHdyb3RlOg0KPiBP
biBGcmksIE1heSAxNSwgMjAyMCBhdCAwNDoxMzoxOFBNICswMzAwLCBKYW5pIE5pa3VsYSB3cm90
ZToNCj4gPiBPbiBGcmksIDE1IE1heSAyMDIwLCBWaWxsZSBTeXJqw6Rsw6QgPHZpbGxlLnN5cmph
bGFAbGludXguaW50ZWwuY29tPg0KPiA+IHdyb3RlOg0KPiA+ID4gT24gVGh1LCBNYXkgMTQsIDIw
MjAgYXQgMDI6MTk6MjNQTSArMDMwMCwgSmFuaSBOaWt1bGEgd3JvdGU6DQo+ID4gPiA+IE9uIFRo
dSwgMTQgTWF5IDIwMjAsIEd3YW4tZ3llb25nIE11biA8Z3dhbi1neWVvbmcubXVuQGludGVsLmNv
bQ0KPiA+ID4gPiA+IHdyb3RlOg0KPiA+ID4gPiA+IEluIG9yZGVyIHRvIHJlYWRvdXQgRFAgU0RQ
cyAoU2Vjb25kYXJ5IERhdGEgUGFja2V0OiBEUCBIRFINCj4gPiA+ID4gPiBNZXRhZGF0YQ0KPiA+
ID4gPiA+IEluZm9mcmFtZSBTRFAsIERQIFZTQyBTRFApLCBpdCByZWZhY3RvcnMgaGFuZGxpbmcg
RFAgU0RQcw0KPiA+ID4gPiA+IGNvZGVzLg0KPiA+ID4gPiA+IEl0IGFkZHMgbmV3IGNvbXB1dGUg
cm91dGluZXMgZm9yIERQIEhEUiBNZXRhZGF0YSBJbmZvZnJhbWUNCj4gPiA+ID4gPiBTRFANCj4g
PiA+ID4gPiBhbmQgRFAgVlNDIFNEUC4gDQo+ID4gPiA+ID4gQW5kIG5ldyB3cml0aW5nIHJvdXRp
bmVzIG9mIERQIFNEUHMgKFNlY29uZGFyeSBEYXRhIFBhY2tldCkNCj4gPiA+ID4gPiB0aGF0IHVz
ZXMNCj4gPiA+ID4gPiBjb21wdXRlZCBjb25maWdzLg0KPiA+ID4gPiA+IE5ldyByZWFkaW5nIHJv
dXRpbmVzIG9mIERQIFNEUHMgYXJlIGFkZGVkIGZvciByZWFkb3V0Lg0KPiA+ID4gPiA+IEl0IGFk
ZHMgYSBsb2dnaW5nIGZ1bmN0aW9uIGZvciBEUCBWU0MgU0RQLg0KPiA+ID4gPiA+IFdoZW4gcmVj
ZWl2aW5nIHZpZGVvIGl0IGlzIHZlcnkgdXNlZnVsIHRvIGJlIGFibGUgdG8gbG9nIERQDQo+ID4g
PiA+ID4gVlNDIFNEUC4NCj4gPiA+ID4gPiBUaGlzIGdyZWF0bHkgc2ltcGxpZmllcyBkZWJ1Z2dp
bmcuDQo+ID4gPiA+ID4gSW4gb3JkZXIgdG8gdXNlIGEgY29tbW9uIFZTQyBTRFAgQ29sb3JpbWV0
cnkgY2FsY3VsYXRpbmcgY29kZQ0KPiA+ID4gPiA+IG9uIFBTUiwNCj4gPiA+ID4gPiBpdCB1c2Vz
IGEgbmV3IHBzciB2c2Mgc2RwIGNvbXB1dGUgcm91dGluZS4NCj4gPiA+ID4gDQo+ID4gPiA+IFB1
c2hlZCB0aGUgc2VyaWVzIHRvIGRybS1pbnRlbC1uZXh0LXF1ZXVlZCB3aXRoIERhbmllbCdzIGly
Yw0KPiA+ID4gPiBhY2sgZm9yDQo+ID4gPiA+IG1lcmdpbmcgdGhlIHR3byBub24taTkxNSBwYXRj
aGVzIHRoYXQgcm91dGUgdG9vLg0KPiA+ID4gDQo+ID4gPiBmaS1oc3ctNDc3MCBub3cgb29wc2Vz
IGF0IGJvb3Q6DQo+ID4gDQo+ID4gL29cDQo+ID4gDQo+ID4gV2hhdCBkaWQgSSBtaXNzPyBXaGF0
IHBhcnQgYWJvdXQgdGhlIENJIHJlcG9ydCBkaWQgSSBvdmVybG9vaz8NCj4gDQo+IFBhcnRpY2lw
YXRpbmcgaG9zdHMgKDQ4IC0+IDQ1KQ0KPiAtLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0N
Cj4gDQo+ICAgQWRkaXRpb25hbCAoMSk6IGZpLWtibC03NTYwdSANCj4gICBNaXNzaW5nICAgICg0
KTogZmktYnl0LWNsYXBwZXIgZmktYnl0LXNxdWF3a3MgZmktYnN3LWN5YW4gZmktaHN3LQ0KPiA0
MjAwdQ0KPiANCj4gDQo+IFlvdSBraWxsIG1hY2hpbmVzIGF0IGJvb3QsIENJIHdvbid0IHRlbGwg
eW91Lg0KPiANCj4gVGhpcyBpcyAob3IgYXQgbGVhc3Qgd2FzKSBiZWNhdXNlIHRoZSBuZXR3b3Jr
IGlzIHNoaXR0eSBlbm91Z2ggdGhhdA0KPiB3ZQ0KPiBoYXZlIG1vcmUgc3B1cmlvdXMgZmFpbHVy
ZXMgYmVjYXVzZSB0aGUgZXRoZXJuZXQgd2VudCBpbnRvIHRoZSBldGhlcg0KPiB0aGFuDQo+IGJl
Y2F1c2Ugb2YgcGVvcGxlIGhhdmluZyBraWxsZWQgdGhlIG1hY2hpbmUgd2l0aCB0aGVpciBwYXRj
aGVzIGZvcg0KPiByZWFsLg0KPiBBbHNvIGl0J3MgaGFyZCB0byBncmFiIGxvZ3MgaWYgdGhlIHRo
aW5nIGRvZXNuJ3Qgd29yayBhdCBhbGwsIHNvIGNhbnQNCj4gZ2l2ZQ0KPiB5b3UgYW55IG1vcmUg
ZGF0YSB0aGFuIHRoZSBhYm92ZS4NCj4gDQo+IFllcyB0aGlzIHN1Y2tzIDotLw0KPiANCj4gQ2hl
ZXJzLCBEYW5pZWwNCj4gDQo+ID4gQlIsDQo+ID4gSmFuaS4NCj4gPiANCj4gPiANCj4gPiA+IDwx
PlsgICAgMy43MzY5MDNdIEJVRzoga2VybmVsIE5VTEwgcG9pbnRlciBkZXJlZmVyZW5jZSwgYWRk
cmVzczoNCj4gPiA+IDAwMDAwMDAwMDAwMDAwMDANCj4gPiA+IDwxPlsgICAgMy43MzY5MTZdICNQ
Rjogc3VwZXJ2aXNvciByZWFkIGFjY2VzcyBpbiBrZXJuZWwgbW9kZQ0KPiA+ID4gPDE+WyAgICAz
LjczNjkxNl0gI1BGOiBlcnJvcl9jb2RlKDB4MDAwMCkgLSBub3QtcHJlc2VudCBwYWdlDQo+ID4g
PiA8Nj5bICAgIDMuNzM2OTE3XSBQR0QgMCBQNEQgMCANCj4gPiA+IDw0PlsgICAgMy43MzY5MTld
IE9vcHM6IDAwMDAgWyMxXSBQUkVFTVBUIFNNUCBQVEkNCj4gPiA+IDw0PlsgICAgMy43MzY5MjFd
IENQVTogMCBQSUQ6IDM2MyBDb21tOiBzeXN0ZW1kLXVkZXZkIE5vdCB0YWludGVkDQo+ID4gPiA1
LjcuMC1yYzUtQ0ktQ0lfRFJNXzg0ODUrICMxDQo+ID4gPiA8ND5bICAgIDMuNzM2OTIyXSBIYXJk
d2FyZSBuYW1lOiBMRU5PVk8gMTBBR1MwMDYwMS9TSEFSS0JBWSwgQklPUw0KPiA+ID4gRkJLVDM0
QVVTIDA0LzI0LzIwMTMNCj4gPiA+IDw0PlsgICAgMy43MzY5ODZdIFJJUDogMDAxMDppbnRlbF9w
c3JfZW5hYmxlZCsweDgvMHg3MCBbaTkxNV0NCj4gPiA+IDw0PlsgICAgMy43MzY5ODhdIENvZGU6
IDE4IDQ4IGM3IGM2IDQwIDA5IDc5IGEwIGU4IGUzIGUyIDA0IGUxIDBmDQo+ID4gPiBiNiA0NCAy
NCAwMyBlOSBmNCBmZCBmZiBmZiA5MCA2NiAyZSAwZiAxZiA4NCAwMCAwMCAwMCAwMCAwMCA0MSA1
NA0KPiA+ID4gNTUgNTMgNDggODMgZWMgMDggPDQ4PiA4YiA5ZiBkOCBmZSBmZiBmZiBmNiA4MyA1
ZSAwZCAwMCAwMCAyMCA3NA0KPiA+ID4gMDkgODAgYmIgNmMgYjYgMDAgMDANCj4gPiA+IDw0Plsg
ICAgMy43MzcwMzZdIFJTUDogMDAxODpmZmZmYzkwMDAwNDdmOGEwIEVGTEFHUzogMDAwMTAyODYN
Cj4gPiA+IDw0PlsgICAgMy43MzcwNDJdIFJBWDogMDAwMDAwMDAwMDAwMDAwMiBSQlg6IGZmZmY4
ODgzZmZkMDQwMDANCj4gPiA+IFJDWDogMDAwMDAwMDAwMDAwMDAwMQ0KPiA+ID4gPDQ+WyAgICAz
LjczNzA0OF0gUkRYOiAwMDAwMDAwMDAwMDAwMDA3IFJTSTogZmZmZjg4ODNmZmQwNDAwMA0KPiA+
ID4gUkRJOiAwMDAwMDAwMDAwMDAwMTI4DQo+ID4gPiA8ND5bICAgIDMuNzM3MDU1XSBSQlA6IGZm
ZmY4ODg0MDZhZmUyMDAgUjA4OiAwMDAwMDAwMDAwMDAwMDBmDQo+ID4gPiBSMDk6IDAwMDAwMDAw
MDAwMDAwMDENCj4gPiA+IDw0PlsgICAgMy43MzcwNjFdIFIxMDogMDAwMDAwMDAwMDAwMDAwMCBS
MTE6IDAwMDAwMDAwMDAwMDAwMDANCj4gPiA+IFIxMjogMDAwMDAwMDAwMDAwMDAwMA0KPiA+ID4g
PDQ+WyAgICAzLjczNzA2OF0gUjEzOiBmZmZmODg4M2Y3NWQwMDAwIFIxNDogZmZmZjg4ODQwNmFm
ZTIwMA0KPiA+ID4gUjE1OiBmZmZmODg4M2Y3NWQwODcwDQo+ID4gPiA8ND5bICAgIDMuNzM3MDc1
XSBGUzogIDAwMDA3ZjcxNjE4Zjk2ODAoMDAwMCkNCj4gPiA+IEdTOmZmZmY4ODg0MGVjMDAwMDAo
MDAwMCkga25sR1M6MDAwMDAwMDAwMDAwMDAwMA0KPiA+ID4gPDQ+WyAgICAzLjczNzA4Ml0gQ1M6
ICAwMDEwIERTOiAwMDAwIEVTOiAwMDAwIENSMDoNCj4gPiA+IDAwMDAwMDAwODAwNTAwMzMNCj4g
PiA+IDw0PlsgICAgMy43MzcwODhdIENSMjogMDAwMDAwMDAwMDAwMDAwMCBDUjM6IDAwMDAwMDA0
MDI1MTAwMDINCj4gPiA+IENSNDogMDAwMDAwMDAwMDE2MDZmMA0KPiA+ID4gPDQ+WyAgICAzLjcz
NzA5NF0gRFIwOiAwMDAwMDAwMDAwMDAwMDAwIERSMTogMDAwMDAwMDAwMDAwMDAwMA0KPiA+ID4g
RFIyOiAwMDAwMDAwMDAwMDAwMDAwDQo+ID4gPiA8ND5bICAgIDMuNzM3MTAxXSBEUjM6IDAwMDAw
MDAwMDAwMDAwMDAgRFI2OiAwMDAwMDAwMGZmZmUwZmYwDQo+ID4gPiBEUjc6IDAwMDAwMDAwMDAw
MDA0MDANCj4gPiA+IDw0PlsgICAgMy43MzcxMDddIENhbGwgVHJhY2U6DQo+ID4gPiA8ND5bICAg
IDMuNzM3MTc1XSAgaW50ZWxfcmVhZF9kcF9zZHArMHgxYTQvMHgzODAgW2k5MTVdDQo+ID4gPiA8
ND5bICAgIDMuNzM3MjQ2XSAgaHN3X2NydF9nZXRfY29uZmlnKzB4MTIvMHg0MCBbaTkxNV0NCj4g
PiA+IDw0PlsgICAgMy43MzczMTddICBpbnRlbF9tb2Rlc2V0X3NldHVwX2h3X3N0YXRlKzB4M2Iz
LzB4MTZhMA0KPiA+ID4gW2k5MTVdDQo+ID4gPiAuLi4NCj4gPiANCj4gPiAtLSANCj4gPiBKYW5p
IE5pa3VsYSwgSW50ZWwgT3BlbiBTb3VyY2UgR3JhcGhpY3MgQ2VudGVyDQo

WARNING: multiple messages have this Message-ID (diff)
From: "Mun, Gwan-gyeong" <gwan-gyeong.mun@intel.com>
To: "daniel@ffwll.ch" <daniel@ffwll.ch>,
	"jani.nikula@linux.intel.com" <jani.nikula@linux.intel.com>
Cc: "linux-fbdev@vger.kernel.org" <linux-fbdev@vger.kernel.org>,
	"daniel.vetter@ffwll.ch" <daniel.vetter@ffwll.ch>,
	"intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>,
	"dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>,
	"laurent.pinchart@ideasonboard.com"
	<laurent.pinchart@ideasonboard.com>
Subject: Re: [PATCH v12 00/14] In order to readout DP SDPs, refactors the handling of DP SDPs
Date: Fri, 15 May 2020 17:22:43 +0000	[thread overview]
Message-ID: <e2cf963c6db302d94195f4af643d8fbad3ece5dc.camel@intel.com> (raw)
In-Reply-To: <20200515141426.GI206103@phenom.ffwll.local>

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ä <ville.syrjala@linux.intel.com>
> > wrote:
> > > On Thu, May 14, 2020 at 02:19:23PM +0300, Jani Nikula wrote:
> > > > On Thu, 14 May 2020, Gwan-gyeong Mun <gwan-gyeong.mun@intel.com
> > > > > 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:
> > > 0000000000000000
> > > <1>[    3.736916] #PF: supervisor read access in kernel mode
> > > <1>[    3.736916] #PF: error_code(0x0000) - not-present page
> > > <6>[    3.736917] PGD 0 P4D 0 
> > > <4>[    3.736919] Oops: 0000 [#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:ffffc9000047f8a0 EFLAGS: 00010286
> > > <4>[    3.737042] RAX: 0000000000000002 RBX: ffff8883ffd04000
> > > RCX: 0000000000000001
> > > <4>[    3.737048] RDX: 0000000000000007 RSI: ffff8883ffd04000
> > > RDI: 0000000000000128
> > > <4>[    3.737055] RBP: ffff888406afe200 R08: 000000000000000f
> > > R09: 0000000000000001
> > > <4>[    3.737061] R10: 0000000000000000 R11: 0000000000000000
> > > R12: 0000000000000000
> > > <4>[    3.737068] R13: ffff8883f75d0000 R14: ffff888406afe200
> > > R15: ffff8883f75d0870
> > > <4>[    3.737075] FS:  00007f71618f9680(0000)
> > > GS:ffff88840ec00000(0000) knlGS:0000000000000000
> > > <4>[    3.737082] CS:  0010 DS: 0000 ES: 0000 CR0:
> > > 0000000080050033
> > > <4>[    3.737088] CR2: 0000000000000000 CR3: 0000000402510002
> > > CR4: 00000000001606f0
> > > <4>[    3.737094] DR0: 0000000000000000 DR1: 0000000000000000
> > > DR2: 0000000000000000
> > > <4>[    3.737101] DR3: 0000000000000000 DR6: 00000000fffe0ff0
> > > DR7: 0000000000000400
> > > <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

WARNING: multiple messages have this Message-ID (diff)
From: "Mun, Gwan-gyeong" <gwan-gyeong.mun@intel.com>
To: "daniel@ffwll.ch" <daniel@ffwll.ch>,
	"jani.nikula@linux.intel.com" <jani.nikula@linux.intel.com>
Cc: "linux-fbdev@vger.kernel.org" <linux-fbdev@vger.kernel.org>,
	"daniel.vetter@ffwll.ch" <daniel.vetter@ffwll.ch>,
	"intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>,
	"dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>,
	"laurent.pinchart@ideasonboard.com"
	<laurent.pinchart@ideasonboard.com>
Subject: Re: [Intel-gfx] [PATCH v12 00/14] In order to readout DP SDPs, refactors the handling of DP SDPs
Date: Fri, 15 May 2020 17:22:43 +0000	[thread overview]
Message-ID: <e2cf963c6db302d94195f4af643d8fbad3ece5dc.camel@intel.com> (raw)
In-Reply-To: <20200515141426.GI206103@phenom.ffwll.local>

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ä <ville.syrjala@linux.intel.com>
> > wrote:
> > > On Thu, May 14, 2020 at 02:19:23PM +0300, Jani Nikula wrote:
> > > > On Thu, 14 May 2020, Gwan-gyeong Mun <gwan-gyeong.mun@intel.com
> > > > > 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:
> > > 0000000000000000
> > > <1>[    3.736916] #PF: supervisor read access in kernel mode
> > > <1>[    3.736916] #PF: error_code(0x0000) - not-present page
> > > <6>[    3.736917] PGD 0 P4D 0 
> > > <4>[    3.736919] Oops: 0000 [#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:ffffc9000047f8a0 EFLAGS: 00010286
> > > <4>[    3.737042] RAX: 0000000000000002 RBX: ffff8883ffd04000
> > > RCX: 0000000000000001
> > > <4>[    3.737048] RDX: 0000000000000007 RSI: ffff8883ffd04000
> > > RDI: 0000000000000128
> > > <4>[    3.737055] RBP: ffff888406afe200 R08: 000000000000000f
> > > R09: 0000000000000001
> > > <4>[    3.737061] R10: 0000000000000000 R11: 0000000000000000
> > > R12: 0000000000000000
> > > <4>[    3.737068] R13: ffff8883f75d0000 R14: ffff888406afe200
> > > R15: ffff8883f75d0870
> > > <4>[    3.737075] FS:  00007f71618f9680(0000)
> > > GS:ffff88840ec00000(0000) knlGS:0000000000000000
> > > <4>[    3.737082] CS:  0010 DS: 0000 ES: 0000 CR0:
> > > 0000000080050033
> > > <4>[    3.737088] CR2: 0000000000000000 CR3: 0000000402510002
> > > CR4: 00000000001606f0
> > > <4>[    3.737094] DR0: 0000000000000000 DR1: 0000000000000000
> > > DR2: 0000000000000000
> > > <4>[    3.737101] DR3: 0000000000000000 DR6: 00000000fffe0ff0
> > > DR7: 0000000000000400
> > > <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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2020-05-15 17:22 UTC|newest]

Thread overview: 73+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-14  6:07 [PATCH v12 00/14] In order to readout DP SDPs, refactors the handling of DP SDPs Gwan-gyeong Mun
2020-05-14  6:07 ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07 ` Gwan-gyeong Mun
2020-05-14  6:07 ` [PATCH v12 01/14] video/hdmi: Add Unpack only function for DRM infoframe Gwan-gyeong Mun
2020-05-14  6:07   ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07   ` Gwan-gyeong Mun
2020-05-14  6:23   ` [Intel-gfx] " Mun, Gwan-gyeong
2020-05-14  6:23     ` Mun, Gwan-gyeong
2020-05-14  6:23     ` Mun, Gwan-gyeong
2020-05-14  6:07 ` [PATCH v12 02/14] drm/i915/dp: Read out DP SDPs Gwan-gyeong Mun
2020-05-14  6:07   ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07   ` Gwan-gyeong Mun
2020-05-14  6:07 ` [PATCH v12 03/14] drm: Add logging function for DP VSC SDP Gwan-gyeong Mun
2020-05-14  6:07   ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07   ` Gwan-gyeong Mun
2020-05-14  6:07 ` [PATCH v12 04/14] drm/i915: Include HDMI DRM infoframe in the crtc state dump Gwan-gyeong Mun
2020-05-14  6:07   ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07   ` Gwan-gyeong Mun
2020-05-14  6:07 ` [PATCH v12 05/14] drm/i915: Include DP HDR Metadata Infoframe SDP " Gwan-gyeong Mun
2020-05-14  6:07   ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07   ` Gwan-gyeong Mun
2020-05-14  6:07 ` [PATCH v12 06/14] drm/i915: Include DP VSC " Gwan-gyeong Mun
2020-05-14  6:07   ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07   ` Gwan-gyeong Mun
2020-05-14  6:07 ` [PATCH v12 07/14] drm/i915: Program DP SDPs with computed configs Gwan-gyeong Mun
2020-05-14  6:07   ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07   ` Gwan-gyeong Mun
2020-05-14  6:07 ` [PATCH v12 08/14] drm/i915: Add state readout for DP HDR Metadata Infoframe SDP Gwan-gyeong Mun
2020-05-14  6:07   ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07   ` Gwan-gyeong Mun
2020-05-14  6:07 ` [PATCH v12 09/14] drm/i915: Add state readout for DP VSC SDP Gwan-gyeong Mun
2020-05-14  6:07   ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07   ` Gwan-gyeong Mun
2020-05-14  6:07 ` [PATCH v12 10/14] drm/i915: Fix enabled infoframe states of lspcon Gwan-gyeong Mun
2020-05-14  6:07   ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07   ` Gwan-gyeong Mun
2020-05-14  6:07 ` [PATCH v12 11/14] drm/i915: Program DP SDPs on pipe updates Gwan-gyeong Mun
2020-05-14  6:07   ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07   ` Gwan-gyeong Mun
2020-05-14  6:07 ` [PATCH v12 12/14] drm/i915: Stop sending DP SDPs on ddi disable Gwan-gyeong Mun
2020-05-14  6:07   ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07   ` Gwan-gyeong Mun
2020-05-14  6:07 ` [PATCH v12 13/14] drm/i915/dp: Add compute routine for DP PSR VSC SDP Gwan-gyeong Mun
2020-05-14  6:07   ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07   ` Gwan-gyeong Mun
2020-05-14  6:07 ` [PATCH v12 14/14] drm/i915/psr: Use new DP VSC SDP compute routine on PSR Gwan-gyeong Mun
2020-05-14  6:07   ` [Intel-gfx] " Gwan-gyeong Mun
2020-05-14  6:07   ` Gwan-gyeong Mun
2020-05-14  6:24 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for In order to readout DP SDPs, refactors the handling of DP SDPs (rev14) Patchwork
2020-05-14  6:48 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2020-05-14  9:07 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
2020-05-14 11:19 ` [PATCH v12 00/14] In order to readout DP SDPs, refactors the handling of DP SDPs Jani Nikula
2020-05-14 11:19   ` [Intel-gfx] " Jani Nikula
2020-05-14 11:19   ` Jani Nikula
2020-05-15 13:06   ` Ville Syrjälä
2020-05-15 13:06     ` [Intel-gfx] " Ville Syrjälä
2020-05-15 13:06     ` Ville Syrjälä
2020-05-15 13:13     ` Jani Nikula
2020-05-15 13:13       ` [Intel-gfx] " Jani Nikula
2020-05-15 13:13       ` Jani Nikula
2020-05-15 14:14       ` Daniel Vetter
2020-05-15 14:14         ` [Intel-gfx] " Daniel Vetter
2020-05-15 14:14         ` Daniel Vetter
2020-05-15 17:22         ` Mun, Gwan-gyeong [this message]
2020-05-15 17:22           ` [Intel-gfx] " Mun, Gwan-gyeong
2020-05-15 17:22           ` Mun, Gwan-gyeong
2020-05-18  9:43           ` Jani Nikula
2020-05-18  9:43             ` [Intel-gfx] " Jani Nikula
2020-05-18  9:43             ` Jani Nikula
2020-05-15 14:25       ` [Intel-gfx] " Saarinen, Jani
2020-05-15 14:25         ` Saarinen, Jani
2020-05-15 14:25         ` Saarinen, Jani
2020-05-15 14:11 ` [Intel-gfx] ✓ Fi.CI.BAT: success for In order to readout DP SDPs, refactors the handling of DP SDPs (rev14) Patchwork

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=e2cf963c6db302d94195f4af643d8fbad3ece5dc.camel@intel.com \
    --to=gwan-gyeong.mun@intel.com \
    --cc=daniel.vetter@ffwll.ch \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=jani.nikula@linux.intel.com \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-fbdev@vger.kernel.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.