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 09:51:01 +0100 Message-ID: <6dc98aa7-e072-ac70-d314-cccb3924ed52@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> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTPS id 5A0C46E1EC for ; Tue, 5 Jun 2018 08:51:04 +0000 (UTC) In-Reply-To: <152812506459.7032.17958260936760397934@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 Ck9uIDA0LzA2LzIwMTggMTY6MTEsIENocmlzIFdpbHNvbiB3cm90ZToKPiBRdW90aW5nIFR2cnRr byBVcnN1bGluICgyMDE4LTA2LTA0IDE2OjAxOjI2KQo+Pgo+PiBPbiAwNC8wNi8yMDE4IDE0OjAz LCBDaHJpcyBXaWxzb24gd3JvdGU6Cj4+PiBRdW90aW5nIENocmlzIFdpbHNvbiAoMjAxOC0wNi0w NCAxMzo1OToxMikKPj4+PiBRdW90aW5nIFR2cnRrbyBVcnN1bGluICgyMDE4LTA2LTA0IDEzOjUy OjM5KQo+Pj4+PiBGcm9tOiBUdnJ0a28gVXJzdWxpbiA8dHZydGtvLnVyc3VsaW5AaW50ZWwuY29t Pgo+Pj4+Pgo+Pj4+PiBBcyBDaHJpcyBoYXMgZGlzY292ZXJlZCBvbiBoaXMgSXZ5YnJpZGdlLCBh bmQgbGF0ZXIgYXV0b21hdGVkIHRlc3QgcnVucwo+Pj4+PiBoYXZlIGNvbmZpcm1lZCwgb24gbW9z dCBvZiBvdXIgcGxhdGZvcm1zIGhydGltZXIgZmFjZWQgd2l0aCBoZWF2eSBHUFUgbG9hZAo+Pj4+ PiBjYSBvY2Nhc2lvbmFsbHkgYmVjb21lIHN1ZmZpY2llbnRseSBpbXByZWNpc2UgdG8gYWZmZWN0 IFBNVSBzYW1wbGluZwo+Pj4+Cj4+Pj4gcy9jYS9jYW4vCj4+Pj4KPj4+Pj4gY2FsY3VsYXRpb25z Lgo+Pj4+Pgo+Pj4+PiBUaGlzIG1lYW5zIHdlIGNhbm5vdCBhc3N1bWUgc2FtcGxpbmcgZnJlcXVl bmN5IGlzIHdoYXQgd2UgYXNrZWQgZm9yLCBidXQKPj4+Pj4gd2UgbmVlZCB0byBtZWFzdXJlIHRo ZSBpbnRlcnZhbCBvdXJzZWx2ZXMuCj4+Pj4+Cj4+Pj4+IFRoaXMgcGF0Y2ggaXMgc2ltaWxhciB0 byBDaHJpcycgb3JpZ2luYWwgcHJvcG9zYWwgZm9yIHBlci1lbmdpbmUgY291bnRlcnMsCj4+Pj4+ IGJ1dCBpbnN0ZWFkIG9mIGludHJvZHVjaW5nIGEgbmV3IHNldCB0byB3b3JrIGFyb3VuZCB0aGUg cHJvYmxlbSB3aXRoCj4+Pj4+IGZyZXF1ZW5jeSBzYW1wbGluZywgaXQgc3dhcHMgYXJvdW5kIHRo ZSB3YXkgaW50ZXJuYWwgZnJlcXVlbmN5IGFjY291bnRpbmcKPj4+Pj4gaXMgZG9uZS4gSW5zdGVh ZCBvZiBhY2N1bXVsYXRpbmcgY3VycmVudCBmcmVxdWVuY3kgYW5kIGRpdmlkaW5nIGJ5Cj4+Pj4+ IHNhbXBsaW5nIGZyZXF1ZW5jeSBvbiByZWFkb3V0LCBpdCBhY2N1bXVsYXRlcyBmcmVxdWVuY3kg c2NhbGVkIGJ5IGVhY2gKPj4+Pj4gcGVyaW9kLgo+Pj4+Cj4+Pj4gTXkgdWx0ZXJpb3IgbW90aXZl IGZhaWxlZCB0byB3aW4gZmF2b3VyIDspCj4+Pj4gICAgCj4+Pj4+IFRlc3RjYXNlOiBpZ3QvcGVy Zl9wbXUvKmJ1c3kqICMgc25iLCBpdmIsIGhzdwo+Pj4+PiBTaWduZWQtb2ZmLWJ5OiBUdnJ0a28g VXJzdWxpbiA8dHZydGtvLnVyc3VsaW5AaW50ZWwuY29tPgo+Pj4+PiBTdWdnZXN0ZWQtYnk6IENo cmlzIFdpbHNvbiA8Y2hyaXNAY2hyaXMtd2lsc29uLmNvLnVrPgo+Pj4+PiBDYzogQ2hyaXMgV2ls c29uIDxjaHJpc0BjaHJpcy13aWxzb24uY28udWs+Cj4+Pj4gUmV2aWV3ZWQtYnk6IENocmlzIFdp bHNvbiA8Y2hyaXNAY2hyaXMtd2lsc29uLmNvLnVrPgo+Pj4KPj4+IEkgc2hvdWxkIHBvaW50IG91 dCB0aGF0IGV2ZW4gd2l0aCB0aGlzIGZpeCAob3IgcmF0aGVyIG15IHBhdGNoKSwgd2UgY2FuCj4+ Cj4+IEkgZGlkIG5vdCBtZWFuIHRvIHN0ZWFsIGl0LAo+IAo+IERvbid0IG1pbmQsIEkgdmlldyBz dWNoIHBhdGNoZXMgYXMgInRoaXMgaGVyZSBpcyBhIHByb2JsZW0sIGFuZCB3aGF0IEkKPiB0aGlu ayBjYW4gYmUgZG9uZSIuIEknbSBxdWl0ZSBoYXBweSB0byBiZSB0b2xkICJub3BlLCB0aGUgcHJv YmxlbQo+IGlzLi4uIjsgdGhlIGVuZCByZXN1bHQgaXMgd2UgZml4IGFuZCBtb3ZlIG9uLgo+IAo+ PiBqdXN0IHRoYXQgeW91IHNlZW1lZCB2ZXJ5IHVuY29tcHJvbWlzaW5nIG9uCj4+IHRoZSBuZXcg Y291bnRlcnMgYXBwcm9hY2guIElmIHlvdSBwcmVmZXIgeW91IGNhbiByZXNwaW4geW91ciBwYXRj aCB3aXRoCj4+IHRoaXMgYXBwcm9hY2ggZm9yIGZyZXF1ZW5jeSBjb3VudGVycywgaXQgaXMgZmlu ZSBieSBtZS4KPiAKPiBJJ20ganVzdCBub3QgY29udmluY2VkIHlldCBieSB0aGUgZnJlcXVlbmN5 IHNhbXBsZXIsIGFuZCBJIHN0aWxsIGZlZWwKPiB0aGF0IHdlIHdvdWxkIGJlIGJldHRlciBvZmYg d2l0aCBjeWNsZXMuIEkganVzdCBoYXZlbid0IGZvdW5kIGEKPiBjb252aW5jaW5nIGFyZ3VtZW50 IDspCgpKdXN0IGltYWdpbmUgLnVuaXQgZmlsZSBoYXMgTWN5Y2xlcyBpbnN0ZWFkIG9mIE1IeiBp biBpdD8gOikKClNpbmNlIHdlIHdlbnQgd2l0aCB0aGlzIHdoZW4gYWRkaW5nIGl0IGluaXRpYWxs eSBJIHRoaW5rIHdlIHNob3VsZCAKZXhlcmNpc2UgcmVzdHJhaW4gd2l0aCBkZXByZWNhdGluZyBh bmQgYWRkaW5nIHNvIHF1aWNrbHkuCgo+Pj4gc3RpbGwgc2VlIGVycm9ycyBvZiAyNS0zMHVzLCBl bm91Z2ggdG8gZmFpbCB0aGUgdGVzdC4gSG93ZXZlciwgd2l0aG91dAo+Pj4gdGhlIGZpeCBvdXIg ZXJyb3JzIGNhbiBiZSBvcmRlcnMgb2YgbWFnbml0dWRlIHdvcnNlIChlLmcuIHJlcG9ydGluZyA4 MHVzCj4+PiBidXN5IGluc3RlYWQgb2YgNTAwdXMpLgo+Pgo+PiBZZWFoIEkgZ3Vlc3MgaWYgdGhl IHRpbWVyIGlzIGRlbGF5ZWQgaXQgaXMgZGVsYXllZCBhbmQgYWxsIHNhbXBsZXMganVzdAo+PiB3 b24ndCBiZSB0aGVyZS4gSSBkb24ndCBzZWUgYSB3YXkgdG8gZml4IHRoYXQgZWxlZ2FudGx5LiBF dmVuIHByYWN0aWNhbGx5Lgo+IAo+IFJpZ2h0LCBidXQgaXQncyBub3QgdGhlIGNhc2UgdGhhdCB3 ZSBhcmVuJ3Qgc2FtcGxpbmcuIElmIHRoZSB0aW1lciB3YXMKPiBkZWxheWVkIGVudGlyZWx5LCB0 aGVuIHdlIHdvdWxkIHNlZSAwIChzdGFydF92YWwgPT0gZW5kX3ZhbCkuIEknbSBub3QKPiBzdXJl IGV4YWN0bHkgd2hhdCBnb2VzIHdyb25nLCBidXQgaXQgcHJvYmFibHkgaXMgcmVsYXRlZCB0byB0 aGUgdGltZXIKPiBiZWluZyB1bnJlbGlhYmxlLiAoSXQncyBqdXN0IHRoYXQgaW4gdGhpcyBjYXNl IHdlIGFyZSBzYW1wbGluZyBhIHNxdWFyZQo+IHdhdmUsIHNvIHJlYWxseSBoYXJkIHRvIG1lc3Mg dXAhKQoKSG0gbWF5YmUgSSBhbSBub3QgY29tcGxldGVseSB1bmRlcnN0YW5kaW5nIHdoYXQgeW91 IGFyZSBzZWVpbmcuIEkgCnRob3VnaHQgdGhhdCBtdWx0aXBsZSB0aW1lciBvdmVycnVucyAoYWth IGRlbGF5IGJ5IG11bHRpcGxlIHBlcmlvZHMpIApleHBsYWlucyBldmVyeXRoaW5nLgoKVGhlbiBl dmVuIHdpdGggdGhpcyBwYXRjaCwgaWYgdGhleSBoYXBwZW4gdG8gaGFwcGVuICghKSBhdCB0aGUg ZW5kIG9mIGEgCnNhbXBsaW5nIHBlcmlvZCAoYXMgb2JzZXJ2ZWQgZnJvbSB1c2Vyc3BhY2UpLCB0 aGUgbGFyZ2UtaXNoIGNodW5rIG9mIApzYW1wbGVzIChkZXBlbmRpbmcgb24gdGhlIHRvdGFsIHNh bXBsaW5nIGR1cmF0aW9uKSBtYXkgYmUgbWlzc2luZyAKKHBlbmRpbmcpLCBhbmQgc28gc3RpbGwg b2JzZXJ2ZWQgYXMgZmFpbC4KCkRpZmZlcmVuY2UgYWZ0ZXIgdGhpcyBwYXRjaCBpcyB0aGF0IHdo ZW4gdGhlIHRpbWVyIGV2ZW50dWFsbHkgZmlyZXMgdGhlIApjb3VudGVyIHdpbGwgYWNjb3VudCBm b3IgdGhlIGRlbGF5LCBzbyBzdWJzZXF1ZW50IHF1ZXJpZXMgd2lsbCBiZSBmaW5lLgoKSWYgcmVh ZG91dCB3YWl0ZWQgZm9yIGF0IGxlYXN0IG9uZSByZWFsIHNhbXBsaW5nIHRpY2ssIHRoYXQgd291 bGQgYmUgCnNvbHZlZCwgYnV0IGJ5IHNldmVyYWwgb3RoZXIgdW5kZXNpcmFibGUgY29uc2VxdWVu Y2VzLiBBdCBsZWFzdCBpdCB3b3VsZCAKc2xvdyBkb3duIHJlYWRvdXQgdG8gMjAwSHosIGJ1dCBJ IGRvbid0IGtub3cgZnJvbSB0aGUgdG9wIG9mIG15IGhlYWQgaWYgCml0IHdvdWxkbid0IGNhdXNl IHVucmVzb2x2YWJsZSBkZWFkbG9jayBzY2VuYXJpb3MsIG9yIGlmIGl0IGlzIGV2ZW4gCnRlY2hu aWNhbGx5IHBvc3NpYmxlIHdpdGhpbiB0aGUgcGVyZiBmcmFtZXdvcmsuIChTaW5jZSBJIGRvbid0 IHRoaW5rIHdlIApzaG91bGQgZG8gaXQsIEkgZG9uJ3QgZXZlbiB3YW50IHRvIHN0YXJ0IHRoaW5r aW5nIGFib3V0IGl0LikKClJlZ2FyZHMsCgpUdnJ0a28KX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhA bGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxt YW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==