From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH] drm/vblank: Fix race between drm_crtc_arm_vblank_event() and the irq Date: Fri, 30 Jun 2017 16:54:46 +0300 Message-ID: <20170630135446.GY12629@intel.com> References: <20170630122729.16026-1-ville.syrjala@linux.intel.com> <20170630131251.dspibjhjbr5nw4lr@phenom.ffwll.local> <20170630132649.GW12629@intel.com> <20170630133650.5enqocsymovy24np@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" 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 A40D46E7FC for ; Fri, 30 Jun 2017 13:54:49 +0000 (UTC) Content-Disposition: inline In-Reply-To: <20170630133650.5enqocsymovy24np@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Daniel Vetter Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org T24gRnJpLCBKdW4gMzAsIDIwMTcgYXQgMDM6MzY6NTBQTSArMDIwMCwgRGFuaWVsIFZldHRlciB3 cm90ZToKPiBPbiBGcmksIEp1biAzMCwgMjAxNyBhdCAwNDoyNjo0OVBNICswMzAwLCBWaWxsZSBT eXJqw6Rsw6Qgd3JvdGU6Cj4gPiBPbiBGcmksIEp1biAzMCwgMjAxNyBhdCAwMzoxMjo1MVBNICsw MjAwLCBEYW5pZWwgVmV0dGVyIHdyb3RlOgo+ID4gPiBPbiBGcmksIEp1biAzMCwgMjAxNyBhdCAw MzoyNzoyOVBNICswMzAwLCB2aWxsZS5zeXJqYWxhQGxpbnV4LmludGVsLmNvbSB3cm90ZToKPiA+ ID4gPiBGcm9tOiBWaWxsZSBTeXJqw6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29t Pgo+ID4gPiA+IAo+ID4gPiA+IE1ha2Ugc3VyZSB0aGUgYXJtZWQgZXZlbiBkb2Vzbid0IGZpcmUg dW50aWwgdGhlIG5leHQgdmJsYW5rIGJ5IGFkZGluZwo+ID4gPiA+IG9uZSB0byB0aGUgY3VycmVu dCB2YmxhbmsgY291bnRlciB2YWx1ZS4gVGhpcyB3aWxsIHByZXZlbnQgdGhlIGV2ZW50Cj4gPiA+ ID4gYmVpbmcgZmlyZWQgb2ZmIHByZW1hdHVyZWx5IGlmIGRybV9oYW5kbGVfdmJsYW5rKCkgaXMg Y2FsbGVkCj4gPiA+ID4gcmVkdW5kYW50bHksIG9yIGlmIHRoZSBpcnEgaGFuZGxlciBnZXRzIGRl bGF5ZWQgc29tZWhvdy4KPiA+ID4gPiAKPiA+ID4gPiBUaGlzIGlzIGFjdHVhbGx5IGEgcmVhbCBy YWNlIG9uIEludGVsIGdlbjIvMyBoYXJkd2FyZSBkdWUgdG8gdGhlIHZibGFuawo+ID4gPiA+IGlu dGVycnVwdCBmaXJpbmcgYXBwcm94LiBvbmUgb3IgbW9yZSBzY2FubGluZXMgYWZ0ZXIgdGhlIHN0 YXJ0IG9mIHZibGFuawo+ID4gPiA+ICh3aGljaCBpcyB3aGVuIHRoZSBkb3VibGUgYnVmZmVyZWQg cmVnaXN0ZXJzIGdldCBsYXRjaGVkKS4gU28gaWYgd2Ugd2VyZQo+ID4gPiA+IHRvIHBlcmZvcm0g YW4gYXRvbWljIHVwZGF0ZSBqdXN0IGFmdGVyIHRoZSBzdGFydCBvZiB2YmxhbmssIGJ1dCBiZWZv cmUKPiA+ID4gPiB0aGUgaXJxIGhhcyBmaXJlZCwgdGhlIGlycSBoYW5kbGVyIHdvdWxkIHNlbmQg b3V0IHRoZSB0aGUgZXZlbnQgaW1tZWRpYXRlbHkKPiA+ID4gPiBpbnN0ZWFkIG9mIHdhaXRpbmcg Zm9yIHRoZSBuZXh0IHZibGFuayBsaWtlIGl0IHNob3VsZC4KPiA+ID4gPiAKPiA+ID4gPiBUaGlz IGFsc28gbWFrZXMgbG9ncyBsZXNzIGNvbmZ1c2luZyBiZWNhdXNlIG5vdyBpdCBub3JtYWxseSBs b29rcyBsaWtlCj4gPiA+ID4gdGhlIHZibGFuayBpbnRlcnJ1cHQgd2FzIHNvbWVob3cgbWlzc2Vk IGFuZCB0aGUgZXZlbnQgZ2V0cyBzZW50IG9uZQo+ID4gPiA+IGZyYW1lIHRvbyBsYXRlLgo+ID4g PiAKPiA+ID4gU28gdGhpcyBwYXJhIGhlcmUgSSBsaWtlLCBzaW5jZSB0aGUgb3RoZXJzIGFyZSBh bHJlYWR5IHdyaXR0ZW4gb2ZmIGluIHRoZQo+ID4gPiBrZXJuZWwtZG9jIGFzICJkb24ndCB1c2Ug dGhpcyBmdW5jdGlvbiBpZiB0aGlzIGlzIGEgcHJvYmxlbSBmb3IgeW91Ii4gSSdkCj4gPiA+IHNh eSB5b3VyIGNsYWltIGlzIGV2ZW4gd3JvbmcsIHNpbmNlIHdlJ3JlIG5vdCB1c2luZwo+ID4gPiBk cm1fYWNjdXJhdGVfdmJsYW5rX2NvdW50LCBhbmQgc28gdGhlIHJhY2Ugc3RpbGwgZXhpc3RzIChp ZiB5b3UncmUKPiA+ID4gc3VmZmljaWVudGx5IHVubHVja3kgYXQgbGVhc3QsIGFuZCBzaW5jZSB0 aGVyZSBjYW4gYmUgYW4gYXJiaXRyYXJ5IGRlbGF5Cj4gPiA+IGJldHdlZW4gd2hlbiB0aGUgaHcg cmFpc2VzIGFuIGlycSBhbmQgd2hlbiB0aGUga2VybmVsIHJ1bnMgdGhlIGlycQo+ID4gPiBoYW5k bGVyIGl0J3Mgbm90IGFueSBiZXR0ZXIpLgo+ID4gCj4gPiBBdCBsZWFzdCBmb3IgaTkxNSBpdCBz aG91bGQgYmUgYWNjdXJhdGUgaWYgdGhlIGlycSB3YXNuJ3QgZW5hYmxlZAo+ID4gcHJpb3IgdG8g c3RhcnRpbmcgdGhlIHBpcGUgdXBkYXRlIGFzIHdlJ2xsIHVwZGF0ZSB0aGUgY291bnRlciB3aGVu Cj4gPiBlbmFibGluZyB0aGUgaXJxLiBUbyBtYWtlIHN1cmUgaXQncyBhY2N1cmF0ZSB3ZSBzaG91 bGQgcHJvYmFibHkKPiA+IGhhdmUgYSBkcm1fdmJsYW5rX2dldF9hY2N1cmF0ZSgpIG9yIHNvbWV0 aGluZyBsaWtlIHRoYXQuCj4gCj4gV2UgY291bGQgc3dpdGNoIHRoZSBkcm1fdmJsYW5rX2NvdW50 IGluIHRoZSBkaWZmIGJlbG93IHRvCj4gZHJtX2FjY3VyYXRlX3ZibGFua19jb3VudC4KClRoYXQn cyBzb21ld2hhdCBzdWItb3B0aW1hbCBhcyB3ZSBjYW4gYXZvaWQgdGhlIHVwZGF0ZSB3aGVuIHdl IGtub3cgdGhhdApkcm1fdmJsYW5rX2dldCgpIGFscmVhZHkgZGlkIGl0IGZvciB1cy4KCj4gSSB0 eXBlZCB0aGlzIGhlbHBlciBmb3IgZHJpdmVycyB3aGljaCBkb24ndAo+IGhhdmUgYWNjdXJhdGUg dmJsYW5rIHRpbWVzdGFtcGluZyBzdXBwb3J0LCBzbyB0aGUgZGlmZmVyZW5jZSB3YXMgbW9vdC4g QnV0Cj4gd2l0aCBpOTE1IHVzaW5nIGl0IHRoYXQncyBjaGFuZ2VkIChhbmQgaWlyYyB2YzQgdXNl cyBpdCB0b28gb3Igc29tZXRoaW5nCj4gbGlrZSB0aGF0KS4KPiAtRGFuaWVsCj4gCj4gPiAKPiA+ ID4gCj4gPiA+IEFueXdheSwgYXMgbG9uZyBhcyBpdCdzIG5vdCBjYzogc3RhYmxlICh0aGF0IHdv dWxkIG5lZWQgbW9yZSBjb252aW5jaW5nIG9uCj4gPiA+IG15IHNpZGUpOgo+ID4gPiAKPiA+ID4g UmV2aWV3ZWQtYnk6IERhbmllbCBWZXR0ZXIgPGRhbmllbC52ZXR0ZXJAZmZ3bGwuY2g+Cj4gPiA+ ID4gCj4gPiA+ID4gQ2M6IERhbmllbCBWZXR0ZXIgPGRhbmllbEBmZndsbC5jaD4KPiA+ID4gPiBT aWduZWQtb2ZmLWJ5OiBWaWxsZSBTeXJqw6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwu Y29tPgo+ID4gPiA+IC0tLQo+ID4gPiA+ICBkcml2ZXJzL2dwdS9kcm0vZHJtX3ZibGFuay5jIHwg MiArLQo+ID4gPiA+ICAxIGZpbGUgY2hhbmdlZCwgMSBpbnNlcnRpb24oKyksIDEgZGVsZXRpb24o LSkKPiA+ID4gPiAKPiA+ID4gPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2RybV92Ymxh bmsuYyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fdmJsYW5rLmMKPiA+ID4gPiBpbmRleCBiZmU4NmZh MmQzYjEuLjgyM2M4NTNkZTA1MiAxMDA2NDQKPiA+ID4gPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0v ZHJtX3ZibGFuay5jCj4gPiA+ID4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2RybV92YmxhbmsuYwo+ ID4gPiA+IEBAIC04NjksNyArODY5LDcgQEAgdm9pZCBkcm1fY3J0Y19hcm1fdmJsYW5rX2V2ZW50 KHN0cnVjdCBkcm1fY3J0YyAqY3J0YywKPiA+ID4gPiAgCWFzc2VydF9zcGluX2xvY2tlZCgmZGV2 LT5ldmVudF9sb2NrKTsKPiA+ID4gPiAgCj4gPiA+ID4gIAllLT5waXBlID0gcGlwZTsKPiA+ID4g PiAtCWUtPmV2ZW50LnNlcXVlbmNlID0gZHJtX3ZibGFua19jb3VudChkZXYsIHBpcGUpOwo+ID4g PiA+ICsJZS0+ZXZlbnQuc2VxdWVuY2UgPSBkcm1fdmJsYW5rX2NvdW50KGRldiwgcGlwZSkgKyAx Owo+ID4gPiA+ICAJZS0+ZXZlbnQuY3J0Y19pZCA9IGNydGMtPmJhc2UuaWQ7Cj4gPiA+ID4gIAls aXN0X2FkZF90YWlsKCZlLT5iYXNlLmxpbmssICZkZXYtPnZibGFua19ldmVudF9saXN0KTsKPiA+ ID4gPiAgfQo+ID4gPiA+IC0tIAo+ID4gPiA+IDIuMTMuMAo+ID4gPiA+IAo+ID4gPiAKPiA+ID4g LS0gCj4gPiA+IERhbmllbCBWZXR0ZXIKPiA+ID4gU29mdHdhcmUgRW5naW5lZXIsIEludGVsIENv cnBvcmF0aW9uCj4gPiA+IGh0dHA6Ly9ibG9nLmZmd2xsLmNoCj4gPiAKPiA+IC0tIAo+ID4gVmls bGUgU3lyasOkbMOkCj4gPiBJbnRlbCBPVEMKPiAKPiAtLSAKPiBEYW5pZWwgVmV0dGVyCj4gU29m dHdhcmUgRW5naW5lZXIsIEludGVsIENvcnBvcmF0aW9uCj4gaHR0cDovL2Jsb2cuZmZ3bGwuY2gK Ci0tIApWaWxsZSBTeXJqw6Rsw6QKSW50ZWwgT1RDCl9fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVsQGxp c3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFu L2xpc3RpbmZvL2RyaS1kZXZlbAo=