From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH v2] drm: Warn if plane/crtc/encoder/connector index exceeds our 32bit bitmasks Date: Thu, 25 Jan 2018 13:27:27 +0200 Message-ID: <20180125112727.GC5453@intel.com> References: <20180124183752.32716-1-ville.syrjala@linux.intel.com> <20180124214741.12681-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: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Maarten Lankhorst Cc: Daniel Vetter , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T24gVGh1LCBKYW4gMjUsIDIwMTggYXQgMTA6MTc6MjFBTSArMDEwMCwgTWFhcnRlbiBMYW5raG9y c3Qgd3JvdGU6Cj4gT3AgMjQtMDEtMTggb20gMjI6NDcgc2NocmVlZiBWaWxsZSBTeXJqYWxhOgo+ ID4gRnJvbTogVmlsbGUgU3lyasOkbMOkIDx2aWxsZS5zeXJqYWxhQGxpbnV4LmludGVsLmNvbT4K PiA+Cj4gPiBXZSB1c2UgMzJiaXQgYml0bWFza3MgdG8gdHJhY2sgcGxhbmVzL2NydGNzL2VuY29k ZXJzL2Nvbm5lY3RvcnMuCj4gPiBOYXR1cmFsbHkgd2UgY2FuIG9ubHkgZG8gdGhhdCBpZiB0aGUg aW5kZXggb2YgdGhvc2Ugb2JqZWN0cyBzdGF5cwo+ID4gYmVsb3cgMzIuIElzc3VlIGEgd2Fybmlu ZyB3aGVuZXZlciB3ZSBleGNlZWQgdGhhdCBsaW1pdCwgaG9wZWZ1bGx5Cj4gPiBwcm9tcHRpbmcg c29tZW9uZSB0byBmaXggdGhlIHByb2JsZW0uCj4gPgo+ID4gRm9yIGNvbm5lY3RvcnMgdGhlIGlz c3VlIGlzIGEgYml0IG1vcmUgY29tcGxpY2F0ZWQgYXMgdGhleSBjYW4KPiA+IGJlIGNyZWF0ZWQv ZGVzdHJveWVkIGF0IHJ1bnRpbWUgZHVlIHRvIE1TVC4gU28gdGhlIHByb2JsZW0gaXMgbm8KPiA+ IGxvbmdlciBhIHB1cmVseSB0aGVvcmV0aWNhbCBwcm9ncmFtbWVyIGVycm9yLiBBcyB0aGUgY29u bmVjdG9yCj4gPiBpbmRleGVzIGFyZSBhbGxvY2F0ZWQgdmlhIGlkYSwgd2UgY2FuIHNpbXBseSBs aW1pdCB0aGUgbWF4aW11bQo+ID4gdmFsdWUgdGhlIGlkYSBpcyBhbGxvd2VkIHRvIGhhbmQgb3V0 LiBUaGUgZXJyb3IgaGFuZGxpbmcgaXMgYWxyZWFkeQo+ID4gaW4gcGxhY2UuCj4gPgo+ID4gdjI6 IFJldHVybiBhbiBlcnJvciB0byB0aGUgY2FsbGVyIChIYXJyeSkKPiA+Cj4gPiBDYzogSGFycnkg V2VudGxhbmQgPGhhcnJ5LndlbnRsYW5kQGFtZC5jb20+Cj4gPiBDYzogTWFhcnRlbiBMYW5raG9y c3QgPG1hYXJ0ZW4ubGFua2hvcnN0QGxpbnV4LmludGVsLmNvbT4KPiA+IENjOiBEYW5pZWwgVmV0 dGVyIDxkYW5pZWwudmV0dGVyQGZmd2xsLmNoPgo+ID4gU2lnbmVkLW9mZi1ieTogVmlsbGUgU3ly asOkbMOkIDx2aWxsZS5zeXJqYWxhQGxpbnV4LmludGVsLmNvbT4KPiA+IC0tLQo+ID4gIGRyaXZl cnMvZ3B1L2RybS9kcm1fY29ubmVjdG9yLmMgfCAzICsrLQo+ID4gIGRyaXZlcnMvZ3B1L2RybS9k cm1fY3J0Yy5jICAgICAgfCA0ICsrKysKPiA+ICBkcml2ZXJzL2dwdS9kcm0vZHJtX2VuY29kZXIu YyAgIHwgNCArKysrCj4gPiAgZHJpdmVycy9ncHUvZHJtL2RybV9wbGFuZS5jICAgICB8IDQgKysr Kwo+ID4gIDQgZmlsZXMgY2hhbmdlZCwgMTQgaW5zZXJ0aW9ucygrKSwgMSBkZWxldGlvbigtKQo+ ID4KPiA+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vZHJtX2Nvbm5lY3Rvci5jIGIvZHJp dmVycy9ncHUvZHJtL2RybV9jb25uZWN0b3IuYwo+ID4gaW5kZXggYjg1YTc3NDk3MDlkLi5mNGE2 ODlhYjk5MGEgMTAwNjQ0Cj4gPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vZHJtX2Nvbm5lY3Rvci5j Cj4gPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX2Nvbm5lY3Rvci5jCj4gPiBAQCAtMjA1LDcg KzIwNSw4IEBAIGludCBkcm1fY29ubmVjdG9yX2luaXQoc3RydWN0IGRybV9kZXZpY2UgKmRldiwK PiA+ICAJY29ubmVjdG9yLT5kZXYgPSBkZXY7Cj4gPiAgCWNvbm5lY3Rvci0+ZnVuY3MgPSBmdW5j czsKPiA+ICAKPiA+IC0JcmV0ID0gaWRhX3NpbXBsZV9nZXQoJmNvbmZpZy0+Y29ubmVjdG9yX2lk YSwgMCwgMCwgR0ZQX0tFUk5FTCk7Cj4gPiArCS8qIGNvbm5lY3RvciBpbmRleCBpcyB1c2VkIHdp dGggMzJiaXQgYml0bWFza3MgKi8KPiA+ICsJcmV0ID0gaWRhX3NpbXBsZV9nZXQoJmNvbmZpZy0+ Y29ubmVjdG9yX2lkYSwgMCwgMzIsIEdGUF9LRVJORUwpOwo+ID4gIAlpZiAocmV0IDwgMCkKPiA+ ICAJCWdvdG8gb3V0X3B1dDsKPiA+ICAJY29ubmVjdG9yLT5pbmRleCA9IHJldDsKPiBDb3VsZCB3 ZSBhbHNvIHB1dCBhIERSTV9ERUJVR19LTVMgaW4gaGVyZSBpZiBhbGxvY2F0aW9uIGZhaWxzPyBT aW5jZSBpdCdzIG1vcmUgbGlrZWx5IHRvIGhhcHBlbiBub3cuLgoKSSB3YXMgYWN0dWFsbHkgcG9u ZGVyaW5nIHVzaW5nIERSTV9FUlJPUigpLCBvciBtYXliZSBldmVuIFdBUk4oKS4gSXQKc2hvdWxk bid0IGJlIGRpcmVjdGx5IHVzZXIgdHJpZ2dlcmFibGUgc28gSSB0aGluayB3ZSB3YW50IHRvIGdl dCB0aGUKYnVncmVwb3J0IGlmIHdlIGRvIHJ1biBvdXQgb2YgYml0cy4KCj4gCj4gT3RoZXJ3aXNl IGxvb2tzIGdvb2Qgc28gd2l0aCB0aGF0IGNoYW5nZToKPiBSZXZpZXdlZC1ieTogTWFhcnRlbiBM YW5raG9yc3QgPG1hYXJ0ZW4ubGFua2hvcnN0QGxpbnV4LmludGVsLmNvbT4KPiA+IGRpZmYgLS1n aXQgYS9kcml2ZXJzL2dwdS9kcm0vZHJtX2NydGMuYyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fY3J0 Yy5jCj4gPiBpbmRleCBmMDU1NmU2NTQxMTYuLjViNGJlMzgyYTFkNyAxMDA2NDQKPiA+IC0tLSBh L2RyaXZlcnMvZ3B1L2RybS9kcm1fY3J0Yy5jCj4gPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vZHJt X2NydGMuYwo+ID4gQEAgLTI4Miw2ICsyODIsMTAgQEAgaW50IGRybV9jcnRjX2luaXRfd2l0aF9w bGFuZXMoc3RydWN0IGRybV9kZXZpY2UgKmRldiwgc3RydWN0IGRybV9jcnRjICpjcnRjLAo+ID4g IAlXQVJOX09OKHByaW1hcnkgJiYgcHJpbWFyeS0+dHlwZSAhPSBEUk1fUExBTkVfVFlQRV9QUklN QVJZKTsKPiA+ICAJV0FSTl9PTihjdXJzb3IgJiYgY3Vyc29yLT50eXBlICE9IERSTV9QTEFORV9U WVBFX0NVUlNPUik7Cj4gPiAgCj4gPiArCS8qIGNydGMgaW5kZXggaXMgdXNlZCB3aXRoIDMyYml0 IGJpdG1hc2tzICovCj4gPiArCWlmIChXQVJOX09OKGNvbmZpZy0+bnVtX2NydGMgPj0gMzIpKQo+ ID4gKwkJcmV0dXJuIC1FSU5WQUw7Cj4gPiArCj4gPiAgCWNydGMtPmRldiA9IGRldjsKPiA+ICAJ Y3J0Yy0+ZnVuY3MgPSBmdW5jczsKPiA+ICAKPiA+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9k cm0vZHJtX2VuY29kZXIuYyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fZW5jb2Rlci5jCj4gPiBpbmRl eCA1OWUwZWJlNzMzZjguLjI3M2UxYzU5YzU0YSAxMDA2NDQKPiA+IC0tLSBhL2RyaXZlcnMvZ3B1 L2RybS9kcm1fZW5jb2Rlci5jCj4gPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX2VuY29kZXIu Ywo+ID4gQEAgLTExMCw2ICsxMTAsMTAgQEAgaW50IGRybV9lbmNvZGVyX2luaXQoc3RydWN0IGRy bV9kZXZpY2UgKmRldiwKPiA+ICB7Cj4gPiAgCWludCByZXQ7Cj4gPiAgCj4gPiArCS8qIGVuY29k ZXIgaW5kZXggaXMgdXNlZCB3aXRoIDMyYml0IGJpdG1hc2tzICovCj4gPiArCWlmIChXQVJOX09O KGRldi0+bW9kZV9jb25maWcubnVtX2VuY29kZXIgPj0gMzIpKQo+ID4gKwkJcmV0dXJuIC1FSU5W QUw7Cj4gPiArCj4gPiAgCXJldCA9IGRybV9tb2RlX29iamVjdF9hZGQoZGV2LCAmZW5jb2Rlci0+ YmFzZSwgRFJNX01PREVfT0JKRUNUX0VOQ09ERVIpOwo+ID4gIAlpZiAocmV0KQo+ID4gIAkJcmV0 dXJuIHJldDsKPiA+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vZHJtX3BsYW5lLmMgYi9k cml2ZXJzL2dwdS9kcm0vZHJtX3BsYW5lLmMKPiA+IGluZGV4IDJjOTA1MTk1NzZhMy4uMjJiNTQ2 NjNiNmU3IDEwMDY0NAo+ID4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2RybV9wbGFuZS5jCj4gPiAr KysgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX3BsYW5lLmMKPiA+IEBAIC0xNzMsNiArMTczLDEwIEBA IGludCBkcm1fdW5pdmVyc2FsX3BsYW5lX2luaXQoc3RydWN0IGRybV9kZXZpY2UgKmRldiwgc3Ry dWN0IGRybV9wbGFuZSAqcGxhbmUsCj4gPiAgCXVuc2lnbmVkIGludCBmb3JtYXRfbW9kaWZpZXJf Y291bnQgPSAwOwo+ID4gIAlpbnQgcmV0Owo+ID4gIAo+ID4gKwkvKiBwbGFuZSBpbmRleCBpcyB1 c2VkIHdpdGggMzJiaXQgYml0bWFza3MgKi8KPiA+ICsJaWYgKFdBUk5fT04oY29uZmlnLT5udW1f dG90YWxfcGxhbmUgPj0gMzIpKQo+ID4gKwkJcmV0dXJuIC1FSU5WQUw7Cj4gPiArCj4gPiAgCXJl dCA9IGRybV9tb2RlX29iamVjdF9hZGQoZGV2LCAmcGxhbmUtPmJhc2UsIERSTV9NT0RFX09CSkVD VF9QTEFORSk7Cj4gPiAgCWlmIChyZXQpCj4gPiAgCQlyZXR1cm4gcmV0Owo+IAoKLS0gClZpbGxl IFN5cmrDpGzDpApJbnRlbCBPVEMKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRl c2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8v ZHJpLWRldmVsCg==