From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 48/55] drm/i915: Add *_ring_begin() to request allocation Date: Thu, 18 Jun 2015 15:29:37 +0200 Message-ID: References: <1432917856-12261-1-git-send-email-John.C.Harrison@Intel.com> <1432917856-12261-49-git-send-email-John.C.Harrison@Intel.com> <20150617133159.GX23637@phenom.ffwll.local> <20150617142708.GJ24012@nuc-i3427.alporthouse.com> <20150617145442.GA23637@phenom.ffwll.local> <20150617155206.GM24012@nuc-i3427.alporthouse.com> <5582A9D3.8060306@Intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail-ob0-f172.google.com (mail-ob0-f172.google.com [209.85.214.172]) by gabe.freedesktop.org (Postfix) with ESMTP id D3C036ECDB for ; Thu, 18 Jun 2015 06:29:37 -0700 (PDT) Received: by obbgp2 with SMTP id gp2so53980478obb.2 for ; Thu, 18 Jun 2015 06:29:37 -0700 (PDT) In-Reply-To: <5582A9D3.8060306@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 List-Id: intel-gfx@lists.freedesktop.org T24gVGh1LCBKdW4gMTgsIDIwMTUgYXQgMToyMSBQTSwgSm9obiBIYXJyaXNvbgo8Sm9obi5DLkhh cnJpc29uQGludGVsLmNvbT4gd3JvdGU6Cj4gSSdtIHN0aWxsIGNvbmZ1c2VkIGJ5IHdoYXQgeW91 IGFyZSBzYXlpbmcgaW4gdGhlIGFib3ZlIHJlZmVyZW5jZWQgZW1haWwuCj4gUGFydCBvZiBpdCBp cyBhYm91dCB0aGUgc2FuaXR5IGNoZWNrcyBmYWlsaW5nIHRvIGhhbmRsZSB0aGUgd3JhcHBpbmcg Y2FzZQo+IGNvcnJlY3RseSB3aGljaCBoYXMgYmVlbiBmaXhlZCBpbiB0aGUgYmFzZSByZXNlcnZl IHNwYWNlIHBhdGNoIChwYXRjaCAyIGluCj4gdGhlIHNlcmllcykuIFRoZSByZXN0IGlzIGVpdGhl ciBzYXlpbmcgdGhhdCB5b3UgdGhpbmsgd2UgYXJlIHBvdGVudGlhbGx5Cj4gd3JhcHBwaW5nIHRv byBlYXJseSBhbmQgd2FzdGluZyBhIGZldyBieXRlcyBvZiB0aGUgcmluZyBidWZmZXIgb3IgdGhh dAo+IHNvbWV0aGluZyBpcyBhY3R1YWxseSBicm9rZW4/CgpZZWFoIEkgZGlkbid0IHJlYWxpemUg dGhhdCB0aGlzIGNoYW5nZSB3YXMgbWVhbnQgdG8gZml4IHRoZQpyaW5nLT5yZXNlcnZlZF90YWls IGNoZWNrIHNpbmNlIEkgZGlkbid0IG1ha2UgdGhhdCBjb25uZWN0aW9uLiBJdCBpcwpjb3JyZWN0 IHdpdGggdGhhdCBjaGFuZ2UsIGJ1dCB0aGUgcHJvYmxlbSBJIHNlZSBpcyB0aGF0IHRoZQpjb3Jy ZWN0bmVzcyBvZiB0aGF0IGRlYnVnIGFpZCBpc24ndCBhc3N1cmVkIGxvY2FsbHk6IE5vIHdlIGJv dGggbmVlZAp0aGF0IGNoZWNrIF9hbmRfIHRoZSBjb3JyZWN0IGhhbmRsaW5nIG9mIHRoZSByZXNl cnZhdGlvbiB0cmFja2luZyBhdAp3cmFwLWFyb3VuZC4gSWYgdGhlIGNoZWNrIGp1c3QgaGFuZGxl cyB3cmFwcGluZyBpdCdsbCByb2J1c3RseSBzdGF5IGluCndvcmtpbmcgc2hhcGUgZXZlbiB3aGVu IHRoZSB3cmFwcGluZyBiZWhhdmlvdXIgY2hhbmdlcy4KCj4gUG9pbnQgMjogMTAwIGJ5dGVzIG9m IHJlc2VydmUsIDE2MCBieXRlcyBvZiBleGVjYnVmIGFuZCAyMDAgYnl0ZXMgcmVtYWluaW5nLgo+ IFlvdSBzZWVtIHRvIHRoaW5rIHRoaXMgd2lsbCBmYWlsIHNvbWVob3c/IFdoeT8gVGhlIHdhaXRf Zm9yX3NwYWNlKDE2MCkgaW4KPiB0aGUgZXhlY2J1ZiBjb2RlIHdpbGwgY2F1c2UgYSB3cmFwIGJl Y2F1c2UgdGhlIHRoZSAxMDAgYnl0ZXMgZm9yIHRoZQo+IGFkZF9yZXF1ZXN0IHJlc2VydmF0aW9u IGlzIGFkZGVkIG9uIGFuZCB0aGUgd2FpdCBpcyBhY3R1YWxseSBiZWluZyBkb25lIGZvcgo+IDI2 MCBieXRlcy4gU28geWVzLCB3ZSB3cmFwIGVhcmxpZXIgdGhhbiB3b3VsZCBvdGhlcndpc2UgaGF2 ZSBiZWVuIG5lY2Vzc2FyeQo+IGJ1dCB0aGF0IGlzIHRoZSBvbmx5IHdheSB0byBhYnNvbHV0ZWx5 IGd1YXJhbnRlZSB0aGF0IHRoZSBhZGRfcmVxdWVzdCgpIGNhbGwKPiBjYW5ub3QgZmFpbCB3aGVu IHRyeWluZyB0byBkbyB0aGUgd3JhcCBpdHNlbGYuCgpUaGVyZSdzIG5vIHByb2JsZW0gZXhjZXB0 IHRoYXQgaXQncyB3YXN0ZWZ1bC4gQW5kIEkgdHJpZWQgdG8gZXhwbGFpbgp0aGF0IG5vIHVuY29u ZGl0aW9uYWxseSBmb3JjZS13cmFwcGluZyBmb3IgdGhlIGVudGlyZSByZXNlcnZhdGlvbiBpcwph Y3R1YWxseSBub3QgbmVlZGVkLCBzaW5jZSB0aGUgYWRkaXRpb25hbCBzcGFjZSBuZWVkZWQgdG8g YWNjb3VudCBmb3IKdGhlIGV2ZW50dWFsIHdyYXBwaW5nIGlzIGJvdW5kZWQgYnkgYSBmYWN0b3Ig b2YgMi4gSXQncyBtdWNoIGxlc3MgaW4KcHJhY3RpY2Ugc2luY2Ugd2Ugc3BsaXQgdXAgdGhlIGZp bmFsIHJlcXVlc3QgYml0cyBpbnRvIG11bHRpcGxlCnNtYWxsZXIgaW50ZWxfcmluZ19iZWdpbi4g QW5kIGlmIGZlZWxzIGEgYml0IHdhc3RlZnVsIHRvIHRocm93IHRoYXQKc3BhY2UgYXdheSAoYW5k IG1ha2UgdGhlIGdwdSBlYXQgdGhyb3VnaCBNSV9OT1ApIGp1c3QgYmVjYXVzZSBpdCBtYWtlcwpj YXJpbmcgZm9yIHRoZSB3b3JzdC1jYXNlIGhhcmRlci4gQW5kIHdpdGggR3VDIHRoZSAxNjAgZHdv cmRzIGlzCmFjdHVhbGx5IGEgZmFpcmx5IHN1YnN0YW50aWFsIHBhcnQgb2YgdGhlIHJpbmcuCgpF dmVuIG1vcmUgc28gd2hlbiB3ZSBjb21wbGV0ZWx5IHN3aXRjaCB0byBhIHRyYW5zYWN0aW9uIG1v ZGVsIGZvcgpyZXF1ZXN0LCB3aGVyZSB3ZSBvbmx5IG5lZWQgdG8gd3JhcCBmb3IgaW5kaXZpZHVh bCBjb21tYW5kcyBhbmQgaGVuY2UKY291bGQgcGxhY2UgaW50ZWxfcmluZ19iZWluZyBwZXItY21k ICh3aGljaCBpcyBtb3N0bHkgd2hhdCB3ZSBkbwphbHJlYWR5IGFueXdheSkuCgo+IEFzIENocmlz IHNheXMsIGlmIHRoZSBkcml2ZXIgaXMgYXR0ZW1wdGluZyB0byBjcmVhdGUgYSBzaW5nbGUgcmVx dWVzdCB0aGF0Cj4gZmlsbHMgdGhlIGVudGlyZSByaW5nYnVmZmVyIHRoZW4gdGhhdCBpcyBhIGJ1 ZyB0aGF0IHNob3VsZCBiZSBjYXVnaHQgYXMgc29vbgo+IGFzIHBvc3NpYmxlLiBFdmVuIHdpdGgg YSBHdWMsIHRoZSByaW5nIGJ1ZmZlciBpcyBub3Qgc21hbGwgY29tcGFyZWQgdG8gdGhlCj4gc2l6 ZSBvZiByZXF1ZXN0cyB0aGUgZHJpdmVyIGN1cnJlbnRseSBwcm9kdWNlcy4gUGFydCBvZiB0aGUg c2NoZWR1bGVyIHdvcmsKPiBpcyB0byBsaW1pdCB0aGUgbnVtYmVyIG9mIGJhdGNoIGJ1ZmZlcnMg dGhhdCBhIGdpdmVuIGFwcGxpY2F0aW9uL2NvbnRleHQgY2FuCj4gaGF2ZSBvdXRzdGFuZGluZyBp biB0aGUgcmluZyBidWZmZXIgYXQgYW55IGdpdmVuIHRpbWUgaW4gb3JkZXIgdG8gcHJldmVudAo+ IHN0YXJ2YXRpb24gb2YgdGhlIHJlc3Qgb2YgdGhlIHN5c3RlbSBieSBvbmUgYmFkbHkgYmVoYXZl ZCBhcHAuIFRodXMKPiBjb21wbGV0ZWx5IGZpbGxpbmcgYSBsYXJnZSByaW5nIGJ1ZmZlciBiZWNv bWVzIGltcG9zc2libGUgYW55d2F5IC0gdGhlCj4gYXBwbGljYXRpb24gd2lsbCBiZSBibG9ja2Vk IGJlZm9yZSBpdCBnZXRzIHRoYXQgZmFyLgoKTXkgcHJvcG9zYWwgZm9yIHRoaXMgcmVzZXJ2YXRp b24gd3JhcHBpbmcgYnVzaW5lc3Mgd291bGQgaGF2ZSBiZWVuOgotIEluY3JlYXNlIHRoZSByZXNl cnZhdGlvbiBieSAzMSBkd29yZHMgKHRvIGFjY291bnQgZm9yIHRoZSB3b3JzdC1jYXNlCndyYXAg aW4gcGNfcmVuZGVyX2FkZF9yZXF1ZXN0KS4KLSBSZXdvcmsgdGhlIHJlc2VydmF0aW9uIG92ZXJm bG93IFdBUk5fT04gaW4gcmVzZXJ2ZV9zcGFjZV9lbmQgdG8gd29yawpjb3JyZWN0bHkgZXZlbiB3 aGVuIHdyYXBwaW5nIHdoaWxlIHRoZSByZXNlcnZhdGlvbiBoYXMgYmVlbiBpbiB1c2UuCi0gTW92 ZSB0aGUgYWRkaXRpb24gb2YgcmVzZXJ2ZWRfc3BhY2UgYmVsb3cgdGhlIHBvaW50IHdoZXJlIHdl IHdyYXAKdGhlIHJpbmcgYW5kIG9ubHkgY2hlY2sgYWdhaW5zdCB0b3RhbCBmcmVlIHNwYWNlLCBu ZWdsZWN0aW5nIHdyYXBwaW5nLgotIFJlbW92ZSBhbGwgb3RoZXIgY29tcGxpY2F0aW9ucyB5b3Un dmUgYWRkZWQuCgpSZXN1bHQgaXMgbm8gZm9yY2VkIHdyYXBwaW5nIGZvciByZXNlcnZhdGlvbiBh bmQgYSBkZWJ1ZyBjaGVjayB3aGljaApzaG91bGQgZXZlbiBzdXJ2aXZlIHJhbmRvbSBjaGFuZ2Vz IGJ5IG1vbmtleXMgc2luY2UgdGhlIGxvZ2ljIGZvciB0aGF0CmNoZWNrIGlzIGZ1bGx5IGNvbnRh aW5lZCB3aXRoaW4gcmVzZXJ2ZV9zcGFjZV9lbmQuIEFuZCBmb3IgdGhlIGNoZWNrCndlIHNob3Vs ZCBiZSBhYmxlIHRvIHJldXNlIF9faW50ZWxfZnJlZV9zcGFjZS4KCklmIEknbSByZWFkaW5nIHRo aW5ncyBjb3JyZWN0bHkgdGhpcyBzaG91bGRuJ3QgaGF2ZSBhbnkgZWZmZWN0IG91dHNpZGUKb2Yg cGF0Y2ggMiBhbmQgc2hvdWxkbid0IGNhdXNlIGFueSBjb25mbGljdHMuCi1EYW5pZWwKLS0gCkRh bmllbCBWZXR0ZXIKU29mdHdhcmUgRW5naW5lZXIsIEludGVsIENvcnBvcmF0aW9uCis0MSAoMCkg NzkgMzY1IDU3IDQ4IC0gaHR0cDovL2Jsb2cuZmZ3bGwuY2gKX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1n ZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFp bG1hbi9saXN0aW5mby9pbnRlbC1nZngK