From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH v2 2/2] drm/i915: Wait for pending modesets to complete before trying link training, v2. Date: Wed, 14 Jun 2017 16:44:24 +0300 Message-ID: <20170614134424.GN12629@intel.com> References: <20170614090842.9669-1-maarten.lankhorst@linux.intel.com> <20170614090842.9669-2-maarten.lankhorst@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <20170614090842.9669-2-maarten.lankhorst@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Maarten Lankhorst Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T24gV2VkLCBKdW4gMTQsIDIwMTcgYXQgMTE6MDg6NDJBTSArMDIwMCwgTWFhcnRlbiBMYW5raG9y c3Qgd3JvdGU6Cj4gVGhlIG5vbmJsb2NraW5nIG1vZGVzZXQgdGVzdHMgd2VyZSBmYWlsaW5nIG9u IHN5c3RlbXMgd2l0aCBhIERQIG91dHB1dCwKPiBiZWNhdXNlIGxpam5rIHRyYWluaW5nIGNvdWxk IG9jY3VyIGR1cmluZyB0aGUgbW9kZXNldC4KPiAKPiBXaXRoIG5vcm1hbCBtb2Rlc2V0cyB3ZSdy ZSBwcm90ZWN0ZWQgYnkgdGhlIGNvbm5lY3Rpb25fbXV0ZXggbG9jaywKPiBidXQgbm9uYmxvY2tp bmcgbW9kZXNldHMgZHJvcCBsb2NrcyBiZWZvcmUgY29tcGxldGluZy4gVG8gZ2V0IGFyb3VuZAo+ IHRoYXQsIGNoZWNrIGlmIHRoZSBtb3N0IHJlY2VudCBjcnRjX3N0YXRlIGhhcyBhIG1vZGVzZXQs IGFuZCBpZiB0aGlzCj4gaXMgdGhlIGNhc2UsIHdhaXQgZm9yIGFsbCBvdXRzdGFuZGluZyBhdG9t aWMgdXBkYXRlcyBvbiB0aGF0IGNydGMKPiB0byBjb21wbGV0ZS4KPiAKPiBUaGlzIGZpeGVzIHRo ZSBmb2xsb3dpbmcgc3BldyB3aGVuIHJ1bm5pbmcgbm9uYmxvY2tpbmcgdGVzdHM6Cj4gWyAxMDYx LjY1Mzc3OF0gLS0tLS0tLS0tLS0tWyBjdXQgaGVyZSBdLS0tLS0tLS0tLS0tCj4gWyAxMDYxLjY1 Mzc4MV0gV0FSTklORzogQ1BVOiAxIFBJRDogODE2NSBhdAo+IGRyaXZlcnMvZ3B1L2RybS9kcm1f aXJxLmM6MTIxNyBkcm1fd2FpdF9vbmVfdmJsYW5rKzB4MTY1LzB4MWEwCj4gWyAxMDYxLjY1Mzc4 Ml0gdmJsYW5rIG5vdCBhdmFpbGFibGUgb24gY3J0YyAwLCByZXQ9LTIyCj4gWyAxMDYxLjY1Mzc4 M10gTW9kdWxlcyBsaW5rZWQgaW46IHZnZW0gc25kX2hkYV9pbnRlbCBpOTE1Cj4gc25kX2hkYV9j b2RlY19oZG1pIHNuZF9oZGFfY29kZWNfcmVhbHRlayBzbmRfaGRhX2NvZGVjX2dlbmVyaWMKPiB4 ODZfcGtnX3RlbXBfdGhlcm1hbCBpbnRlbF9wb3dlcmNsYW1wIGNvcmV0ZW1wIGNyY3QxMGRpZl9w Y2xtdWwKPiBjcmMzMl9wY2xtdWwgZ2hhc2hfY2xtdWxuaV9pbnRlbCBzbmRfaGRhX2NvZGVjIHNu ZF9od2RlcCBzbmRfaGRhX2NvcmUKPiBzbmRfcGNtIG1laV9tZSBtZWkgZTEwMDBlIHB0cCBwcHNf Y29yZSBpMmNfaGlkIFtsYXN0IHVubG9hZGVkOiB2Z2VtXQo+IFsgMTA2MS42NTM3OTZdIENQVTog MSBQSUQ6IDgxNjUgQ29tbToga3dvcmtlci8xOjMgVGFpbnRlZDogRyAgICAgVSAgVwo+IDQuMTAu MC1yYzEtQ0ktQ3VzdG9tXzExNCsgIzEKPiBbIDEwNjEuNjUzNzk3XSBIYXJkd2FyZSBuYW1lOiBH SUdBQllURSBHQi1CS2k1QS03MjAwL01GTFA1QVAtMDAsIEJJT1MgRjEKPiAwNy8yNy8yMDE2Cj4g WyAxMDYxLjY1MzgxMl0gV29ya3F1ZXVlOiBldmVudHMgaTkxNV9ocGRfcG9sbF9pbml0X3dvcmsg W2k5MTVdCj4gWyAxMDYxLjY1MzgxM10gQ2FsbCBUcmFjZToKPiBbIDEwNjEuNjUzODE2XSAgZHVt cF9zdGFjaysweDY3LzB4OTIKPiBbIDEwNjEuNjUzODE4XSAgX193YXJuKzB4YzYvMHhlMAo+IFsg MTA2MS42NTM4MTldICB3YXJuX3Nsb3dwYXRoX2ZtdCsweDRhLzB4NTAKPiBbIDEwNjEuNjUzODIw XSAgPyBkcm1fdmJsYW5rX2dldCsweDc0LzB4YzAKPiBbIDEwNjEuNjUzODIxXSAgZHJtX3dhaXRf b25lX3ZibGFuaysweDE2NS8weDFhMAo+IFsgMTA2MS42NTM4MjNdICA/IGRybV9kcF9kcGNkX3dy aXRlKzB4MTYvMHgyMAo+IFsgMTA2MS42NTM4MzddICA/IGludGVsX2RwX3NldF9saW5rX3RyYWlu KzB4NDQvMHhlMCBbaTkxNV0KPiBbIDEwNjEuNjUzODUxXSAgaW50ZWxfZHBfY2hlY2tfbGlua19z dGF0dXMrMHgxMTMvMHgxYTAgW2k5MTVdCj4gWyAxMDYxLjY1Mzg2NF0gIGludGVsX2RwX2RldGVj dCsweDVhMS8weGE0MCBbaTkxNV0KPiBbIDEwNjEuNjUzODY2XSAgZHJtX2hlbHBlcl9ocGRfaXJx X2V2ZW50KzB4OGQvMHgxMTAKPiBbIDEwNjEuNjUzODgwXSAgaTkxNV9ocGRfcG9sbF9pbml0X3dv cmsrMHhhYi8weGQwIFtpOTE1XQo+IFsgMTA2MS42NTM4ODJdICBwcm9jZXNzX29uZV93b3JrKzB4 MWY0LzB4NmQwCj4gWyAxMDYxLjY1Mzg4M10gID8gcHJvY2Vzc19vbmVfd29yaysweDE2ZS8weDZk MAo+IFsgMTA2MS42NTM4ODVdICB3b3JrZXJfdGhyZWFkKzB4NDkvMHg0YTAKPiBbIDEwNjEuNjUz ODg2XSAga3RocmVhZCsweDEwNy8weDE0MAo+IFsgMTA2MS42NTM4ODhdICA/IHByb2Nlc3Nfb25l X3dvcmsrMHg2ZDAvMHg2ZDAKPiBbIDEwNjEuNjUzODg5XSAgPyBrdGhyZWFkX2NyZWF0ZV9vbl9u b2RlKzB4NDAvMHg0MAo+IFsgMTA2MS42NTM4OTFdICByZXRfZnJvbV9mb3JrKzB4MjcvMHg0MAo+ IFsgMTA2MS42NTM4OTJdIC0tLVsgZW5kIHRyYWNlIGY2MDM3NjE4NjY4ZDRjYmMgXS0tLQo+IFsg MTA2MS42NTYzNTNdIFtkcm06aW50ZWxfZHBfc2V0X2lkbGVfbGlua190cmFpbiBbaTkxNV1dICpF UlJPUiogVGltZWQgb3V0IHdhaXRpbmcgZm9yIERQIGlkbGUgcGF0dGVybnMKPiAKPiBDaGFuZ2Vz IHNpbmNlIHYxOgo+IC0gaW50ZWxfZHBfZGV0ZWN0KCkgbm93IGhvbGRzIGNvbm5lY3Rpb25fbXV0 ZXggYW5kIGhhcyBhbiBhY3F1aXJlX2N0eCwKPiAgIHJld3JpdGUgcGF0Y2ggdG8gdGFrZSBhZHZh bnRhZ2Ugb2YgaXQuCj4gCj4gU2lnbmVkLW9mZi1ieTogTWFhcnRlbiBMYW5raG9yc3QgPG1hYXJ0 ZW4ubGFua2hvcnN0QGxpbnV4LmludGVsLmNvbT4KPiBCdWd6aWxsYTogaHR0cHM6Ly9idWdzLmZy ZWVkZXNrdG9wLm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTkyNzIKPiBUZXN0Y2FzZToga21zX2F0b21p Y190cmFuc2l0aW9ucy4xeC1tb2Rlc2V0LXRyYW5zaXRpb25zLW5vbmJsb2NraW5nCj4gLS0tCj4g IGRyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2RwLmMgfCA3MiArKysrKysrKysrKysrKysrKysr KysrKysrKysrKysrKysrLS0tLS0tLQo+ICAxIGZpbGUgY2hhbmdlZCwgNjAgaW5zZXJ0aW9ucygr KSwgMTIgZGVsZXRpb25zKC0pCj4gCj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1 L2ludGVsX2RwLmMgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kcC5jCj4gaW5kZXggNjRm YTc3NGM4NTViLi5jN2E2Njg4M2U0NTMgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5 MTUvaW50ZWxfZHAuYwo+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2RwLmMKPiBA QCAtNDI0MywyMSArNDI0MywzMiBAQCBzdGF0aWMgdm9pZAo+ICBpbnRlbF9kcF9jaGVja19saW5r X3N0YXR1cyhzdHJ1Y3QgaW50ZWxfZHAgKmludGVsX2RwKQo+ICB7Cj4gIAlzdHJ1Y3QgaW50ZWxf ZW5jb2RlciAqaW50ZWxfZW5jb2RlciA9ICZkcF90b19kaWdfcG9ydChpbnRlbF9kcCktPmJhc2U7 Cj4gLQlzdHJ1Y3QgZHJtX2RldmljZSAqZGV2ID0gaW50ZWxfZHBfdG9fZGV2KGludGVsX2RwKTsK PiAgCXU4IGxpbmtfc3RhdHVzW0RQX0xJTktfU1RBVFVTX1NJWkVdOwo+ICsJc3RydWN0IGludGVs X2Nvbm5lY3RvciAqaW50ZWxfY29ubmVjdG9yID0gaW50ZWxfZHAtPmF0dGFjaGVkX2Nvbm5lY3Rv cjsKPiArCXN0cnVjdCBkcm1fY3J0YyAqY3J0YzsKPiAgCj4gLQlXQVJOX09OKCFkcm1fbW9kZXNl dF9pc19sb2NrZWQoJmRldi0+bW9kZV9jb25maWcuY29ubmVjdGlvbl9tdXRleCkpOwo+IC0KPiAt CWlmICghaW50ZWxfZHBfZ2V0X2xpbmtfc3RhdHVzKGludGVsX2RwLCBsaW5rX3N0YXR1cykpIHsK PiAtCQlEUk1fRVJST1IoIkZhaWxlZCB0byBnZXQgbGluayBzdGF0dXNcbiIpOwo+ICsJY3J0YyA9 IGludGVsX2Nvbm5lY3Rvci0+YmFzZS5zdGF0ZS0+Y3J0YzsKPiArCWlmICghY3J0YykKPiAgCQly ZXR1cm47Cj4gLQl9Cj4gIAo+IC0JaWYgKCFpbnRlbF9lbmNvZGVyLT5iYXNlLmNydGMpCj4gKwlX QVJOX09OKCFkcm1fbW9kZXNldF9pc19sb2NrZWQoJmNydGMtPm11dGV4KSk7Cj4gKwo+ICsJaWYg KCFjcnRjLT5zdGF0ZS0+YWN0aXZlKQo+ICAJCXJldHVybjsKPiAgCj4gLQlpZiAoIXRvX2ludGVs X2NydGMoaW50ZWxfZW5jb2Rlci0+YmFzZS5jcnRjKS0+YWN0aXZlKQo+ICsJaWYgKGRybV9hdG9t aWNfY3J0Y19uZWVkc19tb2Rlc2V0KGNydGMtPnN0YXRlKSkgewo+ICsJCWludCByZXQ7Cj4gKwo+ ICsJCS8qIHdhaXQgZm9yIGF0b21pYyBtb2Rlc2V0IHRvIGNvbXBsZXRlICovCj4gKwkJcmV0ID0g ZHJtX2F0b21pY19oZWxwZXJfd2FpdF9mb3JfaHdfZG9uZShjcnRjKTsKPiArCQlpZiAocmV0IDwg MCkKPiArCQkJRFJNX0VSUk9SKCJXYWl0aW5nIGZvciBod19kb25lIHRpbWVkIG91dFxuIik7Cj4g Kwl9Cj4gKwo+ICsJaWYgKCFpbnRlbF9kcF9nZXRfbGlua19zdGF0dXMoaW50ZWxfZHAsIGxpbmtf c3RhdHVzKSkgewo+ICsJCURSTV9FUlJPUigiRmFpbGVkIHRvIGdldCBsaW5rIHN0YXR1c1xuIik7 Cj4gIAkJcmV0dXJuOwo+ICsJfQo+ICAKPiAgCS8qCj4gIAkgKiBWYWxpZGF0ZSB0aGUgY2FjaGVk IHZhbHVlcyBvZiBpbnRlbF9kcC0+bGlua19yYXRlIGFuZAo+IEBAIC00MjkxLDcgKzQzMDIsNiBA QCBpbnRlbF9kcF9jaGVja19saW5rX3N0YXR1cyhzdHJ1Y3QgaW50ZWxfZHAgKmludGVsX2RwKQo+ ICBzdGF0aWMgYm9vbAo+ICBpbnRlbF9kcF9zaG9ydF9wdWxzZShzdHJ1Y3QgaW50ZWxfZHAgKmlu dGVsX2RwKQo+ICB7Cj4gLQlzdHJ1Y3QgZHJtX2RldmljZSAqZGV2ID0gaW50ZWxfZHBfdG9fZGV2 KGludGVsX2RwKTsKPiAgCXN0cnVjdCBpbnRlbF9lbmNvZGVyICppbnRlbF9lbmNvZGVyID0gJmRw X3RvX2RpZ19wb3J0KGludGVsX2RwKS0+YmFzZTsKPiAgCXU4IHNpbmtfaXJxX3ZlY3RvciA9IDA7 Cj4gIAl1OCBvbGRfc2lua19jb3VudCA9IGludGVsX2RwLT5zaW5rX2NvdW50Owo+IEBAIC00MzMx LDkgKzQzNDEsOCBAQCBpbnRlbF9kcF9zaG9ydF9wdWxzZShzdHJ1Y3QgaW50ZWxfZHAgKmludGVs X2RwKQo+ICAJCQlEUk1fREVCVUdfRFJJVkVSKCJDUCBvciBzaW5rIHNwZWNpZmljIGlycSB1bmhh bmRsZWRcbiIpOwo+ICAJfQo+ICAKPiAtCWRybV9tb2Rlc2V0X2xvY2soJmRldi0+bW9kZV9jb25m aWcuY29ubmVjdGlvbl9tdXRleCwgTlVMTCk7Cj4gIAlpbnRlbF9kcF9jaGVja19saW5rX3N0YXR1 cyhpbnRlbF9kcCk7Cj4gLQlkcm1fbW9kZXNldF91bmxvY2soJmRldi0+bW9kZV9jb25maWcuY29u bmVjdGlvbl9tdXRleCk7Cj4gKwo+ICAJaWYgKGludGVsX2RwLT5jb21wbGlhbmNlLnRlc3RfdHlw ZSA9PSBEUF9URVNUX0xJTktfVFJBSU5JTkcpIHsKPiAgCQlEUk1fREVCVUdfS01TKCJMaW5rIFRy YWluaW5nIENvbXBsaWFuY2UgVGVzdCByZXF1ZXN0ZWRcbiIpOwo+ICAJCS8qIFNlbmQgYSBIb3Rw bHVnIFVldmVudCB0byB1c2Vyc3BhY2UgdG8gc3RhcnQgbW9kZXNldCAqLwo+IEBAIC00NzM1LDgg KzQ3NDQsMTkgQEAgaW50ZWxfZHBfZGV0ZWN0KHN0cnVjdCBkcm1fY29ubmVjdG9yICpjb25uZWN0 b3IsCj4gIAkJICAgICAgY29ubmVjdG9yLT5iYXNlLmlkLCBjb25uZWN0b3ItPm5hbWUpOwo+ICAK PiAgCS8qIElmIGZ1bGwgZGV0ZWN0IGlzIG5vdCBwZXJmb3JtZWQgeWV0LCBkbyBhIGZ1bGwgZGV0 ZWN0ICovCj4gLQlpZiAoIWludGVsX2RwLT5kZXRlY3RfZG9uZSkKPiArCWlmICghaW50ZWxfZHAt PmRldGVjdF9kb25lKSB7Cj4gKwkJc3RydWN0IGRybV9jcnRjICpjcnRjOwo+ICsJCWludCByZXQ7 Cj4gKwo+ICsJCWNydGMgPSBjb25uZWN0b3ItPnN0YXRlLT5jcnRjOwo+ICsJCWlmIChjcnRjKSB7 Cj4gKwkJCXJldCA9IGRybV9tb2Rlc2V0X2xvY2soJmNydGMtPm11dGV4LCBjdHgpOwo+ICsJCQlp ZiAocmV0KQo+ICsJCQkJcmV0dXJuIHJldDsKPiArCQl9Cj4gKwo+ICAJCXN0YXR1cyA9IGludGVs X2RwX2xvbmdfcHVsc2UoaW50ZWxfZHAtPmF0dGFjaGVkX2Nvbm5lY3Rvcik7Cj4gKwl9Cj4gIAo+ ICAJaW50ZWxfZHAtPmRldGVjdF9kb25lID0gZmFsc2U7Cj4gIAo+IEBAIC01MDI4LDEwICs1MDQ4 LDM4IEBAIGludGVsX2RwX2hwZF9wdWxzZShzdHJ1Y3QgaW50ZWxfZGlnaXRhbF9wb3J0ICppbnRl bF9kaWdfcG9ydCwgYm9vbCBsb25nX2hwZCkKPiAgCX0KPiAgCj4gIAlpZiAoIWludGVsX2RwLT5p c19tc3QpIHsKPiArCQlzdHJ1Y3QgZHJtX21vZGVzZXRfYWNxdWlyZV9jdHggY3R4Owo+ICsJCXN0 cnVjdCBkcm1fY29ubmVjdG9yICpjb25uZWN0b3IgPSAmaW50ZWxfZHAtPmF0dGFjaGVkX2Nvbm5l Y3Rvci0+YmFzZTsKPiArCQlzdHJ1Y3QgZHJtX2NydGMgKmNydGM7Cj4gKwkJaW50IGlyZXQ7Cj4g Kwo+ICsJCWRybV9tb2Rlc2V0X2FjcXVpcmVfaW5pdCgmY3R4LCAwKTsKPiArcmV0cnk6Cj4gKwkJ aXJldCA9IGRybV9tb2Rlc2V0X2xvY2soJmRldi0+bW9kZV9jb25maWcuY29ubmVjdGlvbl9tdXRl eCwgJmN0eCk7Cj4gKwkJaWYgKGlyZXQpCj4gKwkJCWdvdG8gZXJyOwo+ICsKPiArCQljcnRjID0g Y29ubmVjdG9yLT5zdGF0ZS0+Y3J0YzsKPiArCQlpZiAoY3J0Yykgewo+ICsJCQlpcmV0ID0gZHJt X21vZGVzZXRfbG9jaygmY3J0Yy0+bXV0ZXgsICZjdHgpOwoKUGxlYXNlIG5vIG1vZGVzZXQgbG9j a3MgaW4gdGhlIGRpZ193b3JrLiBIbW0uIExvb2tzIGxpa2Ugd2UgYWxyZWFkeSBoYXZlCml0IHRo ZXJlIHRob3VnaCA6KCBJIHRob3VnaHQgd2UgaGFkIG1vdmVkIHRoZSBsaW5rIHJldHJhaW5pbmcg ZnVsbHkgaW50bwp0aGUgaG9wbHVnX3dvcmssIGJ1dCBJIGd1ZXNzIG5vdC4gQ2FuIHlvdSB0cnkg dG8gZG8gdGhhdCBpbnN0ZWFkPwoKPiArCQkJaWYgKGlyZXQpCj4gKwkJCQlnb3RvIGVycjsKPiAr CQl9Cj4gKwo+ICAJCWlmICghaW50ZWxfZHBfc2hvcnRfcHVsc2UoaW50ZWxfZHApKSB7Cj4gIAkJ CWludGVsX2RwLT5kZXRlY3RfZG9uZSA9IGZhbHNlOwo+ICAJCQlnb3RvIHB1dF9wb3dlcjsKPiAg CQl9Cj4gKwo+ICtlcnI6Cj4gKwkJaWYgKGlyZXQgPT0gLUVERUFETEspIHsKPiArCQkJZHJtX21v ZGVzZXRfYmFja29mZigmY3R4KTsKPiArCQkJZ290byByZXRyeTsKPiArCQl9Cj4gKwo+ICsJCWRy bV9tb2Rlc2V0X2Ryb3BfbG9ja3MoJmN0eCk7Cj4gKwkJZHJtX21vZGVzZXRfYWNxdWlyZV9maW5p KCZjdHgpOwo+ICsJCVdBUk4oaXJldCwgIkFjcXVpcmluZyBtb2Rlc2V0IGxvY2tzIGZhaWxlZCB3 aXRoICVpXG4iLCBpcmV0KTsKPiAgCX0KPiAgCj4gIAlyZXQgPSBJUlFfSEFORExFRDsKPiAtLSAK PiAyLjExLjAKPiAKPiBfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fXwo+IEludGVsLWdmeCBtYWlsaW5nIGxpc3QKPiBJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0 b3Aub3JnCj4gaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9p bnRlbC1nZngKCi0tIApWaWxsZSBTeXJqw6Rsw6QKSW50ZWwgT1RDCl9fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50 ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9y Zy9tYWlsbWFuL2xpc3RpbmZvL2ludGVsLWdmeAo=