From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mika Kuoppala Subject: Re: [PATCH 2/5] drm/i915/gtt: Add some range asserts Date: Wed, 21 Aug 2019 19:24:05 +0300 Message-ID: <875zmqv656.fsf@gaia.fi.intel.com> References: <20190821155728.2839-1-chris@chris-wilson.co.uk> <20190821155728.2839-2-chris@chris-wilson.co.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTPS id 1BB186E46D for ; Wed, 21 Aug 2019 16:24:55 +0000 (UTC) In-Reply-To: <20190821155728.2839-2-chris@chris-wilson.co.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Chris Wilson , intel-gfx@lists.freedesktop.org Cc: matthew.auld@intel.com List-Id: intel-gfx@lists.freedesktop.org Q2hyaXMgV2lsc29uIDxjaHJpc0BjaHJpcy13aWxzb24uY28udWs+IHdyaXRlczoKCj4gVGhlc2Ug c2hvdWxkIGhhdmUgYmVlbiB2YWxpZGF0ZWQgaW4gdGhlIHVwcGVyIGxheWVycywgYnV0IGZvciBz YW5pdHkncwo+IHNha2UsIHJlcGVhdCB0aGVtLgo+Cj4gU2lnbmVkLW9mZi1ieTogQ2hyaXMgV2ls c29uIDxjaHJpc0BjaHJpcy13aWxzb24uY28udWs+Cj4gLS0tCj4gIGRyaXZlcnMvZ3B1L2RybS9p OTE1L2k5MTVfZ2VtX2d0dC5jIHwgMTYgKysrKysrKysrKystLS0tLQo+ICAxIGZpbGUgY2hhbmdl ZCwgMTEgaW5zZXJ0aW9ucygrKSwgNSBkZWxldGlvbnMoLSkKPgo+IGRpZmYgLS1naXQgYS9kcml2 ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2dlbV9ndHQuYyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5 MTVfZ2VtX2d0dC5jCj4gaW5kZXggYjA2ZDFkOTA1NGJhLi4wYjgxZTBiNjQzOTMgMTAwNjQ0Cj4g LS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9nZW1fZ3R0LmMKPiArKysgYi9kcml2ZXJz L2dwdS9kcm0vaTkxNS9pOTE1X2dlbV9ndHQuYwo+IEBAIC05NjUsOCArOTY1LDEwIEBAIHN0YXRp YyB1NjQgX19nZW44X3BwZ3R0X2NsZWFyKHN0cnVjdCBpOTE1X2FkZHJlc3Nfc3BhY2UgKiBjb25z dCB2bSwKPiAgCWNvbnN0IHN0cnVjdCBpOTE1X3BhZ2Vfc2NyYXRjaCAqIGNvbnN0IHNjcmF0Y2gg PSAmdm0tPnNjcmF0Y2hbbHZsXTsKPiAgCXVuc2lnbmVkIGludCBpZHgsIGxlbjsKPiAgCj4gKwlH RU1fQlVHX09OKGVuZCA+IHZtLT50b3RhbCA+PiBHRU44X1BURV9TSElGVCk7CgpJIGFtIGhhdmlu ZyB0aG91Z2h0cyB0byBjbGVhbiBhbGwgdW5kZXJseWluZyBoYW5kbGluZyB0bwp1c2UgcGFnZSBp bmRleGVzIGNvbnNpc3RlbnRseS4uLgoKUmV2aWV3ZWQtYnk6IE1pa2EgS3VvcHBhbGEgPG1pa2Eu a3VvcHBhbGFAbGludXguaW50ZWwuY29tPgoKPiArCj4gIAlsZW4gPSBnZW44X3BkX3JhbmdlKHN0 YXJ0LCBlbmQsIGx2bC0tLCAmaWR4KTsKPiAtCURCRygiJXMoJXApOnsgbHZsOiVkLCBzdGFydDol bGx4LCBlbmQ6JWxseCwgaWR4OiVkLCBsZW46JWQsIHVzZWQ6JWR9XG4iLAo+ICsJREJHKCIlcygl cCk6eyBsdmw6JWQsIHN0YXJ0OiVsbHgsIGVuZDolbGx4LCBpZHg6JWQsIGxlbjolZCwgdXNlZDol ZCB9XG4iLAo+ICAJICAgIF9fZnVuY19fLCB2bSwgbHZsICsgMSwgc3RhcnQsIGVuZCwKPiAgCSAg ICBpZHgsIGxlbiwgYXRvbWljX3JlYWQocHhfdXNlZChwZCkpKTsKPiAgCUdFTV9CVUdfT04oIWxl biB8fCBsZW4gPj0gYXRvbWljX3JlYWQocHhfdXNlZChwZCkpKTsKPiBAQCAtOTkyLDcgKzk5NCw3 IEBAIHN0YXRpYyB1NjQgX19nZW44X3BwZ3R0X2NsZWFyKHN0cnVjdCBpOTE1X2FkZHJlc3Nfc3Bh Y2UgKiBjb25zdCB2bSwKPiAgCQkJdTY0ICp2YWRkcjsKPiAgCj4gIAkJCWNvdW50ID0gZ2VuOF9w dF9jb3VudChzdGFydCwgZW5kKTsKPiAtCQkJREJHKCIlcyglcCk6eyBsdmw6JWQsIHN0YXJ0OiVs bHgsIGVuZDolbGx4LCBpZHg6JWQsIGxlbjolZCwgdXNlZDolZH0gcmVtb3ZpbmcgcHRlXG4iLAo+ ICsJCQlEQkcoIiVzKCVwKTp7IGx2bDolZCwgc3RhcnQ6JWxseCwgZW5kOiVsbHgsIGlkeDolZCwg bGVuOiVkLCB1c2VkOiVkIH0gcmVtb3ZpbmcgcHRlXG4iLAo+ICAJCQkgICAgX19mdW5jX18sIHZt LCBsdmwsIHN0YXJ0LCBlbmQsCj4gIAkJCSAgICBnZW44X3BkX2luZGV4KHN0YXJ0LCAwKSwgY291 bnQsCj4gIAkJCSAgICBhdG9taWNfcmVhZCgmcHQtPnVzZWQpKTsKPiBAQCAtMTAyMCw2ICsxMDIy LDcgQEAgc3RhdGljIHZvaWQgZ2VuOF9wcGd0dF9jbGVhcihzdHJ1Y3QgaTkxNV9hZGRyZXNzX3Nw YWNlICp2bSwKPiAgewo+ICAJR0VNX0JVR19PTighSVNfQUxJR05FRChzdGFydCwgQklUX1VMTChH RU44X1BURV9TSElGVCkpKTsKPiAgCUdFTV9CVUdfT04oIUlTX0FMSUdORUQobGVuZ3RoLCBCSVRf VUxMKEdFTjhfUFRFX1NISUZUKSkpOwo+ICsJR0VNX0JVR19PTihyYW5nZV9vdmVyZmxvd3Moc3Rh cnQsIGxlbmd0aCwgdm0tPnRvdGFsKSk7Cj4gIAo+ICAJc3RhcnQgPj49IEdFTjhfUFRFX1NISUZU Owo+ICAJbGVuZ3RoID4+PSBHRU44X1BURV9TSElGVDsKPiBAQCAtMTAzMSwxNSArMTAzNCwxNyBA QCBzdGF0aWMgdm9pZCBnZW44X3BwZ3R0X2NsZWFyKHN0cnVjdCBpOTE1X2FkZHJlc3Nfc3BhY2Ug KnZtLAo+ICAKPiAgc3RhdGljIGludCBfX2dlbjhfcHBndHRfYWxsb2Moc3RydWN0IGk5MTVfYWRk cmVzc19zcGFjZSAqIGNvbnN0IHZtLAo+ICAJCQkgICAgICBzdHJ1Y3QgaTkxNV9wYWdlX2RpcmVj dG9yeSAqIGNvbnN0IHBkLAo+IC0JCQkgICAgICB1NjQgKiBjb25zdCBzdGFydCwgdTY0IGVuZCwg aW50IGx2bCkKPiArCQkJICAgICAgdTY0ICogY29uc3Qgc3RhcnQsIGNvbnN0IHU2NCBlbmQsIGlu dCBsdmwpCj4gIHsKPiAgCWNvbnN0IHN0cnVjdCBpOTE1X3BhZ2Vfc2NyYXRjaCAqIGNvbnN0IHNj cmF0Y2ggPSAmdm0tPnNjcmF0Y2hbbHZsXTsKPiAgCXN0cnVjdCBpOTE1X3BhZ2VfdGFibGUgKmFs bG9jID0gTlVMTDsKPiAgCXVuc2lnbmVkIGludCBpZHgsIGxlbjsKPiAgCWludCByZXQgPSAwOwo+ ICAKPiArCUdFTV9CVUdfT04oZW5kID4gdm0tPnRvdGFsID4+IEdFTjhfUFRFX1NISUZUKTsKPiAr Cj4gIAlsZW4gPSBnZW44X3BkX3JhbmdlKCpzdGFydCwgZW5kLCBsdmwtLSwgJmlkeCk7Cj4gLQlE QkcoIiVzKCVwKTp7bHZsOiVkLCBzdGFydDolbGx4LCBlbmQ6JWxseCwgaWR4OiVkLCBsZW46JWQs IHVzZWQ6JWR9XG4iLAo+ICsJREJHKCIlcyglcCk6eyBsdmw6JWQsIHN0YXJ0OiVsbHgsIGVuZDol bGx4LCBpZHg6JWQsIGxlbjolZCwgdXNlZDolZCB9XG4iLAo+ICAJICAgIF9fZnVuY19fLCB2bSwg bHZsICsgMSwgKnN0YXJ0LCBlbmQsCj4gIAkgICAgaWR4LCBsZW4sIGF0b21pY19yZWFkKHB4X3Vz ZWQocGQpKSk7Cj4gIAlHRU1fQlVHX09OKCFsZW4gfHwgKGlkeCArIGxlbiAtIDEpID4+IGdlbjhf cGRfc2hpZnQoMSkpOwo+IEBAIC0xMTA1LDcgKzExMTAsNyBAQCBzdGF0aWMgaW50IF9fZ2VuOF9w cGd0dF9hbGxvYyhzdHJ1Y3QgaTkxNV9hZGRyZXNzX3NwYWNlICogY29uc3Qgdm0sCj4gIAkJfSBl bHNlIHsKPiAgCQkJdW5zaWduZWQgaW50IGNvdW50ID0gZ2VuOF9wdF9jb3VudCgqc3RhcnQsIGVu ZCk7Cj4gIAo+IC0JCQlEQkcoIiVzKCVwKTp7bHZsOiVkLCBzdGFydDolbGx4LCBlbmQ6JWxseCwg aWR4OiVkLCBsZW46JWQsIHVzZWQ6JWR9IGluc2VydGluZyBwdGVcbiIsCj4gKwkJCURCRygiJXMo JXApOnsgbHZsOiVkLCBzdGFydDolbGx4LCBlbmQ6JWxseCwgaWR4OiVkLCBsZW46JWQsIHVzZWQ6 JWQgfSBpbnNlcnRpbmcgcHRlXG4iLAo+ICAJCQkgICAgX19mdW5jX18sIHZtLCBsdmwsICpzdGFy dCwgZW5kLAo+ICAJCQkgICAgZ2VuOF9wZF9pbmRleCgqc3RhcnQsIDApLCBjb3VudCwKPiAgCQkJ ICAgIGF0b21pY19yZWFkKCZwdC0+dXNlZCkpOwo+IEBAIC0xMTMxLDYgKzExMzYsNyBAQCBzdGF0 aWMgaW50IGdlbjhfcHBndHRfYWxsb2Moc3RydWN0IGk5MTVfYWRkcmVzc19zcGFjZSAqdm0sCj4g IAo+ICAJR0VNX0JVR19PTighSVNfQUxJR05FRChzdGFydCwgQklUX1VMTChHRU44X1BURV9TSElG VCkpKTsKPiAgCUdFTV9CVUdfT04oIUlTX0FMSUdORUQobGVuZ3RoLCBCSVRfVUxMKEdFTjhfUFRF X1NISUZUKSkpOwo+ICsJR0VNX0JVR19PTihyYW5nZV9vdmVyZmxvd3Moc3RhcnQsIGxlbmd0aCwg dm0tPnRvdGFsKSk7Cj4gIAo+ICAJc3RhcnQgPj49IEdFTjhfUFRFX1NISUZUOwo+ICAJbGVuZ3Ro ID4+PSBHRU44X1BURV9TSElGVDsKPiAtLSAKPiAyLjIzLjAucmMxCl9fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50 ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9y Zy9tYWlsbWFuL2xpc3RpbmZvL2ludGVsLWdmeA==