From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matt Roper Subject: Re: [Intel-gfx] [PATCH v2 06/10] drm: Avoid atomic commit path for CRTC property (Gamma) Date: Fri, 19 Jun 2015 15:50:40 -0700 Message-ID: <20150619225040.GC24601@intel.com> References: <1433425361-17864-1-git-send-email-Kausal.Malladi@intel.com> <1433425361-17864-7-git-send-email-Kausal.Malladi@intel.com> <20150606010102.GI1099@intel.com> <5572E1DD.5020809@intel.com> <20150609005823.GC7449@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <20150609005823.GC7449@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: "Sharma, Shashank" Cc: annie.j.matheson@intel.com, intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, vijay.a.purushothaman@intel.com, dhanya.p.r@intel.com, daniel.vetter@intel.com, jesse.barnes@intel.com List-Id: intel-gfx@lists.freedesktop.org T24gTW9uLCBKdW4gMDgsIDIwMTUgYXQgMDU6NTg6MjNQTSAtMDcwMCwgTWF0dCBSb3BlciB3cm90 ZToKPiBPbiBTYXQsIEp1biAwNiwgMjAxNSBhdCAwNTozNDo0NVBNICswNTMwLCBTaGFybWEsIFNo YXNoYW5rIHdyb3RlOgo+ID4gUmVnYXJkcwo+ID4gU2hhc2hhbmsKPiA+IAo+ID4gT24gNi82LzIw MTUgNjozMSBBTSwgTWF0dCBSb3BlciB3cm90ZToKPiA+ID5PbiBUaHUsIEp1biAwNCwgMjAxNSBh dCAwNzoxMjozN1BNICswNTMwLCBLYXVzYWwgTWFsbGFkaSB3cm90ZToKPiA+ID4+RnJvbTogS2F1 c2FsIE1hbGxhZGkgPEthdXNhbC5NYWxsYWRpQGludGVsLmNvbT4KPiA+ID4+Cj4gPiA+PlRoZSBh dG9taWMgQ1JUQyBzZXQgaW5mcmFzdHJ1Y3R1cmUgaXMgbm90IGF2YWlsYWJsZSB5ZXQuIFNvLCB0 aGUgYXRvbWljCj4gPiA+PmNoZWNrIHBhdGggdGhyb3dzIGVycm9yIGZvciBhbnkgbm9uLXBsYW5l IHByb3BlcnR5Lgo+ID4gPj4KPiA+ID4+VGhpcyBwYXRjaCBhZGRzIGEgZGl2ZXJzaW9uIHRvIGF2 b2lkIGNvbW1pdCBwYXRoIGZvciBEUk0gYXRvbWljIHNldCBHYW1tYQo+ID4gPj5wcm9wZXJ0eSwg dW50aWwgYXRvbWljIENSVEMgaW5mcmFzdHJ1Y3R1cmUgaXMgcmVhZHkuCj4gPiA+Cj4gPiA+VGhp cyBkb2Vzbid0IGxvb2sgcmlnaHQsIGJ1dCBJIGRvbid0IHF1aXRlIHVuZGVyc3RhbmQgd2hhdCB5 b3UncmUgdHJ5aW5nCj4gPiA+dG8gZG8gZnJvbSB0aGUgY29tbWl0IG1lc3NhZ2UuCj4gPiA+Cj4g PiA+VGhpcyBmdW5jdGlvbiBpcyB3aGF0IHdpbGwgaW1wbGVtZW50IGxlZ2FjeSBzZXRfcHJvcGVy dHkgaW9jdGwncyBvZiBhCj4gPiA+Q1JUQyBvbiBhbiBhdG9taWMtYmFzZWQgZHJpdmVyLiAgVGhl IGlkZWEgaXMgdGhhdCB3aGVuIHRoZSBpb2N0bCBpcwo+ID4gPmlzc3VlZCwgd2Ugc2hvdWxkIGdl dCAoaS5lLiwgbWFrZSBhIGR1cGxpY2F0ZSBvZikgdGhlIGN1cnJlbnQgQ1JUQwo+ID4gPnN0YXRl LCBjaGFuZ2Ugc29tZSBvZiB0aGUgdmFsdWVzIGluIHRoYXQgc3RhdGUgKHdoaWNoIGlzIHdoYXQg dGhlCj4gPiA+LmF0b21pY19zZXRfcHJvcGVydHkgZnVuY3Rpb24gdGFrZXMgY2FyZSBvZiksIGFu ZCB0aGVuIHN1Ym1pdCB0aGF0Cj4gPiA+bW9kaWZpZWQgc3RhdGUgdG8gdGhlIGRyaXZlciBmb3Ig Y2hlY2tpbmcgYW5kIGh3LXByb2dyYW1taW5nLgo+ID4gPgo+ID4gPk9rYXksIEkganVzdCB0b29r IGEgcXVpY2sgcGVlayBhaGVhZCBpbiB5b3VyIHBhdGNoIHNldCBhbmQgSSB0aGluayBJCj4gPiA+ dW5kZXJzdGFuZCB0aGUgY29uZnVzaW9uIG5vdy4uLml0IGxvb2tzIGxpa2UgeW91J3JlIHRyeWlu ZyB0byBhY3R1YWxseQo+ID4gPnBlcmZvcm0gdGhlIGZ1bGwgaGFyZHdhcmUgdXBkYXRlIGluIHRo ZSAuYXRvbWljX3NldF9wcm9wZXJ0eSBjYWxsIGNoYWluLAo+ID4gPndoaWNoIGlzbid0IHdoYXQg d2Ugd2FudCB0byBiZSBkb2luZyBpbiBhbiBhdG9taWMgZHJpdmVyLgo+ID4gPi5hdG9taWNfc2V0 X3Byb3BlcnR5KCkgaXMganVzdCBhIG1hdHRlciBvZiB1cGRhdGluZyB0aGUgc3RhdGUgc3RydWN0 dXJlcwo+ID4gPnRvIHJlZmxlY3QgdGhlIGNoYW5nZXMgeW91ICp3YW50KiB0byBtYWtlIChidXQg bm90IGFjdHVhbGx5IHBlcmZvcm1pbmcKPiA+ID50aG9zZSBjaGFuZ2VzIHJpZ2h0IGF3YXkpLiAg SXQgaXNuJ3QgdW50aWwgZHJtX2F0b21pY19jb21taXQoKSBnZXRzCj4gPiA+Y2FsbGVkIHRoYXQg d2UgdmFsaWRhdGUgdGhlIG5ldyBzdGF0ZSBzdHJ1Y3R1cmUgYW5kIHRoZW4gd3JpdGUgaXQgdG8g dGhlCj4gPiA+aGFyZHdhcmUgaWYgaXQgbG9va3Mgb2theS4KPiA+ID4KPiA+ID5LZWVwIGluIG1p bmQgdGhhdCB0aGUgb3ZlcmFsbCBnb2FsIGJlaGluZCBhdG9taWMgaXMgdGhhdCB3ZSB3YW50IHRv IGJlCj4gPiA+YWJsZSB0byBzdXBwbHkgc2V2ZXJhbCBpdGVtcyB0byBiZSB1cGRhdGVkIChlLmcu LCBtb2RlLCBDU0MsIGdhbW1hLAo+ID4gPmV0Yy4pIHZpYSB0aGUgYXRvbWljIGlvY3RsIGFuZCB0 aGVuIGhhdmUgdGhlbSBhbGwgYWNjZXB0ZWQgKGFuZAo+ID4gPnByb2dyYW1tZWQpIGJ5IHRoZSBk cml2ZXIsIG9yIGFsbCByZWplY3RlZCAoc28gbm9uZSBnZXQgcHJvZ3JhbW1lZCkgaWYKPiA+ID5h bnkgb2YgdGhlbSBhcmUgaW52YWxpZC4gIEFsc28sIGlmIHRoZSBjb2xsZWN0aW9uIG9mIHByb3Bl cnRpZXMgdGhhdAo+ID4gPnlvdSdyZSB1cGRhdGluZyBmYWxsIHdpdGhpbiB0aGUgIm51Y2xlYXIg cGFnZWZsaXAiIHN1YnNldCBvZgo+ID4gPmZ1bmN0aW9uYWxpdHksIHlvdSdsbCBhbHNvIGdldCBh IGd1YXJhbnRlZSB0aGF0IHRob3NlIGl0ZW1zIGFsbCBnZXQKPiA+ID51cGRhdGVkIHdpdGhpbiB0 aGUgc2FtZSB2Ymxhbms7IHVwZGF0ZXMgdGhhdCBzdHJhZGRsZSBhIHZibGFuayBjb3VsZAo+ID4g PmNhdXNlIHVud2FudGVkIGZsaWNrZXJpbmcgb3Igb3RoZXIgYXJ0aWZhY3RzLiAgVGhlIGhlbHBl ciBmdW5jdGlvbgo+ID4gPnlvdSdyZSB1cGRhdGluZyBoZXJlIG9ubHkgaGFwcGVucyB0byB1cGRh dGUgYSBzaW5nbGUgc3RhdGUgdmFsdWUgYXQgYQo+ID4gPnRpbWUsIGJ1dCB0aGUgc2FtZSAuYXRv bWljX3NldF9wcm9wZXJ0eSgpIGlzIGFsc28gdXNlZCBieSB0aGUgYXRvbWljCj4gPiA+aW9jdGws IHNvIHdlIG5lZWQgdG8gbWFrZSBzdXJlIGl0J3MgZm9sbG93aW5nIHRoZSBleHBlY3RlZCBkZXNp Z24uCj4gPiA+Cj4gPiA+RmluYWxseSwga2VlcCBpbiBtaW5kIHRoYXQgdGhlIGZ1bmN0aW9uIHlv dSdyZSB1cGRhdGluZyBoZXJlIGlzIGEgRFJNCj4gPiA+Y29yZSBmdW5jdGlvbi4gIEV2ZW4gdGhv dWdoIGk5MTUgaXNuJ3QgcXVpdGUgcmVhZHkgZm9yIGZ1bGwgYXRvbWljIHlldAo+ID4gPmFuZCBt aWdodCBoYXZlIGEgYml0IG9mIGJyYWluIGRhbWFnZSBpbiBhcmVhcywgb3RoZXIgdmVuZG9ycycg ZHJpdmVycyBkbwo+ID4gPmhhdmUgY29tcGxldGUgYXRvbWljIG1vZGVzZXR0aW5nIHN1cHBvcnQg KEkgdGhpbms/KSwgc28gYWRkaW5nCj4gPiA+aTkxNS1zcGVjaWZpYyB3b3JrYXJvdW5kcyBsaWtl IHRoaXMgdG8gdGhlIGNvcmUgZnVuY3Rpb24gY291bGQgYWN0dWFsbHkKPiA+ID5oYW1wZXIgdGhl bS4KPiA+ID4KPiA+ID4KPiA+ID5NYXR0Cj4gPiA+Cj4gPiBJIHVuZGVyc3Rvb2QgdGhpcyBwb2lu dC4gQnV0IGluIHRoaXMgY2FzZSwgd2Ugd2lsbCBiZSBibG9ja2VkIHVudGlsCj4gPiBzZXQgQ1JU QyBpbXBsZW1lbnRhdGlvbiBjb21lcyBpbi4gQ2FuIHlvdSBzdWdnZXN0IGEgYmV0dGVyIHdheSB0 bwo+ID4gaGFuZGxlIHRoaXMgd2l0aG91dCBnZXR0aW5nIGJsb2NrZWQgZm9yIHNldF9jcnRjKCkg Pwo+IAo+IElzIHRoZSBmdW5jdGlvbmFsaXR5IHlvdSBuZWVkIHN0aWxsIGxhY2tpbmcgd2l0aCBN YWFydGVuIExhbmtob3JzdCdzCj4gImNvbnZlcnQgdG8gYXRvbWljLCBwYXJ0IDIiIHNlcmllcyB0 aGF0IHJlY2VudGx5IGxhbmRlZCB1cHN0cmVhbT8gIElmCj4gdGhhdCBzZXJpZXMgaXNuJ3Qgc3Vm ZmljaWVudCwgdGhlbSBJJ20gcHJldHR5IHN1cmUgdGhhdCBoaXMgInBhcnQgMyIKPiBzZXJpZXMg b24gdGhlIG1haWxpbmcgbGlzdCB3aWxsIGZpeCBpdDsgdGhhdCB3aWxsIGhvcGVmdWxseSBiZSBy ZWFkeSB0bwo+IG1lcmdlIGFzIHNvb24gYXMgSSBnZXQgYSBjaGFuY2UgdG8gZG8gYSByZXZpZXcg b2YgaXQuCj4gCj4gSSdsbCBhZGQgTWFhcnRlbiB0byB0aGUgQ2MgaGVyZSBpbiBjYXNlIGhlIHdh bnRzIHRvIGNvbW1lbnQuLi4KPiAKPiAKPiBNYXR0CgpLYXVzYWwvU2hhc2hhbmssIGhhdmUgeW91 IGd1eXMgaGFkIGEgY2hhbmNlIHRvIGNoZWNrIG91dCBNYWFydGVuJ3MKbGF0ZXN0IGk5MTUgYXRv bWljIHdvcms/ICBIaXMgInBhcnQgMiIgc2VyaWVzIGlzIGFscmVhZHkgbWVyZ2VkIGFuZAoicGFy dCAzIiBzaG91bGQgYmUgbWVyZ2VkIHNvb24gYW5kIHRob3NlIHJlYWxseSBoZWxwIGNsZWFuIHVw IGEgbG90IG9mCnRoZSBJbnRlbC1zcGVjaWZpYyBwYWluIHBvaW50cyBjYXVzZWQgYnkgdGhlIGF0 b21pYyB0cmFuc2l0aW9uLiAgSSB0aGluawp3ZSBuZWVkIHRvIG1ha2Ugc3VyZSB5b3UgY2FuIGhh bmRsZSBwcm9wZXJ0aWVzIHByb3Blcmx5IGluIGFuIGF0b21pYwptYW5uZXIsIHNvIGlmIHlvdSBm aW5kIHlvdSdyZSBzdGlsbCBibG9ja2VkIG9uIHRoZSBjdXJyZW50IGNvZGUsIHdlCnNob3VsZCBs b29rIGludG8gaG93IHRvIGFkZHJlc3MgdGhhdCBzbyB0aGF0IHdlIGRvbid0IGhhdmUgdG8gYnJp bmcgYW55CnRlbXBvcmFyeSBJbnRlbCBoYWNraW5lc3MgaW50byB0aGUgc2hhcmVkIERSTSBjb3Jl IGNvZGUuCgoKTWF0dAoKPiAKPiA+ID4+Cj4gPiA+PlNpZ25lZC1vZmYtYnk6IFNoYXNoYW5rIFNo YXJtYSA8c2hhc2hhbmsuc2hhcm1hQGludGVsLmNvbT4KPiA+ID4+U2lnbmVkLW9mZi1ieTogS2F1 c2FsIE1hbGxhZGkgPEthdXNhbC5NYWxsYWRpQGludGVsLmNvbT4KPiA+ID4+LS0tCj4gPiA+PiAg ZHJpdmVycy9ncHUvZHJtL2RybV9hdG9taWNfaGVscGVyLmMgfCAxNyArKysrKysrKysrKysrKy0t LQo+ID4gPj4gIDEgZmlsZSBjaGFuZ2VkLCAxNCBpbnNlcnRpb25zKCspLCAzIGRlbGV0aW9ucygt KQo+ID4gPj4KPiA+ID4+ZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9kcm1fYXRvbWljX2hl bHBlci5jIGIvZHJpdmVycy9ncHUvZHJtL2RybV9hdG9taWNfaGVscGVyLmMKPiA+ID4+aW5kZXgg YTkwMDg1OC4uMzdkYmE1NSAxMDA2NDQKPiA+ID4+LS0tIGEvZHJpdmVycy9ncHUvZHJtL2RybV9h dG9taWNfaGVscGVyLmMKPiA+ID4+KysrIGIvZHJpdmVycy9ncHUvZHJtL2RybV9hdG9taWNfaGVs cGVyLmMKPiA+ID4+QEAgLTE2OTYsNiArMTY5Niw4IEBAIGRybV9hdG9taWNfaGVscGVyX2NydGNf c2V0X3Byb3BlcnR5KHN0cnVjdCBkcm1fY3J0YyAqY3J0YywKPiA+ID4+ICB7Cj4gPiA+PiAgCXN0 cnVjdCBkcm1fYXRvbWljX3N0YXRlICpzdGF0ZTsKPiA+ID4+ICAJc3RydWN0IGRybV9jcnRjX3N0 YXRlICpjcnRjX3N0YXRlOwo+ID4gPj4rCXN0cnVjdCBkcm1fZGV2aWNlICpkZXYgPSBjcnRjLT5k ZXY7Cj4gPiA+PisJc3RydWN0IGRybV9tb2RlX2NvbmZpZyAqY29uZmlnID0gJmRldi0+bW9kZV9j b25maWc7Cj4gPiA+PiAgCWludCByZXQgPSAwOwo+ID4gPj4KPiA+ID4+ICAJc3RhdGUgPSBkcm1f YXRvbWljX3N0YXRlX2FsbG9jKGNydGMtPmRldik7Cj4gPiA+PkBAIC0xNzE2LDkgKzE3MTgsMTgg QEAgcmV0cnk6Cj4gPiA+PiAgCWlmIChyZXQpCj4gPiA+PiAgCQlnb3RvIGZhaWw7Cj4gPiA+Pgo+ ID4gPj4tCXJldCA9IGRybV9hdG9taWNfY29tbWl0KHN0YXRlKTsKPiA+ID4+LQlpZiAocmV0ICE9 IDApCj4gPiA+Pi0JCWdvdG8gZmFpbDsKPiA+ID4+KwkvKioKPiA+ID4+KwkgKiBGSVhNRSA6IFRo aXMgaXMgYSBoYWNrLCB0byBhdm9pZCBhdG9taWMgY29tbWl0Cj4gPiA+PisJICogZm9yIENSVEMs IGJlY2F1c2UgaTkxNSBzdXBwb3J0cyBvbmx5Cj4gPiA+PisJICogYXRvbWljIHBsYW5lIG9wZXJh dGlvbnMgYXQgdGhlIG1vbWVudAo+ID4gPj4rCSAqLwo+ID4gPj4rCWlmIChwcm9wZXJ0eSA9PSBj b25maWctPmdhbW1hX3Byb3BlcnR5KQo+ID4gPj4rCQlyZXQgPSAwOwo+ID4gPj4rCWVsc2Ugewo+ ID4gPj4rCQlyZXQgPSBkcm1fYXRvbWljX2NvbW1pdChzdGF0ZSk7Cj4gPiA+PisJCWlmIChyZXQg IT0gMCkKPiA+ID4+KwkJCWdvdG8gZmFpbDsKPiA+ID4+Kwl9Cj4gPiA+Pgo+ID4gPj4gIAkvKiBE cml2ZXIgdGFrZXMgb3duZXJzaGlwIG9mIHN0YXRlIG9uIHN1Y2Nlc3NmdWwgY29tbWl0LiAqLwo+ ID4gPj4gIAlyZXR1cm4gMDsKPiA+ID4+LS0KPiA+ID4+Mi40LjIKPiA+ID4+Cj4gPiA+Cj4gCj4g LS0gCj4gTWF0dCBSb3Blcgo+IEdyYXBoaWNzIFNvZnR3YXJlIEVuZ2luZWVyCj4gSW9URyBQbGF0 Zm9ybSBFbmFibGluZyAmIERldmVsb3BtZW50Cj4gSW50ZWwgQ29ycG9yYXRpb24KPiAoOTE2KSAz NTYtMjc5NQo+IF9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f Cj4gSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdAo+IEludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5v cmcKPiBodHRwOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwt Z2Z4CgotLSAKTWF0dCBSb3BlcgpHcmFwaGljcyBTb2Z0d2FyZSBFbmdpbmVlcgpJb1RHIFBsYXRm b3JtIEVuYWJsaW5nICYgRGV2ZWxvcG1lbnQKSW50ZWwgQ29ycG9yYXRpb24KKDkxNikgMzU2LTI3 OTUKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRl dmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9s aXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK