From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 1/3] drm: Defer disabling the vblank IRQ until the next interrupt (for instant-off) Date: Wed, 22 Mar 2017 22:02:00 +0200 Message-ID: <20170322200200.GM31595@intel.com> References: <20170315204027.20160-1-chris@chris-wilson.co.uk> <20170315210031.GY31595@intel.com> <08524408-b648-7c67-9ea4-a8c58f92f424@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <08524408-b648-7c67-9ea4-a8c58f92f424@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Mario Kleiner Cc: Michel =?iso-8859-1?Q?D=E4nzer?= , dri-devel@lists.freedesktop.org, Laurent Pinchart , Dave Airlie , intel-gfx@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org T24gV2VkLCBNYXIgMjIsIDIwMTcgYXQgMDQ6MDY6MzJQTSArMDEwMCwgTWFyaW8gS2xlaW5lciB3 cm90ZToKPiBPbiAwMy8xNS8yMDE3IDEwOjAwIFBNLCBWaWxsZSBTeXJqw6Rsw6Qgd3JvdGU6Cj4g PiBPbiBXZWQsIE1hciAxNSwgMjAxNyBhdCAwODo0MDoyNVBNICswMDAwLCBDaHJpcyBXaWxzb24g d3JvdGU6Cj4gPj4gT24gdmJsYW5rIGluc3RhbnQtb2ZmIHN5c3RlbXMsIHdlIGNhbiBnZXQgaW50 byBhIHNpdHVhdGlvbiB3aGVyZSB0aGUgY29zdAo+ID4+IG9mIGVuYWJsaW5nIGFuZCBkaXNhYmxp bmcgdGhlIHZibGFuayBJUlEgYXJvdW5kIGEgZHJtV2FpdFZibGFuayBxdWVyeQo+ID4+IGRvbWlu YXRlcy4gQW5kIHdpdGggdGhlIGFkdmVudCBvZiBldmVuIGRlZXBlciBoYXJkd2FyZSBzbGVlcCBz dGF0ZSwKPiA+PiB0b3VjaGluZyByZWdpc3RlcnMgYmVjb21lcyBldmVyIG1vcmUgZXhwZW5zaXZl LiAgSG93ZXZlciwgd2Uga25vdyB0aGF0IGlmCj4gPj4gdGhlIHVzZXIgd2FudHMgdGhlIGN1cnJl bnQgdmJsYW5rIGNvdW50ZXIsIHRoZXkgYXJlIGFsc28gdmVyeSBsaWtlbHkgdG8KPiA+PiBpbW1l ZGlhdGVseSBxdWV1ZSBhIHZibGFuayB3YWl0IGFuZCBzbyB3ZSBjYW4ga2VlcCB0aGUgaW50ZXJy dXB0IGFyb3VuZAo+ID4+IGFuZCBvbmx5IHR1cm4gaXQgb2ZmIGlmIHdlIGhhdmUgbm8gZnVydGhl ciB2YmxhbmsgcmVxdWVzdHMgcXVldWVkIHdpdGhpbgo+ID4+IHRoZSBpbnRlcnJ1cHQgaW50ZXJ2 YWwuCj4gPj4KPiA+PiBBZnRlciB2YmxhbmsgZXZlbnQgZGVsaXZlcnksIHRoaXMgcGF0Y2ggYWRk cyBhIHNoYWRvdyBvZiBvbmUgdmJsYW5rIHdoZXJlCj4gPj4gdGhlIGludGVycnVwdCBpcyBrZXB0 IGFsaXZlIGZvciB0aGUgdXNlciB0byBxdWVyeSBhbmQgcXVldWUgYW5vdGhlciB2YmxhbmsKPiA+ PiBldmVudC4gU2ltaWxhcmx5LCBpZiB0aGUgdXNlciBpcyB1c2luZyBibG9ja2luZyBkcm1XYWl0 VmJsYW5rcywgdGhlCj4gPj4gaW50ZXJydXB0IHdpbGwgYmUgZGlzYWJsZWQgb24gdGhlIElSUSBm b2xsb3dpbmcgdGhlIHdhaXQgY29tcGxldGlvbi4KPiA+PiBIb3dldmVyLCBpZiB0aGUgdXNlciBp cyBzaW1wbHkgcXVlcnlpbmcgdGhlIGN1cnJlbnQgdmJsYW5rIGNvdW50ZXIgYW5kCj4gPj4gdGlt ZXN0YW1wLCB0aGUgaW50ZXJydXB0IHdpbGwgYmUgZGlzYWJsZWQgYWZ0ZXIgZXZlcnkgSVJRIGFu ZCB0aGUgdXNlcgo+ID4+IHdpbGwgZW5hYmxlZCBpdCBhZ2FpbiBvbiB0aGUgZmlyc3QgcXVlcnkg Zm9sbG93aW5nIHRoZSBJUlEuCj4gPj4KPiA+PiB2MjogTWFyaW8gS2xlaW5lciAtCj4gPj4gQWZ0 ZXIgdGVzdGluZyB0aGlzLCBvbmUgbW9yZSB0aGluZyB0aGF0IHdvdWxkIG1ha2Ugc2Vuc2UgaXMg dG8gbW92ZQo+ID4+IHRoZSBkaXNhYmxlIGJsb2NrIGF0IHRoZSBlbmQgb2YgZHJtX2hhbmRsZV92 YmxhbmsoKSBpbnN0ZWFkIG9mIGF0IHRoZQo+ID4+IHRvcC4KPiA+Pgo+ID4+IFR1cm5zIG91dCB0 aGF0IGlmIGhpZ2ggcHJlY2lzaW9uIHRpbWVzdGFtaW5nIGlzIGRpc2FibGVkIG9yIGRvZXNuJ3QK PiA+PiB3b3JrIGZvciBzb21lIHJlYXNvbiAoYXMgY2FuIGJlIHNpbXVsYXRlZCBieSBlY2hvIDAg Pgo+ID4+IC9zeXMvbW9kdWxlL2RybS9wYXJhbWV0ZXJzL3RpbWVzdGFtcF9wcmVjaXNpb25fdXNl YyksIHRoZW4gd2l0aCB5b3VyCj4gPj4gZGVsYXllZCBkaXNhYmxlIGNvZGUgYXQgaXRzIGN1cnJl bnQgcGxhY2UsIHRoZSB2YmxhbmsgY291bnRlciB3b24ndAo+ID4+IGluY3JlbWVudCBhbnltb3Jl IGF0IGFsbCBmb3IgaW5zdGFudCBxdWVyaWVzLCBpZS4gd2l0aCB5b3VyIG90aGVyCj4gPj4gImlu c3RhbnQgcXVlcnkiIHBhdGNoZXMuIENsaWVudHMgd2hpY2ggcmVwZWF0ZWRseSBxdWVyeSB0aGUg Y291bnRlcgo+ID4+IGFuZCB3YWl0IGZvciBpdCB0byBwcm9ncmVzcyB3aWxsIHNpbXBseSBoYW5n LCBzcGlubmluZyBpbiBhbiBlbmRsZXNzCj4gPj4gcXVlcnkgbG9vcC4gVGhlcmUncyB0aGF0IGNv bW1lbnQgaW4gdmJsYW5rX2Rpc2FibGVfYW5kX3NhdmU6Cj4gPj4KPiA+PiAiKiBTa2lwIHRoaXMg c3RlcCBpZiB0aGVyZSBpc24ndCBhbnkgaGlnaCBwcmVjaXNpb24gdGltZXN0YW1wCj4gPj4gICog YXZhaWxhYmxlLiBJbiB0aGF0IGNhc2Ugd2UgY2FuJ3QgYWNjb3VudCBmb3IgdGhpcyBhbmQganVz dAo+ID4+ICAqIGhvcGUgZm9yIHRoZSBiZXN0Lgo+ID4+ICAqLwo+ID4+Cj4gPj4gV2l0aCB0aGUg ZGlzYWJsZSBoYXBwZW5pbmcgYWZ0ZXIgbGVhZGluZyBlZGdlIG9mIHZibGFuayAoPT0gaHcgY291 bnRlcgo+ID4+IGluY3JlbWVudCBhbHJlYWR5IGhhcHBlbmVkKSBidXQgYmVmb3JlIHRoZSB2Ymxh bmsgY291bnRlci90aW1lc3RhbXAKPiA+PiBoYW5kbGluZyBpbiBkcm1faGFuZGxlX3ZibGFuaywg dGhhdCBzdGVwIGlzIG5lZWRlZCB0byBrZWVwIHRoZSBjb3VudGVyCj4gPj4gcHJvZ3Jlc3Npbmcs IHNvIHNraXBwaW5nIGl0IGlzIGJhZC4KPiA+Pgo+ID4+IE5vdyB3aXRob3V0IGhpZ2ggcHJlY2lz aW9uIHRpbWVzdGFtcGluZyBzdXBwb3J0LCBhIGttcyBkcml2ZXIgbXVzdCBub3QKPiA+PiBzZXQg ZGV2LT52YmxhbmtfZGlzYWJsZV9pbW1lZGlhdGUgPSB0cnVlLCBhcyB0aGlzIHdvdWxkIGNhdXNl IHByb2JsZW1zCj4gPj4gZm9yIGNsaWVudHMsIHNvIHRoaXMgc2hvdWxkbid0IG1hdHRlciwgYnV0 IGl0IHdvdWxkIGJlIGdvb2QgdG8gc3RpbGwKPiA+PiBtYWtlIHRoaXMgcm9idXN0IGFnYWluc3Qg YSBmdXR1cmUga21zIGRyaXZlciB3aGljaCBtaWdodCBoYXZlCj4gPj4gdW5yZWxpYWJsZSBoaWdo IHByZWNpc2lvbiB0aW1lc3RhbXBpbmcsIGUuZy4sIGhpZ2ggcHJlY2lzaW9uCj4gPj4gdGltZXN0 YW1waW5nIHRoYXQgaW50ZXJtaXR0ZW50bHkgZG9lc24ndCB3b3JrLgo+ID4+Cj4gPj4gdjM6IFBh dGNoIGJlZm9yZSBjb2ZmZWUgbmVlZHMgZXh0cmEgY29mZmVlLgo+ID4+Cj4gPj4gVGVzdGNhc2U6 IGlndC9rbXNfdmJsYW5rCj4gPj4gU2lnbmVkLW9mZi1ieTogQ2hyaXMgV2lsc29uIDxjaHJpc0Bj aHJpcy13aWxzb24uY28udWs+Cj4gPj4gQ2M6IFZpbGxlIFN5cmrDpGzDpCA8dmlsbGUuc3lyamFs YUBsaW51eC5pbnRlbC5jb20+Cj4gPj4gQ2M6IERhbmllbCBWZXR0ZXIgPGRhbmllbEBmZndsbC5j aD4KPiA+PiBDYzogTWljaGVsIETDpG56ZXIgPG1pY2hlbEBkYWVuemVyLm5ldD4KPiA+PiBDYzog TGF1cmVudCBQaW5jaGFydCA8bGF1cmVudC5waW5jaGFydEBpZGVhc29uYm9hcmQuY29tPgo+ID4+ IENjOiBEYXZlIEFpcmxpZSA8YWlybGllZEByZWRoYXQuY29tPiwKPiA+PiBDYzogTWFyaW8gS2xl aW5lciA8bWFyaW8ua2xlaW5lci5kZUBnbWFpbC5jb20+Cj4gPgo+ID4gWWVwLiBUaGlzIHNlZW1z IGxpa2UgYSBnb29kIGlkZWEgdG8gbWUuIEkganVzdCBuZWdsZWN0ZWQgdG8gcmV2aWV3IGl0Cj4g PiBsYXN0IHRpbWUgYXJvdW5kIChhbmQgbWF5YmUgZXZlbiBiZWZvcmUgdGhhdD8pIGZvciBzb21l IHJlYXNvbi4gTG9ja3MKPiA+IHNlZW0gdG8gYmUgdGFrZW4gaW4gdGhlIHJpZ2h0IG9yZGVyLCBz byBpdCBhdCBsZWFzdCBsb29rcyBzYWZlIHRvIG1lLgo+ID4KPiA+IFJldmlld2VkLWJ5OiBWaWxs ZSBTeXJqw6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29tPgo+ID4KPiAKPiBIaSwK PiAKPiBhcyBhIGZvbGxvd3VwIHRvIHRoaXMgb25lLCBtYXliZSB3ZSBzaG91bGQgbW92ZSB0aGUg Cj4gZHJtX2hhbmRsZV92YmxhbmtfZXZlbnRzKGRldiwgcGlwZSk7IGRvd24sIGltbWVkaWF0ZWx5 IGFmdGVyIENocmlzIG5ldyAKPiBkZWxheWVkIGRpc2FibGUgY29kZT8KPiAKPiBUaGUgaWRlYSB3 YXMgdG8gYXZvaWQgbG90cyBvZiByZWR1bmRhbnQgZW5hYmxlLT5kaXNhYmxlLT5lbmFibGUuLi4g Y2FsbHMgCj4gYnkgaGF2aW5nIHNvbWUgMSBmcmFtZSBkZWxheSBiZWZvcmUgZGlzYWJsZS4gVGhp cyB3b3JrcyBmb3IgcHVyZSB2YmxhbmsgCj4gY291bnQvdHMgcXVlcmllcy4KPiAKPiBCdXQgYm90 aCBEUkkyIGFuZCBEUkkzL1ByZXNlbnQgdXNlIHZibGFuayBldmVudHMgdG8gdHJpZ2dlciBhIAo+ IHBhZ2VmbGlwLWlvY3RsIGF0IHRoZSByaWdodCB0YXJnZXQgdmJsYW5rLiBXaXRoIHRoZSBjdXJy ZW50IG9yZGVyaW5nIHdlIAo+IG1heSBkaXNwYXRjaCB0aGUgdmJsYW5rIHN3YXAgdHJpZ2dlciBl dmVudCB0byB0aGUgWC1TZXJ2ZXIgYW5kIGRyb3AgdGhlIAo+IHZibGFuayByZWZjb3VudCB0byB6 ZXJvIGR1ZSB0byB0aGUgdmJsYW5rX3B1dCBpbnNpZGUgCj4gZHJtX2hhbmRsZV92YmxhbmtfZXZl bnRzIGZvciB0aGUgZGlzcGF0Y2hlZCBldmVudCwgdGhlbiBkZXRlY3QgaW4gdGhpcyAKPiBwYXRj aCB0aGF0IHJlZmNvdW50ID09IDAgYW5kIGRpc2FibGUgdmJsYW5rcywgYnV0IGEgZmV3IG1pY3Jv c2Vjb25kcyAKPiBsYXRlciB0aGUgc2VydmVyIHdpbGwgcXVldWUgYSBwYWdlZmxpcCBpb2N0bCB3 aGljaCBidW1wcyB0aGUgcmVmY291bnQgCj4gYW5kIHJlZW5hYmxlcyB2YmxhbmsgaXJxcywgc28g d2UgaGF2ZSBhIHJlZHVuZGFudCBkaXNhYmxlLT5lbmFibGUuCj4gCj4gQWxzbyBtYW55IGttcyBk cml2ZXJzIG5vdyB1c2UgZHJtX2NydGNfYXJtX3ZibGFua19ldmVudCgpIGZvciBwYWdlZmxpcCAK PiBjb21wbGV0aW9uIGhhbmRsaW5nIGF0IHZibGFuaywgdGhlIHBhZ2VmbGlwIGNvbXBsZXRpb24g ZXZlbnRzIGFyZSBhbHNvIAo+IGRpc3BhdGNoZWQgdmlhIGRybV9oYW5kbGVfdmJsYW5rX2V2ZW50 cygpLiBBZnRlciBhIHBhZ2VmbGlwIGNvbXBsZXRlcywgCj4gaXQgbWFrZXMgc2Vuc2UgdG8gaGF2 ZSB0aGlzICJzd2FwIHNoYWRvdyIgb2YgMSBmdWxsIGZyYW1lLCBhcyBhbmltYXRpb25zIAo+IHdv dWxkIGxpa2VseSBxdWV1ZSBhIG5ldyB2YmxhbmsgcXVlcnkvZXZlbnQgaW1tZWRpYXRlbHkgZm9y IHRoZSBuZXh0IAo+IGFuaW1hdGlvbiBmcmFtZS4KClRoYXQgZG9lcyBzZWVtIGxpa2UgYSBkZWNl bnQgaWRlYS4gSXQgd29uJ3QgYWN0dWFsbHkgY2hhbmdlIGFueXRoaW5nIGZvcgppOTE1IHBhZ2Ug ZmxpcHMgc2luY2Ugd2Ugc3RpbGwgaGFuZyBvbiB0byBvdXIgdmJsYW5rIHJlZmVyZW5jZSBhZnRl cgpkcm1faGFuZGxlX3ZibGFuaygpIHJldHVybnMuIEJ1dCBpZiB5b3UsIGZvciBleGFtcGxlLCBq dXN0IGNhbGwKZ2xYV2FpdFZpZGVvU3luY1NHSSgxLDAsLi4uKSBpbiBhIGxvb3AgdGhlIGN1cnJl bnQgY29kZSB3aWxsIHN0aWxsCnJlc3VsdCBvbiBlbmFibGU8LT5kaXNhYmxlIHBpbmctcG9uZywg d2hlcmVhcyB3aXRoIHlvdXIgcHJvcG9zZWQKcmVvcmRlcmluZyB3ZSdkIGtlZXAgdGhlIHZibGFu ayBpbnRlcnJ1cHQgZW5hYmxlZCBhbGwgdGhlIHRpbWUuCgpDaHJpcywgYW55IHRob3VnaHRzPwoK LS0gClZpbGxlIFN5cmrDpGzDpApJbnRlbCBPVEMKX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlz dHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4v bGlzdGluZm8vaW50ZWwtZ2Z4Cg==