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: Tue, 25 Nov 2014 08:43:02 +0100 Message-ID: <54743306.8080907@canonical.com> References: <546C5085.1020300@canonical.com> <546DAC16.80704@canonical.com> <5470C02F.6080103@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 6F1526E2ED for ; Mon, 24 Nov 2014 23:43:05 -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 SGV5LAoKT3AgMjItMTEtMTQgb20gMjE6MTYgc2NocmVlZiBNaWNoYWVsIE1hcmluZWF1Ogo+IE9u IE5vdiAyMiwgMjAxNCAxMTo0NSBBTSwgIk1pY2hhZWwgTWFyaW5lYXUiIDxtaWtlQG1hcmluZWF1 Lm9yZz4gd3JvdGU6Cj4+Cj4+IE9uIE5vdiAyMiwgMjAxNCA4OjU2IEFNLCAiTWFhcnRlbiBMYW5r aG9yc3QiIDwKPiBtYWFydGVuLmxhbmtob3JzdEBjYW5vbmljYWwuY29tPiB3cm90ZToKPj4+IEhl eSwKPj4+Cj4+PiBPcCAyMi0xMS0xNCBvbSAwMToxOSBzY2hyZWVmIE1pY2hhZWwgTWFyaW5lYXU6 Cj4+Pj4gT24gVGh1LCBOb3YgMjAsIDIwMTQgYXQgMTI6NTMgQU0sIE1hYXJ0ZW4gTGFua2hvcnN0 Cj4+Pj4gPG1hYXJ0ZW4ubGFua2hvcnN0QGNhbm9uaWNhbC5jb20+IHdyb3RlOgo+Pj4+PiBPcCAy MC0xMS0xNCBvbSAwNTowNiBzY2hyZWVmIE1pY2hhZWwgTWFyaW5lYXU6Cj4+Pj4+PiBPbiBXZWQs IE5vdiAxOSwgMjAxNCBhdCAxMjoxMCBBTSwgTWFhcnRlbiBMYW5raG9yc3QKPj4+Pj4+IDxtYWFy dGVuLmxhbmtob3JzdEBjYW5vbmljYWwuY29tPiB3cm90ZToKPj4+Pj4+PiBIZXksCj4+Pj4+Pj4K Pj4+Pj4+PiBPbiAxOS0xMS0xNCAwNzo0MywgTWljaGFlbCBNYXJpbmVhdSB3cm90ZToKPj4+Pj4+ Pj4gT24gMy4xOC1yYyBrZXJuZWwncyBJIGhhdmUgYmVlbiBpbnRlcm1pdHRlbnRseSBleHBlcmll bmNpbmcgR1BVCj4+Pj4+Pj4+IGxvY2t1cHMgc2hvcnRseSBhZnRlciBzdGFydHVwLCBhY2NvbXBh bmllZCB3aXRoIG9uZSBvciBib3RoIG9mIHRoZQo+Pj4+Pj4+PiBmb2xsb3dpbmcgZXJyb3JzOgo+ Pj4+Pj4+Pgo+Pj4+Pj4+PiBub3V2ZWF1IEVbICAgUEZJRk9dWzAwMDA6MDE6MDAuMF0gcmVhZCBm YXVsdCBhdCAweDAwMDczNGEwMDAgW1BURV0KPj4+Pj4+Pj4gZnJvbSBQQkRNQTAvSE9TVF9DUFUg b24gY2hhbm5lbCAweDAwN2ZhYTMwMDAgW3Vua25vd25dCj4+Pj4+Pj4+IG5vdXZlYXUgRVsgICAg IERSTV0gR1BVIGxvY2t1cCAtIHN3aXRjaGluZyB0byBzb2Z0d2FyZSBmYmNvbgo+Pj4+Pj4+Pgo+ Pj4+Pj4+PiBJIHdhcyBhYmxlIHRvIHRyYWNlIHRoZSBpc3N1ZSB3aXRoIGJpc2VjdCB0byBjb21t aXQKPj4+Pj4+Pj4gODA5ZTk0NDdiOTJmZmUxMzQ2YjJkNmVjMzkwZTIxMmQ1MzA3ZjYxYyAiZHJt L25vdXZlYXU6IHVzZSBzaGFyZWQKPj4+Pj4+Pj4gZmVuY2VzIGZvciByZWFkYWJsZSBvYmplY3Rz Ii4gVGhlIGxvY2t1cHMgYXBwZWFyIHRvIGhhdmUgY2xlYXJlZAo+IHVwCj4+Pj4+Pj4+IHNpbmNl IHJldmVydGluZyB0aGF0IGFuZCBhIGZldyByZWxhdGVkIGZvbGxvd3VwIGNvbW1pdHM6Cj4+Pj4+ Pj4+Cj4+Pj4+Pj4+IDgwOWU5NDQ3OiAiZHJtL25vdXZlYXU6IHVzZSBzaGFyZWQgZmVuY2VzIGZv ciByZWFkYWJsZSBvYmplY3RzIgo+Pj4+Pj4+PiAwNTVkZmZkZjogImRybS9ub3V2ZWF1OiBidW1w IGRyaXZlciBwYXRjaGxldmVsIHRvIDEuMi4xIgo+Pj4+Pj4+PiBlM2JlNGMyMzogImRybS9ub3V2 ZWF1OiBzcGVjaWZ5IGlmIGludGVycnVwdGlibGUgd2FpdCBpcyBkZXNpcmVkCj4gaW4KPj4+Pj4+ Pj4gbm91dmVhdV9mZW5jZV9zeW5jIgo+Pj4+Pj4+PiAxNWE5OTZiYjogImRybS9ub3V2ZWF1OiBh c3NpZ24gZmVuY2VfY2hhbi0+bmFtZSBjb3JyZWN0bHkiCj4+Pj4+Pj4gV2VpcmQuIEknbSBub3Qg c3VyZSB5ZXQgd2hhdCBjYXVzZXMgaXQuCj4+Pj4+Pj4KPj4+Pj4+Pgo+IGh0dHA6Ly9jZ2l0LmZy ZWVkZXNrdG9wLm9yZy9+bWxhbmtob3JzdC9saW51eC9jb21taXQvP2g9Zml4ZWQtZmVuY2VzLWZv ci1iaXNlY3QmaWQ9ODZiZTRmMjE2YmJiOWVhMzMzOTg0M2E1NjU4ZDRjMjExNjJjN2VlMgo+Pj4+ Pj4gQnVpbGRpbmcgYSBrZXJuZWwgZnJvbSB0aGF0IGNvbW1pdCBnaXZlcyBtZSBhbiBlbnRpcmVs eSBuZXcKPiBiZWhhdmlvcjoKPj4+Pj4+IFggaGFuZ3MgZm9yIGF0IGxlYXN0IDEwLTIwIHNlY29u ZHMgYXQgYSB0aW1lIHdpdGggYnJpZWYgbW9tZW50cyBvZgo+Pj4+Pj4gcmVzcG9uc2l2ZW5lc3Mg YmVmb3JlIGhhbmdpbmcgYWdhaW4gd2hpbGUgZ2l0ayBvbiB0aGUga2VybmVsIHJlcG8KPj4+Pj4+ IGxvYWRzLiBPdGhlcndpc2UgdGhlIHN5c3RlbSBpcyByZXNwb25zaXZlLiBUaGUgaGVhZCBvZiB0 aGF0Cj4+Pj4+PiBmaXhlZC1mZW5jZXMtZm9yLWJpc2VjdCBicmFuY2ggKDFjNmFhZmI1KSB3aGlj aCBpcyB0aGUgInVzZSBzaGFyZWQKPj4+Pj4+IGZlbmNlcyBmb3IgcmVhZGFibGUgb2JqZWN0cyIg Y29tbWl0IEkgb3JpZ2luYWxseSBiaXNlY3RlZCB0byBkb2VzCj4+Pj4+PiBmZWF0dXJlIHRoZSBj b21wbGV0ZSBsb2NrdXBzIEkgd2FzIHNlZWluZyBiZWZvcmUuCj4+Pj4+IE9rIGZvciB0aGUgc2Fr ZSBvZiBhcmd1bWVudCBsZXRzIGp1c3QgYXNzdW1lIHRoZXkncmUgc2VwYXJhdGUgYnVncywKPiBh bmQgd2Ugc2hvdWxkIGxvb2sgYXQgeG9yZwo+Pj4+PiBoYW5naW5nIGZpcnN0Lgo+Pj4+Pgo+Pj4+ PiBJcyB0aGVyZSBhbnl0aGluZyBpbiB0aGUgZG1lc2cgd2hlbiB0aGUgaGFuZ2luZyBoYXBwZW5z Pwo+Pj4+Pgo+Pj4+PiBBbmQgaXQncyBwcm9iYWJseSAxNSBzZWNvbmRzLCBpZiBpdCdzIGNhbGxl ZCB0aHJvdWdoCj4gbm91dmVhdV9mZW5jZV93YWl0Lgo+Pj4+PiBUcnkgY2hhbmdpbmcgZWxzZSBp ZiAoIXJldCkgdG8gZWxzZSBpZiAoV0FSTl9PTighcmV0KSkgaW4gdGhhdAo+IGZ1bmN0aW9uLCBh bmQgc2VlIGlmIHlvdSBnZXQgc29tZSBkbWVzZyBzcGFtLiA6KQo+Pj4+IEFkZGluZyB0aGUgV0FS Tl9PTiB0byA4NmJlNGYyMSByZXBvdHMgdGhlIGZvbGxvd2luZzoKPj4+Pgo+Pj4+IFsgMTE4OC42 NzYwNzNdIC0tLS0tLS0tLS0tLVsgY3V0IGhlcmUgXS0tLS0tLS0tLS0tLQo+Pj4+IFsgMTE4OC42 NzYxNjFdIFdBUk5JTkc6IENQVTogMSBQSUQ6IDQ3NCBhdAo+Pj4+IGRyaXZlcnMvZ3B1L2RybS9u b3V2ZWF1L25vdXZlYXVfZmVuY2UuYzozNTkKPj4+PiBub3V2ZWF1X2ZlbmNlX3dhaXQucGFydC45 KzB4MzMvMHg0MCBbbm91dmVhdV0oKQo+Pj4+IFsgMTE4OC42NzYxNjZdIE1vZHVsZXMgbGlua2Vk IGluOiBybmRpc19ob3N0IGNkY19ldGhlciB1c2JuZXQgbWlpIGJuZXAKPj4+PiBlY2IgYnR1c2Ig Ymx1ZXRvb3RoIHJma2lsbCBicmlkZ2Ugc3RwIGxsYyBoaWRfZ2VuZXJpYyB1c2Jfc3RvcmFnZQo+ Pj4+IGpveWRldiBtb3VzZWRldiBoaWRfYXBwbGUgdXNiaGlkIGJjbTU5NzQgbmxzX2lzbzg4NTlf MSBubHNfY3A0MzcgdmZhdAo+Pj4+IGZhdCBub3V2ZWF1IHNuZF9oZGFfY29kZWNfaGRtaSBjb3Jl dGVtcCB4ODZfcGtnX3RlbXBfdGhlcm1hbAo+Pj4+IGludGVsX3Bvd2VyY2xhbXAga3ZtX2ludGVs IGt2bSBpVENPX3dkdCBjcmN0MTBkaWZfcGNsbXVsCj4+Pj4gaVRDT192ZW5kb3Jfc3VwcG9ydCBj cmMzMmNfaW50ZWwgZXZkZXYgYWVzbmlfaW50ZWwgbWFjX2hpZCBhZXNfeDg2XzY0Cj4+Pj4gbHJ3 IGdsdWVfaGVscGVyIGFibGtfaGVscGVyIGFwcGxlc21jIHNuZF9oZGFfY29kZWNfY2lycnVzIGNy eXB0ZAo+Pj4+IGlucHV0X3BvbGxkZXYgc25kX2hkYV9jb2RlY19nZW5lcmljIG14bV93bWkgbGVk X2NsYXNzIHdtaSBtaWNyb2NvZGUKPj4+PiBod21vbiBzbmRfaGRhX2ludGVsIHR0bSBzbmRfaGRh X2NvbnRyb2xsZXIgbHBjX2ljaCBpMmNfaTgwMSBtZmRfY29yZQo+Pj4+IHNuZF9oZGFfY29kZWMg aTJjX2FsZ29fYml0IHNuZF9od2RlcCBkcm1fa21zX2hlbHBlciBzbmRfcGNtIHNicyBkcm0KPj4+ PiBhcHBsZV9nbXV4IGkyY2NvcmUgc25kX3RpbWVyIHNuZCBhZ3BnYXJ0IG1laV9tZSBzb3VuZGNv cmUgc2JzaGMgbWVpCj4+Pj4gdmlkZW8geGhjaV9oY2QgdXNiY29yZSB1c2JfY29tbW9uIGFwcGxl X2JsIGJ1dHRvbiBiYXR0ZXJ5IGFjIGVmaXZhcnMKPj4+PiBhdXRvZnM0Cj4+Pj4gWyAxMTg4LjY3 NjMwMF0gIGVmaXZhcmZzCj4+Pj4gWyAxMTg4LjY3NjMwOF0gQ1BVOiAxIFBJRDogNDc0IENvbW06 IFhvcmcgVGFpbnRlZDogRyAgICAgICAgVwo+Pj4+IDMuMTcuMC1yYzItbnZ0ZXN0KyAjMTQ3Cj4+ Pj4gWyAxMTg4LjY3NjMxM10gSGFyZHdhcmUgbmFtZTogQXBwbGUgSW5jLgo+Pj4+IE1hY0Jvb2tQ cm8xMSwzL01hYy0yQkQxQjMxOTgzRkUxNjYzLCBCSU9TCj4+Pj4gTUJQMTEyLjg4Wi4wMTM4LkIx MS4xNDA4MjkxNTAzIDA4LzI5LzIwMTQKPj4+PiBbIDExODguNjc2MzE2XSAgMDAwMDAwMDAwMDAw MDAwOSBmZmZmODgwNDVkYWViY2U4IGZmZmZmZmZmODE0ZjBjMDkKPj4+PiAwMDAwMDAwMDAwMDAw MDAwCj4+Pj4gWyAxMTg4LjY3NjMyNV0gIGZmZmY4ODA0NWRhZWJkMjAgZmZmZmZmZmY4MTA0ZWE1 ZCBmZmZmODgwMDZhNmMxNDY4Cj4+Pj4gMDAwMDAwMDBmZmZmZmZmMAo+Pj4+IFsgMTE4OC42NzYz MzNdICAwMDAwMDAwMDAwMDAwMDAwIDAwMDAwMDAwMDAwMDAwMDAgZmZmZjg4MDA2YTZjMTAwMAo+ Pj4+IGZmZmY4ODA0NWRhZWJkMzAKPj4+PiBbIDExODguNjc2MzQxXSBDYWxsIFRyYWNlOgo+Pj4+ IFsgMTE4OC42NzYzNTZdICBbPGZmZmZmZmZmODE0ZjBjMDk+XSBkdW1wX3N0YWNrKzB4NGQvMHg2 Ngo+Pj4+IFsgMTE4OC42NzYzNjldICBbPGZmZmZmZmZmODEwNGVhNWQ+XSB3YXJuX3Nsb3dwYXRo X2NvbW1vbisweDdkLzB4YTAKPj4+PiBbIDExODguNjc2Mzc3XSAgWzxmZmZmZmZmZjgxMDRlYjNh Pl0gd2Fybl9zbG93cGF0aF9udWxsKzB4MWEvMHgyMAo+Pj4+IFsgMTE4OC42NzY0MzldICBbPGZm ZmZmZmZmYzA0ZGQ1MjM+XQo+Pj4+IG5vdXZlYXVfZmVuY2Vfd2FpdC5wYXJ0LjkrMHgzMy8weDQw IFtub3V2ZWF1XQo+Pj4+IFsgMTE4OC42NzY0OTZdICBbPGZmZmZmZmZmYzA0ZGRmZTY+XSBub3V2 ZWF1X2ZlbmNlX3dhaXQrMHgxNi8weDMwCj4gW25vdXZlYXVdCj4+Pj4gWyAxMTg4LjY3NjU1Ml0g IFs8ZmZmZmZmZmZjMDRlNTk4Zj5dCj4+Pj4gbm91dmVhdV9nZW1faW9jdGxfY3B1X3ByZXArMHhl Zi8weDFmMCBbbm91dmVhdV0KPj4+PiBbIDExODguNjc2NTc4XSAgWzxmZmZmZmZmZmMwMWMyZjRj Pl0gZHJtX2lvY3RsKzB4MWVjLzB4NjYwIFtkcm1dCj4+Pj4gWyAxMTg4LjY3NjU5MF0gIFs8ZmZm ZmZmZmY4MTRmOTAyNj5dID8KPiBfcmF3X3NwaW5fdW5sb2NrX2lycXJlc3RvcmUrMHgzNi8weDcw Cj4+Pj4gWyAxMTg4LjY3NjYwMF0gIFs8ZmZmZmZmZmY4MTA5NGY2ZD5dID8gdHJhY2VfaGFyZGly cXNfb24rMHhkLzB4MTAKPj4+PiBbIDExODguNjc2NjU1XSAgWzxmZmZmZmZmZmMwNGRhNWI0Pl0g bm91dmVhdV9kcm1faW9jdGwrMHg1NC8weGMwCj4gW25vdXZlYXVdCj4+Pj4gWyAxMTg4LjY3NjY2 M10gIFs8ZmZmZmZmZmY4MTFhODk1MD5dIGRvX3Zmc19pb2N0bCsweDMwMC8weDUyMAo+Pj4+IFsg MTE4OC42NzY2NzFdICBbPGZmZmZmZmZmODE0ZjllNTU+XSA/IHN5c3JldF9jaGVjaysweDIyLzB4 NWQKPj4+PiBbIDExODguNjc2Njc3XSAgWzxmZmZmZmZmZjgxMWE4YmIxPl0gU3lTX2lvY3RsKzB4 NDEvMHg4MAo+Pj4+IFsgMTE4OC42NzY2ODNdICBbPGZmZmZmZmZmODE0ZjllMjk+XSBzeXN0ZW1f Y2FsbF9mYXN0cGF0aCsweDE2LzB4MWIKPj4+PiBbIDExODguNjc2Njg4XSAtLS1bIGVuZCB0cmFj ZSA2ZjdhNTEwODY1YjQ2NzRmIF0tLS0KPj4+Pgo+Pj4+IEhlcmUgYXJlIHRoZSBmZW5jZSBldmVu dHMgdGhhdCBmaXJlZCBkdXJpbmcgdGhhdCBwYXJ0aWN1bGFyCj4gZmVuY2Vfd2FpdDoKPj4+PiAg ICAgICAgICAgICBYb3JnICAgNDc0IFswMDRdICAxMTczLjY2NzY0NTogZmVuY2U6ZmVuY2Vfd2Fp dF9zdGFydDoKPj4+PiBkcml2ZXI9bm91dmVhdSB0aW1lbGluZT1Yb3JnWzQ3NF0gY29udGV4dD0y IHNlcW5vPTU2OTEwCj4+Pj4gICAgICAgICAgICAgWG9yZyAgIDQ3NCBbMDA0XSAgMTE3My42Njc2 NDc6IGZlbmNlOmZlbmNlX2VuYWJsZV9zaWduYWw6Cj4+Pj4gZHJpdmVyPW5vdXZlYXUgdGltZWxp bmU9WG9yZ1s0NzRdIGNvbnRleHQ9MiBzZXFubz01NjkxMAo+Pj4+ICAgICAgICAgIHN3YXBwZXIg ICAgIDAgWzAwN10gIDExNzMuNjY3Njg4OiBmZW5jZTpmZW5jZV9zaWduYWxlZDoKPj4+PiBkcml2 ZXI9bm91dmVhdSB0aW1lbGluZT1Yb3JnWzQ3NF0gY29udGV4dD0yIHNlcW5vPTU2OTAwCj4+Pj4g ICAgICAgICAgc3dhcHBlciAgICAgMCBbMDA3XSAgMTE3My42Njc2OTI6IGZlbmNlOmZlbmNlX2Rl c3Ryb3k6Cj4+Pj4gZHJpdmVyPW5vdXZlYXUgdGltZWxpbmU9WG9yZ1s0NzRdIGNvbnRleHQ9MiBz ZXFubz01NjkwMAo+Pj4+ICAgICAgICAgIHN3YXBwZXIgICAgIDAgWzAwN10gIDExNzMuNjY3ODM5 OiBmZW5jZTpmZW5jZV9zaWduYWxlZDoKPj4+PiBkcml2ZXI9bm91dmVhdSB0aW1lbGluZT1Yb3Jn WzQ3NF0gY29udGV4dD0yIHNlcW5vPTU2OTAxCj4+Pj4gICAgICAgICAgc3dhcHBlciAgICAgMCBb MDA3XSAgMTE3My42Njc4NDI6IGZlbmNlOmZlbmNlX2Rlc3Ryb3k6Cj4+Pj4gZHJpdmVyPW5vdXZl YXUgdGltZWxpbmU9WG9yZ1s0NzRdIGNvbnRleHQ9MiBzZXFubz01NjkwMQo+Pj4+ICAgICAgICAg IHN3YXBwZXIgICAgIDAgWzAwN10gIDExNzMuNjY4MDIxOiBmZW5jZTpmZW5jZV9zaWduYWxlZDoK Pj4+PiBkcml2ZXI9bm91dmVhdSB0aW1lbGluZT1Yb3JnWzQ3NF0gY29udGV4dD0yIHNlcW5vPTU2 OTAyCj4+Pj4gICAgICAgICAgc3dhcHBlciAgICAgMCBbMDA3XSAgMTE3My42Njg0ODI6IGZlbmNl OmZlbmNlX3NpZ25hbGVkOgo+Pj4+IGRyaXZlcj1ub3V2ZWF1IHRpbWVsaW5lPVhvcmdbNDc0XSBj b250ZXh0PTIgc2Vxbm89NTY5MDMKPj4+PiAgICAgICAgICBzd2FwcGVyICAgICAwIFswMDddICAx MTczLjY2ODQ4NTogZmVuY2U6ZmVuY2VfZGVzdHJveToKPj4+PiBkcml2ZXI9bm91dmVhdSB0aW1l bGluZT1Yb3JnWzQ3NF0gY29udGV4dD0yIHNlcW5vPTU2OTAzCj4+Pj4gICAgICAgICAgc3dhcHBl ciAgICAgMCBbMDA3XSAgMTE3My42Njg0ODk6IGZlbmNlOmZlbmNlX3NpZ25hbGVkOgo+Pj4+IGRy aXZlcj1ub3V2ZWF1IHRpbWVsaW5lPVhvcmdbNDc0XSBjb250ZXh0PTIgc2Vxbm89NTY5MDQKPj4+ PiAgICAgICAgICBzd2FwcGVyICAgICAwIFswMDddICAxMTczLjY2ODQ5NjogZmVuY2U6ZmVuY2Vf c2lnbmFsZWQ6Cj4+Pj4gZHJpdmVyPW5vdXZlYXUgdGltZWxpbmU9WG9yZ1s0NzRdIGNvbnRleHQ9 MiBzZXFubz01NjkwNQo+Pj4+ICAgICAgICAgIHN3YXBwZXIgICAgIDAgWzAwN10gIDExNzMuNjY4 NDk5OiBmZW5jZTpmZW5jZV9kZXN0cm95Ogo+Pj4+IGRyaXZlcj1ub3V2ZWF1IHRpbWVsaW5lPVhv cmdbNDc0XSBjb250ZXh0PTIgc2Vxbm89NTY5MDUKPj4+PiAgICAgICAgICBzd2FwcGVyICAgICAw IFswMDddICAxMTczLjY2ODUwMjogZmVuY2U6ZmVuY2Vfc2lnbmFsZWQ6Cj4+Pj4gZHJpdmVyPW5v dXZlYXUgdGltZWxpbmU9WG9yZ1s0NzRdIGNvbnRleHQ9MiBzZXFubz01NjkwNgo+Pj4+ICAgICAg ICAgIHN3YXBwZXIgICAgIDAgWzAwN10gIDExNzMuNjY4NTA1OiBmZW5jZTpmZW5jZV9zaWduYWxl ZDoKPj4+PiBkcml2ZXI9bm91dmVhdSB0aW1lbGluZT1Yb3JnWzQ3NF0gY29udGV4dD0yIHNlcW5v PTU2OTA3Cj4+Pj4gICAgICAgICAgc3dhcHBlciAgICAgMCBbMDA3XSAgMTE3My42Njg1MDg6IGZl bmNlOmZlbmNlX2Rlc3Ryb3k6Cj4+Pj4gZHJpdmVyPW5vdXZlYXUgdGltZWxpbmU9WG9yZ1s0NzRd IGNvbnRleHQ9MiBzZXFubz01NjkwNwo+Pj4+ICAgICAgICAgIHN3YXBwZXIgICAgIDAgWzAwN10g IDExNzMuNjY4NTExOiBmZW5jZTpmZW5jZV9zaWduYWxlZDoKPj4+PiBkcml2ZXI9bm91dmVhdSB0 aW1lbGluZT1Yb3JnWzQ3NF0gY29udGV4dD0yIHNlcW5vPTU2OTA4Cj4+Pj4gICAgICAgICAgc3dh cHBlciAgICAgMCBbMDA3XSAgMTE3My42Njg1MTM6IGZlbmNlOmZlbmNlX2Rlc3Ryb3k6Cj4+Pj4g ZHJpdmVyPW5vdXZlYXUgdGltZWxpbmU9WG9yZ1s0NzRdIGNvbnRleHQ9MiBzZXFubz01NjkwOAo+ Pj4+ICAgICAga3dvcmtlci80OjEgICAgODAgWzAwNF0gIDExNzMuNjc2MjY1OiBmZW5jZTpmZW5j ZV9kZXN0cm95Ogo+Pj4+IGRyaXZlcj1ub3V2ZWF1IHRpbWVsaW5lPVhvcmdbNDc0XSBjb250ZXh0 PTIgc2Vxbm89NTY4OTYKPj4+PiAgICAgIGt3b3JrZXIvNDoxICAgIDgwIFswMDRdICAxMTczLjY3 NjI3MzogZmVuY2U6ZmVuY2VfZGVzdHJveToKPj4+PiBkcml2ZXI9bm91dmVhdSB0aW1lbGluZT1Y b3JnWzQ3NF0gY29udGV4dD0yIHNlcW5vPTU2ODk4Cj4+Pj4gICAgICBrd29ya2VyLzQ6MSAgICA4 MCBbMDA0XSAgMTE3My42NzYyNzc6IGZlbmNlOmZlbmNlX2Rlc3Ryb3k6Cj4+Pj4gZHJpdmVyPW5v dXZlYXUgdGltZWxpbmU9WG9yZ1s0NzRdIGNvbnRleHQ9MiBzZXFubz01NjkwMgo+Pj4+ICAgICAg a3dvcmtlci80OjEgICAgODAgWzAwNF0gIDExNzMuNjc2MjgwOiBmZW5jZTpmZW5jZV9kZXN0cm95 Ogo+Pj4+IGRyaXZlcj1ub3V2ZWF1IHRpbWVsaW5lPVhvcmdbNDc0XSBjb250ZXh0PTIgc2Vxbm89 NTY5MDQKPj4+PiAgICAgICAgICAgICBYb3JnICAgNDc0IFswMDFdICAxMTg4LjY3NjA2NzogZmVu Y2U6ZmVuY2Vfd2FpdF9lbmQ6Cj4+Pj4gZHJpdmVyPW5vdXZlYXUgdGltZWxpbmU9WG9yZ1s0NzRd IGNvbnRleHQ9MiBzZXFubz01NjkxMAo+Pj4+Cj4+Pj4gSSBhc3N1bWUgdGhhdCBleGNsdWRlcyB0 aGUgY29udGV4dCB5b3UgcmVhbGx5IHdhbnQgc28gdGhlIGZ1bGwgZmVuY2UKPj4+PiBldmVudCBs b2cgYW5kIGNvcnJlc3BvbmRpbmcgZG1lc2cgb3V0cHV0IGFyZSBhdHRhY2hlZC4KPj4+IFllcCwg dGhlIHRyYWNlIGV2ZW50cyBhcmUgdXNlZnVsLiBUaGUgZmVuY2UgaXMgZW1pdHRlZCBhbmQgcHJl c3VtYWJseQo+IG5vIGV2ZW50IGlzIGZpcmVkIGFmdGVyIGVtaXNzaW9uLgo+Pj4gTGV0cyBmaW5k IG91dCBpZiB0aGUgbnZpZiBjcmFwIGlzIGJ1Z2d5IG9yIGl0J3MgYSByZXN1bHQgb2Ygc29tZSBv dGhlcgo+IGlzc3VlLCB3aGF0IGhhcHBlbnMgd2hlbiB5b3UgY2hhbmdlOgo+Pj4gLndhaXQgPSBm ZW5jZV9kZWZhdWx0X3dhaXQsCj4+PiB0bwo+Pj4gLndhaXQgPSBub3V2ZWF1X2ZlbmNlX3dhaXRf bGVnYWN5LAo+Pj4gaW4gbm91dmVhdV9mZW5jZS5jPwo+PiBUaGF0IGNoYW5nZSB3b3JrcyBqdXN0 IGZpbmUuCj4gVGhlIHhvcmcgaGFuZ3MgYWxzbyBhcHBlYXIgdG8gYmUgcmVzb2x2ZWQgYnkgZGIx Y2Y0NiAiZHJtL25vdXZlYXU6IHVzZSByY3UKPiBpbiBub3V2ZWF1X2dlbV9pb2N0bF9jcHVfcHJl cCIKPgpPaCByaWdodCwgSSBqdXN0IHJlbWVtYmVyZWQgd2hhdCB3YXMgZ29pbmcgb24gdGhlcmUu Lgp0aGUgLndhaXQgd2FzIGEgd29ya2Fyb3VuZCBmb3IgYSBidWcgaW4gdGhlIG52aWYgaW50ZXJm YWNlLiBUaGUgaGFuZ3Mgc2hvdWxkIGJlIHJlc29sdmVkIGJ5IGFwcGx5aW5nCjdjYWE2M2MwNDAy OWM1ODgyODY1ODY3NDcwYzEwNmVmNzNlYWZiMmIKCkNhbiB5b3UgY29uZmlybT8KCkNvdWxkIHlv dSBhbHNvIGNvbmZpcm0gdGhhdCAxYzZhYWZiNWYxYjg0ZmE1ICJkcm0vbm91dmVhdTogdXNlIHNo YXJlZCBmZW5jZXMgZm9yIHJlYWRhYmxlIG9iamVjdHMsIEVESVRFRCIgd2l0aCB0aGUKYWJvdmUg Y29tbWl0IGFwcGxpZWQgaXMgdGhlIGZpcnN0IGJyb2tlbiBjb21taXQ/Cgp+TWFhcnRlbgoKX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1h aWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5m cmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752115AbaKYHnJ (ORCPT ); Tue, 25 Nov 2014 02:43:09 -0500 Received: from youngberry.canonical.com ([91.189.89.112]:57137 "EHLO youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751209AbaKYHnH (ORCPT ); Tue, 25 Nov 2014 02:43:07 -0500 Message-ID: <54743306.8080907@canonical.com> Date: Tue, 25 Nov 2014 08:43:02 +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: David Airlie , linux-kernel@vger.kernel.org, Ben Skeggs , dri-devel@lists.freedesktop.org Subject: Re: 3.18-rc regression: drm/nouveau: use shared fences for readable objects References: <546C5085.1020300@canonical.com> <546DAC16.80704@canonical.com> <5470C02F.6080103@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 21:16 schreef Michael Marineau: > On Nov 22, 2014 11:45 AM, "Michael Marineau" wrote: >> >> On Nov 22, 2014 8:56 AM, "Maarten Lankhorst" < > maarten.lankhorst@canonical.com> wrote: >>> 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? >> That change works just fine. > The xorg hangs also appear to be resolved by db1cf46 "drm/nouveau: use rcu > in nouveau_gem_ioctl_cpu_prep" > Oh right, I just remembered what was going on there.. the .wait was a workaround for a bug in the nvif interface. The hangs should be resolved by applying 7caa63c04029c5882865867470c106ef73eafb2b Can you confirm? Could you also confirm that 1c6aafb5f1b84fa5 "drm/nouveau: use shared fences for readable objects, EDITED" with the above commit applied is the first broken commit? ~Maarten