From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tvrtko Ursulin Subject: Re: [PATCH] drm/i915/pmu: Do not assume fixed hrtimer period Date: Tue, 5 Jun 2018 12:35:36 +0100 Message-ID: <5ff983bd-b7bd-6ed8-d199-16b368bc529e@linux.intel.com> References: <20180604125239.5856-1-tvrtko.ursulin@linux.intel.com> <152811715261.7032.18163436727010567693@mail.alporthouse.com> <152811738922.7032.4810588326807036972@mail.alporthouse.com> <2547acb2-c7e0-b427-6ce8-ae1f4e8dbb2e@linux.intel.com> <152812506459.7032.17958260936760397934@mail.alporthouse.com> <6dc98aa7-e072-ac70-d314-cccb3924ed52@linux.intel.com> <152818960780.9058.9489807079671744910@mail.alporthouse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" 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 B0A4A6ECBB for ; Tue, 5 Jun 2018 11:35:39 +0000 (UTC) In-Reply-To: <152818960780.9058.9489807079671744910@mail.alporthouse.com> Content-Language: en-GB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Chris Wilson , Tvrtko Ursulin , Intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org Ck9uIDA1LzA2LzIwMTggMTA6MDYsIENocmlzIFdpbHNvbiB3cm90ZToKPiBRdW90aW5nIFR2cnRr byBVcnN1bGluICgyMDE4LTA2LTA1IDA5OjUxOjAxKQo+Pgo+PiBPbiAwNC8wNi8yMDE4IDE2OjEx LCBDaHJpcyBXaWxzb24gd3JvdGU6Cj4+PiBRdW90aW5nIFR2cnRrbyBVcnN1bGluICgyMDE4LTA2 LTA0IDE2OjAxOjI2KQo+Pj4+Cj4+Pj4gT24gMDQvMDYvMjAxOCAxNDowMywgQ2hyaXMgV2lsc29u IHdyb3RlOgo+Pj4+PiBRdW90aW5nIENocmlzIFdpbHNvbiAoMjAxOC0wNi0wNCAxMzo1OToxMikK Pj4+Pj4+IFF1b3RpbmcgVHZydGtvIFVyc3VsaW4gKDIwMTgtMDYtMDQgMTM6NTI6MzkpCj4+Pj4+ Pj4gRnJvbTogVHZydGtvIFVyc3VsaW4gPHR2cnRrby51cnN1bGluQGludGVsLmNvbT4KPj4+Pj4+ Pgo+Pj4+Pj4+IEFzIENocmlzIGhhcyBkaXNjb3ZlcmVkIG9uIGhpcyBJdnlicmlkZ2UsIGFuZCBs YXRlciBhdXRvbWF0ZWQgdGVzdCBydW5zCj4+Pj4+Pj4gaGF2ZSBjb25maXJtZWQsIG9uIG1vc3Qg b2Ygb3VyIHBsYXRmb3JtcyBocnRpbWVyIGZhY2VkIHdpdGggaGVhdnkgR1BVIGxvYWQKPj4+Pj4+ PiBjYSBvY2Nhc2lvbmFsbHkgYmVjb21lIHN1ZmZpY2llbnRseSBpbXByZWNpc2UgdG8gYWZmZWN0 IFBNVSBzYW1wbGluZwo+Pj4+Pj4KPj4+Pj4+IHMvY2EvY2FuLwo+Pj4+Pj4KPj4+Pj4+PiBjYWxj dWxhdGlvbnMuCj4+Pj4+Pj4KPj4+Pj4+PiBUaGlzIG1lYW5zIHdlIGNhbm5vdCBhc3N1bWUgc2Ft cGxpbmcgZnJlcXVlbmN5IGlzIHdoYXQgd2UgYXNrZWQgZm9yLCBidXQKPj4+Pj4+PiB3ZSBuZWVk IHRvIG1lYXN1cmUgdGhlIGludGVydmFsIG91cnNlbHZlcy4KPj4+Pj4+Pgo+Pj4+Pj4+IFRoaXMg cGF0Y2ggaXMgc2ltaWxhciB0byBDaHJpcycgb3JpZ2luYWwgcHJvcG9zYWwgZm9yIHBlci1lbmdp bmUgY291bnRlcnMsCj4+Pj4+Pj4gYnV0IGluc3RlYWQgb2YgaW50cm9kdWNpbmcgYSBuZXcgc2V0 IHRvIHdvcmsgYXJvdW5kIHRoZSBwcm9ibGVtIHdpdGgKPj4+Pj4+PiBmcmVxdWVuY3kgc2FtcGxp bmcsIGl0IHN3YXBzIGFyb3VuZCB0aGUgd2F5IGludGVybmFsIGZyZXF1ZW5jeSBhY2NvdW50aW5n Cj4+Pj4+Pj4gaXMgZG9uZS4gSW5zdGVhZCBvZiBhY2N1bXVsYXRpbmcgY3VycmVudCBmcmVxdWVu Y3kgYW5kIGRpdmlkaW5nIGJ5Cj4+Pj4+Pj4gc2FtcGxpbmcgZnJlcXVlbmN5IG9uIHJlYWRvdXQs IGl0IGFjY3VtdWxhdGVzIGZyZXF1ZW5jeSBzY2FsZWQgYnkgZWFjaAo+Pj4+Pj4+IHBlcmlvZC4K Pj4+Pj4+Cj4+Pj4+PiBNeSB1bHRlcmlvciBtb3RpdmUgZmFpbGVkIHRvIHdpbiBmYXZvdXIgOykK Pj4+Pj4+ICAgICAKPj4+Pj4+PiBUZXN0Y2FzZTogaWd0L3BlcmZfcG11LypidXN5KiAjIHNuYiwg aXZiLCBoc3cKPj4+Pj4+PiBTaWduZWQtb2ZmLWJ5OiBUdnJ0a28gVXJzdWxpbiA8dHZydGtvLnVy c3VsaW5AaW50ZWwuY29tPgo+Pj4+Pj4+IFN1Z2dlc3RlZC1ieTogQ2hyaXMgV2lsc29uIDxjaHJp c0BjaHJpcy13aWxzb24uY28udWs+Cj4+Pj4+Pj4gQ2M6IENocmlzIFdpbHNvbiA8Y2hyaXNAY2hy aXMtd2lsc29uLmNvLnVrPgo+Pj4+Pj4gUmV2aWV3ZWQtYnk6IENocmlzIFdpbHNvbiA8Y2hyaXNA Y2hyaXMtd2lsc29uLmNvLnVrPgo+Pj4+Pgo+Pj4+PiBJIHNob3VsZCBwb2ludCBvdXQgdGhhdCBl dmVuIHdpdGggdGhpcyBmaXggKG9yIHJhdGhlciBteSBwYXRjaCksIHdlIGNhbgo+Pj4+Cj4+Pj4g SSBkaWQgbm90IG1lYW4gdG8gc3RlYWwgaXQsCj4+Pgo+Pj4gRG9uJ3QgbWluZCwgSSB2aWV3IHN1 Y2ggcGF0Y2hlcyBhcyAidGhpcyBoZXJlIGlzIGEgcHJvYmxlbSwgYW5kIHdoYXQgSQo+Pj4gdGhp bmsgY2FuIGJlIGRvbmUiLiBJJ20gcXVpdGUgaGFwcHkgdG8gYmUgdG9sZCAibm9wZSwgdGhlIHBy b2JsZW0KPj4+IGlzLi4uIjsgdGhlIGVuZCByZXN1bHQgaXMgd2UgZml4IGFuZCBtb3ZlIG9uLgo+ Pj4KPj4+PiBqdXN0IHRoYXQgeW91IHNlZW1lZCB2ZXJ5IHVuY29tcHJvbWlzaW5nIG9uCj4+Pj4g dGhlIG5ldyBjb3VudGVycyBhcHByb2FjaC4gSWYgeW91IHByZWZlciB5b3UgY2FuIHJlc3BpbiB5 b3VyIHBhdGNoIHdpdGgKPj4+PiB0aGlzIGFwcHJvYWNoIGZvciBmcmVxdWVuY3kgY291bnRlcnMs IGl0IGlzIGZpbmUgYnkgbWUuCj4+Pgo+Pj4gSSdtIGp1c3Qgbm90IGNvbnZpbmNlZCB5ZXQgYnkg dGhlIGZyZXF1ZW5jeSBzYW1wbGVyLCBhbmQgSSBzdGlsbCBmZWVsCj4+PiB0aGF0IHdlIHdvdWxk IGJlIGJldHRlciBvZmYgd2l0aCBjeWNsZXMuIEkganVzdCBoYXZlbid0IGZvdW5kIGEKPj4+IGNv bnZpbmNpbmcgYXJndW1lbnQgOykKPj4KPj4gSnVzdCBpbWFnaW5lIC51bml0IGZpbGUgaGFzIE1j eWNsZXMgaW5zdGVhZCBvZiBNSHogaW4gaXQ/IDopCj4+Cj4+IFNpbmNlIHdlIHdlbnQgd2l0aCB0 aGlzIHdoZW4gYWRkaW5nIGl0IGluaXRpYWxseSBJIHRoaW5rIHdlIHNob3VsZAo+PiBleGVyY2lz ZSByZXN0cmFpbiB3aXRoIGRlcHJlY2F0aW5nIGFuZCBhZGRpbmcgc28gcXVpY2tseS4KPj4KPj4+ Pj4gc3RpbGwgc2VlIGVycm9ycyBvZiAyNS0zMHVzLCBlbm91Z2ggdG8gZmFpbCB0aGUgdGVzdC4g SG93ZXZlciwgd2l0aG91dAo+Pj4+PiB0aGUgZml4IG91ciBlcnJvcnMgY2FuIGJlIG9yZGVycyBv ZiBtYWduaXR1ZGUgd29yc2UgKGUuZy4gcmVwb3J0aW5nIDgwdXMKPj4+Pj4gYnVzeSBpbnN0ZWFk IG9mIDUwMHVzKS4KPj4+Pgo+Pj4+IFllYWggSSBndWVzcyBpZiB0aGUgdGltZXIgaXMgZGVsYXll ZCBpdCBpcyBkZWxheWVkIGFuZCBhbGwgc2FtcGxlcyBqdXN0Cj4+Pj4gd29uJ3QgYmUgdGhlcmUu IEkgZG9uJ3Qgc2VlIGEgd2F5IHRvIGZpeCB0aGF0IGVsZWdhbnRseS4gRXZlbiBwcmFjdGljYWxs eS4KPj4+Cj4+PiBSaWdodCwgYnV0IGl0J3Mgbm90IHRoZSBjYXNlIHRoYXQgd2UgYXJlbid0IHNh bXBsaW5nLiBJZiB0aGUgdGltZXIgd2FzCj4+PiBkZWxheWVkIGVudGlyZWx5LCB0aGVuIHdlIHdv dWxkIHNlZSAwIChzdGFydF92YWwgPT0gZW5kX3ZhbCkuIEknbSBub3QKPj4+IHN1cmUgZXhhY3Rs eSB3aGF0IGdvZXMgd3JvbmcsIGJ1dCBpdCBwcm9iYWJseSBpcyByZWxhdGVkIHRvIHRoZSB0aW1l cgo+Pj4gYmVpbmcgdW5yZWxpYWJsZS4gKEl0J3MganVzdCB0aGF0IGluIHRoaXMgY2FzZSB3ZSBh cmUgc2FtcGxpbmcgYSBzcXVhcmUKPj4+IHdhdmUsIHNvIHJlYWxseSBoYXJkIHRvIG1lc3MgdXAh KQo+Pgo+PiBIbSBtYXliZSBJIGFtIG5vdCBjb21wbGV0ZWx5IHVuZGVyc3RhbmRpbmcgd2hhdCB5 b3UgYXJlIHNlZWluZy4gSQo+PiB0aG91Z2h0IHRoYXQgbXVsdGlwbGUgdGltZXIgb3ZlcnJ1bnMg KGFrYSBkZWxheSBieSBtdWx0aXBsZSBwZXJpb2RzKQo+PiBleHBsYWlucyBldmVyeXRoaW5nLgo+ Pgo+PiBUaGVuIGV2ZW4gd2l0aCB0aGlzIHBhdGNoLCBpZiB0aGV5IGhhcHBlbiB0byBoYXBwZW4g KCEpIGF0IHRoZSBlbmQgb2YgYQo+PiBzYW1wbGluZyBwZXJpb2QgKGFzIG9ic2VydmVkIGZyb20g dXNlcnNwYWNlKSwgdGhlIGxhcmdlLWlzaCBjaHVuayBvZgo+PiBzYW1wbGVzIChkZXBlbmRpbmcg b24gdGhlIHRvdGFsIHNhbXBsaW5nIGR1cmF0aW9uKSBtYXkgYmUgbWlzc2luZwo+PiAocGVuZGlu ZyksIGFuZCBzbyBzdGlsbCBvYnNlcnZlZCBhcyBmYWlsLgo+IAo+IEFoLCBidXQgd2l0aCB0aGUg dmFyaWFibGUgcGVyaW9kIHRpbWVyLCB3ZSBjYW4gZm9yY2UgdGhlIHRpbWVyIHRvIHJ1bgo+IGJl Zm9yZSB3ZSByZXBvcnQgdGhlIHNhbWUgKHBlcmZfZXZlbnRfcmVhZCkuIFRoYXQgc2hvdWxkIHN0 aWxsIGJlIGluc2lkZQo+IHRoZSBidXN5IGJhdGNoLCBhbmQgaXQgc3RpbGwgaGFzIHRoZSBzYW1l IGluYWNjdXJhY3kgKH4yOHVzLCBmb3IgYSBzbWFsbAo+IHNhbXBsZSBwb3B1bGF0aW9uLCBtZWFz dXJpbmcgdGhlIHRhaWwgaXMgb24gbXkgd2lzaGxpc3QpOgo+IAo+IGRpZmYgLS1naXQgYS9kcml2 ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X3BtdS5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9w bXUuYwo+IGluZGV4IDM0Y2M2ZjYuLjg1NzhhNDMgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUv ZHJtL2k5MTUvaTkxNV9wbXUuYwo+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfcG11 LmMKPiBAQCAtNTM0LDYgKzUzNCwxMiBAQCBzdGF0aWMgdTY0IF9faTkxNV9wbXVfZXZlbnRfcmVh ZChzdHJ1Y3QgcGVyZl9ldmVudCAqZXZlbnQpCj4gICAgICAgICAgICAgICAgICAgICAgICAgIHZh bCA9IGVuZ2luZS0+cG11LnNhbXBsZVtzYW1wbGVdLmN1cjsKPiAgICAgICAgICAgICAgICAgIH0K PiAgICAgICAgICB9IGVsc2Ugewo+ICsgICAgICAgICAgICAgICBpZiAoaHJ0aW1lcl9jYW5jZWwo Jmk5MTUtPnBtdS50aW1lcikpIHsKPiArICAgICAgICAgICAgICAgICAgICAgICBpOTE1X3NhbXBs ZSgmaTkxNS0+cG11LnRpbWVyKTsKPiArICAgICAgICAgICAgICAgICAgICAgICBocnRpbWVyX3N0 YXJ0X2V4cGlyZXMoJmk5MTUtPnBtdS50aW1lciwKPiArICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgSFJUSU1FUl9NT0RFX0FCUyk7Cj4gKyAgICAgICAgICAgICAg IH0KPiArCj4gICAgICAgICAgICAgICAgICBzd2l0Y2ggKGV2ZW50LT5hdHRyLmNvbmZpZykgewo+ ICAgICAgICAgICAgICAgICAgY2FzZSBJOTE1X1BNVV9BQ1RVQUxfRlJFUVVFTkNZOgo+ICAgICAg ICAgICAgICAgICAgICAgICAgICB2YWwgPQo+IAo+IEkgaGF2ZW4ndCB0aG91Z2h0IGVub3VnaCBh Ym91dCB3aHkgaXQgc3RpbGwgaGFzIHRoZSB0YWlsLCBub3IgbWVhc3VyZWQKPiB0aGUgZGlzdHJp YnV0aW9uIG9mIGVycm9ycy4gSSBqdXN0IHdhbnRlZCB0byB3YXJuIHRoYXQgd2UgYXJlbid0IHF1 aXRlCj4gb3V0IG9mIHRoZSBtZXNzIHlldCwgYnV0IHdpdGggdGhlIHZhcmlhYmxlIHBlcmlvZCB3 ZSBkbyBhdCBsZWFzdCBzdG9wCj4gZHJvd25pbmchCj4gICAKPj4gRGlmZmVyZW5jZSBhZnRlciB0 aGlzIHBhdGNoIGlzIHRoYXQgd2hlbiB0aGUgdGltZXIgZXZlbnR1YWxseSBmaXJlcyB0aGUKPj4g Y291bnRlciB3aWxsIGFjY291bnQgZm9yIHRoZSBkZWxheSwgc28gc3Vic2VxdWVudCBxdWVyaWVz IHdpbGwgYmUgZmluZS4KPj4KPj4gSWYgcmVhZG91dCB3YWl0ZWQgZm9yIGF0IGxlYXN0IG9uZSBy ZWFsIHNhbXBsaW5nIHRpY2ssIHRoYXQgd291bGQgYmUKPj4gc29sdmVkLCBidXQgYnkgc2V2ZXJh bCBvdGhlciB1bmRlc2lyYWJsZSBjb25zZXF1ZW5jZXMuIEF0IGxlYXN0IGl0IHdvdWxkCj4+IHNs b3cgZG93biByZWFkb3V0IHRvIDIwMEh6LCBidXQgSSBkb24ndCBrbm93IGZyb20gdGhlIHRvcCBv ZiBteSBoZWFkIGlmCj4+IGl0IHdvdWxkbid0IGNhdXNlIHVucmVzb2x2YWJsZSBkZWFkbG9jayBz Y2VuYXJpb3MsIG9yIGlmIGl0IGlzIGV2ZW4KPj4gdGVjaG5pY2FsbHkgcG9zc2libGUgd2l0aGlu IHRoZSBwZXJmIGZyYW1ld29yay4gKFNpbmNlIEkgZG9uJ3QgdGhpbmsgd2UKPj4gc2hvdWxkIGRv IGl0LCBJIGRvbid0IGV2ZW4gd2FudCB0byBzdGFydCB0aGlua2luZyBhYm91dCBpdC4pCj4gCj4g VGhlIGFib3ZlIHNob3VsZCBkbyB3aGF0IHlvdSBoYXZlIGluIG1pbmQsIEkgdGhpbmsuCgpJIHRo aW5rIHlvdSBnb3QgdGhlIGRpcmVjdCBzYW1wbGUgY3JpdGVyaWEgcmV2ZXJzZWQgLSBpdCBzaG91 bGQgYmUgaWYgCnRoZSB0aW1lciB3YXNuJ3QgcnVubmluZy4gSWYgaXQgd2FzLCBpdCBtZWFucyBp dCBqdXN0IGV4aXRlZCBzbyB0aGUgCnNhbXBsaW5nIGlzIGFscmVhZHkgcHJldHR5IGNvcnJlY3Qu CgpBbHNvIEkgd2FzIHRoaW5raW5nIHRvIGxpbWl0IHRoaXMgdG8gb25seSB3aGVuIHRoZSB0aW1l ciB3YXMgZGVsYXllZCBzbyAKdG8gYXZvaWQgbWFueSByZWFkZXJzIG1lc3Npbmcgd2l0aCBhIHNp bmdsZSB0aW1lciBhbG90LiBJbiBvdGhlciB3b3JkcyAKd2Uga2VlcCBhY2NlcHRpbmcgdG8gc2Ft cGxpbmcgZXJyb3IgKDAuNSUgSSB0aGluayBmb3Igc3RlYWR5IGlucHV0IApzaWduYWwpIGFuZCBv bmx5IGltcHJvdmUgdGhlIHRpbWVyIGRlbGF5IHNjZW5hcmlvLgoKSSdsbCBzZW5kIGEgcGF0Y2gg dG8gZGlzY3VzcyBvbiBpbiBtb3JlIGRldGFpbC4KClJlZ2FyZHMsCgpUdnJ0a28KX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcg bGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRl c2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==