From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joonas Lahtinen Subject: Re: [PATCH 2/2] drm/i915: Move the modulus for ring emission to the register write Date: Thu, 28 Jul 2016 12:59:52 +0300 Message-ID: <1469699992.3897.48.camel@linux.intel.com> References: <1469692366.3897.19.camel@linux.intel.com> <1469697366-27175-1-git-send-email-chris@chris-wilson.co.uk> <1469697366-27175-2-git-send-email-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 ESMTP id EA40C6E7B9 for ; Thu, 28 Jul 2016 09:59:55 +0000 (UTC) In-Reply-To: <1469697366-27175-2-git-send-email-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 List-Id: intel-gfx@lists.freedesktop.org T24gdG8sIDIwMTYtMDctMjggYXQgMTA6MTYgKzAxMDAsIENocmlzIFdpbHNvbiB3cm90ZToKPiBT cGFjZSByZXNlcnZhdGlvbiBpcyBhbHJlYWR5IHNhZmUgd2l0aCByZXNwZWN0IHRvIHRoZSByaW5n LT5zaXplCj4gbW9kdWx1cywgYnV0IGhhcmR3YXJlIG9ubHkgZXhwZWN0cyB0byBzZWUgdmFsdWVz IGluIHRoZSByYW5nZQo+IDAuLi5yaW5nLT5zaXplLTEgKGluY2x1c2l2ZSkgYW5kIHNvIHJlcXVp cmVzIHRoZSBtb2R1bHVzIHRvIHByZXZlbnQgdXMKPiB3cml0aW5nIHRoZSB2YWx1ZSByaW5nLT5z aXplIGluc3RlYWQgb2YgMC4gQXMgdGhpcyBpcyBvbmx5IHJlcXVpcmVkIGZvcgo+IHRoZSByZWdp c3RlciBpdHNlbGYsIHdlIGNhbiBkZWZlciB0aGUgbW9kdWx1cyB0byB0aGUgcmVnaXN0ZXIgdXBk YXRlIGFuZAo+IG5vdCBwZXJmb3JtIGl0IGFmdGVyIGV2ZXJ5IGNvbW1hbmQgcGFja2V0LiBXZSBr ZWVwIHRoZQo+IGludGVsX3JpbmdfYWR2YW5jZSgpIGFyb3VuZCBpbiB0aGUgY29kZSB0byBwcm92 aWRlIGRlbWFyY2F0aW9uIGZvciB0aGUKPiBlbmQtb2YtcGFja2V0ICh3aXRoIHRoZW4gY2FuIGJl IGNvbXBhcmVkIGFnYWluc3QgaW50ZWxfcmluZ19iZWdpbigpIGFzCj4gdGhlIG51bWJlciBvZiBk d29yZHMgZW1pdHRlZCBtdXN0IG1hdGNoIHRoZSByZXNlcnZlZCBzcGFjZSkuCj4gCj4gU2lnbmVk LW9mZi1ieTogQ2hyaXMgV2lsc29uIDxjaHJpc0BjaHJpcy13aWxzb24uY28udWs+Cj4gQ2M6IEpv b25hcyBMYWh0aW5lbiA8am9vbmFzLmxhaHRpbmVuQGxpbnV4LmludGVsLmNvbT4KPiBDYzogRGF2 ZSBHb3Jkb24gPGRhdmlkLnMuZ29yZG9uQGludGVsLmNvbT4KCkxHVE0sCgpSZXZpZXdlZC1ieTog Sm9vbmFzIExhaHRpbmVuIDxqb29uYXMubGFodGluZW5AbGludXguaW50ZWwuY29tPgoKPiAtLS0K PiDCoGRyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2xyYy5jwqDCoMKgwqDCoMKgwqDCoHzCoMKg MiArLQo+IMKgZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcmluZ2J1ZmZlci5jIHzCoMKgNiAr KysrLS0KPiDCoGRyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX3JpbmdidWZmZXIuaCB8IDE3ICsr KysrKysrKysrKystLS0tCj4gwqAzIGZpbGVzIGNoYW5nZWQsIDE4IGluc2VydGlvbnMoKyksIDcg ZGVsZXRpb25zKC0pCj4gCj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVs X2xyYy5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfbHJjLmMKPiBpbmRleCBiZjQyYTY2 ZDY2MjQuLjgyNGY3ZWZlNGU2NCAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9p bnRlbF9scmMuYwo+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2xyYy5jCj4gQEAg LTM3Myw3ICszNzMsNyBAQCBzdGF0aWMgdm9pZCBleGVjbGlzdHNfdXBkYXRlX2NvbnRleHQoc3Ry dWN0IGRybV9pOTE1X2dlbV9yZXF1ZXN0ICpycSkKPiDCoAlzdHJ1Y3QgaTkxNV9od19wcGd0dCAq cHBndHQgPSBycS0+Y3R4LT5wcGd0dDsKPiDCoAl1aW50MzJfdCAqcmVnX3N0YXRlID0gcnEtPmN0 eC0+ZW5naW5lW2VuZ2luZS0+aWRdLmxyY19yZWdfc3RhdGU7Cj4gwqAKPiAtCXJlZ19zdGF0ZVtD VFhfUklOR19UQUlMKzFdID0gcnEtPnRhaWw7Cj4gKwlyZWdfc3RhdGVbQ1RYX1JJTkdfVEFJTCsx XSA9IGludGVsX3Jpbmdfb2Zmc2V0KHJxLT5yaW5nLCBycS0+dGFpbCk7Cj4gwqAKPiDCoAkvKiBU cnVlIDMyYiBQUEdUVCB3aXRoIGR5bmFtaWMgcGFnZSBhbGxvY2F0aW9uOiB1cGRhdGUgUERQCj4g wqAJwqAqIHJlZ2lzdGVycyBhbmQgcG9pbnQgdGhlIHVuYWxsb2NhdGVkIFBEUHMgdG8gc2NyYXRj aCBwYWdlLgo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9yaW5nYnVm ZmVyLmMgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9yaW5nYnVmZmVyLmMKPiBpbmRleCAz MTQyMDg1YjVjYzAuLjIxZDVlODIwOTQwMCAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0v aTkxNS9pbnRlbF9yaW5nYnVmZmVyLmMKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRl bF9yaW5nYnVmZmVyLmMKPiBAQCAtMTcxOCw3ICsxNzE4LDggQEAgc3RhdGljIHZvaWQgaTl4eF9z dWJtaXRfcmVxdWVzdChzdHJ1Y3QgZHJtX2k5MTVfZ2VtX3JlcXVlc3QgKnJlcXVlc3QpCj4gwqB7 Cj4gwqAJc3RydWN0IGRybV9pOTE1X3ByaXZhdGUgKmRldl9wcml2ID0gcmVxdWVzdC0+aTkxNTsK PiDCoAo+IC0JSTkxNV9XUklURV9UQUlMKHJlcXVlc3QtPmVuZ2luZSwgcmVxdWVzdC0+dGFpbCk7 Cj4gKwlJOTE1X1dSSVRFX1RBSUwocmVxdWVzdC0+ZW5naW5lLAo+ICsJCQlpbnRlbF9yaW5nX29m ZnNldChyZXF1ZXN0LT5yaW5nLCByZXF1ZXN0LT50YWlsKSk7Cj4gwqB9Cj4gwqAKPiDCoHN0YXRp YyB2b2lkCj4gQEAgLTI1MDUsNyArMjUwNiw4IEBAIHN0YXRpYyB2b2lkIGdlbjZfYnNkX3N1Ym1p dF9yZXF1ZXN0KHN0cnVjdCBkcm1faTkxNV9nZW1fcmVxdWVzdCAqcmVxdWVzdCkKPiDCoAkJRFJN X0VSUk9SKCJ0aW1lZCBvdXQgd2FpdGluZyBmb3IgdGhlIEJTRCByaW5nIHRvIHdha2UgdXBcbiIp Owo+IMKgCj4gwqAJLyogTm93IHRoYXQgdGhlIHJpbmcgaXMgZnVsbHkgcG93ZXJlZCB1cCwgdXBk YXRlIHRoZSB0YWlsICovCj4gLQlJOTE1X1dSSVRFX0ZXKFJJTkdfVEFJTChyZXF1ZXN0LT5lbmdp bmUtPm1taW9fYmFzZSksIHJlcXVlc3QtPnRhaWwpOwo+ICsJSTkxNV9XUklURV9GVyhSSU5HX1RB SUwocmVxdWVzdC0+ZW5naW5lLT5tbWlvX2Jhc2UpLAo+ICsJCcKgwqDCoMKgwqDCoGludGVsX3Jp bmdfb2Zmc2V0KHJlcXVlc3QtPnJpbmcsIHJlcXVlc3QtPnRhaWwpKTsKPiDCoAlQT1NUSU5HX1JF QURfRlcoUklOR19UQUlMKHJlcXVlc3QtPmVuZ2luZS0+bW1pb19iYXNlKSk7Cj4gwqAKPiDCoAkv KiBMZXQgdGhlIHJpbmcgc2VuZCBJRExFIG1lc3NhZ2VzIHRvIHRoZSBHVCBhZ2FpbiwKPiBkaWZm IC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcmluZ2J1ZmZlci5oIGIvZHJpdmVy cy9ncHUvZHJtL2k5MTUvaW50ZWxfcmluZ2J1ZmZlci5oCj4gaW5kZXggMTRkMmVhMzZmYjg4Li4x OThiNTQxZjliMjIgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcmlu Z2J1ZmZlci5oCj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcmluZ2J1ZmZlci5o Cj4gQEAgLTQ2MCwxNCArNDYwLDIzIEBAIHN0YXRpYyBpbmxpbmUgdm9pZCBpbnRlbF9yaW5nX2Vt aXRfcmVnKHN0cnVjdCBpbnRlbF9yaW5nICpyaW5nLCBpOTE1X3JlZ190IHJlZykKPiDCoAo+IMKg c3RhdGljIGlubGluZSB2b2lkIGludGVsX3JpbmdfYWR2YW5jZShzdHJ1Y3QgaW50ZWxfcmluZyAq cmluZykKPiDCoHsKPiArCS8qIER1bW15IGZ1bmN0aW9uLgo+ICsJwqAqCj4gKwnCoCogVGhpcyBz ZXJ2ZXMgYXMgYSBwbGFjZWhvbGRlciBpbiB0aGUgY29kZSBzbyB0aGF0IHRoZSByZWFkZXIKPiAr CcKgKiBjYW4gY29tcGFyZSBhZ2FpbnN0IHRoZSBwcmVjZWVkaW5nIGludGVsX3JpbmdfYmVnaW4o KSBhbmQKPiArCcKgKiBjaGVjayB0aGF0IHRoZSBudW1iZXIgb2YgZHdvcmRzIGVtaXR0ZWQgbWF0 Y2hlcyB0aGUgc3BhY2UKPiArCcKgKiByZXNlcnZlZCBmb3IgdGhlIGNvbW1hbmQgcGFja2V0IChp LmUuIHRoZSB2YWx1ZSBwYXNzZWQgdG8KPiArCcKgKiBpbnRlbF9yaW5nX2JlZ2luKCkpLgo+ICsJ wqAqLwo+ICt9Cj4gKwo+ICtzdGF0aWMgaW5saW5lIHUzMiBpbnRlbF9yaW5nX29mZnNldChzdHJ1 Y3QgaW50ZWxfcmluZyAqcmluZywgdTMyIHZhbHVlKQo+ICt7Cj4gwqAJLyogVGhlIG1vZHVsdXMg aXMgcmVxdWlyZWQgc28gdGhhdCB3ZSBhdm9pZCB3cml0aW5nCj4gwqAJwqAqIHJlcXVlc3QtPnRh aWwgPT0gcmluZy0+c2l6ZSwgcmF0aGVyIHRoYW4gdGhlIGV4cGVjdGVkIDAsCj4gwqAJwqAqIGlu dG8gdGhlIFJJTkdfVEFJTCByZWdpc3RlciBhcyB0aGF0IGNhbiBjYXVzZSBhIEdQVSBoYW5nLgo+ IC0JwqAqIEFzIHRoaXMgaXMgb25seSBzdHJpY3RseSByZXF1aXJlZCBmb3IgdGhlIHJlcXVlc3Qt PnRhaWwsCj4gLQnCoCogYW5kIG9ubHkgdGhlbiBhcyB3ZSB3cml0ZSB0aGUgdmFsdWUgaW50byBo YXJkd2FyZSwgd2UgY2FuCj4gLQnCoCogb25lIGRheSByZW1vdmUgdGhlIG1vZHVsdXMgYWZ0ZXIg ZXZlcnkgY29tbWFuZCBwYWNrZXQuCj4gwqAJwqAqLwo+IC0JcmluZy0+dGFpbCAmPSByaW5nLT5z aXplIC0gMTsKPiArCXJldHVybiB2YWx1ZSAmIChyaW5nLT5zaXplIC0gMSk7Cj4gwqB9Cj4gwqAK PiDCoGludCBfX2ludGVsX3Jpbmdfc3BhY2UoaW50IGhlYWQsIGludCB0YWlsLCBpbnQgc2l6ZSk7 Ci0tIApKb29uYXMgTGFodGluZW4KT3BlbiBTb3VyY2UgVGVjaG5vbG9neSBDZW50ZXIKSW50ZWwg Q29ycG9yYXRpb24KX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3Jn Cmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4 Cg==