From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tvrtko Ursulin Subject: Re: [PATCH 1/2] drm/i915: Fallback to single PAGE_SIZE segments for DMA remapping Date: Tue, 20 Dec 2016 11:13:43 +0000 Message-ID: <85bebef1-9cb8-2d81-2f97-0c5f8204a823@linux.intel.com> References: <20161219124346.550-1-chris@chris-wilson.co.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20161219124346.550-1-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: drm-intel-fixes@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org Ck9uIDE5LzEyLzIwMTYgMTI6NDMsIENocmlzIFdpbHNvbiB3cm90ZToKPiBJZiB3ZSBhdCBmaXJz dCBkbyBub3Qgc3VjY2VlZCB3aXRoIGF0dGVtcHRpbmcgdG8gcmVtYXAgb3VyIHBoeXNpY2FsCj4g cGFnZXMgdXNpbmcgYSBjb2FsZXNjZWQgc2NhdHRlcmdhdGhlciBsaXN0LCB0cnkgYWdhaW4gd2l0 aCBvbmUKPiBzY2F0dGVyZ2F0aGVyIGVudHJ5IHBlciBwYWdlLiBUaGlzIHNob3VsZCBoZWxwIHdp dGggc3dpb3RsYiBhcyBpdCB1c2VzIGEKPiBsaW1pdGVkIGJ1ZmZlciBzaXplIGFuZCBvbmx5IHNl YXJjaGVzIGZvciBjb250aWd1b3VzIGNodW5rcyB3aXRoaW4gaXRzCj4gYnVmZmVyIGFsaWduZWQg dXAgdG8gdGhlIG5leHQgYm91bmRhcnkgLSBpLmUuIHdlIG1heSBwcmVtYXR1cmVseSBjYXVzZSBh Cj4gZmFpbHVyZSBhcyB3ZSBhcmUgdW5hYmxlIHRvIHV0aWxpemUgdGhlIHVudXNlZCBzcGFjZSBi ZXR3ZWVuIGxhcmdlCj4gY2h1bmtzIGFuZCB0cmlnZ2VyIGFuIGVycm9yIHN1Y2ggYXM6Cj4KPiAJ IGk5MTUgMDAwMDowMDowMi4wOiBzd2lvdGxiIGJ1ZmZlciBpcyBmdWxsIChzejogMTYzMDIwOCBi eXRlcykKPgo+IFJlcG9ydGVkLWJ5OiBKdWVyZ2VuIEdyb3NzIDxqZ3Jvc3NAc3VzZS5jb20+Cj4g Rml4ZXM6IDg3MWRmYmQ2N2Q0ZSAoImRybS9pOTE1OiBBbGxvdyBjb21wYWN0aW9uIHVwdG8gU1dJ T1RMQiBtYXggc2VnbWVudCBzaXplIikKPiBTaWduZWQtb2ZmLWJ5OiBDaHJpcyBXaWxzb24gPGNo cmlzQGNocmlzLXdpbHNvbi5jby51az4KPiBDYzogVHZydGtvIFVyc3VsaW4gPHR2cnRrby51cnN1 bGluQGludGVsLmNvbT4KPiBDYzogSW1yZSBEZWFrIDxpbXJlLmRlYWtAaW50ZWwuY29tPgo+IENj OiA8ZHJtLWludGVsLWZpeGVzQGxpc3RzLmZyZWVkZXNrdG9wLm9yZz4KPiAtLS0KPiAgZHJpdmVy cy9ncHUvZHJtL2k5MTUvaTkxNV9nZW0uYyB8IDI2ICsrKysrKysrKysrKysrKysrKysrKystLS0t Cj4gIDEgZmlsZSBjaGFuZ2VkLCAyMiBpbnNlcnRpb25zKCspLCA0IGRlbGV0aW9ucygtKQo+Cj4g ZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZ2VtLmMgYi9kcml2ZXJzL2dw dS9kcm0vaTkxNS9pOTE1X2dlbS5jCj4gaW5kZXggNDEyZjM1MTNmMjY5Li40ZTI2M2RmMmFmYzMg MTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9nZW0uYwo+ICsrKyBiL2Ry aXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZ2VtLmMKPiBAQCAtMjMyNiw3ICsyMzI2LDggQEAgc3Rh dGljIHN0cnVjdCBzZ190YWJsZSAqCj4gIGk5MTVfZ2VtX29iamVjdF9nZXRfcGFnZXNfZ3R0KHN0 cnVjdCBkcm1faTkxNV9nZW1fb2JqZWN0ICpvYmopCj4gIHsKPiAgCXN0cnVjdCBkcm1faTkxNV9w cml2YXRlICpkZXZfcHJpdiA9IHRvX2k5MTUob2JqLT5iYXNlLmRldik7Cj4gLQlpbnQgcGFnZV9j b3VudCwgaTsKPiArCWNvbnN0IHVuc2lnbmVkIGxvbmcgcGFnZV9jb3VudCA9IG9iai0+YmFzZS5z aXplIC8gUEFHRV9TSVpFOwo+ICsJdW5zaWduZWQgbG9uZyBpOwo+ICAJc3RydWN0IGFkZHJlc3Nf c3BhY2UgKm1hcHBpbmc7Cj4gIAlzdHJ1Y3Qgc2dfdGFibGUgKnN0Owo+ICAJc3RydWN0IHNjYXR0 ZXJsaXN0ICpzZzsKPiBAQCAtMjM1Miw3ICsyMzUzLDcgQEAgaTkxNV9nZW1fb2JqZWN0X2dldF9w YWdlc19ndHQoc3RydWN0IGRybV9pOTE1X2dlbV9vYmplY3QgKm9iaikKPiAgCWlmIChzdCA9PSBO VUxMKQo+ICAJCXJldHVybiBFUlJfUFRSKC1FTk9NRU0pOwo+Cj4gLQlwYWdlX2NvdW50ID0gb2Jq LT5iYXNlLnNpemUgLyBQQUdFX1NJWkU7Cj4gK3JlYnVpbGRfc3Q6Cj4gIAlpZiAoc2dfYWxsb2Nf dGFibGUoc3QsIHBhZ2VfY291bnQsIEdGUF9LRVJORUwpKSB7Cj4gIAkJa2ZyZWUoc3QpOwo+ICAJ CXJldHVybiBFUlJfUFRSKC1FTk9NRU0pOwo+IEBAIC0yNDExLDggKzI0MTIsMjUgQEAgaTkxNV9n ZW1fb2JqZWN0X2dldF9wYWdlc19ndHQoc3RydWN0IGRybV9pOTE1X2dlbV9vYmplY3QgKm9iaikK PiAgCWk5MTVfc2dfdHJpbShzdCk7Cj4KPiAgCXJldCA9IGk5MTVfZ2VtX2d0dF9wcmVwYXJlX3Bh Z2VzKG9iaiwgc3QpOwo+IC0JaWYgKHJldCkKPiAtCQlnb3RvIGVycl9wYWdlczsKPiArCWlmIChy ZXQpIHsKPiArCQkvKiBETUEgcmVtYXBwaW5nIGZhaWxlZD8gT25lIHBvc3NpYmxlIGNhdXNlIGlz IHRoYXQKPiArCQkgKiBpdCBjb3VsZCBub3QgcmVzZXJ2ZSBlbm91Z2ggbGFyZ2UgZW50cmllcywg YXNraW5nCj4gKwkJICogZm9yIFBBR0VfU0laRSBjaHVua3MgaW5zdGVhZCBtYXkgYmUgaGVscGZ1 bC4KPiArCQkgKi8KPiArCQlpZiAobWF4X3NlZ21lbnQgPiBQQUdFX1NJWkUpIHsKPiArCQkJZm9y X2VhY2hfc2d0X3BhZ2UocGFnZSwgc2d0X2l0ZXIsIHN0KQo+ICsJCQkJcHV0X3BhZ2UocGFnZSk7 Cj4gKwkJCXNnX2ZyZWVfdGFibGUoc3QpOwo+ICsKPiArCQkJbWF4X3NlZ21lbnQgPSBQQUdFX1NJ WkU7Cj4gKwkJCWdvdG8gcmVidWlsZF9zdDsKPiArCQl9IGVsc2Ugewo+ICsJCQlkZXZfd2Fybigm ZGV2X3ByaXYtPmRybS5wZGV2LT5kZXYsCj4gKwkJCQkgIkZhaWxlZCB0byBETUEgcmVtYXAgJWx1 IHBhZ2VzXG4iLAo+ICsJCQkJIHBhZ2VfY291bnQpOwo+ICsJCQlnb3RvIGVycl9wYWdlczsKPiAr CQl9Cj4gKwl9Cj4KPiAgCWlmIChpOTE1X2dlbV9vYmplY3RfbmVlZHNfYml0MTdfc3dpenpsZShv YmopKQo+ICAJCWk5MTVfZ2VtX29iamVjdF9kb19iaXRfMTdfc3dpenpsZShvYmosIHN0KTsKPgoK SG93IG11Y2ggaXMgdGhlIGNvc3Qgb2YgZnJlZWluZyBhbmQgcmUtYWNxdWlyaW5nIHBhZ2VzIGlu IHRoZSBmYWxsIGJhY2sgCmNhc2U/IEl0IGNvdWxkIGJlIGF2b2lkYWJsZSBieSB1c2luZyB0aGUg dGFibGUgYW5kIGFkZGluZyBzb21ldGhpbmcgbGlrZSAKc2d0ID0gaTkxNV9zZ19jb3B5KHNndCwg dGFibGVfbWF4X3NlZ21lbnQpLiBCdXQgaXQgZGVwZW5kcyBvbiBob3cgbGlrZWx5IAppcyB0aGlz IHBhdGggdG8gYmUgaGl0IG9uIHN3aW90bGIgcGxhdGZvcm1zLiBJIGhhdmUgbm8gaWRlYS4gT3Vy IApkYXRhc2V0cyBhcmUgbXVjaCBiaWdnZXIgdGhhbiB0aGUgc3dpb3RsYiBzcGFjZSAtIGlmIHRo YXQgaXMgdHJ1ZSBvbiAKc3VjaCBwbGF0Zm9ybXM/CgpSZWdhcmRzLAoKVHZydGtvCl9fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5n IGxpc3QKSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVk ZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2ludGVsLWdmeAo=