From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ashutosh Dixit Subject: Re: [PATCH 2/3] drm/i915/perf: Add support for report sizes that are not power of 2 Date: Mon, 16 Sep 2019 21:11:58 -0700 Message-ID: <87tv9b36k1.wl-ashutosh.dixit@intel.com> References: <20190913230620.15906-1-umesh.nerlige.ramappa@intel.com> <20190913230620.15906-3-umesh.nerlige.ramappa@intel.com> <814defdf-c6d6-c009-f454-67b2a09b32a1@intel.com> <20190916191754.GA72596@orsosgc001.amr.corp.intel.com> Mime-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by gabe.freedesktop.org (Postfix) with ESMTPS id E5B806EAE9 for ; Tue, 17 Sep 2019 04:12:00 +0000 (UTC) In-Reply-To: <20190916191754.GA72596@orsosgc001.amr.corp.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Umesh Nerlige Ramappa Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T24gTW9uLCAxNiBTZXAgMjAxOSAxMjoxNzo1NCAtMDcwMCwgVW1lc2ggTmVybGlnZSBSYW1hcHBh IHdyb3RlOgo+Cj4gT24gU3VuLCBTZXAgMTUsIDIwMTkgYXQgMDI6MjQ6NDFQTSArMDMwMCwgTGlv bmVsIExhbmR3ZXJsaW4gd3JvdGU6Cj4gPiBPbiAxNC8wOS8yMDE5IDAyOjA2LCBVbWVzaCBOZXJs aWdlIFJhbWFwcGEgd3JvdGU6Cj4gPj4gT0EgcGVyZiB1bml0IHN1cHBvcnRzIG5vbi1wb3dlciBv ZiAyIHJlcG9ydCBzaXplcy4gRW5hYmxlIHN1cHBvcnQgZm9yCj4gPj4gdGhlc2Ugc2l6ZXMgaW4g dGhlIGRyaXZlci4KPiA+Pgo+ID4+IFNpZ25lZC1vZmYtYnk6IFVtZXNoIE5lcmxpZ2UgUmFtYXBw YSA8dW1lc2gubmVybGlnZS5yYW1hcHBhQGludGVsLmNvbT4KPiA+PiAtLS0KPiA+PiAgZHJpdmVy cy9ncHUvZHJtL2k5MTUvaTkxNV9wZXJmLmMgfCA1OSArKysrKysrKysrKystLS0tLS0tLS0tLS0t LS0tLS0tLQo+ID4+ICAxIGZpbGUgY2hhbmdlZCwgMjEgaW5zZXJ0aW9ucygrKSwgMzggZGVsZXRp b25zKC0pCj4gPj4KPiA+PiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9w ZXJmLmMgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X3BlcmYuYwo+ID4+IGluZGV4IDUwYjZk MTU0ZmQ0Ni4uNDgyZmNhM2RhN2RlIDEwMDY0NAo+ID4+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9p OTE1L2k5MTVfcGVyZi5jCj4gPj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9wZXJm LmMKPiA+PiBAQCAtNDUwLDcgKzQ1MCw3IEBAIHN0YXRpYyBib29sIG9hX2J1ZmZlcl9jaGVja191 bmxvY2tlZChzdHJ1Y3QgaTkxNV9wZXJmX3N0cmVhbSAqc3RyZWFtKQo+ID4+CXUzMiBndHRfb2Zm c2V0ID0gaTkxNV9nZ3R0X29mZnNldChzdHJlYW0tPm9hX2J1ZmZlci52bWEpOwo+ID4+CWludCBy ZXBvcnRfc2l6ZSA9IHN0cmVhbS0+b2FfYnVmZmVyLmZvcm1hdF9zaXplOwo+ID4+CXVuc2lnbmVk IGxvbmcgZmxhZ3M7Cj4gPj4gLQl1MzIgaHdfdGFpbDsKPiA+PiArCXUzMiBod190YWlsLCBhZ2lu Z190YWlsOwo+ID4+CXU2NCBub3c7Cj4gPj4JLyogV2UgaGF2ZSB0byBjb25zaWRlciB0aGUgKHVu bGlrZWx5KSBwb3NzaWJpbGl0eSB0aGF0IHJlYWQoKSBlcnJvcnMKPiA+PiBAQCAtNDU5LDE2ICs0 NTksMTcgQEAgc3RhdGljIGJvb2wgb2FfYnVmZmVyX2NoZWNrX3VubG9ja2VkKHN0cnVjdCBpOTE1 X3BlcmZfc3RyZWFtICpzdHJlYW0pCj4gPj4JICovCj4gPj4Jc3Bpbl9sb2NrX2lycXNhdmUoJnN0 cmVhbS0+b2FfYnVmZmVyLnB0cl9sb2NrLCBmbGFncyk7Cj4gPj4gLQlod190YWlsID0gZGV2X3By aXYtPnBlcmYub3BzLm9hX2h3X3RhaWxfcmVhZChzdHJlYW0pOwo+ID4+ICsJaHdfdGFpbCA9IGRl dl9wcml2LT5wZXJmLm9wcy5vYV9od190YWlsX3JlYWQoc3RyZWFtKSAtIGd0dF9vZmZzZXQ7Cj4g Pj4gKwlhZ2luZ190YWlsID0gc3RyZWFtLT5vYV9idWZmZXIuYWdpbmdfdGFpbCAtIGd0dF9vZmZz ZXQ7Cj4gPj4JLyogVGhlIHRhaWwgcG9pbnRlciBpbmNyZWFzZXMgaW4gNjQgYnl0ZSBpbmNyZW1l bnRzLAo+ID4+CSAqIG5vdCBpbiByZXBvcnRfc2l6ZSBzdGVwcy4uLgo+ID4+CSAqLwo+ID4+IC0J aHdfdGFpbCAmPSB+KHJlcG9ydF9zaXplIC0gMSk7Cj4gPj4gKwlod190YWlsID0gT0FfVEFLRU4o aHdfdGFpbCwgKE9BX1RBS0VOKGh3X3RhaWwsIGFnaW5nX3RhaWwpICUgcmVwb3J0X3NpemUpKTsK PiA+Cj4gPgo+ID4gSSdtIHN0cnVnZ2xpbmcgdG8gcGFyc2UgdGhpcyBsaW5lIGFib3ZlIGFuZCBJ J20gbm90IDEwMCUgc3VyZSBpdCdzIGNvcnJlY3QuCj4gPgo+ID4gQ291bGQgYWRkIGEgY29tbWVu dCB0byBleHBsYWluIHdoYXQgaXMgZ29pbmcgb24/Cj4KPiBUaGUgYWdpbmcgdGFpbCBpcyBhbHdh eXMgcG9pbnRpbmcgdG8gdGhlIGJvdW5kYXJ5IG9mIGEgcmVwb3J0IHdoZXJlYXMKPiB0aGUgaHdf dGFpbCBpcyBhZHZhbmNpbmcgaW4gNjQgYnl0ZSBpbmNyZW1lbnRzLgo+Cj4gVGhlIGlubmVybW9z dCBPQV9UQUtFTiBpcyByZXR1cm5pbmcgdGhlIG51bWJlciBvZiBieXRlcyBiZXR3ZWVuIHRoZQo+ IGh3X3RhaWwgYW5kIHRoZSBhZ2luZ190YWlsLiBUaGUgbW9kdWxvIGlzIGdldHRpbmcgdGhlIHNp emUgb2YgdGhlCj4gcGFydGlhbCByZXBvcnQgKGlmIGFueSkuCj4KPiBUaGUgb3V0ZXJtb3N0IE9B X1RBS0VOIGlzIHN1YnRyYWN0aW5nIHRoZSBzaXplIG9mIHBhcnRpYWwgcmVwb3J0IGZyb20KPiB0 aGUgaHdfdGFpbCB0byBnZXQgYSBod190YWlsIHRoYXQgcG9pbnRzIHRvIHRoZSBib3VuZGFyeSBv ZiB0aGUgbGFzdAo+IGZ1bGwgcmVwb3J0Lgo+Cj4gVGhlIHZhbHVlIG9mIGh3X3RhaWwgd291bGQg YmUgdGhlIHNhbWUgYXMgZnJvbSB0aGUgZGVsZXRlZCBsaW5lIG9mIGNvZGUKPiBhYm92ZSB0aGlz IGxpbmUuCgpMb29rcyBsaWtlIGFnaW5nX3RhaWwgc2hvdWxkIG5vdCBiZSBuZWVkZWQgKGl0IGlz IGNvbXBsaWNhdGluZyB0aGUKZXhwcmVzc2lvbiBhbmQgaXMgbm90IHRoZXJlIGluIHRoZSBvcmln aW5hbCBleHByZXNzaW9uKS4gQWxsIHdlIG5lZWQgaXMgYQoiY2lyY3VsYXIgbW9kdWx1cyIuIEZv ciBleGFtcGxlIHdvdWxkIHRoZSBmb2xsb3dpbmcgd29yaz8KCiAgICBpZiAoaHdfdGFpbCA8IHJl cG9ydF9zaXplKQogICAgICAgIGh3X3RhaWwgKz0gT0FfQlVGRkVSX1NJWkU7CiAgICBod190YWls ID0gcm91bmRkb3duKGh3X3RhaWwsIHJlcG9ydF9zaXplKTsKCkFub3RoZXIgd2F5IChpZiB3ZSB3 YW50IHRvIGF2b2lkIHRoZSBkaXZpc2lvbikgd291bGQgYmUgdG8gbWFpbnRhaW4gYQpjYWNoZWQg Y29weSBvZiBod190YWlsIGluIFNXIHdoaWNoIGlzIHN1Y2Nlc3NpdmVseSAoYW5kIGNpcmN1bGFy bHkpCmluY3JlbWVudGVkIGJ5IHJlcG9ydF9zaXplIHRpbGwgaXQgY2F0Y2hlcyB1cCB3aXRoIGh3 X3RhaWwgcmVhZCBmcm9tCkhXLiBCdXQgcHJvYmFibHkgdGhlIGZpcnN0IG1ldGhvZCBhYm92ZSBp cyBzaW1wbGVyLgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f XwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcK aHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZng=