From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH] drm/i915: sync dp link status checks against atomic commmits Date: Wed, 8 Nov 2017 15:26:15 +0200 Message-ID: <20171108132615.GB10981@intel.com> References: <20171108105450.20339-2-daniel.vetter@ffwll.ch> <20171108125750.1663-1-daniel.vetter@ffwll.ch> <20171108130458.GA10981@intel.com> <20171108131146.2m3e4opmxgoaplb7@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <20171108131146.2m3e4opmxgoaplb7@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Daniel Vetter Cc: Daniel Vetter , Intel Graphics Development , DRI Development , Daniel Vetter List-Id: intel-gfx@lists.freedesktop.org T24gV2VkLCBOb3YgMDgsIDIwMTcgYXQgMDI6MTE6NDZQTSArMDEwMCwgRGFuaWVsIFZldHRlciB3 cm90ZToKPiBPbiBXZWQsIE5vdiAwOCwgMjAxNyBhdCAwMzowNDo1OFBNICswMjAwLCBWaWxsZSBT eXJqw6Rsw6Qgd3JvdGU6Cj4gPiBPbiBXZWQsIE5vdiAwOCwgMjAxNyBhdCAwMTo1Nzo1MFBNICsw MTAwLCBEYW5pZWwgVmV0dGVyIHdyb3RlOgo+ID4gPiBUd28gYml0czoKPiA+ID4gLSBjaGVjayBh Y3R1YWwgYXRvbWljIHN0YXRlLCB0aGUgbGVnYWN5IHN0dWZmIGNhbiBvbmx5IGJlIGxvb2tlZCBh dAo+ID4gPiAgIGZyb20gd2l0aGluIHRoZSBhdG9taWNfY29tbWl0X3RhaWwgZnVuY3Rpb24sIHNp bmNlIGl0J3Mgb25seQo+ID4gPiAgIHByb3RlY3RlZCBieSBvcmRlcmluZyBhbmQgbm90IGJ5IGFu eSBsb2Nrcy4KPiA+ID4gCj4gPiA+IC0gTWFrZSBzdXJlIHdlIGRvbid0IHdyZWFrIHRoZSB3b3Jr IGFuIG9uZ29pbmcgbm9uYmxvY2tpbmcgY29tbWl0IGlzCj4gPiA+ICAgZG9pbmcuCj4gPiAKPiA+ IEkgc3RpbGwgdGhpbmsgd2UgbmVlZCB0byBtb3ZlIHRoZSByZXRyYWluaW5nIHRvIHRoZSBob3Rw bHVnIHdvcmsuCj4gCj4gV2h5PyBPbmUgb2YgdGhlIHBhdGNoIHNlcmllcyBNYWFydGVuIG1lbnRp b25lZCBzYXlzIHRoZXJlJ3MgYSBkZWFkbG9jawo+IHdpdGggZHAgYXV4LCBidXQgaXQgc2VlbXMg dG8gYmUgYWxsIGp1c3QgZmluZSB3aGVuIENJIHJldHJhaW5zLgoKSSBndWVzcyB0aGUgZGVhZGxv Y2sgaXMgcG9zc2libGUgb25seSB3aXRoIE1TVCwgbWF5YmUuIEN1cnJlbnRseSBNU1QKaGFzIGl0 J3Mgb3duIGNvcHkgb2YgdGhlIHJldHJhaW5pbmcgY29kZSB3aXRob3V0IHRoZSBsb2Nrcy4KCkF0 IG9uZSBwb2ludCBJIHN0YXJ0ZWQgdG8gcmV3cml0ZSB0aGUgZW50aXJlIHNpbmsgaXJxIGhhbmRs aW5nIGludG8gYQptdWNoIG5pY2VyIHNoYXBlLCBhbHNvIHVuaWZ5aW5nIHRoZSBhcHByb2FjaCBi ZXR3ZWVuIE1TVCBhbmQgU1NULiBJSVJDCkkgZGlkIHN0aWxsIG1ha2UgdGhlIG1pc3Rha2Ugb2Yg aGF2aW5nIHNvbWUga2luZCBvZiBoaWdoZXIgbGV2ZWwgTVNUCnZzLiBTU1Qgc3BsaXQsIGJ1dCBJ IHRoaW5rIHRoZSBwcm9wZXIgc29sdXRpb24gaXMgdG8gY29tYmluZSB0aGUgdHdvCmFsbW9zdCBl bnRpcmVseS4gSSB0aGluayB3ZSBzaG91bGQgZXZlbiBiZSB1c2luZyB0aGUgRVNJIHJlZ2lzdGVy cwp3aXRoIFNTVCBmb3IgRFBDRCAxLjIrLiBDdXJyZW50bHkgd2UgdXNlIEVTSSBmb3IgTVNUIGFu ZCBub24tRVNJIGZvcgpTU1QuIFNhZGx5IEkndmUgbm90IGZvdW5kIHRoZSB0aW1lIHRvIGNvbnRp bnVlIHRoYXQgd29yayAoc2FtZSBzdG9yeQp3aXRoIHRoZSAic2h1dGRvd24gZGlzcGxheXMgcHJp b3IgdG8gcmVib290aW5nIHRvIGtlZXAgRGVsbCBNU1QKbW9uaXRvcnMgd29ya2luZyIgdGhpbmcp LgoKPiAKPiBBbmQgZXZlbiBpZiB0aGVyZSBpcyBhIGRlYWRsb2NrLCBpdCdzIHRoZXJlIGFscmVh ZHksIHNvIG5vdCBzdXJlIHdoeSB3ZQo+IG5lZWQgdG8gYmxvY2sgdGhpcyBidWdmaXggb24gYSBk aWZmZXJlbnQgYnVnZml4LiBXaGljaCBzZWVtcyB0byBiZSB3aGF0Cj4geW91J3JlIHN1Z2dlc3Rp bmcgaGVyZSAoYnV0IGl0J3MgYSBiaXQgc3BhcnNlKS4KCkkgZ3Vlc3Mgd2hhdCBJJ20gcmVhbGx5 IHNheWluZyBpcyB0aGF0IHdlIHNob3VsZG4ndCBzdG9wIGhlcmUuCgo+IC1EYW5pZWwKPiAKPiA+ ID4gdjI6IFdlIG5lZWQgdGhlIGNydGMgbG9jayB0b28sIGJlY2F1c2UgYSBwbGFuZSB1cGRhdGUg bWlnaHQgY2hhbmdlIGl0Cj4gPiA+IHdpdGhvdXQgaGF2aW5nIHRvIGFjcXVpcmUgdGhlIGNvbm5l Y3Rpb25fbXV0ZXggKE1hYXJ0ZW4pLiBVc2UKPiA+ID4gTWFhcnRlbidzIGNoYW5nZXMgZm9yIHRo aXMgbG9ja2luZywgd2hpbGUga2VlcGluZyB0aGUgbG9naWMgdGhhdCB1c2VzCj4gPiA+IHRoZSBj b25uZWN0aW9uLT5jb21taXQtPmh3X2RvbmUgc2lnbmFsIGZvciBzeW5jaW5nIHdpdGggbm9uYmxv Y2tpbmcKPiA+ID4gY29tbWl0cy4KPiA+ID4gCj4gPiA+IENjOiBNYW5hc2kgTmF2YXJlIDxtYW5h c2kuZC5uYXZhcmVAaW50ZWwuY29tPgo+ID4gPiBDYzogTWFhcnRlbiBMYW5raG9yc3QgPG1hYXJ0 ZW4ubGFua2hvcnN0QGxpbnV4LmludGVsLmNvbT4KPiA+ID4gQ2M6IFZpbGxlIFN5cmrDpGzDpCA8 dmlsbGUuc3lyamFsYUBsaW51eC5pbnRlbC5jb20+Cj4gPiA+IFJlZmVyZW5jZXM6IGh0dHBzOi8v YnVncy5mcmVlZGVza3RvcC5vcmcvc2hvd19idWcuY2dpP2lkPTEwMzMzNgo+ID4gPiBCdWd6aWxs YTogaHR0cHM6Ly9idWdzLmZyZWVkZXNrdG9wLm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTkyNzIKPiA+ ID4gU2lnbmVkLW9mZi1ieTogRGFuaWVsIFZldHRlciA8ZGFuaWVsLnZldHRlckBpbnRlbC5jb20+ Cj4gPiA+IC0tLQo+ID4gPiAgZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZHAuYyB8IDU2ICsr KysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKy0tLS0tCj4gPiA+ICAxIGZpbGUgY2hh bmdlZCwgNTAgaW5zZXJ0aW9ucygrKSwgNiBkZWxldGlvbnMoLSkKPiA+ID4gCj4gPiA+IGRpZmYg LS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kcC5jIGIvZHJpdmVycy9ncHUvZHJt L2k5MTUvaW50ZWxfZHAuYwo+ID4gPiBpbmRleCBkMjdjMDE0NWFjOTEuLjdjZDdhYjRmYjQzMSAx MDA2NDQKPiA+ID4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZHAuYwo+ID4gPiAr KysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kcC5jCj4gPiA+IEBAIC00MzE5LDYgKzQz MTksOCBAQCBzdGF0aWMgdm9pZAo+ID4gPiAgaW50ZWxfZHBfY2hlY2tfbGlua19zdGF0dXMoc3Ry dWN0IGludGVsX2RwICppbnRlbF9kcCkKPiA+ID4gIHsKPiA+ID4gIAlzdHJ1Y3QgaW50ZWxfZW5j b2RlciAqaW50ZWxfZW5jb2RlciA9ICZkcF90b19kaWdfcG9ydChpbnRlbF9kcCktPmJhc2U7Cj4g PiA+ICsJc3RydWN0IGRybV9jb25uZWN0b3Jfc3RhdGUgKmNvbm5fc3RhdGUgPQo+ID4gPiArCQlp bnRlbF9kcC0+YXR0YWNoZWRfY29ubmVjdG9yLT5iYXNlLnN0YXRlOwo+ID4gPiAgCXN0cnVjdCBk cm1fZGV2aWNlICpkZXYgPSBpbnRlbF9kcF90b19kZXYoaW50ZWxfZHApOwo+ID4gPiAgCXU4IGxp bmtfc3RhdHVzW0RQX0xJTktfU1RBVFVTX1NJWkVdOwo+ID4gPiAgCj4gPiA+IEBAIC00MzI5LDEw ICs0MzMxLDE1IEBAIGludGVsX2RwX2NoZWNrX2xpbmtfc3RhdHVzKHN0cnVjdCBpbnRlbF9kcCAq aW50ZWxfZHApCj4gPiA+ICAJCXJldHVybjsKPiA+ID4gIAl9Cj4gPiA+ICAKPiA+ID4gLQlpZiAo IWludGVsX2VuY29kZXItPmJhc2UuY3J0YykKPiA+ID4gKwlpZiAoIWNvbm5fc3RhdGUtPmNydGMp Cj4gPiA+ICAJCXJldHVybjsKPiA+ID4gIAo+ID4gPiAtCWlmICghdG9faW50ZWxfY3J0YyhpbnRl bF9lbmNvZGVyLT5iYXNlLmNydGMpLT5hY3RpdmUpCj4gPiA+ICsJV0FSTl9PTighZHJtX21vZGVz ZXRfaXNfbG9ja2VkKCZjb25uX3N0YXRlLT5jcnRjLT5tdXRleCkpOwo+ID4gPiArCj4gPiA+ICsJ aWYgKCFjb25uX3N0YXRlLT5jcnRjLT5zdGF0ZS0+YWN0aXZlKQo+ID4gPiArCQlyZXR1cm47Cj4g PiA+ICsKPiA+ID4gKwlpZiAoIXRyeV93YWl0X2Zvcl9jb21wbGV0aW9uKCZjb25uX3N0YXRlLT5j b21taXQtPmh3X2RvbmUpKQo+ID4gPiAgCQlyZXR1cm47Cj4gPiA+ICAKPiA+ID4gIAkvKgo+ID4g PiBAQCAtNDM2OCw3ICs0Mzc1LDYgQEAgaW50ZWxfZHBfY2hlY2tfbGlua19zdGF0dXMoc3RydWN0 IGludGVsX2RwICppbnRlbF9kcCkKPiA+ID4gIHN0YXRpYyBib29sCj4gPiA+ICBpbnRlbF9kcF9z aG9ydF9wdWxzZShzdHJ1Y3QgaW50ZWxfZHAgKmludGVsX2RwKQo+ID4gPiAgewo+ID4gPiAtCXN0 cnVjdCBkcm1fZGV2aWNlICpkZXYgPSBpbnRlbF9kcF90b19kZXYoaW50ZWxfZHApOwo+ID4gPiAg CXN0cnVjdCBpbnRlbF9lbmNvZGVyICppbnRlbF9lbmNvZGVyID0gJmRwX3RvX2RpZ19wb3J0KGlu dGVsX2RwKS0+YmFzZTsKPiA+ID4gIAl1OCBzaW5rX2lycV92ZWN0b3IgPSAwOwo+ID4gPiAgCXU4 IG9sZF9zaW5rX2NvdW50ID0gaW50ZWxfZHAtPnNpbmtfY291bnQ7Cj4gPiA+IEBAIC00NDA4LDkg KzQ0MTQsOCBAQCBpbnRlbF9kcF9zaG9ydF9wdWxzZShzdHJ1Y3QgaW50ZWxfZHAgKmludGVsX2Rw KQo+ID4gPiAgCQkJRFJNX0RFQlVHX0RSSVZFUigiQ1Agb3Igc2luayBzcGVjaWZpYyBpcnEgdW5o YW5kbGVkXG4iKTsKPiA+ID4gIAl9Cj4gPiA+ICAKPiA+ID4gLQlkcm1fbW9kZXNldF9sb2NrKCZk ZXYtPm1vZGVfY29uZmlnLmNvbm5lY3Rpb25fbXV0ZXgsIE5VTEwpOwo+ID4gPiAgCWludGVsX2Rw X2NoZWNrX2xpbmtfc3RhdHVzKGludGVsX2RwKTsKPiA+ID4gLQlkcm1fbW9kZXNldF91bmxvY2so JmRldi0+bW9kZV9jb25maWcuY29ubmVjdGlvbl9tdXRleCk7Cj4gPiA+ICsKPiA+ID4gIAlpZiAo aW50ZWxfZHAtPmNvbXBsaWFuY2UudGVzdF90eXBlID09IERQX1RFU1RfTElOS19UUkFJTklORykg ewo+ID4gPiAgCQlEUk1fREVCVUdfS01TKCJMaW5rIFRyYWluaW5nIENvbXBsaWFuY2UgVGVzdCBy ZXF1ZXN0ZWRcbiIpOwo+ID4gPiAgCQkvKiBTZW5kIGEgSG90cGx1ZyBVZXZlbnQgdG8gdXNlcnNw YWNlIHRvIHN0YXJ0IG1vZGVzZXQgKi8KPiA+ID4gQEAgLTQ4NjAsOCArNDg2NSwxOSBAQCBpbnRl bF9kcF9kZXRlY3Qoc3RydWN0IGRybV9jb25uZWN0b3IgKmNvbm5lY3RvciwKPiA+ID4gIAkJICAg ICAgY29ubmVjdG9yLT5iYXNlLmlkLCBjb25uZWN0b3ItPm5hbWUpOwo+ID4gPiAgCj4gPiA+ICAJ LyogSWYgZnVsbCBkZXRlY3QgaXMgbm90IHBlcmZvcm1lZCB5ZXQsIGRvIGEgZnVsbCBkZXRlY3Qg Ki8KPiA+ID4gLQlpZiAoIWludGVsX2RwLT5kZXRlY3RfZG9uZSkKPiA+ID4gKwlpZiAoIWludGVs X2RwLT5kZXRlY3RfZG9uZSkgewo+ID4gPiArCQlzdHJ1Y3QgZHJtX2NydGMgKmNydGM7Cj4gPiA+ ICsJCWludCByZXQ7Cj4gPiA+ICsKPiA+ID4gKwkJY3J0YyA9IGNvbm5lY3Rvci0+c3RhdGUtPmNy dGM7Cj4gPiA+ICsJCWlmIChjcnRjKSB7Cj4gPiA+ICsJCQlyZXQgPSBkcm1fbW9kZXNldF9sb2Nr KCZjcnRjLT5tdXRleCwgY3R4KTsKPiA+ID4gKwkJCWlmIChyZXQpCj4gPiA+ICsJCQkJcmV0dXJu IHJldDsKPiA+ID4gKwkJfQo+ID4gPiArCj4gPiA+ICAJCXN0YXR1cyA9IGludGVsX2RwX2xvbmdf cHVsc2UoaW50ZWxfZHAtPmF0dGFjaGVkX2Nvbm5lY3Rvcik7Cj4gPiA+ICsJfQo+ID4gPiAgCj4g PiA+ICAJaW50ZWxfZHAtPmRldGVjdF9kb25lID0gZmFsc2U7Cj4gPiA+ICAKPiA+ID4gQEAgLTUx NDYsMTAgKzUxNjIsMzggQEAgaW50ZWxfZHBfaHBkX3B1bHNlKHN0cnVjdCBpbnRlbF9kaWdpdGFs X3BvcnQgKmludGVsX2RpZ19wb3J0LCBib29sIGxvbmdfaHBkKQo+ID4gPiAgCX0KPiA+ID4gIAo+ ID4gPiAgCWlmICghaW50ZWxfZHAtPmlzX21zdCkgewo+ID4gPiArCQlzdHJ1Y3QgZHJtX21vZGVz ZXRfYWNxdWlyZV9jdHggY3R4Owo+ID4gPiArCQlzdHJ1Y3QgZHJtX2Nvbm5lY3RvciAqY29ubmVj dG9yID0gJmludGVsX2RwLT5hdHRhY2hlZF9jb25uZWN0b3ItPmJhc2U7Cj4gPiA+ICsJCXN0cnVj dCBkcm1fY3J0YyAqY3J0YzsKPiA+ID4gKwkJaW50IGlyZXQ7Cj4gPiA+ICsKPiA+ID4gKwkJZHJt X21vZGVzZXRfYWNxdWlyZV9pbml0KCZjdHgsIDApOwo+ID4gPiArcmV0cnk6Cj4gPiA+ICsJCWly ZXQgPSBkcm1fbW9kZXNldF9sb2NrKCZkZXYtPm1vZGVfY29uZmlnLmNvbm5lY3Rpb25fbXV0ZXgs ICZjdHgpOwo+ID4gPiArCQlpZiAoaXJldCkKPiA+ID4gKwkJCWdvdG8gZXJyOwo+ID4gPiArCj4g PiA+ICsJCWNydGMgPSBjb25uZWN0b3ItPnN0YXRlLT5jcnRjOwo+ID4gPiArCQlpZiAoY3J0Yykg ewo+ID4gPiArCQkJaXJldCA9IGRybV9tb2Rlc2V0X2xvY2soJmNydGMtPm11dGV4LCAmY3R4KTsK PiA+ID4gKwkJCWlmIChpcmV0KQo+ID4gPiArCQkJCWdvdG8gZXJyOwo+ID4gPiArCQl9Cj4gPiA+ ICsKPiA+ID4gIAkJaWYgKCFpbnRlbF9kcF9zaG9ydF9wdWxzZShpbnRlbF9kcCkpIHsKPiA+ID4g IAkJCWludGVsX2RwLT5kZXRlY3RfZG9uZSA9IGZhbHNlOwo+ID4gPiAgCQkJZ290byBwdXRfcG93 ZXI7Cj4gPiA+ICAJCX0KPiA+ID4gKwo+ID4gPiArZXJyOgo+ID4gPiArCQlpZiAoaXJldCA9PSAt RURFQURMSykgewo+ID4gPiArCQkJZHJtX21vZGVzZXRfYmFja29mZigmY3R4KTsKPiA+ID4gKwkJ CWdvdG8gcmV0cnk7Cj4gPiA+ICsJCX0KPiA+ID4gKwo+ID4gPiArCQlkcm1fbW9kZXNldF9kcm9w X2xvY2tzKCZjdHgpOwo+ID4gPiArCQlkcm1fbW9kZXNldF9hY3F1aXJlX2ZpbmkoJmN0eCk7Cj4g PiA+ICsJCVdBUk4oaXJldCwgIkFjcXVpcmluZyBtb2Rlc2V0IGxvY2tzIGZhaWxlZCB3aXRoICVp XG4iLCBpcmV0KTsKPiA+ID4gIAl9Cj4gPiA+ICAKPiA+ID4gIAlyZXQgPSBJUlFfSEFORExFRDsK PiA+ID4gLS0gCj4gPiA+IDIuMTUuMAo+ID4gCj4gPiAtLSAKPiA+IFZpbGxlIFN5cmrDpGzDpAo+ ID4gSW50ZWwgT1RDCj4gCj4gLS0gCj4gRGFuaWVsIFZldHRlcgo+IFNvZnR3YXJlIEVuZ2luZWVy LCBJbnRlbCBDb3Jwb3JhdGlvbgo+IGh0dHA6Ly9ibG9nLmZmd2xsLmNoCgotLSAKVmlsbGUgU3ly asOkbMOkCkludGVsIE9UQwpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVza3Rv cC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRl bC1nZngK