From mboxrd@z Thu Jan 1 00:00:00 1970 From: Imre Deak Subject: Re: [PATCH 2/2] drm/i915: Use atomic waits for short non-atomic ones Date: Tue, 28 Jun 2016 20:45:03 +0300 Message-ID: <1467135903.13066.13.camel@intel.com> References: <1467114710-29989-1-git-send-email-tvrtko.ursulin@linux.intel.com> <1467114710-29989-2-git-send-email-tvrtko.ursulin@linux.intel.com> <1467116344.20290.40.camel@intel.com> <57727BBD.7040500@linux.intel.com> <1467121982.20290.57.camel@intel.com> <57728BC9.1090701@linux.intel.com> Reply-To: imre.deak@intel.com 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 34D976E62C for ; Tue, 28 Jun 2016 17:45:11 +0000 (UTC) In-Reply-To: <57728BC9.1090701@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: Tvrtko Ursulin , Intel-gfx@lists.freedesktop.org Cc: Mika Kuoppala List-Id: intel-gfx@lists.freedesktop.org T24gVHVlLCAyMDE2LTA2LTI4IGF0IDE1OjM4ICswMTAwLCBUdnJ0a28gVXJzdWxpbiB3cm90ZToK PiBPbiAyOC8wNi8xNiAxNDo1MywgSW1yZSBEZWFrIHdyb3RlOgo+ID4gT24gdGksIDIwMTYtMDYt MjggYXQgMTQ6MjkgKzAxMDAsIFR2cnRrbyBVcnN1bGluIHdyb3RlOgo+ID4gPiBPbiAyOC8wNi8x NiAxMzoxOSwgSW1yZSBEZWFrIHdyb3RlOgo+ID4gPiA+IE9uIHRpLCAyMDE2LTA2LTI4IGF0IDEy OjUxICswMTAwLCBUdnJ0a28gVXJzdWxpbiB3cm90ZToKPiA+ID4gPiA+IEZyb206IFR2cnRrbyBV cnN1bGluIDx0dnJ0a28udXJzdWxpbkBpbnRlbC5jb20+Cj4gPiA+ID4gPiAKPiA+ID4gPiA+IHVz bGVlcF9yYW5nZSBpcyBub3QgcmVjb21tZW5kZWQgZm9yIHdhaXRzIHNob3J0ZW4gdGhhbiAxMHVz Lgo+ID4gPiA+ID4gCj4gPiA+ID4gPiBNYWtlIHRoZSB3YWl0X2Zvcl91cyB1c2UgdGhlIGF0b21p YyB2YXJpYW50IGZvciBzdWNoIHdhaXRzLgo+ID4gPiA+ID4gCj4gPiA+ID4gPiBUbyBkbyBzbyB3 ZSBuZWVkIHRvIGRpc2FibGUgdGhlICFpbl9hdG9taWMgd2FybmluZyBmb3Igc3VjaCB1c2VzCj4g PiA+ID4gPiBhbmQgYWxzbyBkaXNhYmxlIHByZWVtcHRpb24gc2luY2UgdGhlIG1hY3JvIGlzIHdy aXR0ZW4gaW4gYSB3YXkKPiA+ID4gPiA+IHRvIG9ubHkgYmUgc2FmZSB0byBiZSB1c2VkIGluIGF0 b21pYyBjb250ZXh0IChsb2NhbF9jbG9jaygpIGFuZAo+ID4gPiA+ID4gbm8gc2Vjb25kIENPTkQg Y2hlY2sgYWZ0ZXIgdGhlIHRpbWVvdXQpLgo+ID4gPiA+ID4gCj4gPiA+ID4gPiBTaWduZWQtb2Zm LWJ5OiBUdnJ0a28gVXJzdWxpbiA8dHZydGtvLnVyc3VsaW5AaW50ZWwuY29tPgo+ID4gPiA+ID4g Q2M6IENocmlzIFdpbHNvbiA8Y2hyaXNAY2hyaXMtd2lsc29uLmNvLnVrPgo+ID4gPiA+ID4gQ2M6 IEltcmUgRGVhayA8aW1yZS5kZWFrQGludGVsLmNvbT4KPiA+ID4gPiA+IENjOiBNaWthIEt1b3Bw YWxhIDxtaWthLmt1b3BwYWxhQGludGVsLmNvbT4KPiA+ID4gPiA+IC0tLQo+ID4gPiA+ID4gwqDC oMKgZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZHJ2LmggfCAyOSArKysrKysrKysrKysrKysr KysrKystLS0tLS0tLQo+ID4gPiA+ID4gwqDCoMKgMSBmaWxlIGNoYW5nZWQsIDIxIGluc2VydGlv bnMoKyksIDggZGVsZXRpb25zKC0pCj4gPiA+ID4gPiAKPiA+ID4gPiA+IGRpZmYgLS1naXQgYS9k cml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kcnYuaCBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2lu dGVsX2Rydi5oCj4gPiA+ID4gPiBpbmRleCAzMTU2ZDhkZjc5MjEuLmUyMWJmNmU2ZjExOSAxMDA2 NDQKPiA+ID4gPiA+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2Rydi5oCj4gPiA+ ID4gPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kcnYuaAo+ID4gPiA+ID4gQEAg LTY5LDIwICs2OSwyMSBAQAo+ID4gPiA+ID4gwqDCoMKgfSkKPiA+ID4gPiA+IAo+ID4gPiA+ID4g wqDCoMKgI2RlZmluZSB3YWl0X2ZvcihDT05ELCBNUykJwqDCoAlfd2FpdF9mb3IoKENPTkQpLCAo TVMpICogMTAwMCwgMTAwMCkKPiA+ID4gPiA+IC0jZGVmaW5lIHdhaXRfZm9yX3VzKENPTkQsIFVT KQnCoMKgCV93YWl0X2ZvcigoQ09ORCksIChVUyksIDEpCj4gPiA+ID4gPiAKPiA+ID4gPiA+IMKg wqDCoC8qIElmIENPTkZJR19QUkVFTVBUX0NPVU5UIGlzIGRpc2FibGVkLCBpbl9hdG9taWMoKSBh bHdheXMgcmVwb3J0cyBmYWxzZS4gKi8KPiA+ID4gPiA+IMKgwqDCoCNpZiBkZWZpbmVkKENPTkZJ R19EUk1fSTkxNV9ERUJVRykgJiYgZGVmaW5lZChDT05GSUdfUFJFRU1QVF9DT1VOVCkKPiA+ID4g PiA+IC0jIGRlZmluZSBfV0FJVF9GT1JfQVRPTUlDX0NIRUNLIFdBUk5fT05fT05DRSghaW5fYXRv bWljKCkpCj4gPiA+ID4gPiArIyBkZWZpbmUgX1dBSVRfRk9SX0FUT01JQ19DSEVDSyhBVE9NSUMp IFdBUk5fT05fT05DRSgoQVRPTUlDKSAmJiAhaW5fYXRvbWljKCkpCj4gPiA+ID4gPiDCoMKgwqAj ZWxzZQo+ID4gPiA+ID4gLSMgZGVmaW5lIF9XQUlUX0ZPUl9BVE9NSUNfQ0hFQ0sgZG8geyB9IHdo aWxlICgwKQo+ID4gPiA+ID4gKyMgZGVmaW5lIF9XQUlUX0ZPUl9BVE9NSUNfQ0hFQ0soQVRPTUlD KSBkbyB7IH0gd2hpbGUgKDApCj4gPiA+ID4gPiDCoMKgwqAjZW5kaWYKPiA+ID4gPiA+IAo+ID4g PiA+ID4gLSNkZWZpbmUgX3dhaXRfZm9yX2F0b21pYyhDT05ELCBVUykgKHsgXAo+ID4gPiA+ID4g KyNkZWZpbmUgX3dhaXRfZm9yX2F0b21pYyhDT05ELCBVUywgQVRPTUlDKSAoeyBcCj4gPiA+ID4g PiDCoMKgwqAJdW5zaWduZWQgbG9uZyBlbmRfXzsgXAo+ID4gPiA+ID4gwqDCoMKgCWludCByZXRf XyA9IDA7IFwKPiA+ID4gPiA+IC0JX1dBSVRfRk9SX0FUT01JQ19DSEVDSzsgXAo+ID4gPiA+ID4g LQlCVUlMRF9CVUdfT04oKFVTKSA+IDUwMDAwKTsgXAo+ID4gPiA+ID4gKwlfV0FJVF9GT1JfQVRP TUlDX0NIRUNLKEFUT01JQyk7IFwKPiA+ID4gPiA+ICsJQlVJTERfQlVHX09OKChBVE9NSUMpICYm IChVUykgPiA1MDAwMCk7IFwKPiA+ID4gPiA+ICsJaWYgKCEoQVRPTUlDKSkgXAo+ID4gPiA+ID4g KwkJcHJlZW1wdF9kaXNhYmxlKCk7IFwKPiA+ID4gPiAKPiA+ID4gPiBEaXNhYmxpbmcgcHJlZW1w dGlvbiBmb3IgdGhpcyBwdXJwb3NlIChzY2hlZHVsaW5nIGEgdGltZW91dCkgY291bGQgYmUKPiA+ ID4gPiBmcm93bmVkIHVwb24sIGFsdGhvdWdoIGZvciAxMHVzIG1heSBiZSBub3QgYW4gaXNzdWUu IEFub3RoZXIKPiA+ID4gCj4gPiA+IFBvc3NpYmx5LCBidXQgSSBkb24ndCBzZWUgaG93IHRvIG90 aGVyd2lzZSBkbyBpdC4KPiA+ID4gCj4gPiA+IEFuZCBhYm91dCB0aGUgbnVtYmVyIGl0c2VsZiAt IEkgY2hvc2UgMTB1cyBqdXN0IGJlY2F1c2UgdXNsZWVwX3JhbmdlIGlzCj4gPiA+IG5vdCByZWNv bW1lbmRlZCBmb3IgPDEwdXMgZHVlIHNldHVwIG92ZXJoZWFkLgo+ID4gPiAKPiA+ID4gPiBwb3Nz aWJpbGl0eSB3b3VsZCBiZSB0byB1c2UgY3B1X2Nsb2NrKCkgaW5zdGVhZCB3aGljaCB3b3VsZCBo YXZlIHNvbWUKPiA+ID4gPiBvdmVyaGVhZCBpbiBjYXNlIG9mIHNjaGVkdWxpbmcgYXdheSBmcm9t IHRoZSBpbml0aWFsIENQVSwgYnV0IHdlJ2Qgb25seQo+ID4gPiA+IGluY3VyIGl0IGZvciB0aGUg bm9uLWF0b21pYyA8MTB1cyBjYXNlLCBzbyB3b3VsZCBiZSBuZWdsaWdpYmxlIGltby4KPiA+ID4g PiBZb3UnZCBhbHNvIGhhdmUgdG8gcmUtY2hlY2sgdGhlIGNvbmRpdGlvbiB3aXRoIHRoYXQgc29s dXRpb24uCj4gPiA+IAo+ID4gPiBIb3cgd291bGQgeW91IGltcGxlbWVudCBpdCB3aXRoIGNwdV9j bG9jaz8gV2hhdCB3b3VsZCB5b3UgZG8gd2hlbgo+ID4gPiByZS1zY2hlZHVsZWQ/Cj4gPiAKPiA+ IEJ5IGNhbGN1bGF0aW5nIHRoZSBleHBpcnkgaW4gdGhlIGJlZ2lubmluZyB3aXRoIGNwdV9jbG9j aygpCj4gPiB1c2luZyByYXdfc21wX3Byb2Nlc3Nvcl9pZCgpIGFuZCB0aGVuIGNhbGxpbmcgY3B1 X2Nsb2NrKCkgaW4KPiA+IHRpbWVfYWZ0ZXIoKSB3aXRoIHRoZSBzYW1lIENQVSBpZC4gY3B1X2Ns b2NrKCkgd291bGQgdGhlbiBpbnRlcm5hbGx5Cj4gPiBoYW5kbGUgdGhlIHNjaGVkdWxpbmcgYXdh eSBzY2VuYXJpby4KPiAKPiBSaWdodCwgYnV0IHRoYXQgaXMgYWxzbyBub3QgaWRlYWwgc2luY2Ug aWYgdGhlIHR3byBjcHVfY2xvY2tzIGRpZmZlciB0aGXCoAo+IHJ1bm5pbmcgdGltZSBkb21haW4g aXMgbm90IGlkZW50aWNhbCB0byB0aGUgdGltZW91dCBvbmUuCgpIbSwgdGhpcyBpcyB3aGF0IEkg bWVhbnQ6CgppbnQgY3B1ID3CoHJhd19zbXBfcHJvY2Vzc29yX2lkKCk7CmVuZCA9wqBjcHVfY2xv Y2soY3B1KSArIHRpbWVvdXQ7CndoaWxlICghdGltZV9hZnRlcihjcHVfY2xvY2soY3B1KSwgZW5k KSkKwqAgwqBjaGVjayBjb25kaXRpb24uLi4KClNvIGNwdV9jbG9jaygpIHdvdWxkIGJlIGFsd2F5 cyBjYWxsZWQgd2l0aCB0aGUgc2FtZSBDUFUgaWQgYW5kIGhlbmNlIGl0CndvdWxkIHJldHVybiB0 aW1lc3RhbXBzIGZyb20gdGhlIHNhbWUgdGltZSBkb21haW4uIEluIGNhc2Ugb2YKc2NoZWR1bGlu ZyBhd2F5IGF0IGFueSBwb2ludCBjcHVfY2xvY2soKSB3b3VsZCBpbnRlcm5hbGx5IGRldGVjdCB0 aGlzCmFuZCByZXR1cm4gdGhlIHRpbWVzdGFtcCBmcm9tIHRoZSBvcmlnaW5hbCBDUFUgdGltZSBk b21haW4uCgpUaGlzIGFsc28gaGFzIHRoZSBhZHZhbnRhZ2UgdGhhdCBpbiBjYXNlIG9mIGEgc3Rh YmxlLCBzeW5jaHJvbml6ZWQgVFNDCi0gd2hpY2ggc2hvdWxkIGJlIHRoZSBnZW5lcmljIGNhc2Ug LSBjcHVfY2xvY2soKSBzaW1wbHkgdHJhbnNsYXRlcyB0bwpyZWFkaW5nIHRoZSBUU0MsIHdpdGhv dXQgZGlzYWJsaW5nIHByZS1lbXB0aW9uLgoKSSBkb24ndCBzZWUgYW55IHByb2JsZW0gd2l0aCBD aHJpcycgYXBwcm9hY2ggZWl0aGVyIHRob3VnaC4KCj4gUHJvYmFibHkgd291bGTCoG5vdCBtYXR0 ZXIgYnV0IGZlZWxzIGhhY2t5Lgo+IAo+ID4gPiA+IEFsc28gY291bGQgeW91IGV4cGxhaW4gaG93 IGNhbiB3ZSBpZ25vcmUgaGFyZCBJUlFzIGFzIGhpbnRlZCBieSB0aGUKPiA+ID4gPiBjb21tZW50 IGluIF93YWl0X2Zvcl9hdG9taWMoKT8KPiA+ID4gCj4gPiA+IEhtLCBpbiByZXRyb3NwZWN0IGl0 IGRvZXMgbm90IGxvb2sgc2FmZS4gVXBzaWRlIHRoYXQgYWZ0ZXIgeW91ciBmaXhlcwo+ID4gPiBm cm9tIHRvZGF5IGl0IHdpbGwgYmUsIHNpbmNlIGFsbCByZW1haW5pbmcgY2FsbGVycyBhcmUgd2l0 aCBpbnRlcnJ1cHRzCj4gPiA+IGRpc2FibGVkLgo+ID4gCj4gPiBXZWxsLCBleGNlcHQgZm9yIHRo ZSBHdUMgcGF0aCwgYnV0IHRoYXQncyBmb3IgYSAxMG1zIHRpbWVvdXQsIHNvCj4gPiBwcm9iYWJs eSBkb2Vzbid0IG1hdHRlciAob3IgZWxzZSB3ZSBoYXZlIGEgYmlnZ2VyIHByb2JsZW0pLgo+IAo+ IEkndmUganVzdCBzZW50IGEgcGF0Y2ggZm9yIHRoYXQuCj4gCj4gPiA+IEFuZCBkb3duc2lkZSB0 aGF0IHRoZSBwYXRjaCBmcm9tIHRoaXMgdGhyZWFkIGlzIG5vdCBzYWZlIHRoZW4KPiA+ID4gYW5k IHdvdWxkIG5lZWQgdGhlIGNvbmRpdGlvbiBwdXQgYmFjayBpbi4gUG9zc2libHkgb25seSBpbiB0 aGUgIUFUT01JQwo+ID4gPiBjYXNlIGJ1dCB0aGF0IG1pZ2h0IGJlIHRvbyBmcmFnaWxlIGZvciB0 aGUgZnV0dXJlLgo+ID4gCj4gPiBJJ2Qgc2F5IHdlJ2QgbmVlZCB0aGUgZXh0cmEgY2hlY2sgYXQg bGVhc3Qgd2hlbmV2ZXIgaGFyZCBJUlFzIGFyZSBub3QKPiA+IGRpc2FibGVkLiBFdmVuIHRoZW4g dGhlcmUgY291bGQgYmUgTk1JcyBvciBzb21lIG90aGVyIGJhY2tncm91bmQgc3R1ZmYKPiA+IChN RSkgdGhhdCBjb3VsZCBiZSBhIHByb2JsZW0uIE9UT0ggd2UnZCBpbmN1ciB0aGUgb3ZlcmhlYWQg ZnJvbSB0aGUKPiA+IGV4dHJhIGNoZWNrIG9ubHkgaW4gdGhlIGV4Y2VwdGlvbmFsIHRpbWVvdXQg Y2FzZSwgc28gSSB0aGluayBkb2luZyBpdAo+ID4gaW4gYWxsIGNhc2VzIHdvdWxkbid0IGJlIGEg YmlnIHByb2JsZW0uCj4gCj4gWWVhaCBJJ2xsIHB1dCBpdCBpbi4KPiAKPiBSZWdhcmRzLAo+IAo+ IFR2cnRrbwo+IAo+IAo+IApfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVza3Rv cC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRl bC1nZngK