From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH v3] drm: Warn if plane/crtc/encoder/connector index exceeds our 32bit bitmasks Date: Mon, 29 Jan 2018 19:21:05 +0200 Message-ID: <20180129172105.GS5453@intel.com> References: <20180124183752.32716-1-ville.syrjala@linux.intel.com> <20180125133020.23845-1-ville.syrjala@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: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Harry Wentland Cc: Daniel Vetter , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T24gVGh1LCBKYW4gMjUsIDIwMTggYXQgMTA6MTI6NTJBTSAtMDUwMCwgSGFycnkgV2VudGxhbmQg d3JvdGU6Cj4gT24gMjAxOC0wMS0yNSAwODozMCBBTSwgVmlsbGUgU3lyamFsYSB3cm90ZToKPiA+ IEZyb206IFZpbGxlIFN5cmrDpGzDpCA8dmlsbGUuc3lyamFsYUBsaW51eC5pbnRlbC5jb20+Cj4g PiAKPiA+IFdlIHVzZSAzMmJpdCBiaXRtYXNrcyB0byB0cmFjayBwbGFuZXMvY3J0Y3MvZW5jb2Rl cnMvY29ubmVjdG9ycy4KPiA+IE5hdHVyYWxseSB3ZSBjYW4gb25seSBkbyB0aGF0IGlmIHRoZSBp bmRleCBvZiB0aG9zZSBvYmplY3RzIHN0YXlzCj4gPiBiZWxvdyAzMi4gSXNzdWUgYSB3YXJuaW5n IHdoZW5ldmVyIHdlIGV4Y2VlZCB0aGF0IGxpbWl0LCBob3BlZnVsbHkKPiA+IHByb21wdGluZyBz b21lb25lIHRvIGZpeCB0aGUgcHJvYmxlbS4KPiA+IAo+ID4gRm9yIGNvbm5lY3RvcnMgdGhlIGlz c3VlIGlzIGEgYml0IG1vcmUgY29tcGxpY2F0ZWQgYXMgdGhleSBjYW4KPiA+IGJlIGNyZWF0ZWQv ZGVzdHJveWVkIGF0IHJ1bnRpbWUgZHVlIHRvIE1TVC4gU28gdGhlIHByb2JsZW0gaXMgbm8KPiA+ IGxvbmdlciBhIHB1cmVseSB0aGVvcmV0aWNhbCBwcm9ncmFtbWVyIGVycm9yLiBBcyB0aGUgY29u bmVjdG9yCj4gPiBpbmRleGVzIGFyZSBhbGxvY2F0ZWQgdmlhIGlkYSwgd2UgY2FuIHNpbXBseSBs aW1pdCB0aGUgbWF4aW11bQo+ID4gdmFsdWUgdGhlIGlkYSBpcyBhbGxvd2VkIHRvIGhhbmQgb3V0 LiBUaGUgZXJyb3IgaGFuZGxpbmcgaXMgYWxyZWFkeQo+ID4gaW4gcGxhY2UuCj4gPiAKPiA+IHYy OiBSZXR1cm4gYW4gZXJyb3IgdG8gdGhlIGNhbGxlciAoSGFycnkpCj4gPiB2MzogUHJpbnQgYSBk ZWJ1ZyBtZXNzYWdlIHNvIHRoYXQgd2Uga25vdyB3aGF0IGhhcHBlbmVkIChNYWFydGVuKQo+ID4g Cj4gPiBDYzogSGFycnkgV2VudGxhbmQgPGhhcnJ5LndlbnRsYW5kQGFtZC5jb20+Cj4gPiBDYzog TWFhcnRlbiBMYW5raG9yc3QgPG1hYXJ0ZW4ubGFua2hvcnN0QGxpbnV4LmludGVsLmNvbT4KPiA+ IENjOiBEYW5pZWwgVmV0dGVyIDxkYW5pZWwudmV0dGVyQGZmd2xsLmNoPgo+ID4gUmV2aWV3ZWQt Ynk6IE1hYXJ0ZW4gTGFua2hvcnN0IDxtYWFydGVuLmxhbmtob3JzdEBsaW51eC5pbnRlbC5jb20+ Cj4gCj4gUmV2aWV3ZWQtYnk6IEhhcnJ5IFdlbnRsYW5kIDxoYXJyeS53ZW50bGFuZEBhbWQuY29t PgoKVGhhbmtzIGZvciB0aGUgcmV2aWV3LiBQdXNoZWQgdG8gZHJtLW1pc2MtbmV4dC4KCj4gCj4g SGFycnkKPiAKPiA+IFNpZ25lZC1vZmYtYnk6IFZpbGxlIFN5cmrDpGzDpCA8dmlsbGUuc3lyamFs YUBsaW51eC5pbnRlbC5jb20+Cj4gPiAtLS0KPiA+ICBkcml2ZXJzL2dwdS9kcm0vZHJtX2Nvbm5l Y3Rvci5jIHwgOSArKysrKysrLS0KPiA+ICBkcml2ZXJzL2dwdS9kcm0vZHJtX2NydGMuYyAgICAg IHwgNCArKysrCj4gPiAgZHJpdmVycy9ncHUvZHJtL2RybV9lbmNvZGVyLmMgICB8IDQgKysrKwo+ ID4gIGRyaXZlcnMvZ3B1L2RybS9kcm1fcGxhbmUuYyAgICAgfCA0ICsrKysKPiA+ICA0IGZpbGVz IGNoYW5nZWQsIDE5IGluc2VydGlvbnMoKyksIDIgZGVsZXRpb25zKC0pCj4gPiAKPiA+IGRpZmYg LS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vZHJtX2Nvbm5lY3Rvci5jIGIvZHJpdmVycy9ncHUvZHJt L2RybV9jb25uZWN0b3IuYwo+ID4gaW5kZXggYjg1YTc3NDk3MDlkLi4xNmI5YzM4MTBhZjIgMTAw NjQ0Cj4gPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vZHJtX2Nvbm5lY3Rvci5jCj4gPiArKysgYi9k cml2ZXJzL2dwdS9kcm0vZHJtX2Nvbm5lY3Rvci5jCj4gPiBAQCAtMjA1LDkgKzIwNSwxNCBAQCBp bnQgZHJtX2Nvbm5lY3Rvcl9pbml0KHN0cnVjdCBkcm1fZGV2aWNlICpkZXYsCj4gPiAgCWNvbm5l Y3Rvci0+ZGV2ID0gZGV2Owo+ID4gIAljb25uZWN0b3ItPmZ1bmNzID0gZnVuY3M7Cj4gPiAgCj4g PiAtCXJldCA9IGlkYV9zaW1wbGVfZ2V0KCZjb25maWctPmNvbm5lY3Rvcl9pZGEsIDAsIDAsIEdG UF9LRVJORUwpOwo+ID4gLQlpZiAocmV0IDwgMCkKPiA+ICsJLyogY29ubmVjdG9yIGluZGV4IGlz IHVzZWQgd2l0aCAzMmJpdCBiaXRtYXNrcyAqLwo+ID4gKwlyZXQgPSBpZGFfc2ltcGxlX2dldCgm Y29uZmlnLT5jb25uZWN0b3JfaWRhLCAwLCAzMiwgR0ZQX0tFUk5FTCk7Cj4gPiArCWlmIChyZXQg PCAwKSB7Cj4gPiArCQlEUk1fREVCVUdfS01TKCJGYWlsZWQgdG8gYWxsb2NhdGUgJXMgY29ubmVj dG9yIGluZGV4OiAlZFxuIiwKPiA+ICsJCQkgICAgICBkcm1fY29ubmVjdG9yX2VudW1fbGlzdFtj b25uZWN0b3JfdHlwZV0ubmFtZSwKPiA+ICsJCQkgICAgICByZXQpOwo+ID4gIAkJZ290byBvdXRf cHV0Owo+ID4gKwl9Cj4gPiAgCWNvbm5lY3Rvci0+aW5kZXggPSByZXQ7Cj4gPiAgCXJldCA9IDA7 Cj4gPiAgCj4gPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2RybV9jcnRjLmMgYi9kcml2 ZXJzL2dwdS9kcm0vZHJtX2NydGMuYwo+ID4gaW5kZXggZjA1NTZlNjU0MTE2Li41YjRiZTM4MmEx ZDcgMTAwNjQ0Cj4gPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vZHJtX2NydGMuYwo+ID4gKysrIGIv ZHJpdmVycy9ncHUvZHJtL2RybV9jcnRjLmMKPiA+IEBAIC0yODIsNiArMjgyLDEwIEBAIGludCBk cm1fY3J0Y19pbml0X3dpdGhfcGxhbmVzKHN0cnVjdCBkcm1fZGV2aWNlICpkZXYsIHN0cnVjdCBk cm1fY3J0YyAqY3J0YywKPiA+ICAJV0FSTl9PTihwcmltYXJ5ICYmIHByaW1hcnktPnR5cGUgIT0g RFJNX1BMQU5FX1RZUEVfUFJJTUFSWSk7Cj4gPiAgCVdBUk5fT04oY3Vyc29yICYmIGN1cnNvci0+ dHlwZSAhPSBEUk1fUExBTkVfVFlQRV9DVVJTT1IpOwo+ID4gIAo+ID4gKwkvKiBjcnRjIGluZGV4 IGlzIHVzZWQgd2l0aCAzMmJpdCBiaXRtYXNrcyAqLwo+ID4gKwlpZiAoV0FSTl9PTihjb25maWct Pm51bV9jcnRjID49IDMyKSkKPiA+ICsJCXJldHVybiAtRUlOVkFMOwo+ID4gKwo+ID4gIAljcnRj LT5kZXYgPSBkZXY7Cj4gPiAgCWNydGMtPmZ1bmNzID0gZnVuY3M7Cj4gPiAgCj4gPiBkaWZmIC0t Z2l0IGEvZHJpdmVycy9ncHUvZHJtL2RybV9lbmNvZGVyLmMgYi9kcml2ZXJzL2dwdS9kcm0vZHJt X2VuY29kZXIuYwo+ID4gaW5kZXggNTllMGViZTczM2Y4Li4yNzNlMWM1OWM1NGEgMTAwNjQ0Cj4g PiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vZHJtX2VuY29kZXIuYwo+ID4gKysrIGIvZHJpdmVycy9n cHUvZHJtL2RybV9lbmNvZGVyLmMKPiA+IEBAIC0xMTAsNiArMTEwLDEwIEBAIGludCBkcm1fZW5j b2Rlcl9pbml0KHN0cnVjdCBkcm1fZGV2aWNlICpkZXYsCj4gPiAgewo+ID4gIAlpbnQgcmV0Owo+ ID4gIAo+ID4gKwkvKiBlbmNvZGVyIGluZGV4IGlzIHVzZWQgd2l0aCAzMmJpdCBiaXRtYXNrcyAq Lwo+ID4gKwlpZiAoV0FSTl9PTihkZXYtPm1vZGVfY29uZmlnLm51bV9lbmNvZGVyID49IDMyKSkK PiA+ICsJCXJldHVybiAtRUlOVkFMOwo+ID4gKwo+ID4gIAlyZXQgPSBkcm1fbW9kZV9vYmplY3Rf YWRkKGRldiwgJmVuY29kZXItPmJhc2UsIERSTV9NT0RFX09CSkVDVF9FTkNPREVSKTsKPiA+ICAJ aWYgKHJldCkKPiA+ICAJCXJldHVybiByZXQ7Cj4gPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUv ZHJtL2RybV9wbGFuZS5jIGIvZHJpdmVycy9ncHUvZHJtL2RybV9wbGFuZS5jCj4gPiBpbmRleCAy YzkwNTE5NTc2YTMuLjIyYjU0NjYzYjZlNyAxMDA2NDQKPiA+IC0tLSBhL2RyaXZlcnMvZ3B1L2Ry bS9kcm1fcGxhbmUuYwo+ID4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2RybV9wbGFuZS5jCj4gPiBA QCAtMTczLDYgKzE3MywxMCBAQCBpbnQgZHJtX3VuaXZlcnNhbF9wbGFuZV9pbml0KHN0cnVjdCBk cm1fZGV2aWNlICpkZXYsIHN0cnVjdCBkcm1fcGxhbmUgKnBsYW5lLAo+ID4gIAl1bnNpZ25lZCBp bnQgZm9ybWF0X21vZGlmaWVyX2NvdW50ID0gMDsKPiA+ICAJaW50IHJldDsKPiA+ICAKPiA+ICsJ LyogcGxhbmUgaW5kZXggaXMgdXNlZCB3aXRoIDMyYml0IGJpdG1hc2tzICovCj4gPiArCWlmIChX QVJOX09OKGNvbmZpZy0+bnVtX3RvdGFsX3BsYW5lID49IDMyKSkKPiA+ICsJCXJldHVybiAtRUlO VkFMOwo+ID4gKwo+ID4gIAlyZXQgPSBkcm1fbW9kZV9vYmplY3RfYWRkKGRldiwgJnBsYW5lLT5i YXNlLCBEUk1fTU9ERV9PQkpFQ1RfUExBTkUpOwo+ID4gIAlpZiAocmV0KQo+ID4gIAkJcmV0dXJu IHJldDsKPiA+IAoKLS0gClZpbGxlIFN5cmrDpGzDpApJbnRlbCBPVEMKX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJ bnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Au b3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==