From mboxrd@z Thu Jan 1 00:00:00 1970 From: John Harrison Subject: Re: [PATCH 02/55] drm/i915: Reserve ring buffer space for i915_add_request() commands Date: Tue, 23 Jun 2015 16:43:24 +0100 Message-ID: <55897E9C.2060604@Intel.com> 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> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTP id 4BD6A6E9E0 for ; Tue, 23 Jun 2015 08:43:27 -0700 (PDT) In-Reply-To: <20150623132406.GA25769@phenom.ffwll.local> 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: Intel-GFX@Lists.FreeDesktop.Org List-Id: intel-gfx@lists.freedesktop.org T24gMjMvMDYvMjAxNSAxNDoyNCwgRGFuaWVsIFZldHRlciB3cm90ZToKPiBPbiBUdWUsIEp1biAy MywgMjAxNSBhdCAxMjozODowMVBNICswMTAwLCBKb2huIEhhcnJpc29uIHdyb3RlOgo+PiBPbiAy Mi8wNi8yMDE1IDIxOjEyLCBEYW5pZWwgVmV0dGVyIHdyb3RlOgo+Pj4gT24gRnJpLCBKdW4gMTks IDIwMTUgYXQgMDU6MzQ6MTJQTSArMDEwMCwgSm9obi5DLkhhcnJpc29uQEludGVsLmNvbSB3cm90 ZToKPj4+PiBGcm9tOiBKb2huIEhhcnJpc29uIDxKb2huLkMuSGFycmlzb25ASW50ZWwuY29tPgo+ Pj4+Cj4+Pj4gSXQgaXMgYSBiYWQgaWRlYSBmb3IgaTkxNV9hZGRfcmVxdWVzdCgpIHRvIGZhaWwu IFRoZSB3b3JrIHdpbGwgYWxyZWFkeSBoYXZlIGJlZW4KPj4+PiBzZW5kIHRvIHRoZSByaW5nIGFu ZCB3aWxsIGJlIHByb2Nlc3NlZCwgYnV0IHRoZXJlIHdpbGwgbm90IGJlIGFueSB0cmFja2luZyBv cgo+Pj4+IG1hbmFnZW1lbnQgb2YgdGhhdCB3b3JrLgo+Pj4+Cj4+Pj4gVGhlIG9ubHkgd2F5IHRo ZSBhZGQgcmVxdWVzdCBjYWxsIGNhbiBmYWlsIGlzIGlmIGl0IGNhbid0IHdyaXRlIGl0cyBlcGls b2d1ZQo+Pj4+IGNvbW1hbmRzIHRvIHRoZSByaW5nIChjYWNoZSBmbHVzaGluZywgc2Vxbm8gdXBk YXRlcywgaW50ZXJydXB0IHNpZ25hbGxpbmcpLiBUaGUKPj4+PiByZWFzb25zIGZvciB0aGF0IGFy ZSBtb3N0bHkgZG93biB0byBydW5uaW5nIG91dCBvZiByaW5nIGJ1ZmZlciBzcGFjZSBhbmQgdGhl Cj4+Pj4gcHJvYmxlbXMgYXNzb2NpYXRlZCB3aXRoIHRyeWluZyB0byBnZXQgc29tZSBtb3JlLiBU aGlzIHBhdGNoIHByZXZlbnRzIHRoYXQKPj4+PiBzaXR1YXRpb24gZnJvbSBoYXBwZW5pbmcgaW4g dGhlIGZpcnN0IHBsYWNlLgo+Pj4+Cj4+Pj4gV2hlbiBhIHJlcXVlc3QgaXMgY3JlYXRlZCwgaXQg bWFya3Mgc3VmZmljaWVudCBzcGFjZSBhcyByZXNlcnZlZCBmb3IgdGhlCj4+Pj4gZXBpbG9ndWUg Y29tbWFuZHMuIFRodXMgZ3VhcmFudGVlaW5nIHRoYXQgYnkgdGhlIHRpbWUgdGhlIGVwaWxvZ3Vl IGlzIHdyaXR0ZW4sCj4+Pj4gdGhlcmUgd2lsbCBiZSBwbGVudHkgb2Ygc3BhY2UgZm9yIGl0LiBO b3RlIHRoYXQgYSByaW5nX2JlZ2luKCkgY2FsbCBpcyByZXF1aXJlZAo+Pj4+IHRvIGFjdHVhbGx5 IHJlc2VydmUgdGhlIHNwYWNlIChhbmQgZG8gYW55IHBvdGVudGlhbCB3YWl0aW5nKS4gSG93ZXZl ciwgdGhhdCBpcwo+Pj4+IG5vdCBjdXJyZW50bHkgZG9uZSBhdCByZXF1ZXN0IGNyZWF0aW9uIHRp bWUuIFRoaXMgaXMgYmVjYXVzZSB0aGUgcmluZ19iZWdpbigpCj4+Pj4gY29kZSBjYW4gYWxsb2Nh dGUgYSByZXF1ZXN0LiBIZW5jZSBjYWxsaW5nIGJlZ2luKCkgZnJvbSB0aGUgcmVxdWVzdCBhbGxv Y2F0aW9uCj4+Pj4gY29kZSB3b3VsZCBsZWFkIHRvIGluZmluaXRlIHJlY3Vyc2lvbiEgTGF0ZXIg cGF0Y2hlcyBpbiB0aGlzIHNlcmllcyByZW1vdmUgdGhlCj4+Pj4gbmVlZCBmb3IgYmVnaW4oKSB0 byBkbyB0aGUgYWxsb2NhdGUuIEF0IHRoYXQgcG9pbnQsIGl0IGJlY29tZXMgc2FmZSBmb3IgdGhl Cj4+Pj4gYWxsb2NhdGUgdG8gY2FsbCBiZWdpbigpIGFuZCByZWFsbHkgcmVzZXJ2ZSB0aGUgc3Bh Y2UuCj4+Pj4KPj4+PiBVbnRpbCB0aGVuLCB0aGVyZSBpcyBhIHBvdGVudGlhbCBmb3IgaW5zdWZm aWNpZW50IHNwYWNlIHRvIGJlIGF2YWlsYWJsZSBhdCB0aGUKPj4+PiBwb2ludCBvZiBjYWxsaW5n IGk5MTVfYWRkX3JlcXVlc3QoKS4gSG93ZXZlciwgdGhhdCB3b3VsZCBvbmx5IGJlIGluIHRoZSBj YXNlCj4+Pj4gd2hlcmUgdGhlIHJlcXVlc3Qgd2FzIGNyZWF0ZWQgYW5kIGltbWVkaWF0ZWx5IHN1 Ym1pdHRlZCB3aXRob3V0IGV2ZXIgY2FsbGluZwo+Pj4+IHJpbmdfYmVnaW4oKSBhbmQgYWRkaW5n IGFueSB3b3JrIHRvIHRoYXQgcmVxdWVzdC4gV2hpY2ggc2hvdWxkIG5ldmVyIGhhcHBlbi4gQW5k Cj4+Pj4gZXZlbiBpZiBpdCBkb2VzLCBhbmQgaWYgdGhhdCByZXF1ZXN0IGhhcHBlbnMgdG8gZmFs bCBkb3duIHRoZSB0aW55IHdpbmRvdyBvZgo+Pj4+IG9wcG9ydHVuaXR5IGZvciBmYWlsaW5nIGR1 ZSB0byBiZWluZyBvdXQgb2YgcmluZyBzcGFjZSB0aGVuIGRvZXMgaXQgcmVhbGx5Cj4+Pj4gbWF0 dGVyIGJlY2F1c2UgdGhlIHJlcXVlc3Qgd2Fzbid0IGRvaW5nIGFueXRoaW5nIGluIHRoZSBmaXJz dCBwbGFjZT8KPj4+Pgo+Pj4+IHYyOiBVcGRhdGVkIHRoZSAncmVzZXJ2ZWQgc3BhY2UgdG9vIHNt YWxsJyB3YXJuaW5nIHRvIGluY2x1ZGUgdGhlIG9mZmVuZGluZwo+Pj4+IHNpemVzLiBBZGRlZCBh ICdjYW5jZWwnIG9wZXJhdGlvbiB0byBjbGVhbiB1cCB3aGVuIGEgcmVxdWVzdCBpcyBhYmFuZG9u ZWQuIEFkZGVkCj4+Pj4gcmUtaW5pdGlhbGlzYXRpb24gb2YgdHJhY2tpbmcgc3RhdGUgYWZ0ZXIg YSBidWZmZXIgd3JhcCB0byBrZWVwIHRoZSBzYW5pdHkKPj4+PiBjaGVja3MgYWNjdXJhdGUuCj4+ Pj4KPj4+PiB2MzogSW5jcmVtZW50ZWQgdGhlIHJlc2VydmVkIHNpemUgdG8gYWNjb21tb2RhdGUg SXJvbmxha2UgKGFmdGVyIGZpbmFsbHkKPj4+PiBtYW5hZ2luZyB0byBydW4gb24gYW4gSUxLIHN5 c3RlbSkuIEFsc28gZml4ZWQgbWlzc2luZyB3cmFwIGNvZGUgaW4gTFJDIG1vZGUuCj4+Pj4KPj4+ PiB2NDogQWRkZWQgZXh0cmEgY29tbWVudCBhbmQgcmVtb3ZlZCBkdXBsaWNhdGUgV0FSTiAoZmVl ZGJhY2sgZnJvbSBUb21hcykuCj4+Pj4KPj4+PiB2NTogUmUtd3JpdGUgb2Ygd3JhcCBoYW5kbGlu ZyB0byBwcmV2ZW50IHVubmVjZXNzYXJ5IGVhcmx5IHdyYXBzIChmZWVkYmFjayBmcm9tCj4+Pj4g RGFuaWVsIFZldHRlcikuCj4+PiBUaGlzIGRpZG4ndCBhY3R1YWxseSBpbXBsZW1lbnQgd2hhdCBJ IHN1Z2dlc3RlZCAod3JhcHBpbmcgaXMgdGhlIHdvcnN0Cj4+PiBjYXNlLCBoZW5jZSBza2lwcGlu ZyB0aGUgY2hlY2sgZm9yIHRoYXQgaXMgYnJlYWtpbmcgdGhlIHNhbml0eSBjaGVjaykgYW5kCj4+ PiBzbyBjaGFuZ2VkIHRoZSBwYXRjaCBmcm9tICJjb3JyZWN0LCBidXQgYSBiaXQgZnJhZ2lsZSIg dG8gYnJva2VuLiBJJ3ZlCj4+PiBtZXJnZWQgdGhlIHByZXZpb3VzIHZlcnNpb24gaW5zdGVhZC4K Pj4+IC1EYW5pZWwKPj4gSSdtIGNvbmZ1c2VkLiBJIHRob3VnaHQgeW91ciBtYWluIGlzc3VlIHdh cyB0aGUgZWFybHkgd3JhcHBpbmcgbm90IHRoZQo+PiBzYW5pdHkgY2hlY2suIFRoZSBjaGVjayBp cyB0byBlbnN1cmUgdGhhdCB0aGUgcmVzZXJ2YXRpb24gaXMgbGFyZ2UgZW5vdWdoIHRvCj4+IGNv dmVyIGFsbCB0aGUgY29tbWFuZHMgd3JpdHRlbiBkdXJpbmcgcmVxdWVzdCBzdWJtaXNzaW9uLiBU aGF0IHNob3VsZCBub3QgYmUKPj4gYWZmZWN0ZWQgYnkgd2hldGhlciBhIHdyYXAgb2NjdXJzIG9y IG5vdC4gV3JhcHBpbmcgZG9lcyBub3QgbWFnaWNhbGx5IGFkZCBhbgo+PiBleHRyYSBidW5jaCBv ZiBkd29yZHMgdG8gdGhlIGVtaXRfcmVxdWVzdCgpIGNhbGwuIFdoZXJlYXMgbWFraW5nIHRoZSBj aGVjawo+PiB3b3JrIHdpdGggdGhlIHdyYXAgY29uZGl0aW9uIHJlcXVpcmVzIGFkZGluZyBpbiBl eHRyYSB0cmFja2luZyBzdGF0ZSBvZgo+PiBleGFjdGx5IHdoZXJlIHRoZSB3cmFwIG9jY3VycmVk LiBUaGF0IGlzIGV4dHJhIGNvZGUgdGhhdCBvbmx5IGV4aXN0cyB0bwo+PiBjYXRjaCBzb21ldGhp bmcgaW4gdGhlIHZlcnkgcmFyZSBjYXNlIHdoaWNoIHNob3VsZCBhbHJlYWR5IGhhdmUgYmVlbiBj YXVnaHQKPj4gaW4gdGhlIHZlcnkgY29tbW9uIGNhc2UuIEkuZS4gaWYgeW91ciByZXNlcnZlZCBz aXplIGlzIHRvbyBzbWFsbCB0aGVuIHlvdQo+PiB3aWxsIGhpdCB0aGUgd2FybmluZyBvbiBldmVy eSBiYXRjaCBidWZmZXIgc3VibWlzc2lvbi4KPiBUaGUgcHJvYmxlbSBpcyB0aGF0IGlmIHlvdSBh bGxvdyBhIHdyYXAgaW4gdGhlIHJlc2VydmUgc2l6ZSB0aGVuIHRoZQo+IHJpbmdzcGFjZSByZXF1 aXJlbWVudHMgYXJlIGJpZ2dlciB0aGFuIGlmIHlvdSBkb24ndCB3cmFwLiBBbmQgc2luY2UgdGhl Cj4gYWRkIHJlcXVlc3QgaXMgc3BsaXQgdXAgaW50byBtYW55IGludGVsX3JpbmdfYmVnaW4gdGhh dCdzIHBvc3NpYmxlLiBIZW5jZQo+IGlmIHlvdSBhbGxvdyB3cmFwcGluZyBpbiB0aGUgcmVzZXJ2 ZWQgc3BhY2UsIHRoZW4gdGhlIG1vc3QgaW1wb3J0YW50IGNhc2UKPiBmb3IgdGhlIGRlYnVnIGNo ZWNrIGlzIHRvIG1ha2Ugc3VyZSB0aGF0IGl0IGNhdGNoZXMgYW55IGtpbmQgb2YKPiByZXNlcnZh dGlvbiBvdmVyZmxvdyB3aGlsZSB3cmFwcGluZy4gVGhlIG5vdC13cmFwcGVkIGNhc2UgaXMgcHJv YmFibHkgdGhlCj4gYm9yaW5nIG9uZS4KPgo+IEFuZCBpbmRlZWQgZXZlbnR1YWxseSB3ZSBzaG91 bGQgb3ZlcmZsb3cgc2luY2UgYWNjb3JkaW5nIHRvIHlvdXIgY29tbWVudAo+IHRoZSB3b3JzdCBj YXNlIGFkZCByZXF1ZXN0IG9uIGlsayBpcyAxMzYgZHdvcmRzLiBBbmQgdGhlIGxhcmdlc3QKPiBp bnRlbF9yaW5nX2JlZ2luIGluIHRoZXJlIGlzIDMyIGR3b3Jkcywgd2hpY2ggbWVhbnMgYXQgbW9z dCB3ZSdsbCB0aHJvdwo+IGF3YXkgMzEgZHdvcmRzIHdoZW4gd3JhcHBpbmcuIFdoaWNoIG1lYW5z IHRoZSAxNjAgZHdvcmRzIG9mIHJlc2VydmF0aW9uCj4gYXJlIG5vdCBlbm91Z2ggc2luY2Ugd2Un ZCBuZWVkIDE2NyBkd29yZHMgb2Ygc3BhY2UgZm9yIHRoZSB3b3JzdCBjYXNlLiBCdXQKPiBzaW5j ZSB0aGUgc3BhY2VfZW5kIGRlYnVnIGNoZWNrIHdhcyBhIG5vLW9wIGZvciB0aGUgd3JhcHBlZCBj YXNlIHlvdSB3b24ndAo+IGNhdGNoIHRoaXMgb25lLgoKVGhlIG1pbmltdW0gcmVzZXJ2YXRpb24g c2l6ZSBpbiB0aGlzIGNhc2UgaXMgc3RpbGwgb25seSAxMzYuIFRoZSBwcmVwYXJlIApjb2RlIGNo ZWNrcyBmb3IgdGhlIDMyIHdvcmRzIGFjdHVhbGx5IHJlcXVlc3RlZCBhbmQgd3JhcHMgaWYgbmVj ZXNzYXJ5LiAKSXQgdGhlbiBjaGVja3MgZm9yIDEzNiszMiB3b3JkcyBvZiBzcGFjZS4gSWYgdGhh dCB3b3VsZCBjYXVzZSBhIHdyYXAgaXQgCndpbGwgdGhlbiBhZGQgb24gdGhlIGFtb3VudCBvZiBz cGFjZSBhY3R1YWxseSBsZWZ0IGluIHRoZSByaW5nIGFuZCB3YWl0IApmb3IgdGhhdCBiaWdnZXIg dG90YWwuIFRoYXQgZ3VhcmFudGVlcyB0aGF0IGl0IGhhcyB3YWl0ZWQgZm9yIHRoZSAxMzYgYXQg CnRoZSBzdGFydCBvZiB0aGUgcmluZy4gVGhlIGNhbGxlciBpcyB0aGVuIGZyZWUgdG8gZmlsbCBp biB0aGUgMzIgd29yZHMgCmFuZCB0aGVyZSBpcyBzdGlsbCBndWFyYW50ZWVkIHRvIGJlIGEgbWlu aW11bSBvZiAxMzYgd29yZHMgYXZhaWxhYmxlIAood2l0aCBvciB3aXRob3V0IHdyYXBwaW5nKSBi ZWZvcmUgYW55IGZ1cnRoZXIgd2FpdCBmb3Igc3BhY2UgaXMgCm5lY2Vzc2FyeS4gVGh1cyB0aGUg YWRkX3JlcXVlc3QoKSBjb2RlIGlzIHNhZmUgZnJvbSBmZWFyIG9mIGZhaWx1cmUgCmlycmVzcGVj dGl2ZSBvZiB3aGVyZSBhbnkgd3JhcCBtaWdodCBvY2N1ci4KCgo+Cj4gV3J0IGtlZXBpbmcgdHJh Y2sgb2Ygd3JhcHBpbmcgd2hpbGUgdGhlIHJlc2VydmF0aW9uIGlzIGluIHVzZSwgdGhlCj4gZm9s bG93aW5nIHNob3VsZCBkbyB0aGF0IHdpdGhvdXQgYW55IG5lZWQgb2YgYWRkaXRpb25hbCB0cmFj a2luZzoKPgo+Cj4gCWludCB1c2VkX3NpemUgPSByaW5nYnVmLT50YWlsIC0gcmluZ2J1Zi0+cmVz ZXJ2ZWRfdGFpbDsKPgo+IAlpZiAodXNlZF9zaXplIDwgMCkKPiAJCXVzZWRfc2l6ZSArPSByaW5n YnVmLT5zaXplOwo+Cj4gCVdBUk4odXNlZF9zaXplIDwgcmluZ2J1Zi0+cmVzZXJ2ZWRfc2l6ZSwK PiAJICAgICAicmVxdWVzdCByZXNlcnZlZCBzaXplIHRvbyBzbWFsbDogJWQgdnMgJWQhXG4iLAo+ IAkgICAgIHVzZWRfc2l6ZSwgcmluZ2J1Zi0+cmVzZXJ2ZWRfc2l6ZSk7Cj4KPiBJIHdhcyBtaXN0 YWtlbiB0aGF0IHlvdSBjYW4gcmV1c2UgX19pbnRlbF9yaW5nX3NwYWNlIChzaW5jZSB0aGF0IGhh cwo+IHNsaWdodGx5IGRpZmZlcmVudCByZXF1aXJlbWVudHMpLCBidXQgdGhpcyBnaXZlcyB5b3Ug YSBuaWNlbHkgbG9jYWxpemVkCj4gY2hlY2sgZm9yIHJlc2VydmF0aW9uIG92ZXJmbG93IHdoaWNo IHdvcmtzIGV2ZW4gd2hlbiB5b3Ugd3JhcC4gT2ZjIGl0Cj4gd29uJ3Qgd29yayBpZiBhbiBhZGRf cmVxdWVzdCBpcyBiaWdnZXIgdGhhbiB0aGUgZW50aXJlIHJpbmcsIGJ1dCB0aGF0J3MKPiBpbXBv c3NpYmxlIGFueXdheSBzaW5jZSB3ZSBjYW4gYXQgbW9zdCByZXNlcnZlIHJpbmdidWYtPnNpemUg LQo+IEk5MTVfUklOR19GUkVFX1NQQUNFLgpUaGUgcHJvYmxlbSB3aXRoIHRoZSBhYm92ZSBjYWxj dWxhdGlvbiBpcyB0aGF0IGl0IGluY2x1ZGVzIHRoZSB3YXN0ZWQgCnNwYWNlIGF0IHRoZSBlbmQg b2YgdGhlIHJpbmcuIFRodXMgaXQgd2lsbCBjb21wbGFpbiB0aGUgcmVzZXJ2ZWQgc2l6ZSAKd2Fz IHRvbyBzbWFsbCB3aGVuIGluIGZhY3QgaXQgd2FzIGp1c3QgZmluZS4KCgo+IE9yIGRvIEkgc3Rp bGwgbWlzcyBzb21ldGhpbmc/Cj4gLURhbmllbAoKX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlz dHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9s aXN0aW5mby9pbnRlbC1nZngK