From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Sharma, Shashank" Subject: Re: [PATCH v2 06/10] drm: Avoid atomic commit path for CRTC property (Gamma) Date: Sat, 06 Jun 2015 17:34:45 +0530 Message-ID: <5572E1DD.5020809@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> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20150606010102.GI1099@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Matt Roper , Kausal Malladi Cc: annie.j.matheson@intel.com, dhanya.p.r@intel.com, avinash.reddy.palleti@intel.com, dri-devel@lists.freedesktop.org, vijay.a.purushothaman@intel.com, indranil.mukherjee@intel.com, jesse.barnes@intel.com, daniel.vetter@intel.com, sunil.kamath@intel.com, intel-gfx@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org UmVnYXJkcwpTaGFzaGFuawoKT24gNi82LzIwMTUgNjozMSBBTSwgTWF0dCBSb3BlciB3cm90ZToK PiBPbiBUaHUsIEp1biAwNCwgMjAxNSBhdCAwNzoxMjozN1BNICswNTMwLCBLYXVzYWwgTWFsbGFk aSB3cm90ZToKPj4gRnJvbTogS2F1c2FsIE1hbGxhZGkgPEthdXNhbC5NYWxsYWRpQGludGVsLmNv bT4KPj4KPj4gVGhlIGF0b21pYyBDUlRDIHNldCBpbmZyYXN0cnVjdHVyZSBpcyBub3QgYXZhaWxh YmxlIHlldC4gU28sIHRoZSBhdG9taWMKPj4gY2hlY2sgcGF0aCB0aHJvd3MgZXJyb3IgZm9yIGFu eSBub24tcGxhbmUgcHJvcGVydHkuCj4+Cj4+IFRoaXMgcGF0Y2ggYWRkcyBhIGRpdmVyc2lvbiB0 byBhdm9pZCBjb21taXQgcGF0aCBmb3IgRFJNIGF0b21pYyBzZXQgR2FtbWEKPj4gcHJvcGVydHks IHVudGlsIGF0b21pYyBDUlRDIGluZnJhc3RydWN0dXJlIGlzIHJlYWR5Lgo+Cj4gVGhpcyBkb2Vz bid0IGxvb2sgcmlnaHQsIGJ1dCBJIGRvbid0IHF1aXRlIHVuZGVyc3RhbmQgd2hhdCB5b3UncmUg dHJ5aW5nCj4gdG8gZG8gZnJvbSB0aGUgY29tbWl0IG1lc3NhZ2UuCj4KPiBUaGlzIGZ1bmN0aW9u IGlzIHdoYXQgd2lsbCBpbXBsZW1lbnQgbGVnYWN5IHNldF9wcm9wZXJ0eSBpb2N0bCdzIG9mIGEK PiBDUlRDIG9uIGFuIGF0b21pYy1iYXNlZCBkcml2ZXIuICBUaGUgaWRlYSBpcyB0aGF0IHdoZW4g dGhlIGlvY3RsIGlzCj4gaXNzdWVkLCB3ZSBzaG91bGQgZ2V0IChpLmUuLCBtYWtlIGEgZHVwbGlj YXRlIG9mKSB0aGUgY3VycmVudCBDUlRDCj4gc3RhdGUsIGNoYW5nZSBzb21lIG9mIHRoZSB2YWx1 ZXMgaW4gdGhhdCBzdGF0ZSAod2hpY2ggaXMgd2hhdCB0aGUKPiAuYXRvbWljX3NldF9wcm9wZXJ0 eSBmdW5jdGlvbiB0YWtlcyBjYXJlIG9mKSwgYW5kIHRoZW4gc3VibWl0IHRoYXQKPiBtb2RpZmll ZCBzdGF0ZSB0byB0aGUgZHJpdmVyIGZvciBjaGVja2luZyBhbmQgaHctcHJvZ3JhbW1pbmcuCj4K PiBPa2F5LCBJIGp1c3QgdG9vayBhIHF1aWNrIHBlZWsgYWhlYWQgaW4geW91ciBwYXRjaCBzZXQg YW5kIEkgdGhpbmsgSQo+IHVuZGVyc3RhbmQgdGhlIGNvbmZ1c2lvbiBub3cuLi5pdCBsb29rcyBs aWtlIHlvdSdyZSB0cnlpbmcgdG8gYWN0dWFsbHkKPiBwZXJmb3JtIHRoZSBmdWxsIGhhcmR3YXJl IHVwZGF0ZSBpbiB0aGUgLmF0b21pY19zZXRfcHJvcGVydHkgY2FsbCBjaGFpbiwKPiB3aGljaCBp c24ndCB3aGF0IHdlIHdhbnQgdG8gYmUgZG9pbmcgaW4gYW4gYXRvbWljIGRyaXZlci4KPiAuYXRv bWljX3NldF9wcm9wZXJ0eSgpIGlzIGp1c3QgYSBtYXR0ZXIgb2YgdXBkYXRpbmcgdGhlIHN0YXRl IHN0cnVjdHVyZXMKPiB0byByZWZsZWN0IHRoZSBjaGFuZ2VzIHlvdSAqd2FudCogdG8gbWFrZSAo YnV0IG5vdCBhY3R1YWxseSBwZXJmb3JtaW5nCj4gdGhvc2UgY2hhbmdlcyByaWdodCBhd2F5KS4g IEl0IGlzbid0IHVudGlsIGRybV9hdG9taWNfY29tbWl0KCkgZ2V0cwo+IGNhbGxlZCB0aGF0IHdl IHZhbGlkYXRlIHRoZSBuZXcgc3RhdGUgc3RydWN0dXJlIGFuZCB0aGVuIHdyaXRlIGl0IHRvIHRo ZQo+IGhhcmR3YXJlIGlmIGl0IGxvb2tzIG9rYXkuCj4KPiBLZWVwIGluIG1pbmQgdGhhdCB0aGUg b3ZlcmFsbCBnb2FsIGJlaGluZCBhdG9taWMgaXMgdGhhdCB3ZSB3YW50IHRvIGJlCj4gYWJsZSB0 byBzdXBwbHkgc2V2ZXJhbCBpdGVtcyB0byBiZSB1cGRhdGVkIChlLmcuLCBtb2RlLCBDU0MsIGdh bW1hLAo+IGV0Yy4pIHZpYSB0aGUgYXRvbWljIGlvY3RsIGFuZCB0aGVuIGhhdmUgdGhlbSBhbGwg YWNjZXB0ZWQgKGFuZAo+IHByb2dyYW1tZWQpIGJ5IHRoZSBkcml2ZXIsIG9yIGFsbCByZWplY3Rl ZCAoc28gbm9uZSBnZXQgcHJvZ3JhbW1lZCkgaWYKPiBhbnkgb2YgdGhlbSBhcmUgaW52YWxpZC4g IEFsc28sIGlmIHRoZSBjb2xsZWN0aW9uIG9mIHByb3BlcnRpZXMgdGhhdAo+IHlvdSdyZSB1cGRh dGluZyBmYWxsIHdpdGhpbiB0aGUgIm51Y2xlYXIgcGFnZWZsaXAiIHN1YnNldCBvZgo+IGZ1bmN0 aW9uYWxpdHksIHlvdSdsbCBhbHNvIGdldCBhIGd1YXJhbnRlZSB0aGF0IHRob3NlIGl0ZW1zIGFs bCBnZXQKPiB1cGRhdGVkIHdpdGhpbiB0aGUgc2FtZSB2Ymxhbms7IHVwZGF0ZXMgdGhhdCBzdHJh ZGRsZSBhIHZibGFuayBjb3VsZAo+IGNhdXNlIHVud2FudGVkIGZsaWNrZXJpbmcgb3Igb3RoZXIg YXJ0aWZhY3RzLiAgVGhlIGhlbHBlciBmdW5jdGlvbgo+IHlvdSdyZSB1cGRhdGluZyBoZXJlIG9u bHkgaGFwcGVucyB0byB1cGRhdGUgYSBzaW5nbGUgc3RhdGUgdmFsdWUgYXQgYQo+IHRpbWUsIGJ1 dCB0aGUgc2FtZSAuYXRvbWljX3NldF9wcm9wZXJ0eSgpIGlzIGFsc28gdXNlZCBieSB0aGUgYXRv bWljCj4gaW9jdGwsIHNvIHdlIG5lZWQgdG8gbWFrZSBzdXJlIGl0J3MgZm9sbG93aW5nIHRoZSBl eHBlY3RlZCBkZXNpZ24uCj4KPiBGaW5hbGx5LCBrZWVwIGluIG1pbmQgdGhhdCB0aGUgZnVuY3Rp b24geW91J3JlIHVwZGF0aW5nIGhlcmUgaXMgYSBEUk0KPiBjb3JlIGZ1bmN0aW9uLiAgRXZlbiB0 aG91Z2ggaTkxNSBpc24ndCBxdWl0ZSByZWFkeSBmb3IgZnVsbCBhdG9taWMgeWV0Cj4gYW5kIG1p Z2h0IGhhdmUgYSBiaXQgb2YgYnJhaW4gZGFtYWdlIGluIGFyZWFzLCBvdGhlciB2ZW5kb3JzJyBk cml2ZXJzIGRvCj4gaGF2ZSBjb21wbGV0ZSBhdG9taWMgbW9kZXNldHRpbmcgc3VwcG9ydCAoSSB0 aGluaz8pLCBzbyBhZGRpbmcKPiBpOTE1LXNwZWNpZmljIHdvcmthcm91bmRzIGxpa2UgdGhpcyB0 byB0aGUgY29yZSBmdW5jdGlvbiBjb3VsZCBhY3R1YWxseQo+IGhhbXBlciB0aGVtLgo+Cj4KPiBN YXR0Cj4KSSB1bmRlcnN0b29kIHRoaXMgcG9pbnQuIEJ1dCBpbiB0aGlzIGNhc2UsIHdlIHdpbGwg YmUgYmxvY2tlZCB1bnRpbCBzZXQgCkNSVEMgaW1wbGVtZW50YXRpb24gY29tZXMgaW4uIENhbiB5 b3Ugc3VnZ2VzdCBhIGJldHRlciB3YXkgdG8gaGFuZGxlIAp0aGlzIHdpdGhvdXQgZ2V0dGluZyBi bG9ja2VkIGZvciBzZXRfY3J0YygpID8KPj4KPj4gU2lnbmVkLW9mZi1ieTogU2hhc2hhbmsgU2hh cm1hIDxzaGFzaGFuay5zaGFybWFAaW50ZWwuY29tPgo+PiBTaWduZWQtb2ZmLWJ5OiBLYXVzYWwg TWFsbGFkaSA8S2F1c2FsLk1hbGxhZGlAaW50ZWwuY29tPgo+PiAtLS0KPj4gICBkcml2ZXJzL2dw dS9kcm0vZHJtX2F0b21pY19oZWxwZXIuYyB8IDE3ICsrKysrKysrKysrKysrLS0tCj4+ICAgMSBm aWxlIGNoYW5nZWQsIDE0IGluc2VydGlvbnMoKyksIDMgZGVsZXRpb25zKC0pCj4+Cj4+IGRpZmYg LS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vZHJtX2F0b21pY19oZWxwZXIuYyBiL2RyaXZlcnMvZ3B1 L2RybS9kcm1fYXRvbWljX2hlbHBlci5jCj4+IGluZGV4IGE5MDA4NTguLjM3ZGJhNTUgMTAwNjQ0 Cj4+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9kcm1fYXRvbWljX2hlbHBlci5jCj4+ICsrKyBiL2Ry aXZlcnMvZ3B1L2RybS9kcm1fYXRvbWljX2hlbHBlci5jCj4+IEBAIC0xNjk2LDYgKzE2OTYsOCBA QCBkcm1fYXRvbWljX2hlbHBlcl9jcnRjX3NldF9wcm9wZXJ0eShzdHJ1Y3QgZHJtX2NydGMgKmNy dGMsCj4+ICAgewo+PiAgIAlzdHJ1Y3QgZHJtX2F0b21pY19zdGF0ZSAqc3RhdGU7Cj4+ICAgCXN0 cnVjdCBkcm1fY3J0Y19zdGF0ZSAqY3J0Y19zdGF0ZTsKPj4gKwlzdHJ1Y3QgZHJtX2RldmljZSAq ZGV2ID0gY3J0Yy0+ZGV2Owo+PiArCXN0cnVjdCBkcm1fbW9kZV9jb25maWcgKmNvbmZpZyA9ICZk ZXYtPm1vZGVfY29uZmlnOwo+PiAgIAlpbnQgcmV0ID0gMDsKPj4KPj4gICAJc3RhdGUgPSBkcm1f YXRvbWljX3N0YXRlX2FsbG9jKGNydGMtPmRldik7Cj4+IEBAIC0xNzE2LDkgKzE3MTgsMTggQEAg cmV0cnk6Cj4+ICAgCWlmIChyZXQpCj4+ICAgCQlnb3RvIGZhaWw7Cj4+Cj4+IC0JcmV0ID0gZHJt X2F0b21pY19jb21taXQoc3RhdGUpOwo+PiAtCWlmIChyZXQgIT0gMCkKPj4gLQkJZ290byBmYWls Owo+PiArCS8qKgo+PiArCSAqIEZJWE1FIDogVGhpcyBpcyBhIGhhY2ssIHRvIGF2b2lkIGF0b21p YyBjb21taXQKPj4gKwkgKiBmb3IgQ1JUQywgYmVjYXVzZSBpOTE1IHN1cHBvcnRzIG9ubHkKPj4g KwkgKiBhdG9taWMgcGxhbmUgb3BlcmF0aW9ucyBhdCB0aGUgbW9tZW50Cj4+ICsJICovCj4+ICsJ aWYgKHByb3BlcnR5ID09IGNvbmZpZy0+Z2FtbWFfcHJvcGVydHkpCj4+ICsJCXJldCA9IDA7Cj4+ ICsJZWxzZSB7Cj4+ICsJCXJldCA9IGRybV9hdG9taWNfY29tbWl0KHN0YXRlKTsKPj4gKwkJaWYg KHJldCAhPSAwKQo+PiArCQkJZ290byBmYWlsOwo+PiArCX0KPj4KPj4gICAJLyogRHJpdmVyIHRh a2VzIG93bmVyc2hpcCBvZiBzdGF0ZSBvbiBzdWNjZXNzZnVsIGNvbW1pdC4gKi8KPj4gICAJcmV0 dXJuIDA7Cj4+IC0tCj4+IDIuNC4yCj4+Cj4KX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMu ZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0 aW5mby9kcmktZGV2ZWwK