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: Tue, 17 Sep 2019 22:38:31 -0700 Message-ID: <875zlq2mg8.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> <87tv9b36k1.wl-ashutosh.dixit@intel.com> <20190917173351.GA1279@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 mga18.intel.com (mga18.intel.com [134.134.136.126]) by gabe.freedesktop.org (Postfix) with ESMTPS id 96A016EDF3 for ; Wed, 18 Sep 2019 05:38:32 +0000 (UTC) In-Reply-To: <20190917173351.GA1279@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 T24gVHVlLCAxNyBTZXAgMjAxOSAxMDozMzo1MSAtMDcwMCwgVW1lc2ggTmVybGlnZSBSYW1hcHBh IHdyb3RlOgo+Cj4gT24gTW9uLCBTZXAgMTYsIDIwMTkgYXQgMDk6MTE6NThQTSAtMDcwMCwgQXNo dXRvc2ggRGl4aXQgd3JvdGU6Cj4gPiBPbiBNb24sIDE2IFNlcCAyMDE5IDEyOjE3OjU0IC0wNzAw LCBVbWVzaCBOZXJsaWdlIFJhbWFwcGEgd3JvdGU6Cj4gPj4KPiA+PiBPbiBTdW4sIFNlcCAxNSwg MjAxOSBhdCAwMjoyNDo0MVBNICswMzAwLCBMaW9uZWwgTGFuZHdlcmxpbiB3cm90ZToKPiA+PiA+ IE9uIDE0LzA5LzIwMTkgMDI6MDYsIFVtZXNoIE5lcmxpZ2UgUmFtYXBwYSB3cm90ZToKPiA+PiA+ PiBPQSBwZXJmIHVuaXQgc3VwcG9ydHMgbm9uLXBvd2VyIG9mIDIgcmVwb3J0IHNpemVzLiBFbmFi bGUgc3VwcG9ydCBmb3IKPiA+PiA+PiB0aGVzZSBzaXplcyBpbiB0aGUgZHJpdmVyLgo+ID4+ID4+ Cj4gPj4gPj4gU2lnbmVkLW9mZi1ieTogVW1lc2ggTmVybGlnZSBSYW1hcHBhIDx1bWVzaC5uZXJs aWdlLnJhbWFwcGFAaW50ZWwuY29tPgo+ID4+ID4+IC0tLQo+ID4+ID4+ICBkcml2ZXJzL2dwdS9k cm0vaTkxNS9pOTE1X3BlcmYuYyB8IDU5ICsrKysrKysrKysrKy0tLS0tLS0tLS0tLS0tLS0tLS0t Cj4gPj4gPj4gIDEgZmlsZSBjaGFuZ2VkLCAyMSBpbnNlcnRpb25zKCspLCAzOCBkZWxldGlvbnMo LSkKPiA+PiA+Pgo+ID4+ID4+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1 X3BlcmYuYyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfcGVyZi5jCj4gPj4gPj4gaW5kZXgg NTBiNmQxNTRmZDQ2Li40ODJmY2EzZGE3ZGUgMTAwNjQ0Cj4gPj4gPj4gLS0tIGEvZHJpdmVycy9n cHUvZHJtL2k5MTUvaTkxNV9wZXJmLmMKPiA+PiA+PiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkx NS9pOTE1X3BlcmYuYwo+ID4+ID4+IEBAIC00NTAsNyArNDUwLDcgQEAgc3RhdGljIGJvb2wgb2Ff YnVmZmVyX2NoZWNrX3VubG9ja2VkKHN0cnVjdCBpOTE1X3BlcmZfc3RyZWFtICpzdHJlYW0pCj4g Pj4gPj4JdTMyIGd0dF9vZmZzZXQgPSBpOTE1X2dndHRfb2Zmc2V0KHN0cmVhbS0+b2FfYnVmZmVy LnZtYSk7Cj4gPj4gPj4JaW50IHJlcG9ydF9zaXplID0gc3RyZWFtLT5vYV9idWZmZXIuZm9ybWF0 X3NpemU7Cj4gPj4gPj4JdW5zaWduZWQgbG9uZyBmbGFnczsKPiA+PiA+PiAtCXUzMiBod190YWls Owo+ID4+ID4+ICsJdTMyIGh3X3RhaWwsIGFnaW5nX3RhaWw7Cj4gPj4gPj4JdTY0IG5vdzsKPiA+ PiA+PgkvKiBXZSBoYXZlIHRvIGNvbnNpZGVyIHRoZSAodW5saWtlbHkpIHBvc3NpYmlsaXR5IHRo YXQgcmVhZCgpIGVycm9ycwo+ID4+ID4+IEBAIC00NTksMTYgKzQ1OSwxNyBAQCBzdGF0aWMgYm9v bCBvYV9idWZmZXJfY2hlY2tfdW5sb2NrZWQoc3RydWN0IGk5MTVfcGVyZl9zdHJlYW0gKnN0cmVh bSkKPiA+PiA+PgkgKi8KPiA+PiA+PglzcGluX2xvY2tfaXJxc2F2ZSgmc3RyZWFtLT5vYV9idWZm ZXIucHRyX2xvY2ssIGZsYWdzKTsKPiA+PiA+PiAtCWh3X3RhaWwgPSBkZXZfcHJpdi0+cGVyZi5v cHMub2FfaHdfdGFpbF9yZWFkKHN0cmVhbSk7Cj4gPj4gPj4gKwlod190YWlsID0gZGV2X3ByaXYt PnBlcmYub3BzLm9hX2h3X3RhaWxfcmVhZChzdHJlYW0pIC0gZ3R0X29mZnNldDsKPiA+PiA+PiAr CWFnaW5nX3RhaWwgPSBzdHJlYW0tPm9hX2J1ZmZlci5hZ2luZ190YWlsIC0gZ3R0X29mZnNldDsK PiA+PiA+PgkvKiBUaGUgdGFpbCBwb2ludGVyIGluY3JlYXNlcyBpbiA2NCBieXRlIGluY3JlbWVu dHMsCj4gPj4gPj4JICogbm90IGluIHJlcG9ydF9zaXplIHN0ZXBzLi4uCj4gPj4gPj4JICovCj4g Pj4gPj4gLQlod190YWlsICY9IH4ocmVwb3J0X3NpemUgLSAxKTsKPiA+PiA+PiArCWh3X3RhaWwg PSBPQV9UQUtFTihod190YWlsLCAoT0FfVEFLRU4oaHdfdGFpbCwgYWdpbmdfdGFpbCkgJSByZXBv cnRfc2l6ZSkpOwo+ID4+ID4KPiA+PiA+Cj4gPj4gPiBJJ20gc3RydWdnbGluZyB0byBwYXJzZSB0 aGlzIGxpbmUgYWJvdmUgYW5kIEknbSBub3QgMTAwJSBzdXJlIGl0J3MgY29ycmVjdC4KPiA+PiA+ Cj4gPj4gPiBDb3VsZCBhZGQgYSBjb21tZW50IHRvIGV4cGxhaW4gd2hhdCBpcyBnb2luZyBvbj8K PiA+Pgo+ID4+IFRoZSBhZ2luZyB0YWlsIGlzIGFsd2F5cyBwb2ludGluZyB0byB0aGUgYm91bmRh cnkgb2YgYSByZXBvcnQgd2hlcmVhcwo+ID4+IHRoZSBod190YWlsIGlzIGFkdmFuY2luZyBpbiA2 NCBieXRlIGluY3JlbWVudHMuCj4gPj4KPiA+PiBUaGUgaW5uZXJtb3N0IE9BX1RBS0VOIGlzIHJl dHVybmluZyB0aGUgbnVtYmVyIG9mIGJ5dGVzIGJldHdlZW4gdGhlCj4gPj4gaHdfdGFpbCBhbmQg dGhlIGFnaW5nX3RhaWwuIFRoZSBtb2R1bG8gaXMgZ2V0dGluZyB0aGUgc2l6ZSBvZiB0aGUKPiA+ PiBwYXJ0aWFsIHJlcG9ydCAoaWYgYW55KS4KPiA+Pgo+ID4+IFRoZSBvdXRlcm1vc3QgT0FfVEFL RU4gaXMgc3VidHJhY3RpbmcgdGhlIHNpemUgb2YgcGFydGlhbCByZXBvcnQgZnJvbQo+ID4+IHRo ZSBod190YWlsIHRvIGdldCBhIGh3X3RhaWwgdGhhdCBwb2ludHMgdG8gdGhlIGJvdW5kYXJ5IG9m IHRoZSBsYXN0Cj4gPj4gZnVsbCByZXBvcnQuCj4gPj4KPiA+PiBUaGUgdmFsdWUgb2YgaHdfdGFp bCB3b3VsZCBiZSB0aGUgc2FtZSBhcyBmcm9tIHRoZSBkZWxldGVkIGxpbmUgb2YgY29kZQo+ID4+ IGFib3ZlIHRoaXMgbGluZS4KPiA+Cj4gPiBMb29rcyBsaWtlIGFnaW5nX3RhaWwgc2hvdWxkIG5v dCBiZSBuZWVkZWQgKGl0IGlzIGNvbXBsaWNhdGluZyB0aGUKPiA+IGV4cHJlc3Npb24gYW5kIGlz IG5vdCB0aGVyZSBpbiB0aGUgb3JpZ2luYWwgZXhwcmVzc2lvbikuIEFsbCB3ZSBuZWVkIGlzIGEK PiA+ICJjaXJjdWxhciBtb2R1bHVzIi4gRm9yIGV4YW1wbGUgd291bGQgdGhlIGZvbGxvd2luZyB3 b3JrPwo+Cj4gb3JpZ2luYWwgZXhwcmVzc2lvbiBhc3N1bWVzIGFsbCByZXBvcnQgc2l6ZXMgYXJl IHBvd2VycyBvZiAyIGFuZCBoZW5jZSBkb2VzCj4gbm90IG5lZWQgYSByZWZlcmVuY2UgKGxpa2Ug YWdpbmdfdGFpbCkgdG8gcm91bmRkb3duIHRoZSBod190YWlsLgo+Cj4gPgo+ID4gICAgaWYgKGh3 X3RhaWwgPCByZXBvcnRfc2l6ZSkKPiA+ICAgICAgICBod190YWlsICs9IE9BX0JVRkZFUl9TSVpF Owo+Cj4gQXNzdW1pbmcgdGhhdCB0aGlzIGNvbmRpdGlvbiBpcyBkZXRlY3RpbmcgYSByZXBvcnQg c3BsaXQgYWNyb3NzIHRoZSBPQQo+IGJ1ZmZlciBib3VuZGFyeSwgdGhlIGFib3ZlIGNoZWNrIHdp bGwgbm90IGNhdGNoIHRoZSBzcGxpdCBpbiBjYXNlcyB3aGVyZQo+IHRoZXJlIGFyZSBtdWx0aXBs ZSByZXBvcnRzIGdlbmVyYXRlZCBiZWZvcmUgd2UgcmVhZCB0aGUgaHdfdGFpbC4KPgo+ID4gICAg aHdfdGFpbCA9IHJvdW5kZG93bihod190YWlsLCByZXBvcnRfc2l6ZSk7Cj4gPgo+ID4gQW5vdGhl ciB3YXkgKGlmIHdlIHdhbnQgdG8gYXZvaWQgdGhlIGRpdmlzaW9uKSB3b3VsZCBiZSB0byBtYWlu dGFpbiBhCj4gPiBjYWNoZWQgY29weSBvZiBod190YWlsIGluIFNXIHdoaWNoIGlzIHN1Y2Nlc3Np dmVseSAoYW5kIGNpcmN1bGFybHkpCj4gPiBpbmNyZW1lbnRlZCBieSByZXBvcnRfc2l6ZSB0aWxs IGl0IGNhdGNoZXMgdXAgd2l0aCBod190YWlsIHJlYWQgZnJvbQo+ID4gSFcuIEJ1dCBwcm9iYWJs eSB0aGUgZmlyc3QgbWV0aG9kIGFib3ZlIGlzIHNpbXBsZXIuCj4KPiBhZ2luZ190YWlsIGlzIGEg Y2FjaGVkIGNvcHkgb2YgdGhlIGh3X3RhaWwgdGhhdCBhZHZhbmNlcyBvbmx5IGluIHJlcG9ydAo+ IHNpemUgaW5jcmVtZW50cy4KClVtZXNoIGlzIHJpZ2h0LCB0aGUgcHJldmlvdXMgYWdpbmdfdGFp bCBuZWVkcyB0byBiZSB0YWtlbiBpbnRvCmFjY291bnQuIEJhc2ljYWxseSB3ZSBuZWVkIHRvIHN0 YXJ0IGZyb20gdGhlIHByZXZpb3VzIGFnaW5nX3RhaWwgYW5kCmNvbnRpbnVlIGluY3JlbWVudGlu ZyBieSByZXBvcnRfc2l6ZSB0aWxsIHdlIGNhdGNoIHVwIHdpdGggdGhlIG5ldyBod190YWlsCihh dCB0aGUgcHJldmlvdXMgcmVwb3J0X3NpemUgYm91bmRhcnksIHdoaWNoIGdpdmVzIHRoZSB2YWx1 ZSBvZiB0aGUgbmV3CmFnaW5nX3RhaWwpLgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0cy5m cmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0 aW5mby9pbnRlbC1nZng=