From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 02/55] drm/i915: Reserve ring buffer space for i915_add_request() commands Date: Tue, 23 Jun 2015 22:00:15 +0200 Message-ID: <20150623200015.GJ25769@phenom.ffwll.local> References: <1432917856-12261-3-git-send-email-John.C.Harrison@Intel.com> <1434731652-15672-1-git-send-email-John.C.Harrison@Intel.com> <20150622201233.GK25769@phenom.ffwll.local> <55894519.5010906@Intel.com> <20150623132406.GA25769@phenom.ffwll.local> <55897E9C.2060604@Intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail-wi0-f174.google.com (mail-wi0-f174.google.com [209.85.212.174]) by gabe.freedesktop.org (Postfix) with ESMTP id CAF9C6EA59 for ; Tue, 23 Jun 2015 12:57:28 -0700 (PDT) Received: by wicnd19 with SMTP id nd19so116169748wic.1 for ; Tue, 23 Jun 2015 12:57:28 -0700 (PDT) Content-Disposition: inline In-Reply-To: <55897E9C.2060604@Intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: John Harrison Cc: Intel-GFX@Lists.FreeDesktop.Org List-Id: intel-gfx@lists.freedesktop.org T24gVHVlLCBKdW4gMjMsIDIwMTUgYXQgMDQ6NDM6MjRQTSArMDEwMCwgSm9obiBIYXJyaXNvbiB3 cm90ZToKPiBPbiAyMy8wNi8yMDE1IDE0OjI0LCBEYW5pZWwgVmV0dGVyIHdyb3RlOgo+ID5PbiBU dWUsIEp1biAyMywgMjAxNSBhdCAxMjozODowMVBNICswMTAwLCBKb2huIEhhcnJpc29uIHdyb3Rl Ogo+ID4+T24gMjIvMDYvMjAxNSAyMToxMiwgRGFuaWVsIFZldHRlciB3cm90ZToKPiA+Pj5PbiBG cmksIEp1biAxOSwgMjAxNSBhdCAwNTozNDoxMlBNICswMTAwLCBKb2huLkMuSGFycmlzb25ASW50 ZWwuY29tIHdyb3RlOgo+ID4+Pj5Gcm9tOiBKb2huIEhhcnJpc29uIDxKb2huLkMuSGFycmlzb25A SW50ZWwuY29tPgo+ID4+Pj4KPiA+Pj4+SXQgaXMgYSBiYWQgaWRlYSBmb3IgaTkxNV9hZGRfcmVx dWVzdCgpIHRvIGZhaWwuIFRoZSB3b3JrIHdpbGwgYWxyZWFkeSBoYXZlIGJlZW4KPiA+Pj4+c2Vu ZCB0byB0aGUgcmluZyBhbmQgd2lsbCBiZSBwcm9jZXNzZWQsIGJ1dCB0aGVyZSB3aWxsIG5vdCBi ZSBhbnkgdHJhY2tpbmcgb3IKPiA+Pj4+bWFuYWdlbWVudCBvZiB0aGF0IHdvcmsuCj4gPj4+Pgo+ ID4+Pj5UaGUgb25seSB3YXkgdGhlIGFkZCByZXF1ZXN0IGNhbGwgY2FuIGZhaWwgaXMgaWYgaXQg Y2FuJ3Qgd3JpdGUgaXRzIGVwaWxvZ3VlCj4gPj4+PmNvbW1hbmRzIHRvIHRoZSByaW5nIChjYWNo ZSBmbHVzaGluZywgc2Vxbm8gdXBkYXRlcywgaW50ZXJydXB0IHNpZ25hbGxpbmcpLiBUaGUKPiA+ Pj4+cmVhc29ucyBmb3IgdGhhdCBhcmUgbW9zdGx5IGRvd24gdG8gcnVubmluZyBvdXQgb2Ygcmlu ZyBidWZmZXIgc3BhY2UgYW5kIHRoZQo+ID4+Pj5wcm9ibGVtcyBhc3NvY2lhdGVkIHdpdGggdHJ5 aW5nIHRvIGdldCBzb21lIG1vcmUuIFRoaXMgcGF0Y2ggcHJldmVudHMgdGhhdAo+ID4+Pj5zaXR1 YXRpb24gZnJvbSBoYXBwZW5pbmcgaW4gdGhlIGZpcnN0IHBsYWNlLgo+ID4+Pj4KPiA+Pj4+V2hl biBhIHJlcXVlc3QgaXMgY3JlYXRlZCwgaXQgbWFya3Mgc3VmZmljaWVudCBzcGFjZSBhcyByZXNl cnZlZCBmb3IgdGhlCj4gPj4+PmVwaWxvZ3VlIGNvbW1hbmRzLiBUaHVzIGd1YXJhbnRlZWluZyB0 aGF0IGJ5IHRoZSB0aW1lIHRoZSBlcGlsb2d1ZSBpcyB3cml0dGVuLAo+ID4+Pj50aGVyZSB3aWxs IGJlIHBsZW50eSBvZiBzcGFjZSBmb3IgaXQuIE5vdGUgdGhhdCBhIHJpbmdfYmVnaW4oKSBjYWxs IGlzIHJlcXVpcmVkCj4gPj4+PnRvIGFjdHVhbGx5IHJlc2VydmUgdGhlIHNwYWNlIChhbmQgZG8g YW55IHBvdGVudGlhbCB3YWl0aW5nKS4gSG93ZXZlciwgdGhhdCBpcwo+ID4+Pj5ub3QgY3VycmVu dGx5IGRvbmUgYXQgcmVxdWVzdCBjcmVhdGlvbiB0aW1lLiBUaGlzIGlzIGJlY2F1c2UgdGhlIHJp bmdfYmVnaW4oKQo+ID4+Pj5jb2RlIGNhbiBhbGxvY2F0ZSBhIHJlcXVlc3QuIEhlbmNlIGNhbGxp bmcgYmVnaW4oKSBmcm9tIHRoZSByZXF1ZXN0IGFsbG9jYXRpb24KPiA+Pj4+Y29kZSB3b3VsZCBs ZWFkIHRvIGluZmluaXRlIHJlY3Vyc2lvbiEgTGF0ZXIgcGF0Y2hlcyBpbiB0aGlzIHNlcmllcyBy ZW1vdmUgdGhlCj4gPj4+Pm5lZWQgZm9yIGJlZ2luKCkgdG8gZG8gdGhlIGFsbG9jYXRlLiBBdCB0 aGF0IHBvaW50LCBpdCBiZWNvbWVzIHNhZmUgZm9yIHRoZQo+ID4+Pj5hbGxvY2F0ZSB0byBjYWxs IGJlZ2luKCkgYW5kIHJlYWxseSByZXNlcnZlIHRoZSBzcGFjZS4KPiA+Pj4+Cj4gPj4+PlVudGls IHRoZW4sIHRoZXJlIGlzIGEgcG90ZW50aWFsIGZvciBpbnN1ZmZpY2llbnQgc3BhY2UgdG8gYmUg YXZhaWxhYmxlIGF0IHRoZQo+ID4+Pj5wb2ludCBvZiBjYWxsaW5nIGk5MTVfYWRkX3JlcXVlc3Qo KS4gSG93ZXZlciwgdGhhdCB3b3VsZCBvbmx5IGJlIGluIHRoZSBjYXNlCj4gPj4+PndoZXJlIHRo ZSByZXF1ZXN0IHdhcyBjcmVhdGVkIGFuZCBpbW1lZGlhdGVseSBzdWJtaXR0ZWQgd2l0aG91dCBl dmVyIGNhbGxpbmcKPiA+Pj4+cmluZ19iZWdpbigpIGFuZCBhZGRpbmcgYW55IHdvcmsgdG8gdGhh dCByZXF1ZXN0LiBXaGljaCBzaG91bGQgbmV2ZXIgaGFwcGVuLiBBbmQKPiA+Pj4+ZXZlbiBpZiBp dCBkb2VzLCBhbmQgaWYgdGhhdCByZXF1ZXN0IGhhcHBlbnMgdG8gZmFsbCBkb3duIHRoZSB0aW55 IHdpbmRvdyBvZgo+ID4+Pj5vcHBvcnR1bml0eSBmb3IgZmFpbGluZyBkdWUgdG8gYmVpbmcgb3V0 IG9mIHJpbmcgc3BhY2UgdGhlbiBkb2VzIGl0IHJlYWxseQo+ID4+Pj5tYXR0ZXIgYmVjYXVzZSB0 aGUgcmVxdWVzdCB3YXNuJ3QgZG9pbmcgYW55dGhpbmcgaW4gdGhlIGZpcnN0IHBsYWNlPwo+ID4+ Pj4KPiA+Pj4+djI6IFVwZGF0ZWQgdGhlICdyZXNlcnZlZCBzcGFjZSB0b28gc21hbGwnIHdhcm5p bmcgdG8gaW5jbHVkZSB0aGUgb2ZmZW5kaW5nCj4gPj4+PnNpemVzLiBBZGRlZCBhICdjYW5jZWwn IG9wZXJhdGlvbiB0byBjbGVhbiB1cCB3aGVuIGEgcmVxdWVzdCBpcyBhYmFuZG9uZWQuIEFkZGVk Cj4gPj4+PnJlLWluaXRpYWxpc2F0aW9uIG9mIHRyYWNraW5nIHN0YXRlIGFmdGVyIGEgYnVmZmVy IHdyYXAgdG8ga2VlcCB0aGUgc2FuaXR5Cj4gPj4+PmNoZWNrcyBhY2N1cmF0ZS4KPiA+Pj4+Cj4g Pj4+PnYzOiBJbmNyZW1lbnRlZCB0aGUgcmVzZXJ2ZWQgc2l6ZSB0byBhY2NvbW1vZGF0ZSBJcm9u bGFrZSAoYWZ0ZXIgZmluYWxseQo+ID4+Pj5tYW5hZ2luZyB0byBydW4gb24gYW4gSUxLIHN5c3Rl bSkuIEFsc28gZml4ZWQgbWlzc2luZyB3cmFwIGNvZGUgaW4gTFJDIG1vZGUuCj4gPj4+Pgo+ID4+ Pj52NDogQWRkZWQgZXh0cmEgY29tbWVudCBhbmQgcmVtb3ZlZCBkdXBsaWNhdGUgV0FSTiAoZmVl ZGJhY2sgZnJvbSBUb21hcykuCj4gPj4+Pgo+ID4+Pj52NTogUmUtd3JpdGUgb2Ygd3JhcCBoYW5k bGluZyB0byBwcmV2ZW50IHVubmVjZXNzYXJ5IGVhcmx5IHdyYXBzIChmZWVkYmFjayBmcm9tCj4g Pj4+PkRhbmllbCBWZXR0ZXIpLgo+ID4+PlRoaXMgZGlkbid0IGFjdHVhbGx5IGltcGxlbWVudCB3 aGF0IEkgc3VnZ2VzdGVkICh3cmFwcGluZyBpcyB0aGUgd29yc3QKPiA+Pj5jYXNlLCBoZW5jZSBz a2lwcGluZyB0aGUgY2hlY2sgZm9yIHRoYXQgaXMgYnJlYWtpbmcgdGhlIHNhbml0eSBjaGVjaykg YW5kCj4gPj4+c28gY2hhbmdlZCB0aGUgcGF0Y2ggZnJvbSAiY29ycmVjdCwgYnV0IGEgYml0IGZy YWdpbGUiIHRvIGJyb2tlbi4gSSd2ZQo+ID4+Pm1lcmdlZCB0aGUgcHJldmlvdXMgdmVyc2lvbiBp bnN0ZWFkLgo+ID4+Pi1EYW5pZWwKPiA+PkknbSBjb25mdXNlZC4gSSB0aG91Z2h0IHlvdXIgbWFp biBpc3N1ZSB3YXMgdGhlIGVhcmx5IHdyYXBwaW5nIG5vdCB0aGUKPiA+PnNhbml0eSBjaGVjay4g VGhlIGNoZWNrIGlzIHRvIGVuc3VyZSB0aGF0IHRoZSByZXNlcnZhdGlvbiBpcyBsYXJnZSBlbm91 Z2ggdG8KPiA+PmNvdmVyIGFsbCB0aGUgY29tbWFuZHMgd3JpdHRlbiBkdXJpbmcgcmVxdWVzdCBz dWJtaXNzaW9uLiBUaGF0IHNob3VsZCBub3QgYmUKPiA+PmFmZmVjdGVkIGJ5IHdoZXRoZXIgYSB3 cmFwIG9jY3VycyBvciBub3QuIFdyYXBwaW5nIGRvZXMgbm90IG1hZ2ljYWxseSBhZGQgYW4KPiA+ PmV4dHJhIGJ1bmNoIG9mIGR3b3JkcyB0byB0aGUgZW1pdF9yZXF1ZXN0KCkgY2FsbC4gV2hlcmVh cyBtYWtpbmcgdGhlIGNoZWNrCj4gPj53b3JrIHdpdGggdGhlIHdyYXAgY29uZGl0aW9uIHJlcXVp cmVzIGFkZGluZyBpbiBleHRyYSB0cmFja2luZyBzdGF0ZSBvZgo+ID4+ZXhhY3RseSB3aGVyZSB0 aGUgd3JhcCBvY2N1cnJlZC4gVGhhdCBpcyBleHRyYSBjb2RlIHRoYXQgb25seSBleGlzdHMgdG8K PiA+PmNhdGNoIHNvbWV0aGluZyBpbiB0aGUgdmVyeSByYXJlIGNhc2Ugd2hpY2ggc2hvdWxkIGFs cmVhZHkgaGF2ZSBiZWVuIGNhdWdodAo+ID4+aW4gdGhlIHZlcnkgY29tbW9uIGNhc2UuIEkuZS4g aWYgeW91ciByZXNlcnZlZCBzaXplIGlzIHRvbyBzbWFsbCB0aGVuIHlvdQo+ID4+d2lsbCBoaXQg dGhlIHdhcm5pbmcgb24gZXZlcnkgYmF0Y2ggYnVmZmVyIHN1Ym1pc3Npb24uCj4gPlRoZSBwcm9i bGVtIGlzIHRoYXQgaWYgeW91IGFsbG93IGEgd3JhcCBpbiB0aGUgcmVzZXJ2ZSBzaXplIHRoZW4g dGhlCj4gPnJpbmdzcGFjZSByZXF1aXJlbWVudHMgYXJlIGJpZ2dlciB0aGFuIGlmIHlvdSBkb24n dCB3cmFwLiBBbmQgc2luY2UgdGhlCj4gPmFkZCByZXF1ZXN0IGlzIHNwbGl0IHVwIGludG8gbWFu eSBpbnRlbF9yaW5nX2JlZ2luIHRoYXQncyBwb3NzaWJsZS4gSGVuY2UKPiA+aWYgeW91IGFsbG93 IHdyYXBwaW5nIGluIHRoZSByZXNlcnZlZCBzcGFjZSwgdGhlbiB0aGUgbW9zdCBpbXBvcnRhbnQg Y2FzZQo+ID5mb3IgdGhlIGRlYnVnIGNoZWNrIGlzIHRvIG1ha2Ugc3VyZSB0aGF0IGl0IGNhdGNo ZXMgYW55IGtpbmQgb2YKPiA+cmVzZXJ2YXRpb24gb3ZlcmZsb3cgd2hpbGUgd3JhcHBpbmcuIFRo ZSBub3Qtd3JhcHBlZCBjYXNlIGlzIHByb2JhYmx5IHRoZQo+ID5ib3Jpbmcgb25lLgo+ID4KPiA+ QW5kIGluZGVlZCBldmVudHVhbGx5IHdlIHNob3VsZCBvdmVyZmxvdyBzaW5jZSBhY2NvcmRpbmcg dG8geW91ciBjb21tZW50Cj4gPnRoZSB3b3JzdCBjYXNlIGFkZCByZXF1ZXN0IG9uIGlsayBpcyAx MzYgZHdvcmRzLiBBbmQgdGhlIGxhcmdlc3QKPiA+aW50ZWxfcmluZ19iZWdpbiBpbiB0aGVyZSBp cyAzMiBkd29yZHMsIHdoaWNoIG1lYW5zIGF0IG1vc3Qgd2UnbGwgdGhyb3cKPiA+YXdheSAzMSBk d29yZHMgd2hlbiB3cmFwcGluZy4gV2hpY2ggbWVhbnMgdGhlIDE2MCBkd29yZHMgb2YgcmVzZXJ2 YXRpb24KPiA+YXJlIG5vdCBlbm91Z2ggc2luY2Ugd2UnZCBuZWVkIDE2NyBkd29yZHMgb2Ygc3Bh Y2UgZm9yIHRoZSB3b3JzdCBjYXNlLiBCdXQKPiA+c2luY2UgdGhlIHNwYWNlX2VuZCBkZWJ1ZyBj aGVjayB3YXMgYSBuby1vcCBmb3IgdGhlIHdyYXBwZWQgY2FzZSB5b3Ugd29uJ3QKPiA+Y2F0Y2gg dGhpcyBvbmUuCj4gCj4gVGhlIG1pbmltdW0gcmVzZXJ2YXRpb24gc2l6ZSBpbiB0aGlzIGNhc2Ug aXMgc3RpbGwgb25seSAxMzYuIFRoZSBwcmVwYXJlCj4gY29kZSBjaGVja3MgZm9yIHRoZSAzMiB3 b3JkcyBhY3R1YWxseSByZXF1ZXN0ZWQgYW5kIHdyYXBzIGlmIG5lY2Vzc2FyeS4gSXQKPiB0aGVu IGNoZWNrcyBmb3IgMTM2KzMyIHdvcmRzIG9mIHNwYWNlLiBJZiB0aGF0IHdvdWxkIGNhdXNlIGEg d3JhcCBpdCB3aWxsCj4gdGhlbiBhZGQgb24gdGhlIGFtb3VudCBvZiBzcGFjZSBhY3R1YWxseSBs ZWZ0IGluIHRoZSByaW5nIGFuZCB3YWl0IGZvciB0aGF0Cj4gYmlnZ2VyIHRvdGFsLiBUaGF0IGd1 YXJhbnRlZXMgdGhhdCBpdCBoYXMgd2FpdGVkIGZvciB0aGUgMTM2IGF0IHRoZSBzdGFydCBvZgo+ IHRoZSByaW5nLiBUaGUgY2FsbGVyIGlzIHRoZW4gZnJlZSB0byBmaWxsIGluIHRoZSAzMiB3b3Jk cyBhbmQgdGhlcmUgaXMgc3RpbGwKPiBndWFyYW50ZWVkIHRvIGJlIGEgbWluaW11bSBvZiAxMzYg d29yZHMgYXZhaWxhYmxlICh3aXRoIG9yIHdpdGhvdXQgd3JhcHBpbmcpCj4gYmVmb3JlIGFueSBm dXJ0aGVyIHdhaXQgZm9yIHNwYWNlIGlzIG5lY2Vzc2FyeS4gVGh1cyB0aGUgYWRkX3JlcXVlc3Qo KSBjb2RlCj4gaXMgc2FmZSBmcm9tIGZlYXIgb2YgZmFpbHVyZSBpcnJlc3BlY3RpdmUgb2Ygd2hl cmUgYW55IHdyYXAgbWlnaHQgb2NjdXIuCj4gCj4gCj4gPgo+ID5XcnQga2VlcGluZyB0cmFjayBv ZiB3cmFwcGluZyB3aGlsZSB0aGUgcmVzZXJ2YXRpb24gaXMgaW4gdXNlLCB0aGUKPiA+Zm9sbG93 aW5nIHNob3VsZCBkbyB0aGF0IHdpdGhvdXQgYW55IG5lZWQgb2YgYWRkaXRpb25hbCB0cmFja2lu ZzoKPiA+Cj4gPgo+ID4JaW50IHVzZWRfc2l6ZSA9IHJpbmdidWYtPnRhaWwgLSByaW5nYnVmLT5y ZXNlcnZlZF90YWlsOwo+ID4KPiA+CWlmICh1c2VkX3NpemUgPCAwKQo+ID4JCXVzZWRfc2l6ZSAr PSByaW5nYnVmLT5zaXplOwo+ID4KPiA+CVdBUk4odXNlZF9zaXplIDwgcmluZ2J1Zi0+cmVzZXJ2 ZWRfc2l6ZSwKPiA+CSAgICAgInJlcXVlc3QgcmVzZXJ2ZWQgc2l6ZSB0b28gc21hbGw6ICVkIHZz ICVkIVxuIiwKPiA+CSAgICAgdXNlZF9zaXplLCByaW5nYnVmLT5yZXNlcnZlZF9zaXplKTsKPiA+ Cj4gPkkgd2FzIG1pc3Rha2VuIHRoYXQgeW91IGNhbiByZXVzZSBfX2ludGVsX3Jpbmdfc3BhY2Ug KHNpbmNlIHRoYXQgaGFzCj4gPnNsaWdodGx5IGRpZmZlcmVudCByZXF1aXJlbWVudHMpLCBidXQg dGhpcyBnaXZlcyB5b3UgYSBuaWNlbHkgbG9jYWxpemVkCj4gPmNoZWNrIGZvciByZXNlcnZhdGlv biBvdmVyZmxvdyB3aGljaCB3b3JrcyBldmVuIHdoZW4geW91IHdyYXAuIE9mYyBpdAo+ID53b24n dCB3b3JrIGlmIGFuIGFkZF9yZXF1ZXN0IGlzIGJpZ2dlciB0aGFuIHRoZSBlbnRpcmUgcmluZywg YnV0IHRoYXQncwo+ID5pbXBvc3NpYmxlIGFueXdheSBzaW5jZSB3ZSBjYW4gYXQgbW9zdCByZXNl cnZlIHJpbmdidWYtPnNpemUgLQo+ID5JOTE1X1JJTkdfRlJFRV9TUEFDRS4KPiBUaGUgcHJvYmxl bSB3aXRoIHRoZSBhYm92ZSBjYWxjdWxhdGlvbiBpcyB0aGF0IGl0IGluY2x1ZGVzIHRoZSB3YXN0 ZWQgc3BhY2UKPiBhdCB0aGUgZW5kIG9mIHRoZSByaW5nLiBUaHVzIGl0IHdpbGwgY29tcGxhaW4g dGhlIHJlc2VydmVkIHNpemUgd2FzIHRvbwo+IHNtYWxsIHdoZW4gaW4gZmFjdCBpdCB3YXMganVz dCBmaW5lLgoKT2sgSSBhZ2FpbiBtaXN1bmRlcnN0b29kIHlvdXIgcGF0Y2ggYSBiaXQgc2luY2Ug aXQgZGlkbid0IHF1aXRlIGRvIHdoYXQgSQpleHBlY3QsIGFuZCBJIHN0YW5kIGNvcnJlY3RlZCB0 aGF0IHY1IHdvcmtzIHRvby4gQnV0IEkgc3RpbGwgc2VlbSB0byBmYWlsCnRvIGdldCBteSBtYWlu IGNvbmNlcm4gYWNyb3NzLiBJJ2xsIHNlZSB3aGV0aGVyIEkgY2FuIHdoaXAgdXAgYSBwYXRjaCBh cyBhCnNob3J0IGRlbW9uc3RyYXRpb24sIG1heWJlIHRoYXQgaGVscHMgdG8gdW5jb25mdXNlIHRo aXMgZGljdXNzaW9uLgoKRm9yIG5vdyBJIHRoaW5rIHdlJ3JlIGNvdmVyZWQgd2l0aCBlaXRoZXIg djQgb3IgdjUgc28gc3RpY2tpbmcgd2l0aCBlaXRoZXIKaXMgb2sgd2l0aCBtZS4KLURhbmllbAot LSAKRGFuaWVsIFZldHRlcgpTb2Z0d2FyZSBFbmdpbmVlciwgSW50ZWwgQ29ycG9yYXRpb24KaHR0 cDovL2Jsb2cuZmZ3bGwuY2gKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0 b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRl bC1nZngK