From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH v2] drm: fb-helper: Reject all pixel format changing requests Date: Thu, 4 Oct 2018 14:00:43 +0300 Message-ID: <20181004110043.GP9144@intel.com> References: <20181003164538.5534-1-Eugeniy.Paltsev@synopsys.com> <20181004103422.GL9144@intel.com> <20181004104938.GJ31561@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTPS id 42E4B6E5D1 for ; Thu, 4 Oct 2018 11:00:48 +0000 (UTC) Content-Disposition: inline In-Reply-To: <20181004104938.GJ31561@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Eugeniy Paltsev , David Airlie , Alexey Brodkin , linux-kernel@vger.kernel.org, stable@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-snps-arc@lists.infradead.org, Sean Paul List-Id: dri-devel@lists.freedesktop.org T24gVGh1LCBPY3QgMDQsIDIwMTggYXQgMTI6NDk6MzhQTSArMDIwMCwgRGFuaWVsIFZldHRlciB3 cm90ZToKPiBPbiBUaHUsIE9jdCAwNCwgMjAxOCBhdCAwMTozNDoyMlBNICswMzAwLCBWaWxsZSBT eXJqw6Rsw6Qgd3JvdGU6Cj4gPiBPbiBXZWQsIE9jdCAwMywgMjAxOCBhdCAwNzo0NTozOFBNICsw MzAwLCBFdWdlbml5IFBhbHRzZXYgd3JvdGU6Cj4gPiA+IGRybSBmYmRldiBlbXVsYXRpb24gZG9l c24ndCBzdXBwb3J0IGNoYW5naW5nIHRoZSBwaXhlbCBmb3JtYXQgYXQgYWxsLAo+ID4gPiBzbyBy ZWplY3QgYWxsIHBpeGVsIGZvcm1hdCBjaGFuZ2luZyByZXF1ZXN0cy4KPiA+ID4gCj4gPiA+IENj OiBzdGFibGVAdmdlci5rZXJuZWwub3JnCj4gPiA+IFNpZ25lZC1vZmYtYnk6IEV1Z2VuaXkgUGFs dHNldiA8RXVnZW5peS5QYWx0c2V2QHN5bm9wc3lzLmNvbT4KPiA+ID4gLS0tCj4gPiA+IENoYW5n ZXMgdjEtPnYyOgo+ID4gPiAgKiBSZWplY3QgYWxsIHBpeGVsIGZvcm1hdCBjaGFuZ2luZyByZXF1 ZXN0LCBub3QganVzdCB0aGUgaW52YWxpZCBvbmVzLgo+ID4gPiAKPiA+ID4gIGRyaXZlcnMvZ3B1 L2RybS9kcm1fZmJfaGVscGVyLmMgfCA5MSArKysrKysrKysrKystLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLQo+ID4gPiAgMSBmaWxlIGNoYW5nZWQsIDI2IGluc2VydGlvbnMoKyksIDY1IGRl bGV0aW9ucygtKQo+ID4gPiAKPiA+ID4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9kcm1f ZmJfaGVscGVyLmMgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX2ZiX2hlbHBlci5jCj4gPiA+IGluZGV4 IDE2ZWM5M2I3NWRiZi4uNDg1OThkN2Y2NzNmIDEwMDY0NAo+ID4gPiAtLS0gYS9kcml2ZXJzL2dw dS9kcm0vZHJtX2ZiX2hlbHBlci5jCj4gPiA+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fZmJf aGVscGVyLmMKPiA+ID4gQEAgLTE1ODAsNiArMTU4MCwyNSBAQCBpbnQgZHJtX2ZiX2hlbHBlcl9p b2N0bChzdHJ1Y3QgZmJfaW5mbyAqaW5mbywgdW5zaWduZWQgaW50IGNtZCwKPiA+ID4gIH0KPiA+ ID4gIEVYUE9SVF9TWU1CT0woZHJtX2ZiX2hlbHBlcl9pb2N0bCk7Cj4gPiA+ICAKPiA+ID4gK3N0 YXRpYyBib29sIGRybV9mYl9waXhlbF9mb3JtYXRfZXF1YWwoY29uc3Qgc3RydWN0IGZiX3Zhcl9z Y3JlZW5pbmZvICp2YXJfMSwKPiA+ID4gKwkJCQkgICAgICBjb25zdCBzdHJ1Y3QgZmJfdmFyX3Nj cmVlbmluZm8gKnZhcl8yKQo+ID4gPiArewo+ID4gPiArCXJldHVybiB2YXJfMS0+Yml0c19wZXJf cGl4ZWwgPT0gdmFyXzItPmJpdHNfcGVyX3BpeGVsICYmCj4gPiA+ICsJICAgICAgIHZhcl8xLT5n cmF5c2NhbGUgPT0gdmFyXzItPmdyYXlzY2FsZSAmJgo+ID4gPiArCSAgICAgICB2YXJfMS0+cmVk Lm9mZnNldCA9PSB2YXJfMi0+cmVkLm9mZnNldCAmJgo+ID4gPiArCSAgICAgICB2YXJfMS0+cmVk Lmxlbmd0aCA9PSB2YXJfMi0+cmVkLmxlbmd0aCAmJgo+ID4gPiArCSAgICAgICB2YXJfMS0+cmVk Lm1zYl9yaWdodCA9PSB2YXJfMi0+cmVkLm1zYl9yaWdodCAmJgo+ID4gPiArCSAgICAgICB2YXJf MS0+Z3JlZW4ub2Zmc2V0ID09IHZhcl8yLT5ncmVlbi5vZmZzZXQgJiYKPiA+ID4gKwkgICAgICAg dmFyXzEtPmdyZWVuLmxlbmd0aCA9PSB2YXJfMi0+Z3JlZW4ubGVuZ3RoICYmCj4gPiA+ICsJICAg ICAgIHZhcl8xLT5ncmVlbi5tc2JfcmlnaHQgPT0gdmFyXzItPmdyZWVuLm1zYl9yaWdodCAmJgo+ ID4gPiArCSAgICAgICB2YXJfMS0+Ymx1ZS5vZmZzZXQgPT0gdmFyXzItPmJsdWUub2Zmc2V0ICYm Cj4gPiA+ICsJICAgICAgIHZhcl8xLT5ibHVlLmxlbmd0aCA9PSB2YXJfMi0+Ymx1ZS5sZW5ndGgg JiYKPiA+ID4gKwkgICAgICAgdmFyXzEtPmJsdWUubXNiX3JpZ2h0ID09IHZhcl8yLT5ibHVlLm1z Yl9yaWdodCAmJgo+ID4gPiArCSAgICAgICB2YXJfMS0+dHJhbnNwLm9mZnNldCA9PSB2YXJfMi0+ dHJhbnNwLm9mZnNldCAmJgo+ID4gPiArCSAgICAgICB2YXJfMS0+dHJhbnNwLmxlbmd0aCA9PSB2 YXJfMi0+dHJhbnNwLmxlbmd0aCAmJgo+ID4gPiArCSAgICAgICB2YXJfMS0+dHJhbnNwLm1zYl9y aWdodCA9PSB2YXJfMi0+dHJhbnNwLm1zYl9yaWdodDsKPiA+ID4gK30KPiA+IAo+ID4gQ291bGQg aGF2ZSBzaG9ydGVuZWQgYSBiaXQgd2l0aCBtZW1jbXAoKSBidXQgdGhpcyB3b3JrcyB0b28uCj4g Cj4gSSdtIGFsd2F5cyB2YXJ5IG9mIG1lbWNtcCB3aXRoIHN0cnVjdHMgdGhhdCBtaWdodCBoYXZl IHBhZGRpbmcgYW5kIHN0dWZmLgoKVGhlc2UgYXJlIHVhYmkgc28gcGFkZGluZyB3b3VsZCBiZSBy YXRoZXIgYmFkLgoKPiAKPiA+IFJldmlld2VkLWJ5OiBWaWxsZSBTeXJqw6Rsw6QgPHZpbGxlLnN5 cmphbGFAbGludXguaW50ZWwuY29tPgo+ID4gCj4gPiBJIHN1cHBvc2Ugd2UgcmVhbGx5IHNob3Vs ZCBiZSBkb2luZyB0aGUgc2FtZSBmb3IgbW9zdCBvZiB0aGUgcmVzdCBvZgo+ID4gdmFyX3NjcmVl bmluZm8gYXMgd2UgZG9uJ3QgYWxsb3cgY2hhbmdpbmcgdGhlIHRpbWluZ3MvZXRjLiBlaXRoZXIu Cj4gCj4gVGltaW5nIGF0IGxlYXN0IGRvZW5zJ3QgaGF2ZSBhbiBpbW1lZGlhdGUgY29ycmVjdG5l c3MgaW1wYWN0IG9mIHVzZXJzcGFjZQo+IHJlbmRlcmluZyB0aGUgd3JvbmcgZm9ybWF0IDotKQo+ IAo+IFRoYW5rcyBmb3IgcmV2aWV3JnBhdGNoLCBhcHBsaWVkIHRvIGRybS1taXNjLWZpeGVzLgo+ IC1EYW5pZWwKPiAKPiA+IAo+ID4gPiArCj4gPiA+ICAvKioKPiA+ID4gICAqIGRybV9mYl9oZWxw ZXJfY2hlY2tfdmFyIC0gaW1wbGVtZW50YXRpb24gZm9yICZmYl9vcHMuZmJfY2hlY2tfdmFyCj4g PiA+ICAgKiBAdmFyOiBzY3JlZW5pbmZvIHRvIGNoZWNrCj4gPiA+IEBAIC0xNTkwLDcgKzE2MDks NiBAQCBpbnQgZHJtX2ZiX2hlbHBlcl9jaGVja192YXIoc3RydWN0IGZiX3Zhcl9zY3JlZW5pbmZv ICp2YXIsCj4gPiA+ICB7Cj4gPiA+ICAJc3RydWN0IGRybV9mYl9oZWxwZXIgKmZiX2hlbHBlciA9 IGluZm8tPnBhcjsKPiA+ID4gIAlzdHJ1Y3QgZHJtX2ZyYW1lYnVmZmVyICpmYiA9IGZiX2hlbHBl ci0+ZmI7Cj4gPiA+IC0JaW50IGRlcHRoOwo+ID4gPiAgCj4gPiA+ICAJaWYgKHZhci0+cGl4Y2xv Y2sgIT0gMCB8fCBpbl9kYmdfbWFzdGVyKCkpCj4gPiA+ICAJCXJldHVybiAtRUlOVkFMOwo+ID4g PiBAQCAtMTYxMCw3MiArMTYyOCwxNSBAQCBpbnQgZHJtX2ZiX2hlbHBlcl9jaGVja192YXIoc3Ry dWN0IGZiX3Zhcl9zY3JlZW5pbmZvICp2YXIsCj4gPiA+ICAJCXJldHVybiAtRUlOVkFMOwo+ID4g PiAgCX0KPiA+ID4gIAo+ID4gPiAtCXN3aXRjaCAodmFyLT5iaXRzX3Blcl9waXhlbCkgewo+ID4g PiAtCWNhc2UgMTY6Cj4gPiA+IC0JCWRlcHRoID0gKHZhci0+Z3JlZW4ubGVuZ3RoID09IDYpID8g MTYgOiAxNTsKPiA+ID4gLQkJYnJlYWs7Cj4gPiA+IC0JY2FzZSAzMjoKPiA+ID4gLQkJZGVwdGgg PSAodmFyLT50cmFuc3AubGVuZ3RoID4gMCkgPyAzMiA6IDI0Owo+ID4gPiAtCQlicmVhazsKPiA+ ID4gLQlkZWZhdWx0Ogo+ID4gPiAtCQlkZXB0aCA9IHZhci0+Yml0c19wZXJfcGl4ZWw7Cj4gPiA+ IC0JCWJyZWFrOwo+ID4gPiAtCX0KPiA+ID4gLQo+ID4gPiAtCXN3aXRjaCAoZGVwdGgpIHsKPiA+ ID4gLQljYXNlIDg6Cj4gPiA+IC0JCXZhci0+cmVkLm9mZnNldCA9IDA7Cj4gPiA+IC0JCXZhci0+ Z3JlZW4ub2Zmc2V0ID0gMDsKPiA+ID4gLQkJdmFyLT5ibHVlLm9mZnNldCA9IDA7Cj4gPiA+IC0J CXZhci0+cmVkLmxlbmd0aCA9IDg7Cj4gPiA+IC0JCXZhci0+Z3JlZW4ubGVuZ3RoID0gODsKPiA+ ID4gLQkJdmFyLT5ibHVlLmxlbmd0aCA9IDg7Cj4gPiA+IC0JCXZhci0+dHJhbnNwLmxlbmd0aCA9 IDA7Cj4gPiA+IC0JCXZhci0+dHJhbnNwLm9mZnNldCA9IDA7Cj4gPiA+IC0JCWJyZWFrOwo+ID4g PiAtCWNhc2UgMTU6Cj4gPiA+IC0JCXZhci0+cmVkLm9mZnNldCA9IDEwOwo+ID4gPiAtCQl2YXIt PmdyZWVuLm9mZnNldCA9IDU7Cj4gPiA+IC0JCXZhci0+Ymx1ZS5vZmZzZXQgPSAwOwo+ID4gPiAt CQl2YXItPnJlZC5sZW5ndGggPSA1Owo+ID4gPiAtCQl2YXItPmdyZWVuLmxlbmd0aCA9IDU7Cj4g PiA+IC0JCXZhci0+Ymx1ZS5sZW5ndGggPSA1Owo+ID4gPiAtCQl2YXItPnRyYW5zcC5sZW5ndGgg PSAxOwo+ID4gPiAtCQl2YXItPnRyYW5zcC5vZmZzZXQgPSAxNTsKPiA+ID4gLQkJYnJlYWs7Cj4g PiA+IC0JY2FzZSAxNjoKPiA+ID4gLQkJdmFyLT5yZWQub2Zmc2V0ID0gMTE7Cj4gPiA+IC0JCXZh ci0+Z3JlZW4ub2Zmc2V0ID0gNTsKPiA+ID4gLQkJdmFyLT5ibHVlLm9mZnNldCA9IDA7Cj4gPiA+ IC0JCXZhci0+cmVkLmxlbmd0aCA9IDU7Cj4gPiA+IC0JCXZhci0+Z3JlZW4ubGVuZ3RoID0gNjsK PiA+ID4gLQkJdmFyLT5ibHVlLmxlbmd0aCA9IDU7Cj4gPiA+IC0JCXZhci0+dHJhbnNwLmxlbmd0 aCA9IDA7Cj4gPiA+IC0JCXZhci0+dHJhbnNwLm9mZnNldCA9IDA7Cj4gPiA+IC0JCWJyZWFrOwo+ ID4gPiAtCWNhc2UgMjQ6Cj4gPiA+IC0JCXZhci0+cmVkLm9mZnNldCA9IDE2Owo+ID4gPiAtCQl2 YXItPmdyZWVuLm9mZnNldCA9IDg7Cj4gPiA+IC0JCXZhci0+Ymx1ZS5vZmZzZXQgPSAwOwo+ID4g PiAtCQl2YXItPnJlZC5sZW5ndGggPSA4Owo+ID4gPiAtCQl2YXItPmdyZWVuLmxlbmd0aCA9IDg7 Cj4gPiA+IC0JCXZhci0+Ymx1ZS5sZW5ndGggPSA4Owo+ID4gPiAtCQl2YXItPnRyYW5zcC5sZW5n dGggPSAwOwo+ID4gPiAtCQl2YXItPnRyYW5zcC5vZmZzZXQgPSAwOwo+ID4gPiAtCQlicmVhazsK PiA+ID4gLQljYXNlIDMyOgo+ID4gPiAtCQl2YXItPnJlZC5vZmZzZXQgPSAxNjsKPiA+ID4gLQkJ dmFyLT5ncmVlbi5vZmZzZXQgPSA4Owo+ID4gPiAtCQl2YXItPmJsdWUub2Zmc2V0ID0gMDsKPiA+ ID4gLQkJdmFyLT5yZWQubGVuZ3RoID0gODsKPiA+ID4gLQkJdmFyLT5ncmVlbi5sZW5ndGggPSA4 Owo+ID4gPiAtCQl2YXItPmJsdWUubGVuZ3RoID0gODsKPiA+ID4gLQkJdmFyLT50cmFuc3AubGVu Z3RoID0gODsKPiA+ID4gLQkJdmFyLT50cmFuc3Aub2Zmc2V0ID0gMjQ7Cj4gPiA+IC0JCWJyZWFr Owo+ID4gPiAtCWRlZmF1bHQ6Cj4gPiA+ICsJLyoKPiA+ID4gKwkgKiBkcm0gZmJkZXYgZW11bGF0 aW9uIGRvZXNuJ3Qgc3VwcG9ydCBjaGFuZ2luZyB0aGUgcGl4ZWwgZm9ybWF0IGF0IGFsbCwKPiA+ ID4gKwkgKiBzbyByZWplY3QgYWxsIHBpeGVsIGZvcm1hdCBjaGFuZ2luZyByZXF1ZXN0cy4KPiA+ ID4gKwkgKi8KPiA+ID4gKwlpZiAoIWRybV9mYl9waXhlbF9mb3JtYXRfZXF1YWwodmFyLCAmaW5m by0+dmFyKSkgewo+ID4gPiArCQlEUk1fREVCVUcoImZiZGV2IGVtdWxhdGlvbiBkb2Vzbid0IHN1 cHBvcnQgY2hhbmdpbmcgdGhlIHBpeGVsIGZvcm1hdFxuIik7Cj4gPiA+ICAJCXJldHVybiAtRUlO VkFMOwo+ID4gPiAgCX0KPiA+ID4gKwo+ID4gPiAgCXJldHVybiAwOwo+ID4gPiAgfQo+ID4gPiAg RVhQT1JUX1NZTUJPTChkcm1fZmJfaGVscGVyX2NoZWNrX3Zhcik7Cj4gPiA+IC0tIAo+ID4gPiAy LjE0LjQKPiA+ID4gCj4gPiA+IF9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fCj4gPiA+IGRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKPiA+ID4gZHJpLWRldmVsQGxp c3RzLmZyZWVkZXNrdG9wLm9yZwo+ID4gPiBodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9t YWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo+ID4gCj4gPiAtLSAKPiA+IFZpbGxlIFN5cmrDpGzD pAo+ID4gSW50ZWwKPiA+IF9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fCj4gPiBkcmktZGV2ZWwgbWFpbGluZyBsaXN0Cj4gPiBkcmktZGV2ZWxAbGlzdHMuZnJl ZWRlc2t0b3Aub3JnCj4gPiBodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xp c3RpbmZvL2RyaS1kZXZlbAo+IAo+IC0tIAo+IERhbmllbCBWZXR0ZXIKPiBTb2Z0d2FyZSBFbmdp bmVlciwgSW50ZWwgQ29ycG9yYXRpb24KPiBodHRwOi8vYmxvZy5mZndsbC5jaAoKLS0gClZpbGxl IFN5cmrDpGzDpApJbnRlbApfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3Rv cC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmkt ZGV2ZWwK