From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH 13/73] drm/i915: Move the modulus for ring emission to the register write Date: Mon, 1 Aug 2016 17:32:25 +0100 Message-ID: <20160801163225.GD5384@nuc-i3427.alporthouse.com> References: <1470042681-25318-1-git-send-email-chris@chris-wilson.co.uk> <1470042681-25318-14-git-send-email-chris@chris-wilson.co.uk> <1470046075.4174.5.camel@linux.intel.com> <20160801101516.GC4252@nuc-i3427.alporthouse.com> <1470061714.4174.23.camel@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from fireflyinternet.com (mail.fireflyinternet.com [109.228.58.192]) by gabe.freedesktop.org (Postfix) with ESMTPS id B2E236E06B for ; Mon, 1 Aug 2016 16:32:36 +0000 (UTC) Content-Disposition: inline In-Reply-To: <1470061714.4174.23.camel@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Joonas Lahtinen Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T24gTW9uLCBBdWcgMDEsIDIwMTYgYXQgMDU6Mjg6MzRQTSArMDMwMCwgSm9vbmFzIExhaHRpbmVu IHdyb3RlOgo+IE9uIG1hLCAyMDE2LTA4LTAxIGF0IDExOjE1ICswMTAwLCBDaHJpcyBXaWxzb24g d3JvdGU6Cj4gPiBPbiBNb24sIEF1ZyAwMSwgMjAxNiBhdCAwMTowNzo1NVBNICswMzAwLCBKb29u YXMgTGFodGluZW4gd3JvdGU6Cj4gPiA+IAo+ID4gPiBPbiBtYSwgMjAxNi0wOC0wMSBhdCAxMDox MCArMDEwMCwgQ2hyaXMgV2lsc29uIHdyb3RlOgo+ID4gPiA+IAo+ID4gPiA+IFNwYWNlIHJlc2Vy dmF0aW9uIGlzIGFscmVhZHkgc2FmZSB3aXRoIHJlc3BlY3QgdG8gdGhlIHJpbmctPnNpemUKPiA+ ID4gPiBtb2R1bHVzLCBidXQgaGFyZHdhcmUgb25seSBleHBlY3RzIHRvIHNlZSB2YWx1ZXMgaW4g dGhlIHJhbmdlCj4gPiA+ID4gMC4uLnJpbmctPnNpemUtMSAoaW5jbHVzaXZlKSBhbmQgc28gcmVx dWlyZXMgdGhlIG1vZHVsdXMgdG8gcHJldmVudCB1cwo+ID4gPiA+IHdyaXRpbmcgdGhlIHZhbHVl IHJpbmctPnNpemUgaW5zdGVhZCBvZiAwLiBBcyB0aGlzIGlzIG9ubHkgcmVxdWlyZWQgZm9yCj4g PiA+ID4gdGhlIHJlZ2lzdGVyIGl0c2VsZiwgd2UgY2FuIGRlZmVyIHRoZSBtb2R1bHVzIHRvIHRo ZSByZWdpc3RlciB1cGRhdGUgYW5kCj4gPiA+ID4gbm90IHBlcmZvcm0gaXQgYWZ0ZXIgZXZlcnkg Y29tbWFuZCBwYWNrZXQuIFdlIGtlZXAgdGhlCj4gPiA+ID4gaW50ZWxfcmluZ19hZHZhbmNlKCkg YXJvdW5kIGluIHRoZSBjb2RlIHRvIHByb3ZpZGUgZGVtYXJjYXRpb24gZm9yIHRoZQo+ID4gPiA+ IGVuZC1vZi1wYWNrZXQgKHdoaWNoIHRoZW4gY2FuIGJlIGNvbXBhcmVkIGFnYWluc3QgaW50ZWxf cmluZ19iZWdpbigpIGFzCj4gPiA+ID4gdGhlIG51bWJlciBvZiBkd29yZHMgZW1pdHRlZCBtdXN0 IG1hdGNoIHRoZSByZXNlcnZlZCBzcGFjZSkuCj4gPiA+ID4gCj4gPiA+ID4gU2lnbmVkLW9mZi1i eTogQ2hyaXMgV2lsc29uIDxjaHJpc0BjaHJpcy13aWxzb24uY28udWs+Cj4gPiA+ID4gQ2M6IEpv b25hcyBMYWh0aW5lbiA8am9vbmFzLmxhaHRpbmVuQGxpbnV4LmludGVsLmNvbT4KPiA+ID4gPiBD YzogRGF2ZSBHb3Jkb24gPGRhdmlkLnMuZ29yZG9uQGludGVsLmNvbT4KPiA+ID4gPiAtLS0KPiA+ ID4gPiDCoGRyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2xyYy5jwqDCoMKgwqDCoMKgwqDCoHzC oMKgMiArLQo+ID4gPiA+IMKgZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcmluZ2J1ZmZlci5j IHzCoMKgNiArKysrLS0KPiA+ID4gPiDCoGRyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX3Jpbmdi dWZmZXIuaCB8IDE3ICsrKysrKysrKysrKystLS0tCj4gPiA+ID4gwqAzIGZpbGVzIGNoYW5nZWQs IDE4IGluc2VydGlvbnMoKyksIDcgZGVsZXRpb25zKC0pCj4gPiA+ID4gCj4gPiA+ID4gZGlmZiAt LWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2xyYy5jIGIvZHJpdmVycy9ncHUvZHJt L2k5MTUvaW50ZWxfbHJjLmMKPiA+ID4gPiBpbmRleCBiZjQyYTY2ZDY2MjQuLjgyNGY3ZWZlNGU2 NCAxMDA2NDQKPiA+ID4gPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9scmMuYwo+ ID4gPiA+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2xyYy5jCj4gPiA+ID4gQEAg LTM3Myw3ICszNzMsNyBAQCBzdGF0aWMgdm9pZCBleGVjbGlzdHNfdXBkYXRlX2NvbnRleHQoc3Ry dWN0IGRybV9pOTE1X2dlbV9yZXF1ZXN0ICpycSkKPiA+ID4gPiDCoAlzdHJ1Y3QgaTkxNV9od19w cGd0dCAqcHBndHQgPSBycS0+Y3R4LT5wcGd0dDsKPiA+ID4gPiDCoAl1aW50MzJfdCAqcmVnX3N0 YXRlID0gcnEtPmN0eC0+ZW5naW5lW2VuZ2luZS0+aWRdLmxyY19yZWdfc3RhdGU7Cj4gPiA+ID4g wqAKPiA+ID4gPiAtCXJlZ19zdGF0ZVtDVFhfUklOR19UQUlMKzFdID0gcnEtPnRhaWw7Cj4gPiA+ ID4gKwlyZWdfc3RhdGVbQ1RYX1JJTkdfVEFJTCsxXSA9IGludGVsX3Jpbmdfb2Zmc2V0KHJxLT5y aW5nLCBycS0+dGFpbCk7Cj4gPiA+ID4gwqAKPiA+ID4gPiDCoAkvKiBUcnVlIDMyYiBQUEdUVCB3 aXRoIGR5bmFtaWMgcGFnZSBhbGxvY2F0aW9uOiB1cGRhdGUgUERQCj4gPiA+ID4gwqAJwqAqIHJl Z2lzdGVycyBhbmQgcG9pbnQgdGhlIHVuYWxsb2NhdGVkIFBEUHMgdG8gc2NyYXRjaCBwYWdlLgo+ ID4gPiA+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9yaW5nYnVmZmVy LmMgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9yaW5nYnVmZmVyLmMKPiA+ID4gPiBpbmRl eCAzMTQyMDg1YjVjYzAuLjIxZDVlODIwOTQwMCAxMDA2NDQKPiA+ID4gPiAtLS0gYS9kcml2ZXJz L2dwdS9kcm0vaTkxNS9pbnRlbF9yaW5nYnVmZmVyLmMKPiA+ID4gPiArKysgYi9kcml2ZXJzL2dw dS9kcm0vaTkxNS9pbnRlbF9yaW5nYnVmZmVyLmMKPiA+ID4gPiBAQCAtMTcxOCw3ICsxNzE4LDgg QEAgc3RhdGljIHZvaWQgaTl4eF9zdWJtaXRfcmVxdWVzdChzdHJ1Y3QgZHJtX2k5MTVfZ2VtX3Jl cXVlc3QgKnJlcXVlc3QpCj4gPiA+ID4gwqB7Cj4gPiA+ID4gwqAJc3RydWN0IGRybV9pOTE1X3By aXZhdGUgKmRldl9wcml2ID0gcmVxdWVzdC0+aTkxNTsKPiA+ID4gPiDCoAo+ID4gPiA+IC0JSTkx NV9XUklURV9UQUlMKHJlcXVlc3QtPmVuZ2luZSwgcmVxdWVzdC0+dGFpbCk7Cj4gPiA+ID4gKwlJ OTE1X1dSSVRFX1RBSUwocmVxdWVzdC0+ZW5naW5lLAo+ID4gPiA+ICsJCQlpbnRlbF9yaW5nX29m ZnNldChyZXF1ZXN0LT5yaW5nLCByZXF1ZXN0LT50YWlsKSk7Cj4gPiA+ID4gwqB9Cj4gPiA+ID4g wqAKPiA+ID4gPiDCoHN0YXRpYyB2b2lkCj4gPiA+ID4gQEAgLTI1MDUsNyArMjUwNiw4IEBAIHN0 YXRpYyB2b2lkIGdlbjZfYnNkX3N1Ym1pdF9yZXF1ZXN0KHN0cnVjdCBkcm1faTkxNV9nZW1fcmVx dWVzdCAqcmVxdWVzdCkKPiA+ID4gPiDCoAkJRFJNX0VSUk9SKCJ0aW1lZCBvdXQgd2FpdGluZyBm b3IgdGhlIEJTRCByaW5nIHRvIHdha2UgdXBcbiIpOwo+ID4gPiA+IMKgCj4gPiA+ID4gwqAJLyog Tm93IHRoYXQgdGhlIHJpbmcgaXMgZnVsbHkgcG93ZXJlZCB1cCwgdXBkYXRlIHRoZSB0YWlsICov Cj4gPiA+ID4gLQlJOTE1X1dSSVRFX0ZXKFJJTkdfVEFJTChyZXF1ZXN0LT5lbmdpbmUtPm1taW9f YmFzZSksIHJlcXVlc3QtPnRhaWwpOwo+ID4gPiA+ICsJSTkxNV9XUklURV9GVyhSSU5HX1RBSUwo cmVxdWVzdC0+ZW5naW5lLT5tbWlvX2Jhc2UpLAo+ID4gPiA+ICsJCcKgwqDCoMKgwqDCoGludGVs X3Jpbmdfb2Zmc2V0KHJlcXVlc3QtPnJpbmcsIHJlcXVlc3QtPnRhaWwpKTsKPiA+ID4gPiDCoAlQ T1NUSU5HX1JFQURfRlcoUklOR19UQUlMKHJlcXVlc3QtPmVuZ2luZS0+bW1pb19iYXNlKSk7Cj4g PiA+ID4gwqAKPiA+ID4gPiDCoAkvKiBMZXQgdGhlIHJpbmcgc2VuZCBJRExFIG1lc3NhZ2VzIHRv IHRoZSBHVCBhZ2FpbiwKPiA+ID4gPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUv aW50ZWxfcmluZ2J1ZmZlci5oIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcmluZ2J1ZmZl ci5oCj4gPiA+ID4gaW5kZXggMTRkMmVhMzZmYjg4Li45YWM5NmRkYjAxZWUgMTAwNjQ0Cj4gPiA+ ID4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcmluZ2J1ZmZlci5oCj4gPiA+ID4g KysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcmluZ2J1ZmZlci5oCj4gPiA+ID4gQEAg LTQ2MCwxNCArNDYwLDIzIEBAIHN0YXRpYyBpbmxpbmUgdm9pZCBpbnRlbF9yaW5nX2VtaXRfcmVn KHN0cnVjdCBpbnRlbF9yaW5nICpyaW5nLCBpOTE1X3JlZ190IHJlZykKPiA+ID4gPiDCoAo+ID4g PiA+IMKgc3RhdGljIGlubGluZSB2b2lkIGludGVsX3JpbmdfYWR2YW5jZShzdHJ1Y3QgaW50ZWxf cmluZyAqcmluZykKPiA+ID4gPiDCoHsKPiA+ID4gPiArCS8qIER1bW15IGZ1bmN0aW9uLgo+ID4g PiA+ICsJwqAqCj4gPiA+ID4gKwnCoCogVGhpcyBzZXJ2ZXMgYXMgYSBwbGFjZWhvbGRlciBpbiB0 aGUgY29kZSBzbyB0aGF0IHRoZSByZWFkZXIKPiA+ID4gPiArCcKgKiBjYW4gY29tcGFyZSBhZ2Fp bnN0IHRoZSBwcmVjZWRpbmcgaW50ZWxfcmluZ19iZWdpbigpIGFuZAo+ID4gPiA+ICsJwqAqIGNo ZWNrIHRoYXQgdGhlIG51bWJlciBvZiBkd29yZHMgZW1pdHRlZCBtYXRjaGVzIHRoZSBzcGFjZQo+ ID4gPiA+ICsJwqAqIHJlc2VydmVkIGZvciB0aGUgY29tbWFuZCBwYWNrZXQgKGkuZS4gdGhlIHZh bHVlIHBhc3NlZCB0bwo+ID4gPiA+ICsJwqAqIGludGVsX3JpbmdfYmVnaW4oKSkuCj4gPiA+ID4g KwnCoCovCj4gPiA+ID4gK30KPiA+ID4gPiArCj4gPiA+ID4gK3N0YXRpYyBpbmxpbmUgdTMyIGlu dGVsX3Jpbmdfb2Zmc2V0KHN0cnVjdCBpbnRlbF9yaW5nICpyaW5nLCB1MzIgdmFsdWUpCj4gPiA+ ID4gK3sKPiA+ID4gPiDCoAkvKiBUaGUgbW9kdWx1cyBpcyByZXF1aXJlZCBzbyB0aGF0IHdlIGF2 b2lkIHdyaXRpbmcKPiA+ID4gPiDCoAnCoCogcmVxdWVzdC0+dGFpbCA9PSByaW5nLT5zaXplLCBy YXRoZXIgdGhhbiB0aGUgZXhwZWN0ZWQgMCwKPiA+ID4gPiDCoAnCoCogaW50byB0aGUgUklOR19U QUlMIHJlZ2lzdGVyIGFzIHRoYXQgY2FuIGNhdXNlIGEgR1BVIGhhbmcuCj4gPiA+ID4gLQnCoCog QXMgdGhpcyBpcyBvbmx5IHN0cmljdGx5IHJlcXVpcmVkIGZvciB0aGUgcmVxdWVzdC0+dGFpbCwK PiA+ID4gPiAtCcKgKiBhbmQgb25seSB0aGVuIGFzIHdlIHdyaXRlIHRoZSB2YWx1ZSBpbnRvIGhh cmR3YXJlLCB3ZSBjYW4KPiA+ID4gPiAtCcKgKiBvbmUgZGF5IHJlbW92ZSB0aGUgbW9kdWx1cyBh ZnRlciBldmVyeSBjb21tYW5kIHBhY2tldC4KPiA+ID4gPiDCoAnCoCovCj4gPiA+ID4gLQlyaW5n LT50YWlsICY9IHJpbmctPnNpemUgLSAxOwo+ID4gPiA+ICsJcmV0dXJuIHZhbHVlICYgKHJpbmct PnNpemUgLSAxKTsKPiA+ID4gPiDCoH0KPiA+ID4gVGhlIGNvbW1lbnQgc2VlbXMgb3V0ZGF0ZWQt aXNoIGFzIGl0IHNwZWFrcyBvZiBtb2R1bHVzIHdoaWNoIGlzIG5vd2hlcmUKPiA+ID4gdG8gYmUg c2Vlbi4gSSdkIHNwZWFrIG9mICdtYXNraW5nJy4gV2l0aCB0aGF0LAo+ID4gVGhlIG9wZXJhdGlv biBpcyBhIG1vZHVsdXMuIFRoZSBjb21tb24gb3B0aW1pc2F0aW9uIGZvciBtb2R1bGkgd2l0aAo+ ID4gYSBwb3dlci1vZi10d28gZGl2aXNvciBiZWluZyBhcHBsaWVkLgo+IAo+IFllcywgSSdtIHBl cmZlY3RseSBhd2FyZSBvZiB0aGF0LCBidXQgYXMgZGlzY3Vzc2VkIGluIElSQywgdGhhdAo+IGlu Zm9ybWF0aW9uIHdvdWxkIGJlIGdvb2QgdG8gYmUgbG9jYWwuIE1vZHVsdXMgaW4gdGhpcyBjYXNl IGEgdHJpY2sgdG8KPiBhY2hpZXZlIHZhbHVlID09IHJpbmctPnNpemUgPyAwIDogdmFsdWU7IFdo aWNoIHdvdWxkIGFnYWluIGJlIG1vcmUKPiBpbmZvcm1hdGl2ZSwgd2UgZG8gbm90IGV4cGVjdCB3 cmFwLCBidXQgd2UgaGF2ZSBvbmUgc3BlY2lhbCB2YWx1ZS4KCkRpdGNoIGFsbCB0aGUgdmVyYmFn ZSwgYW5kIGdvIHdpdGggdGhlIHNpbmdsZSBsaW5lOgoJLyogRG9uJ3Qgd3JpdGUgcmluZy0+c2l6 ZSAoZXF1aXZhbGVudCB0byAwKSBhcyB0aGF0IGhhbmdzIHNvbWUgR1BVcy4gKi8KdGhhdCBqdXN0 IGZpdHMgaW50byA4MGNvbHMuCgpLZWVwIGl0IHRvIHdoeSBub3QgaG93IGFuZCB3ZSBuZWVkIG5v dCBhcmd1ZSBhYm91dCBsYW5ndWFnZS4KLUNocmlzCgotLSAKQ2hyaXMgV2lsc29uLCBJbnRlbCBP cGVuIFNvdXJjZSBUZWNobm9sb2d5IENlbnRyZQpfX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0 cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9s aXN0aW5mby9pbnRlbC1nZngK