From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 1/2] drm: Avoid calling the cursor_set2 callback from the drm_mode_cursor ioctl Date: Sun, 29 Nov 2015 10:18:38 +0100 Message-ID: <20151129091838.GZ17050@phenom.ffwll.local> References: <1448563935-3157-1-git-send-email-thellstrom@vmware.com> <20151127101133.GW17050@phenom.ffwll.local> <56584197.3020707@vmware.com> <20151127120248.GL4437@intel.com> <56584B6B.7060302@vmware.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail-wm0-f48.google.com (mail-wm0-f48.google.com [74.125.82.48]) by gabe.freedesktop.org (Postfix) with ESMTPS id 647146E0A3 for ; Sun, 29 Nov 2015 01:18:42 -0800 (PST) Received: by wmww144 with SMTP id w144so95911806wmw.1 for ; Sun, 29 Nov 2015 01:18:40 -0800 (PST) Content-Disposition: inline In-Reply-To: <56584B6B.7060302@vmware.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Thomas Hellstrom Cc: airlied@redhat.com, dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org T24gRnJpLCBOb3YgMjcsIDIwMTUgYXQgMDE6MjQ6MTFQTSArMDEwMCwgVGhvbWFzIEhlbGxzdHJv bSB3cm90ZToKPiBPbiAxMS8yNy8yMDE1IDAxOjAyIFBNLCBWaWxsZSBTeXJqw6Rsw6Qgd3JvdGU6 Cj4gPiBPbiBGcmksIE5vdiAyNywgMjAxNSBhdCAxMjo0MjoxNVBNICswMTAwLCBUaG9tYXMgSGVs bHN0cm9tIHdyb3RlOgo+ID4+IE9uIDExLzI3LzIwMTUgMTE6MTEgQU0sIERhbmllbCBWZXR0ZXIg d3JvdGU6Cj4gPj4+IE9uIFRodSwgTm92IDI2LCAyMDE1IGF0IDEwOjUyOjE0QU0gLTA4MDAsIFRo b21hcyBIZWxsc3Ryb20gd3JvdGU6Cj4gPj4+PiBJZiB0aGUgZHJtX21vZGVfY3Vyc29yX2lvY3Rs IGlzIGNhbGxlZCBhbmQgdGhlIGN1cnNvcl9zZXQyIGNhbGxiYWNrIGlzCj4gPj4+PiBpbXBsZW1l bnRlZCwgdGhlIGN1cnNvciBob3RzcG90IGlzIHNldCB0byAoMCwwKSB3aGljaCBpcyBpbmNvbXBh dGlibGUKPiA+Pj4+IHdpdGggdm13Z2Z4IHdoZXJlIHRoZSBob3RzcG90IHNob3VsZCBpbnN0ZWFk IHJlbWFpbiB1bmNoYW5nZWQuCj4gPj4+Pgo+ID4+Pj4gU28gaWYgdGhlIGRyaXZlciBpbXBsZW1l bnRzIGJvdGggY3Vyc29yX3NldDIgYW5kIGN1cnNvcl9zZXQsIHByZWZlciBjYWxsaW5nCj4gPj4+ PiB0aGUgbGF0dGVyIGZyb20gdGhlIGRybV9tb2RlX2N1cnNvciBpb2N0bC4KPiA+Pj4+Cj4gPj4+ PiBTaWduZWQtb2ZmLWJ5OiBUaG9tYXMgSGVsbHN0cm9tIDx0aGVsbHN0cm9tQHZtd2FyZS5jb20+ Cj4gPj4+IFRoYXQgbG9va3MgbGlrZSBwYXBlcmluZyBvdmVyIGEgYnVnIGluIHRoZSBjbGllbnQg LSBpdCBzaW1wbHkgc2hvdWxkbid0Cj4gPj4+IG1peCB0aGVzZSB0d28gaWYgaXQgZXhwZWN0cyB0 aGUgaG90c3BvdCB0byBzdGljayBhcm91bmQuIFRoZXJlIHdhcyBhbHNvCj4gPj4+IHJlY2VudGx5 IGEgYmlnIGRpc2N1c3Npb24gYWJvdXQgaG90c3BvdCBiZWhhdmlvdXIsIGFuZCB0aGUgY29uY2x1 c2lvbiB3YXMKPiA+Pj4gdGhhdCBxeGwgZ290IGl0IGFsbCB3cm9uZy4gU2luY2UgdGhhdCB3YXMg c3BlY2lmaWNhbGx5IGFkZGVkIGZvciBxeGwgSSdtCj4gPj4+IG5vdCBzdXJlIGhvdyB3ZWxsIHRo aXMgd2FzIGV2ZXIgdGVzdGVkIC4uLgo+ID4+IE5vLCB0aGlzIGlzIG5vdCB0aGUgY2FzZSwgVGhp cyBpcyBmb3Igb2xkIFhvcmcgdXNlcnNwYWNlIHRoYXQgZmlyc3Qgc2V0cwo+ID4+IHRoZSBob3Rz cG90IHVzaW5nIG91ciBhbmNpZW50Cj4gPj4gZHJpdmVyLXByaXZhdGUgaW9jdGwgYW5kIHRoZW4g Y2FsbHMgZHJtX21vZGVfY3Vyc29yKCkgdG8gdXBkYXRlIHRoZSBjdXJzb3IuCj4gPj4KPiA+PiBO b3cgaWYgd2Ugd2VyZSB0byBpbXBsZW1lbnQgY3Vyc29yX3NldDIsIHdoaWNoIGlzIGFwcGFyZW50 bHkgbmVlZGVkIHRvCj4gPj4gZ2V0IGdub21lLXNoZWxsL1dheWxhbmQgY3Vyc29ycyBpbiB0aGUg cmlnaHQgcGxhY2UsIHdpdGhvdXQgdGhpcyBmaXgsIGl0Cj4gPj4gd291bGQgYnJlYWsgb2xkIFhv cmcsIHNvIHdlIGRvbid0IGhhdmUgbXVjaCBjaG9pY2UgaW4gdGhpcyBjYXNlIGZyb20KPiA+PiB3 aGF0IEkgY2FuIHRlbGwuCj4gPj4KPiA+PiBUaGUgcm9vdCBwcm9ibGVtIGhlcmUgaXMgdGhhdCB0 aGUgZHJtX21vZGVfY3Vyc29yKCkgYmVoYXZpb3VyIGluIHRoZQo+ID4+IHByZXNlbmNlIG9mIGN1 cnNvcl9zZXQyIGRpZG4ndCB0YWtlIHRoZSBleGlzdGluZyB2bXdhcmUgaG90c3BvdAo+ID4+IHNl bWFudGljcyBpbnRvIGFjY291bnQuCgpVZ2guIEkgdGhpbmsgdGhlIHNpbXBsZXN0IHNvbHV0aW9u IGlzIHRvIG5vdCBtaXggdXAgdGhlIHR3byBob3RzcG90cywgaS5lLgpzZXBhcmF0ZWx5IGtlZXAg dHJhY2sgb2YgYm90aCB0aGUgbGVnYWN5IHZtd2dmeCBob3RzcG90IGFuZCB0aGUgZHJtIGNvcmUK aG90c3BvdC4gVGhlbiBpbiB0aGUgYWN0dWFsIGN1cnNvciBzZXQgY29kZSBhZGQgdGhlbS4gQSBi aXQgb2YgdWdseSBpbiB0aGUKdm13Z2Z4IGNvZGUgKGJ1dCBub3QgbXVjaCksIGluc3RlYWQgb2Yg bGVha2luZyB0aGF0IGRyaXZlciBwcml2YXRlIGxlZ2FjeQppb2N0bCBzZW1hbnRpY3MgaW50byBk cm0gY29yZS4gV291bGQgdGhhdCB3b3JrPwoKPiA+Pj4gVGhlIG90aGVyIHByb2JsZW0gc2VlbXMg dG8gYmUgdGhhdCBYIGlzIHJhY3kgd2l0aCB1cGRhdGluZyBjdXJzb3JzIChzaW5jZQo+ID4+PiBp dCBkb2VzIGEgc2V0UG9zIGFuZCBzZXRDdXJzb3Igc2VwYXJhdGVseSwgZGVzcGl0ZSB0aGF0IHRo aXMgaW9jdGwgY2FuIGRvCj4gPj4+IGl0IGluIG9uZSBnbykgYW5kIHNvIGlmIHlvdSBtb3ZlIHRo ZSBob3RzcG90IHlvdSBnZXQgYSBqdW1weSBjdXJzb3IuCj4gPj4+IFJhZGVvbiB0cmllZCB0byBw YXBlciBvdmVyIHRoYXQgYnV0IGZ1bmRhbWVudGFsbHkgeW91IG5lZWQgdG8gZml4IFggYW5kCj4g Pj4+IHVzZSBhdG9taWMgKG9yIGF0IGxlYXN0IHVuaXZlcnNhbCBwbGFuZSBjdXJzb3Igc3VwcG9y dCkgdG8gZml4IHRoaXMuCj4gPj4+Cj4gPj4+IEdpdmVuIGFsbCB0aGF0IEknbSBsZWFuaW5nIHRv d2FyZHMgInRoZSBmdXR1cmUgaXMgYXRvbWljIiwgbGV0J3MgcGxlYXNlCj4gPj4+IG5vdCBhZGQg aGFja3MgdG8gbGVnYWN5IGNvZGUuIEFzaWRlOiBBdG9taWMgZG9lc24ndCBoYXZlIHRoZSBob3Rz cG90LXgveQo+ID4+PiBwcm9wZXJ0aWVzIHlldCwgYnV0IHJlYWxseSB0aGUgb25seSByZWFzb24g d2FzIHRoYXQgd2UgZG9uJ3QgeWV0IGhhdmUgYW4KPiA+Pj4gYXRvbWljIGRyaXZlciBuZWVkaW5n IHRoZW0uIEFkZGluZyB0aG9zZSBwcm9wcyB3aWxsIGJlIGEgdGlueSBwYXRjaCBvZiBhCj4gPj4+ IGZldyBsaW5lcy4KPiA+PiBTaW5jbGFpciBoYXMgc3RhcnRlZCB3b3JraW5nIG9uIGF0b21pYyBz dXBwb3J0IGZvciB2bXdnZngsIGJ1dCB3ZSByZWFsbHkKPiA+PiBuZWVkIHRvIGZpeCB0aGlzIGFs c28gaW4KPiA+PiB0aGUgbGVnYWN5IGNvZGUsIHVnbHkgb3Igbm90Lgo+ID4gSSB3YXMgdGhpbmtp bmcgbWF5YmUgaXQgd291bGQgYmUgY2xlYW5lciB0byBub3QgcmVzZXQgdGhlIGhvdHNwb3QgdG8K PiA+IDAsMCBpbiBkcm1fbW9kZV9jdXJzb3JfaW9jdGwoKSBhbmQgaW5zdGVhZCBrZWVwIHdoYXQg d2FzIHByZXZpb3VzbHkKPiA+IGNvbmZpZ3VyZWQsIGFuZCBvbmx5IGRvIHRoZSByZXNldCB0byAw LDAgaW4gbGFzdGNsb3NlIChvciBzb21lIG90aGVyCj4gPiBzdWl0YWJsZSBwbGFjZSk/IEJ1dCBJ IHN1cHBvc2UgdGhhdCBjb3VsZCBicmVhayBzb21ldGhpbmcgZWxzZSB0aGF0Cj4gPiBhc3N1bWUg dGhhdCBkcm1fbW9kZV9jdXJzb3JfaW9jdGwoKSB3aWxsIGRvIHRoZSByZXNldC4KPiAKPiBJZiBz b21ldGhpbmcgYXNzdW1lcyB0aGF0IGRybV9tb2RlX2N1cnNvcl9pb2N0bCgpIHdpbGwgcmVzZXQg dGhlCj4gaG90c3BvdCwgdGhlbiB3ZSdyZSBhbHJlYWR5IGluIHRyb3VibGUgYmVjYXVzZSB0aGVu IHdlIGhhdmUgZGlmZmVyZW50Cj4gc2VtYW50aWNzIG9mIHRoZSBzYW1lIGlvY3RsIGFzc3VtZWQg YnkgdXNlci1zcGFjZS4KPiAKPiBIb3dldmVyLCBmcm9tIHdoYXQgSSBjYW4gdGVsbCwgZ2V0dGlu ZyB0aGUgY3Vyc29yIGhvdHNwb3QgcG9zaXRpb24gZnJvbQo+IHdpdGhpbiBjb3JlIERSTSB3b3Vs ZCByZXF1aXJlIG5ldyBsb2NrIHByb3RlY3RlZCBob3RzcG90IGNvb3JkaW5hdGUKPiBtZW1iZXJz IGluIHRoZSBjcnRjIHN0cnVjdHVyZS4uLgoKWWVhaCwgYW5kIGZvciBhdG9taWMgd2UgZGVmaW5p dGVseSB3YW50IHRob3NlIGluIGRybV9jcnRjX3N0YXRlLCBidXQgZm9yCmxlZ2FjeSB3ZSdkIG5l ZWQgaXQgaW4gZHJtX2NydGMuIEknZCBwcmVmZXIgaWYgd2UgZG9uJ3QgbmVlZCB0byBhZGQgaXQg dG8KYm90aCBwbGFjZXMgOy0pCgpDaGVlcnMsIERhbmllbAotLSAKRGFuaWVsIFZldHRlcgpTb2Z0 d2FyZSBFbmdpbmVlciwgSW50ZWwgQ29ycG9yYXRpb24KaHR0cDovL2Jsb2cuZmZ3bGwuY2gKX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1h aWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5m cmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK