From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maarten Lankhorst Subject: Re: 3.18-rc regression: drm/nouveau: use shared fences for readable objects Date: Sat, 22 Nov 2014 17:56:15 +0100 Message-ID: <5470C02F.6080103@canonical.com> References: <546C5085.1020300@canonical.com> <546DAC16.80704@canonical.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from youngberry.canonical.com (youngberry.canonical.com [91.189.89.112]) by gabe.freedesktop.org (Postfix) with ESMTP id 88FD76E00E for ; Sat, 22 Nov 2014 08:56:21 -0800 (PST) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Michael Marineau Cc: linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Ben Skeggs List-Id: dri-devel@lists.freedesktop.org SGV5LAoKT3AgMjItMTEtMTQgb20gMDE6MTkgc2NocmVlZiBNaWNoYWVsIE1hcmluZWF1Ogo+IE9u IFRodSwgTm92IDIwLCAyMDE0IGF0IDEyOjUzIEFNLCBNYWFydGVuIExhbmtob3JzdAo+IDxtYWFy dGVuLmxhbmtob3JzdEBjYW5vbmljYWwuY29tPiB3cm90ZToKPj4gT3AgMjAtMTEtMTQgb20gMDU6 MDYgc2NocmVlZiBNaWNoYWVsIE1hcmluZWF1Ogo+Pj4gT24gV2VkLCBOb3YgMTksIDIwMTQgYXQg MTI6MTAgQU0sIE1hYXJ0ZW4gTGFua2hvcnN0Cj4+PiA8bWFhcnRlbi5sYW5raG9yc3RAY2Fub25p Y2FsLmNvbT4gd3JvdGU6Cj4+Pj4gSGV5LAo+Pj4+Cj4+Pj4gT24gMTktMTEtMTQgMDc6NDMsIE1p Y2hhZWwgTWFyaW5lYXUgd3JvdGU6Cj4+Pj4+IE9uIDMuMTgtcmMga2VybmVsJ3MgSSBoYXZlIGJl ZW4gaW50ZXJtaXR0ZW50bHkgZXhwZXJpZW5jaW5nIEdQVQo+Pj4+PiBsb2NrdXBzIHNob3J0bHkg YWZ0ZXIgc3RhcnR1cCwgYWNjb21wYW5pZWQgd2l0aCBvbmUgb3IgYm90aCBvZiB0aGUKPj4+Pj4g Zm9sbG93aW5nIGVycm9yczoKPj4+Pj4KPj4+Pj4gbm91dmVhdSBFWyAgIFBGSUZPXVswMDAwOjAx OjAwLjBdIHJlYWQgZmF1bHQgYXQgMHgwMDA3MzRhMDAwIFtQVEVdCj4+Pj4+IGZyb20gUEJETUEw L0hPU1RfQ1BVIG9uIGNoYW5uZWwgMHgwMDdmYWEzMDAwIFt1bmtub3duXQo+Pj4+PiBub3V2ZWF1 IEVbICAgICBEUk1dIEdQVSBsb2NrdXAgLSBzd2l0Y2hpbmcgdG8gc29mdHdhcmUgZmJjb24KPj4+ Pj4KPj4+Pj4gSSB3YXMgYWJsZSB0byB0cmFjZSB0aGUgaXNzdWUgd2l0aCBiaXNlY3QgdG8gY29t bWl0Cj4+Pj4+IDgwOWU5NDQ3YjkyZmZlMTM0NmIyZDZlYzM5MGUyMTJkNTMwN2Y2MWMgImRybS9u b3V2ZWF1OiB1c2Ugc2hhcmVkCj4+Pj4+IGZlbmNlcyBmb3IgcmVhZGFibGUgb2JqZWN0cyIuIFRo ZSBsb2NrdXBzIGFwcGVhciB0byBoYXZlIGNsZWFyZWQgdXAKPj4+Pj4gc2luY2UgcmV2ZXJ0aW5n IHRoYXQgYW5kIGEgZmV3IHJlbGF0ZWQgZm9sbG93dXAgY29tbWl0czoKPj4+Pj4KPj4+Pj4gODA5 ZTk0NDc6ICJkcm0vbm91dmVhdTogdXNlIHNoYXJlZCBmZW5jZXMgZm9yIHJlYWRhYmxlIG9iamVj dHMiCj4+Pj4+IDA1NWRmZmRmOiAiZHJtL25vdXZlYXU6IGJ1bXAgZHJpdmVyIHBhdGNobGV2ZWwg dG8gMS4yLjEiCj4+Pj4+IGUzYmU0YzIzOiAiZHJtL25vdXZlYXU6IHNwZWNpZnkgaWYgaW50ZXJy dXB0aWJsZSB3YWl0IGlzIGRlc2lyZWQgaW4KPj4+Pj4gbm91dmVhdV9mZW5jZV9zeW5jIgo+Pj4+ PiAxNWE5OTZiYjogImRybS9ub3V2ZWF1OiBhc3NpZ24gZmVuY2VfY2hhbi0+bmFtZSBjb3JyZWN0 bHkiCj4+Pj4gV2VpcmQuIEknbSBub3Qgc3VyZSB5ZXQgd2hhdCBjYXVzZXMgaXQuCj4+Pj4KPj4+ PiBodHRwOi8vY2dpdC5mcmVlZGVza3RvcC5vcmcvfm1sYW5raG9yc3QvbGludXgvY29tbWl0Lz9o PWZpeGVkLWZlbmNlcy1mb3ItYmlzZWN0JmlkPTg2YmU0ZjIxNmJiYjllYTMzMzk4NDNhNTY1OGQ0 YzIxMTYyYzdlZTIKPj4+IEJ1aWxkaW5nIGEga2VybmVsIGZyb20gdGhhdCBjb21taXQgZ2l2ZXMg bWUgYW4gZW50aXJlbHkgbmV3IGJlaGF2aW9yOgo+Pj4gWCBoYW5ncyBmb3IgYXQgbGVhc3QgMTAt MjAgc2Vjb25kcyBhdCBhIHRpbWUgd2l0aCBicmllZiBtb21lbnRzIG9mCj4+PiByZXNwb25zaXZl bmVzcyBiZWZvcmUgaGFuZ2luZyBhZ2FpbiB3aGlsZSBnaXRrIG9uIHRoZSBrZXJuZWwgcmVwbwo+ Pj4gbG9hZHMuIE90aGVyd2lzZSB0aGUgc3lzdGVtIGlzIHJlc3BvbnNpdmUuIFRoZSBoZWFkIG9m IHRoYXQKPj4+IGZpeGVkLWZlbmNlcy1mb3ItYmlzZWN0IGJyYW5jaCAoMWM2YWFmYjUpIHdoaWNo IGlzIHRoZSAidXNlIHNoYXJlZAo+Pj4gZmVuY2VzIGZvciByZWFkYWJsZSBvYmplY3RzIiBjb21t aXQgSSBvcmlnaW5hbGx5IGJpc2VjdGVkIHRvIGRvZXMKPj4+IGZlYXR1cmUgdGhlIGNvbXBsZXRl IGxvY2t1cHMgSSB3YXMgc2VlaW5nIGJlZm9yZS4KPj4gT2sgZm9yIHRoZSBzYWtlIG9mIGFyZ3Vt ZW50IGxldHMganVzdCBhc3N1bWUgdGhleSdyZSBzZXBhcmF0ZSBidWdzLCBhbmQgd2Ugc2hvdWxk IGxvb2sgYXQgeG9yZwo+PiBoYW5naW5nIGZpcnN0Lgo+Pgo+PiBJcyB0aGVyZSBhbnl0aGluZyBp biB0aGUgZG1lc2cgd2hlbiB0aGUgaGFuZ2luZyBoYXBwZW5zPwo+Pgo+PiBBbmQgaXQncyBwcm9i YWJseSAxNSBzZWNvbmRzLCBpZiBpdCdzIGNhbGxlZCB0aHJvdWdoIG5vdXZlYXVfZmVuY2Vfd2Fp dC4KPj4KPj4gVHJ5IGNoYW5naW5nIGVsc2UgaWYgKCFyZXQpIHRvIGVsc2UgaWYgKFdBUk5fT04o IXJldCkpIGluIHRoYXQgZnVuY3Rpb24sIGFuZCBzZWUgaWYgeW91IGdldCBzb21lIGRtZXNnIHNw YW0uIDopCj4gQWRkaW5nIHRoZSBXQVJOX09OIHRvIDg2YmU0ZjIxIHJlcG90cyB0aGUgZm9sbG93 aW5nOgo+Cj4gWyAxMTg4LjY3NjA3M10gLS0tLS0tLS0tLS0tWyBjdXQgaGVyZSBdLS0tLS0tLS0t LS0tCj4gWyAxMTg4LjY3NjE2MV0gV0FSTklORzogQ1BVOiAxIFBJRDogNDc0IGF0Cj4gZHJpdmVy cy9ncHUvZHJtL25vdXZlYXUvbm91dmVhdV9mZW5jZS5jOjM1OQo+IG5vdXZlYXVfZmVuY2Vfd2Fp dC5wYXJ0LjkrMHgzMy8weDQwIFtub3V2ZWF1XSgpCj4gWyAxMTg4LjY3NjE2Nl0gTW9kdWxlcyBs aW5rZWQgaW46IHJuZGlzX2hvc3QgY2RjX2V0aGVyIHVzYm5ldCBtaWkgYm5lcAo+IGVjYiBidHVz YiBibHVldG9vdGggcmZraWxsIGJyaWRnZSBzdHAgbGxjIGhpZF9nZW5lcmljIHVzYl9zdG9yYWdl Cj4gam95ZGV2IG1vdXNlZGV2IGhpZF9hcHBsZSB1c2JoaWQgYmNtNTk3NCBubHNfaXNvODg1OV8x IG5sc19jcDQzNyB2ZmF0Cj4gZmF0IG5vdXZlYXUgc25kX2hkYV9jb2RlY19oZG1pIGNvcmV0ZW1w IHg4Nl9wa2dfdGVtcF90aGVybWFsCj4gaW50ZWxfcG93ZXJjbGFtcCBrdm1faW50ZWwga3ZtIGlU Q09fd2R0IGNyY3QxMGRpZl9wY2xtdWwKPiBpVENPX3ZlbmRvcl9zdXBwb3J0IGNyYzMyY19pbnRl bCBldmRldiBhZXNuaV9pbnRlbCBtYWNfaGlkIGFlc194ODZfNjQKPiBscncgZ2x1ZV9oZWxwZXIg YWJsa19oZWxwZXIgYXBwbGVzbWMgc25kX2hkYV9jb2RlY19jaXJydXMgY3J5cHRkCj4gaW5wdXRf cG9sbGRldiBzbmRfaGRhX2NvZGVjX2dlbmVyaWMgbXhtX3dtaSBsZWRfY2xhc3Mgd21pIG1pY3Jv Y29kZQo+IGh3bW9uIHNuZF9oZGFfaW50ZWwgdHRtIHNuZF9oZGFfY29udHJvbGxlciBscGNfaWNo IGkyY19pODAxIG1mZF9jb3JlCj4gc25kX2hkYV9jb2RlYyBpMmNfYWxnb19iaXQgc25kX2h3ZGVw IGRybV9rbXNfaGVscGVyIHNuZF9wY20gc2JzIGRybQo+IGFwcGxlX2dtdXggaTJjY29yZSBzbmRf dGltZXIgc25kIGFncGdhcnQgbWVpX21lIHNvdW5kY29yZSBzYnNoYyBtZWkKPiB2aWRlbyB4aGNp X2hjZCB1c2Jjb3JlIHVzYl9jb21tb24gYXBwbGVfYmwgYnV0dG9uIGJhdHRlcnkgYWMgZWZpdmFy cwo+IGF1dG9mczQKPiBbIDExODguNjc2MzAwXSAgZWZpdmFyZnMKPiBbIDExODguNjc2MzA4XSBD UFU6IDEgUElEOiA0NzQgQ29tbTogWG9yZyBUYWludGVkOiBHICAgICAgICBXCj4gMy4xNy4wLXJj Mi1udnRlc3QrICMxNDcKPiBbIDExODguNjc2MzEzXSBIYXJkd2FyZSBuYW1lOiBBcHBsZSBJbmMu Cj4gTWFjQm9va1BybzExLDMvTWFjLTJCRDFCMzE5ODNGRTE2NjMsIEJJT1MKPiBNQlAxMTIuODha LjAxMzguQjExLjE0MDgyOTE1MDMgMDgvMjkvMjAxNAo+IFsgMTE4OC42NzYzMTZdICAwMDAwMDAw MDAwMDAwMDA5IGZmZmY4ODA0NWRhZWJjZTggZmZmZmZmZmY4MTRmMGMwOQo+IDAwMDAwMDAwMDAw MDAwMDAKPiBbIDExODguNjc2MzI1XSAgZmZmZjg4MDQ1ZGFlYmQyMCBmZmZmZmZmZjgxMDRlYTVk IGZmZmY4ODAwNmE2YzE0NjgKPiAwMDAwMDAwMGZmZmZmZmYwCj4gWyAxMTg4LjY3NjMzM10gIDAw MDAwMDAwMDAwMDAwMDAgMDAwMDAwMDAwMDAwMDAwMCBmZmZmODgwMDZhNmMxMDAwCj4gZmZmZjg4 MDQ1ZGFlYmQzMAo+IFsgMTE4OC42NzYzNDFdIENhbGwgVHJhY2U6Cj4gWyAxMTg4LjY3NjM1Nl0g IFs8ZmZmZmZmZmY4MTRmMGMwOT5dIGR1bXBfc3RhY2srMHg0ZC8weDY2Cj4gWyAxMTg4LjY3NjM2 OV0gIFs8ZmZmZmZmZmY4MTA0ZWE1ZD5dIHdhcm5fc2xvd3BhdGhfY29tbW9uKzB4N2QvMHhhMAo+ IFsgMTE4OC42NzYzNzddICBbPGZmZmZmZmZmODEwNGViM2E+XSB3YXJuX3Nsb3dwYXRoX251bGwr MHgxYS8weDIwCj4gWyAxMTg4LjY3NjQzOV0gIFs8ZmZmZmZmZmZjMDRkZDUyMz5dCj4gbm91dmVh dV9mZW5jZV93YWl0LnBhcnQuOSsweDMzLzB4NDAgW25vdXZlYXVdCj4gWyAxMTg4LjY3NjQ5Nl0g IFs8ZmZmZmZmZmZjMDRkZGZlNj5dIG5vdXZlYXVfZmVuY2Vfd2FpdCsweDE2LzB4MzAgW25vdXZl YXVdCj4gWyAxMTg4LjY3NjU1Ml0gIFs8ZmZmZmZmZmZjMDRlNTk4Zj5dCj4gbm91dmVhdV9nZW1f aW9jdGxfY3B1X3ByZXArMHhlZi8weDFmMCBbbm91dmVhdV0KPiBbIDExODguNjc2NTc4XSAgWzxm ZmZmZmZmZmMwMWMyZjRjPl0gZHJtX2lvY3RsKzB4MWVjLzB4NjYwIFtkcm1dCj4gWyAxMTg4LjY3 NjU5MF0gIFs8ZmZmZmZmZmY4MTRmOTAyNj5dID8gX3Jhd19zcGluX3VubG9ja19pcnFyZXN0b3Jl KzB4MzYvMHg3MAo+IFsgMTE4OC42NzY2MDBdICBbPGZmZmZmZmZmODEwOTRmNmQ+XSA/IHRyYWNl X2hhcmRpcnFzX29uKzB4ZC8weDEwCj4gWyAxMTg4LjY3NjY1NV0gIFs8ZmZmZmZmZmZjMDRkYTVi ND5dIG5vdXZlYXVfZHJtX2lvY3RsKzB4NTQvMHhjMCBbbm91dmVhdV0KPiBbIDExODguNjc2NjYz XSAgWzxmZmZmZmZmZjgxMWE4OTUwPl0gZG9fdmZzX2lvY3RsKzB4MzAwLzB4NTIwCj4gWyAxMTg4 LjY3NjY3MV0gIFs8ZmZmZmZmZmY4MTRmOWU1NT5dID8gc3lzcmV0X2NoZWNrKzB4MjIvMHg1ZAo+ IFsgMTE4OC42NzY2NzddICBbPGZmZmZmZmZmODExYThiYjE+XSBTeVNfaW9jdGwrMHg0MS8weDgw Cj4gWyAxMTg4LjY3NjY4M10gIFs8ZmZmZmZmZmY4MTRmOWUyOT5dIHN5c3RlbV9jYWxsX2Zhc3Rw YXRoKzB4MTYvMHgxYgo+IFsgMTE4OC42NzY2ODhdIC0tLVsgZW5kIHRyYWNlIDZmN2E1MTA4NjVi NDY3NGYgXS0tLQo+Cj4gSGVyZSBhcmUgdGhlIGZlbmNlIGV2ZW50cyB0aGF0IGZpcmVkIGR1cmlu ZyB0aGF0IHBhcnRpY3VsYXIgZmVuY2Vfd2FpdDoKPiAgICAgICAgICAgICBYb3JnICAgNDc0IFsw MDRdICAxMTczLjY2NzY0NTogZmVuY2U6ZmVuY2Vfd2FpdF9zdGFydDoKPiBkcml2ZXI9bm91dmVh dSB0aW1lbGluZT1Yb3JnWzQ3NF0gY29udGV4dD0yIHNlcW5vPTU2OTEwCj4gICAgICAgICAgICAg WG9yZyAgIDQ3NCBbMDA0XSAgMTE3My42Njc2NDc6IGZlbmNlOmZlbmNlX2VuYWJsZV9zaWduYWw6 Cj4gZHJpdmVyPW5vdXZlYXUgdGltZWxpbmU9WG9yZ1s0NzRdIGNvbnRleHQ9MiBzZXFubz01Njkx MAo+ICAgICAgICAgIHN3YXBwZXIgICAgIDAgWzAwN10gIDExNzMuNjY3Njg4OiBmZW5jZTpmZW5j ZV9zaWduYWxlZDoKPiBkcml2ZXI9bm91dmVhdSB0aW1lbGluZT1Yb3JnWzQ3NF0gY29udGV4dD0y IHNlcW5vPTU2OTAwCj4gICAgICAgICAgc3dhcHBlciAgICAgMCBbMDA3XSAgMTE3My42Njc2OTI6 IGZlbmNlOmZlbmNlX2Rlc3Ryb3k6Cj4gZHJpdmVyPW5vdXZlYXUgdGltZWxpbmU9WG9yZ1s0NzRd IGNvbnRleHQ9MiBzZXFubz01NjkwMAo+ICAgICAgICAgIHN3YXBwZXIgICAgIDAgWzAwN10gIDEx NzMuNjY3ODM5OiBmZW5jZTpmZW5jZV9zaWduYWxlZDoKPiBkcml2ZXI9bm91dmVhdSB0aW1lbGlu ZT1Yb3JnWzQ3NF0gY29udGV4dD0yIHNlcW5vPTU2OTAxCj4gICAgICAgICAgc3dhcHBlciAgICAg MCBbMDA3XSAgMTE3My42Njc4NDI6IGZlbmNlOmZlbmNlX2Rlc3Ryb3k6Cj4gZHJpdmVyPW5vdXZl YXUgdGltZWxpbmU9WG9yZ1s0NzRdIGNvbnRleHQ9MiBzZXFubz01NjkwMQo+ICAgICAgICAgIHN3 YXBwZXIgICAgIDAgWzAwN10gIDExNzMuNjY4MDIxOiBmZW5jZTpmZW5jZV9zaWduYWxlZDoKPiBk cml2ZXI9bm91dmVhdSB0aW1lbGluZT1Yb3JnWzQ3NF0gY29udGV4dD0yIHNlcW5vPTU2OTAyCj4g ICAgICAgICAgc3dhcHBlciAgICAgMCBbMDA3XSAgMTE3My42Njg0ODI6IGZlbmNlOmZlbmNlX3Np Z25hbGVkOgo+IGRyaXZlcj1ub3V2ZWF1IHRpbWVsaW5lPVhvcmdbNDc0XSBjb250ZXh0PTIgc2Vx bm89NTY5MDMKPiAgICAgICAgICBzd2FwcGVyICAgICAwIFswMDddICAxMTczLjY2ODQ4NTogZmVu Y2U6ZmVuY2VfZGVzdHJveToKPiBkcml2ZXI9bm91dmVhdSB0aW1lbGluZT1Yb3JnWzQ3NF0gY29u dGV4dD0yIHNlcW5vPTU2OTAzCj4gICAgICAgICAgc3dhcHBlciAgICAgMCBbMDA3XSAgMTE3My42 Njg0ODk6IGZlbmNlOmZlbmNlX3NpZ25hbGVkOgo+IGRyaXZlcj1ub3V2ZWF1IHRpbWVsaW5lPVhv cmdbNDc0XSBjb250ZXh0PTIgc2Vxbm89NTY5MDQKPiAgICAgICAgICBzd2FwcGVyICAgICAwIFsw MDddICAxMTczLjY2ODQ5NjogZmVuY2U6ZmVuY2Vfc2lnbmFsZWQ6Cj4gZHJpdmVyPW5vdXZlYXUg dGltZWxpbmU9WG9yZ1s0NzRdIGNvbnRleHQ9MiBzZXFubz01NjkwNQo+ICAgICAgICAgIHN3YXBw ZXIgICAgIDAgWzAwN10gIDExNzMuNjY4NDk5OiBmZW5jZTpmZW5jZV9kZXN0cm95Ogo+IGRyaXZl cj1ub3V2ZWF1IHRpbWVsaW5lPVhvcmdbNDc0XSBjb250ZXh0PTIgc2Vxbm89NTY5MDUKPiAgICAg ICAgICBzd2FwcGVyICAgICAwIFswMDddICAxMTczLjY2ODUwMjogZmVuY2U6ZmVuY2Vfc2lnbmFs ZWQ6Cj4gZHJpdmVyPW5vdXZlYXUgdGltZWxpbmU9WG9yZ1s0NzRdIGNvbnRleHQ9MiBzZXFubz01 NjkwNgo+ICAgICAgICAgIHN3YXBwZXIgICAgIDAgWzAwN10gIDExNzMuNjY4NTA1OiBmZW5jZTpm ZW5jZV9zaWduYWxlZDoKPiBkcml2ZXI9bm91dmVhdSB0aW1lbGluZT1Yb3JnWzQ3NF0gY29udGV4 dD0yIHNlcW5vPTU2OTA3Cj4gICAgICAgICAgc3dhcHBlciAgICAgMCBbMDA3XSAgMTE3My42Njg1 MDg6IGZlbmNlOmZlbmNlX2Rlc3Ryb3k6Cj4gZHJpdmVyPW5vdXZlYXUgdGltZWxpbmU9WG9yZ1s0 NzRdIGNvbnRleHQ9MiBzZXFubz01NjkwNwo+ICAgICAgICAgIHN3YXBwZXIgICAgIDAgWzAwN10g IDExNzMuNjY4NTExOiBmZW5jZTpmZW5jZV9zaWduYWxlZDoKPiBkcml2ZXI9bm91dmVhdSB0aW1l bGluZT1Yb3JnWzQ3NF0gY29udGV4dD0yIHNlcW5vPTU2OTA4Cj4gICAgICAgICAgc3dhcHBlciAg ICAgMCBbMDA3XSAgMTE3My42Njg1MTM6IGZlbmNlOmZlbmNlX2Rlc3Ryb3k6Cj4gZHJpdmVyPW5v dXZlYXUgdGltZWxpbmU9WG9yZ1s0NzRdIGNvbnRleHQ9MiBzZXFubz01NjkwOAo+ICAgICAga3dv cmtlci80OjEgICAgODAgWzAwNF0gIDExNzMuNjc2MjY1OiBmZW5jZTpmZW5jZV9kZXN0cm95Ogo+ IGRyaXZlcj1ub3V2ZWF1IHRpbWVsaW5lPVhvcmdbNDc0XSBjb250ZXh0PTIgc2Vxbm89NTY4OTYK PiAgICAgIGt3b3JrZXIvNDoxICAgIDgwIFswMDRdICAxMTczLjY3NjI3MzogZmVuY2U6ZmVuY2Vf ZGVzdHJveToKPiBkcml2ZXI9bm91dmVhdSB0aW1lbGluZT1Yb3JnWzQ3NF0gY29udGV4dD0yIHNl cW5vPTU2ODk4Cj4gICAgICBrd29ya2VyLzQ6MSAgICA4MCBbMDA0XSAgMTE3My42NzYyNzc6IGZl bmNlOmZlbmNlX2Rlc3Ryb3k6Cj4gZHJpdmVyPW5vdXZlYXUgdGltZWxpbmU9WG9yZ1s0NzRdIGNv bnRleHQ9MiBzZXFubz01NjkwMgo+ICAgICAga3dvcmtlci80OjEgICAgODAgWzAwNF0gIDExNzMu Njc2MjgwOiBmZW5jZTpmZW5jZV9kZXN0cm95Ogo+IGRyaXZlcj1ub3V2ZWF1IHRpbWVsaW5lPVhv cmdbNDc0XSBjb250ZXh0PTIgc2Vxbm89NTY5MDQKPiAgICAgICAgICAgICBYb3JnICAgNDc0IFsw MDFdICAxMTg4LjY3NjA2NzogZmVuY2U6ZmVuY2Vfd2FpdF9lbmQ6Cj4gZHJpdmVyPW5vdXZlYXUg dGltZWxpbmU9WG9yZ1s0NzRdIGNvbnRleHQ9MiBzZXFubz01NjkxMAo+Cj4gSSBhc3N1bWUgdGhh dCBleGNsdWRlcyB0aGUgY29udGV4dCB5b3UgcmVhbGx5IHdhbnQgc28gdGhlIGZ1bGwgZmVuY2UK PiBldmVudCBsb2cgYW5kIGNvcnJlc3BvbmRpbmcgZG1lc2cgb3V0cHV0IGFyZSBhdHRhY2hlZC4K ClllcCwgdGhlIHRyYWNlIGV2ZW50cyBhcmUgdXNlZnVsLiBUaGUgZmVuY2UgaXMgZW1pdHRlZCBh bmQgcHJlc3VtYWJseSBubyBldmVudCBpcyBmaXJlZCBhZnRlciBlbWlzc2lvbi4KCkxldHMgZmlu ZCBvdXQgaWYgdGhlIG52aWYgY3JhcCBpcyBidWdneSBvciBpdCdzIGEgcmVzdWx0IG9mIHNvbWUg b3RoZXIgaXNzdWUsIHdoYXQgaGFwcGVucyB3aGVuIHlvdSBjaGFuZ2U6Ci53YWl0ID0gZmVuY2Vf ZGVmYXVsdF93YWl0LAp0bwoud2FpdCA9IG5vdXZlYXVfZmVuY2Vfd2FpdF9sZWdhY3ksCmluIG5v dXZlYXVfZmVuY2UuYz8KCn5NYWFydGVuCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVsQGxpc3RzLmZy ZWVkZXNrdG9wLm9yZwpodHRwOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGlu Zm8vZHJpLWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752362AbaKVQ4V (ORCPT ); Sat, 22 Nov 2014 11:56:21 -0500 Received: from youngberry.canonical.com ([91.189.89.112]:40967 "EHLO youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752290AbaKVQ4T (ORCPT ); Sat, 22 Nov 2014 11:56:19 -0500 Message-ID: <5470C02F.6080103@canonical.com> Date: Sat, 22 Nov 2014 17:56:15 +0100 From: Maarten Lankhorst User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: Michael Marineau CC: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Ben Skeggs , David Airlie Subject: Re: 3.18-rc regression: drm/nouveau: use shared fences for readable objects References: <546C5085.1020300@canonical.com> <546DAC16.80704@canonical.com> In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hey, Op 22-11-14 om 01:19 schreef Michael Marineau: > On Thu, Nov 20, 2014 at 12:53 AM, Maarten Lankhorst > wrote: >> Op 20-11-14 om 05:06 schreef Michael Marineau: >>> On Wed, Nov 19, 2014 at 12:10 AM, Maarten Lankhorst >>> wrote: >>>> Hey, >>>> >>>> On 19-11-14 07:43, Michael Marineau wrote: >>>>> On 3.18-rc kernel's I have been intermittently experiencing GPU >>>>> lockups shortly after startup, accompanied with one or both of the >>>>> following errors: >>>>> >>>>> nouveau E[ PFIFO][0000:01:00.0] read fault at 0x000734a000 [PTE] >>>>> from PBDMA0/HOST_CPU on channel 0x007faa3000 [unknown] >>>>> nouveau E[ DRM] GPU lockup - switching to software fbcon >>>>> >>>>> I was able to trace the issue with bisect to commit >>>>> 809e9447b92ffe1346b2d6ec390e212d5307f61c "drm/nouveau: use shared >>>>> fences for readable objects". The lockups appear to have cleared up >>>>> since reverting that and a few related followup commits: >>>>> >>>>> 809e9447: "drm/nouveau: use shared fences for readable objects" >>>>> 055dffdf: "drm/nouveau: bump driver patchlevel to 1.2.1" >>>>> e3be4c23: "drm/nouveau: specify if interruptible wait is desired in >>>>> nouveau_fence_sync" >>>>> 15a996bb: "drm/nouveau: assign fence_chan->name correctly" >>>> Weird. I'm not sure yet what causes it. >>>> >>>> http://cgit.freedesktop.org/~mlankhorst/linux/commit/?h=fixed-fences-for-bisect&id=86be4f216bbb9ea3339843a5658d4c21162c7ee2 >>> Building a kernel from that commit gives me an entirely new behavior: >>> X hangs for at least 10-20 seconds at a time with brief moments of >>> responsiveness before hanging again while gitk on the kernel repo >>> loads. Otherwise the system is responsive. The head of that >>> fixed-fences-for-bisect branch (1c6aafb5) which is the "use shared >>> fences for readable objects" commit I originally bisected to does >>> feature the complete lockups I was seeing before. >> Ok for the sake of argument lets just assume they're separate bugs, and we should look at xorg >> hanging first. >> >> Is there anything in the dmesg when the hanging happens? >> >> And it's probably 15 seconds, if it's called through nouveau_fence_wait. >> >> Try changing else if (!ret) to else if (WARN_ON(!ret)) in that function, and see if you get some dmesg spam. :) > Adding the WARN_ON to 86be4f21 repots the following: > > [ 1188.676073] ------------[ cut here ]------------ > [ 1188.676161] WARNING: CPU: 1 PID: 474 at > drivers/gpu/drm/nouveau/nouveau_fence.c:359 > nouveau_fence_wait.part.9+0x33/0x40 [nouveau]() > [ 1188.676166] Modules linked in: rndis_host cdc_ether usbnet mii bnep > ecb btusb bluetooth rfkill bridge stp llc hid_generic usb_storage > joydev mousedev hid_apple usbhid bcm5974 nls_iso8859_1 nls_cp437 vfat > fat nouveau snd_hda_codec_hdmi coretemp x86_pkg_temp_thermal > intel_powerclamp kvm_intel kvm iTCO_wdt crct10dif_pclmul > iTCO_vendor_support crc32c_intel evdev aesni_intel mac_hid aes_x86_64 > lrw glue_helper ablk_helper applesmc snd_hda_codec_cirrus cryptd > input_polldev snd_hda_codec_generic mxm_wmi led_class wmi microcode > hwmon snd_hda_intel ttm snd_hda_controller lpc_ich i2c_i801 mfd_core > snd_hda_codec i2c_algo_bit snd_hwdep drm_kms_helper snd_pcm sbs drm > apple_gmux i2ccore snd_timer snd agpgart mei_me soundcore sbshc mei > video xhci_hcd usbcore usb_common apple_bl button battery ac efivars > autofs4 > [ 1188.676300] efivarfs > [ 1188.676308] CPU: 1 PID: 474 Comm: Xorg Tainted: G W > 3.17.0-rc2-nvtest+ #147 > [ 1188.676313] Hardware name: Apple Inc. > MacBookPro11,3/Mac-2BD1B31983FE1663, BIOS > MBP112.88Z.0138.B11.1408291503 08/29/2014 > [ 1188.676316] 0000000000000009 ffff88045daebce8 ffffffff814f0c09 > 0000000000000000 > [ 1188.676325] ffff88045daebd20 ffffffff8104ea5d ffff88006a6c1468 > 00000000fffffff0 > [ 1188.676333] 0000000000000000 0000000000000000 ffff88006a6c1000 > ffff88045daebd30 > [ 1188.676341] Call Trace: > [ 1188.676356] [] dump_stack+0x4d/0x66 > [ 1188.676369] [] warn_slowpath_common+0x7d/0xa0 > [ 1188.676377] [] warn_slowpath_null+0x1a/0x20 > [ 1188.676439] [] > nouveau_fence_wait.part.9+0x33/0x40 [nouveau] > [ 1188.676496] [] nouveau_fence_wait+0x16/0x30 [nouveau] > [ 1188.676552] [] > nouveau_gem_ioctl_cpu_prep+0xef/0x1f0 [nouveau] > [ 1188.676578] [] drm_ioctl+0x1ec/0x660 [drm] > [ 1188.676590] [] ? _raw_spin_unlock_irqrestore+0x36/0x70 > [ 1188.676600] [] ? trace_hardirqs_on+0xd/0x10 > [ 1188.676655] [] nouveau_drm_ioctl+0x54/0xc0 [nouveau] > [ 1188.676663] [] do_vfs_ioctl+0x300/0x520 > [ 1188.676671] [] ? sysret_check+0x22/0x5d > [ 1188.676677] [] SyS_ioctl+0x41/0x80 > [ 1188.676683] [] system_call_fastpath+0x16/0x1b > [ 1188.676688] ---[ end trace 6f7a510865b4674f ]--- > > Here are the fence events that fired during that particular fence_wait: > Xorg 474 [004] 1173.667645: fence:fence_wait_start: > driver=nouveau timeline=Xorg[474] context=2 seqno=56910 > Xorg 474 [004] 1173.667647: fence:fence_enable_signal: > driver=nouveau timeline=Xorg[474] context=2 seqno=56910 > swapper 0 [007] 1173.667688: fence:fence_signaled: > driver=nouveau timeline=Xorg[474] context=2 seqno=56900 > swapper 0 [007] 1173.667692: fence:fence_destroy: > driver=nouveau timeline=Xorg[474] context=2 seqno=56900 > swapper 0 [007] 1173.667839: fence:fence_signaled: > driver=nouveau timeline=Xorg[474] context=2 seqno=56901 > swapper 0 [007] 1173.667842: fence:fence_destroy: > driver=nouveau timeline=Xorg[474] context=2 seqno=56901 > swapper 0 [007] 1173.668021: fence:fence_signaled: > driver=nouveau timeline=Xorg[474] context=2 seqno=56902 > swapper 0 [007] 1173.668482: fence:fence_signaled: > driver=nouveau timeline=Xorg[474] context=2 seqno=56903 > swapper 0 [007] 1173.668485: fence:fence_destroy: > driver=nouveau timeline=Xorg[474] context=2 seqno=56903 > swapper 0 [007] 1173.668489: fence:fence_signaled: > driver=nouveau timeline=Xorg[474] context=2 seqno=56904 > swapper 0 [007] 1173.668496: fence:fence_signaled: > driver=nouveau timeline=Xorg[474] context=2 seqno=56905 > swapper 0 [007] 1173.668499: fence:fence_destroy: > driver=nouveau timeline=Xorg[474] context=2 seqno=56905 > swapper 0 [007] 1173.668502: fence:fence_signaled: > driver=nouveau timeline=Xorg[474] context=2 seqno=56906 > swapper 0 [007] 1173.668505: fence:fence_signaled: > driver=nouveau timeline=Xorg[474] context=2 seqno=56907 > swapper 0 [007] 1173.668508: fence:fence_destroy: > driver=nouveau timeline=Xorg[474] context=2 seqno=56907 > swapper 0 [007] 1173.668511: fence:fence_signaled: > driver=nouveau timeline=Xorg[474] context=2 seqno=56908 > swapper 0 [007] 1173.668513: fence:fence_destroy: > driver=nouveau timeline=Xorg[474] context=2 seqno=56908 > kworker/4:1 80 [004] 1173.676265: fence:fence_destroy: > driver=nouveau timeline=Xorg[474] context=2 seqno=56896 > kworker/4:1 80 [004] 1173.676273: fence:fence_destroy: > driver=nouveau timeline=Xorg[474] context=2 seqno=56898 > kworker/4:1 80 [004] 1173.676277: fence:fence_destroy: > driver=nouveau timeline=Xorg[474] context=2 seqno=56902 > kworker/4:1 80 [004] 1173.676280: fence:fence_destroy: > driver=nouveau timeline=Xorg[474] context=2 seqno=56904 > Xorg 474 [001] 1188.676067: fence:fence_wait_end: > driver=nouveau timeline=Xorg[474] context=2 seqno=56910 > > I assume that excludes the context you really want so the full fence > event log and corresponding dmesg output are attached. Yep, the trace events are useful. The fence is emitted and presumably no event is fired after emission. Lets find out if the nvif crap is buggy or it's a result of some other issue, what happens when you change: .wait = fence_default_wait, to .wait = nouveau_fence_wait_legacy, in nouveau_fence.c? ~Maarten