From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michel Thierry Subject: Re: [PATCH 2/7] drm/i915: Push i915_sw_fence_wait into the nonblocking atomic commit Date: Mon, 7 Aug 2017 10:06:34 -0700 Message-ID: <1840a5ca-e83c-98e3-17b0-9fdde315dbea@intel.com> References: <20170720175754.30751-1-daniel.vetter@ffwll.ch> <20170720175754.30751-3-daniel.vetter@ffwll.ch> <20170807153325.kdwmhx7lj2p44k2z@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by gabe.freedesktop.org (Postfix) with ESMTPS id 23F5789DB5 for ; Mon, 7 Aug 2017 17:07:07 +0000 (UTC) In-Reply-To: <20170807153325.kdwmhx7lj2p44k2z@phenom.ffwll.local> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Daniel Vetter Cc: "Vetter, Daniel" , Daniel Vetter , Intel Graphics Development , "Kuoppala, Mika" List-Id: intel-gfx@lists.freedesktop.org T24gOC83LzIwMTcgODozMyBBTSwgRGFuaWVsIFZldHRlciB3cm90ZToKPiBPbiBUaHUsIEF1ZyAw MywgMjAxNyBhdCAxMjo0NDo0MFBNIC0wNzAwLCBNaWNoZWwgVGhpZXJyeSB3cm90ZToKPj4gT24g Ny8yMC8yMDE3IDEwOjU3IEFNLCBEYW5pZWwgVmV0dGVyIHdyb3RlOgo+Pj4gQmxvY2tpbmcgaW4g YSB3b3JrZXIgaXMgb2ssIHRoYXQncyB3aGF0IHRoZSB1bmJvdW5kX3dxIGlzIGZvci4gQW5kIGl0 Cj4+PiB1bmlmaWVzIHRoZSBwYXRocyBiZXR3ZWVuIHRoZSBibG9ja2luZyBhbmQgbm9uYmxvY2tp bmcgY29tbWl0LCBnaXZpbmcKPj4+IG1lIGp1c3Qgb25lIHBhdGggd2hlcmUgSSBoYXZlIHRvIGlt cGxlbWVudCB0aGUgZGVhZGxvY2sgYXZvaWRhbmNlCj4+PiB0cmlja2VyeSBpbiB0aGUgbmV4dCBw YXRjaC4KPj4+Cj4+PiBJIGZpcnN0IHRyaWVkIHRvIGltcGxlbWVudCB0aGUgZm9sbG93aW5nIHBh dGNoIHdpdGhvdXQgdGhpcyByZXdvcmssIGJ1dAo+Pj4gZm9yY2UtY29tcGxldGluZyBpOTE1X3N3 X2ZlbmNlIGNyZWF0ZXMgc29tZSBzZXJpb3VzIGNoYWxsZW5nZXMgYXJvdW5kCj4+PiBwcm9wZXJs eSBjbGVhbmluZyB0aGluZ3MgdXAuIFNvIHdhc24ndCBhIGZlYXNpYmxlIHNob3J0LXRlcm0gYXBw cm9hY2guCj4+PiBBbm90aGVyIGFwcHJvYWNoIHdvdWxkIGJlIHRvIHNpbXBsZSBrZWVwIHRyYWNr IG9mIGFsbCBwZW5kaW5nIGF0b21pYwo+Pj4gY29tbWl0IHdvcmsgaXRlbXMgYW5kIG1hbnVhbGx5 IHF1ZXVlIHRoZW0gZnJvbSB0aGUgcmVzZXQgY29kZS4gV2l0aCB0aGUKPj4+IGNhdmVhdCB0aGF0 IGRvdWJsZS1xdWV1ZSBpbiBjYXNlIHdlIHJhY2Ugd2l0aCB0aGUgaTkxNV9zd19mZW5jZSBtdXN0 IGJlCj4+PiBhdm9pZGVkLiBHaXZlbiBhbGwgdGhhdCwgdGFraW5nIHRoZSBjb3N0IG9mIGEgZG91 YmxlIHNjaGVkdWxlIGluIGF0b21pYwo+Pj4gZm9yIHRoZSBzaG9ydC10ZXJtIGZpeCBpcyB0aGUg YmVzdCBhcHByb2FjaCwgYnV0IGNhbiBiZSBjaGFuZ2VkIGluIHRoZQo+Pj4gZnV0dXJlIG9mIGNv dXJzZS4KPj4+Cj4+PiB2MjogQW1lbmQgY29tbWl0IG1lc3NhZ2UgKENocmlzKS4KPj4+Cj4+PiBS ZXZpZXdlZC1ieTogTWFhcnRlbiBMYW5raG9yc3QgPG1hYXJ0ZW4ubGFua2hvcnN0QGxpbnV4Lmlu dGVsLmNvbT4KPj4+IENjOiBDaHJpcyBXaWxzb24gPGNocmlzQGNocmlzLXdpbHNvbi5jby51az4K Pj4+IENjOiBNaWthIEt1b3BwYWxhIDxtaWthLmt1b3BwYWxhQGludGVsLmNvbT4KPj4+IENjOiBK b29uYXMgTGFodGluZW4gPGpvb25hcy5sYWh0aW5lbkBsaW51eC5pbnRlbC5jb20+Cj4+PiBTaWdu ZWQtb2ZmLWJ5OiBEYW5pZWwgVmV0dGVyIDxkYW5pZWwudmV0dGVyQGludGVsLmNvbT4KPj4+IC0t LQo+Pj4gICAgZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jIHwgMTUgKysrKysr Ky0tLS0tLS0tCj4+PiAgICAxIGZpbGUgY2hhbmdlZCwgNyBpbnNlcnRpb25zKCspLCA4IGRlbGV0 aW9ucygtKQo+Pj4KPj4+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9k aXNwbGF5LmMgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMKPj4+IGluZGV4 IDk5NTUyMmU0MGVjMS4uZjZiZDYyODJkN2Y3IDEwMDY0NAo+Pj4gLS0tIGEvZHJpdmVycy9ncHUv ZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jCj4+PiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9p bnRlbF9kaXNwbGF5LmMKPj4+IEBAIC0xMjM5NCw2ICsxMjM5NCw4IEBAIHN0YXRpYyB2b2lkIGlu dGVsX2F0b21pY19jb21taXRfdGFpbChzdHJ1Y3QgZHJtX2F0b21pY19zdGF0ZSAqc3RhdGUpCj4+ PiAgICAgICAgICAgdW5zaWduZWQgY3J0Y192YmxhbmtfbWFzayA9IDA7Cj4+PiAgICAgICAgICAg aW50IGk7Cj4+Pgo+Pj4gKyAgICAgICBpOTE1X3N3X2ZlbmNlX3dhaXQoJmludGVsX3N0YXRlLT5j b21taXRfcmVhZHkpOwo+Pj4gKwo+Pj4gICAgICAgICAgIGRybV9hdG9taWNfaGVscGVyX3dhaXRf Zm9yX2RlcGVuZGVuY2llcyhzdGF0ZSk7Cj4+Pgo+Pj4gICAgICAgICAgIGlmIChpbnRlbF9zdGF0 ZS0+bW9kZXNldCkKPj4+IEBAIC0xMjU2MSwxMCArMTI1NjMsNyBAQCBpbnRlbF9hdG9taWNfY29t bWl0X3JlYWR5KHN0cnVjdCBpOTE1X3N3X2ZlbmNlICpmZW5jZSwKPj4+Cj4+PiAgICAgICAgICAg c3dpdGNoIChub3RpZnkpIHsKPj4+ICAgICAgICAgICBjYXNlIEZFTkNFX0NPTVBMRVRFOgo+Pj4g LSAgICAgICAgICAgICAgIGlmIChzdGF0ZS0+YmFzZS5jb21taXRfd29yay5mdW5jKQo+Pj4gLSAg ICAgICAgICAgICAgICAgICAgICAgcXVldWVfd29yayhzeXN0ZW1fdW5ib3VuZF93cSwgJnN0YXRl LT5iYXNlLmNvbW1pdF93b3JrKTsKPj4KPj4gSSB3b3VsZCBhZGQgYSBzbWFsbCBjb21tZW50IGhl cmUsIGJlY2F1c2UgbGF0ZXItb24gaWYgc29tZW9uZSBoYXMgZG91YnRzCj4+IChhbmQgdXNlIGdp dC1ibGFtZSksIGl0IHdvbid0IGJlIHZpc2libGUgdGhhdCBzb21ldGhpbmcgY2hhbmdlZCAodGhl IGNhc2UKPj4gYW5kIGJyZWFrIHdlcmUgYWRkZWQgYnkgdGhlIHNhbWUgY29tbWl0KS4KPiAKPiBI bSwgbm90IHN1cmUgd2hhdCBjb21tZW50IEkgc2hvdWxkIHB1dCBoZXJlPyBTdWdnZXN0aW9ucz8g T25seSB0aGluZyBJCj4gY291bGQgY29tZSB1cCB3aXRoIHdhcwo+IAo+IAkvKiB3ZSBkbyBibG9j a2luZyB3YWl0cyBpbiB0aGUgd29ya2VyLCBub3RoaW5nIHRvIGRvIGhlcmUgKi8KPiAKPiBCdXQg bm90IHN1cmUgdGhhdCBhZGRzIHRoZSBpbmZvcm1hdGlvbiB5b3UncmUgbG9va2luZyBmb3IuCgpU aGF0IHNvdW5kcyBnb29kIHRvIG1lLCBvciBtYXliZQoiYW55IGJsb2NraW5nIHdhaXRzIGFscmVh ZHkgaGFuZGxlZCBpbiB0aGUgd29ya2VyIgoKQnV0IEkgdGhpbmsgYm90aCBhcmUgb2suCgotTWlj aGVsCgo+IAo+Pgo+Pj4gICAgICAgICAgICAgICAgICAgYnJlYWs7Cj4+PiAtCj4+PiAgICAgICAg ICAgY2FzZSBGRU5DRV9GUkVFOgo+Pj4gICAgICAgICAgICAgICAgICAgewo+Pj4gICAgICAgICAg ICAgICAgICAgICAgICAgICBzdHJ1Y3QgaW50ZWxfYXRvbWljX2hlbHBlciAqaGVscGVyID0KPj4+ IEBAIC0xMjY2OCwxNCArMTI2NjcsMTQgQEAgc3RhdGljIGludCBpbnRlbF9hdG9taWNfY29tbWl0 KHN0cnVjdCBkcm1fZGV2aWNlICpkZXYsCj4+PiAgICAgICAgICAgfQo+Pj4KPj4+ICAgICAgICAg ICBkcm1fYXRvbWljX3N0YXRlX2dldChzdGF0ZSk7Cj4+PiAtICAgICAgIElOSVRfV09SSygmc3Rh dGUtPmNvbW1pdF93b3JrLAo+Pj4gLSAgICAgICAgICAgICAgICAgbm9uYmxvY2sgPyBpbnRlbF9h dG9taWNfY29tbWl0X3dvcmsgOiBOVUxMKTsKPj4+ICsgICAgICAgSU5JVF9XT1JLKCZzdGF0ZS0+ Y29tbWl0X3dvcmssIGludGVsX2F0b21pY19jb21taXRfd29yayk7Cj4+Pgo+Pj4gICAgICAgICAg IGk5MTVfc3dfZmVuY2VfY29tbWl0KCZpbnRlbF9zdGF0ZS0+Y29tbWl0X3JlYWR5KTsKPj4+IC0g ICAgICAgaWYgKCFub25ibG9jaykgewo+Pj4gLSAgICAgICAgICAgICAgIGk5MTVfc3dfZmVuY2Vf d2FpdCgmaW50ZWxfc3RhdGUtPmNvbW1pdF9yZWFkeSk7Cj4+PiArICAgICAgIGlmIChub25ibG9j aykKPj4+ICsgICAgICAgICAgICAgICBxdWV1ZV93b3JrKHN5c3RlbV91bmJvdW5kX3dxLCAmc3Rh dGUtPmNvbW1pdF93b3JrKTsKPj4+ICsgICAgICAgZWxzZQo+Pj4gICAgICAgICAgICAgICAgICAg aW50ZWxfYXRvbWljX2NvbW1pdF90YWlsKHN0YXRlKTsKPj4+IC0gICAgICAgfQo+Pj4gKwo+Pj4K Pj4+ICAgICAgICAgICByZXR1cm4gMDsKPj4+ICAgIH0KPj4KPj4gUmV2aWV3ZWQtYnk6IE1pY2hl bCBUaGllcnJ5IDxtaWNoZWwudGhpZXJyeUBpbnRlbC5jb20+Cj4gCl9fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50 ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9y Zy9tYWlsbWFuL2xpc3RpbmZvL2ludGVsLWdmeAo=