From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mario Kleiner Subject: Re: [Intel-gfx] [PATCH] drm/vblank: Fixup and document timestamp update/read barriers Date: Mon, 04 May 2015 06:52:18 +0200 Message-ID: <5546FB02.4010508@gmail.com> References: <1429082222-20820-1-git-send-email-daniel.vetter@ffwll.ch> <552E60D4.3070307@hurleysoftware.com> <20150415173153.GA6092@phenom.ffwll.local> <552FAB7F.4040707@hurleysoftware.com> <20150416130323.GG5082@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20150416130323.GG5082@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 , Peter Hurley Cc: Daniel Vetter , Intel Graphics Development , =?UTF-8?B?TWljaGVsIETDpG56ZXI=?= , DRI Development , Daniel Vetter List-Id: dri-devel@lists.freedesktop.org T24gMDQvMTYvMjAxNSAwMzowMyBQTSwgRGFuaWVsIFZldHRlciB3cm90ZToKPiBPbiBUaHUsIEFw ciAxNiwgMjAxNSBhdCAwODozMDo1NUFNIC0wNDAwLCBQZXRlciBIdXJsZXkgd3JvdGU6Cj4+IE9u IDA0LzE1LzIwMTUgMDE6MzEgUE0sIERhbmllbCBWZXR0ZXIgd3JvdGU6Cj4+PiBPbiBXZWQsIEFw ciAxNSwgMjAxNSBhdCAwOTowMDowNEFNIC0wNDAwLCBQZXRlciBIdXJsZXkgd3JvdGU6Cj4+Pj4g SGkgRGFuaWVsLAo+Pj4+Cj4+Pj4gT24gMDQvMTUvMjAxNSAwMzoxNyBBTSwgRGFuaWVsIFZldHRl ciB3cm90ZToKPj4+Pj4gVGhpcyB3YXMgYSBiaXQgdG9vIG11Y2ggY2FyZ28tY3VsdGVkLCBzbyBs ZXRzIG1ha2UgaXQgc29saWQ6Cj4+Pj4+IC0gdmJsYW5rLT5jb3VudCBkb2Vzbid0IG5lZWQgdG8g YmUgYW4gYXRvbWljLCB3cml0ZXMgYXJlIGFsd2F5cyBkb25lCj4+Pj4+ICAgIHVuZGVyIHRoZSBw cm90ZWN0aW9uIG9mIGRldi0+dmJsYW5rX3RpbWVfbG9jay4gU3dpdGNoIHRvIGFuIHVuc2lnbmVk Cj4+Pj4+ICAgIGxvbmcgaW5zdGVhZCBhbmQgdXBkYXRlIGNvbW1lbnRzLiBOb3RlIHRoYXQgYXRv bWljX3JlYWQgaXMganVzdCBhCj4+Pj4+ICAgIG5vcm1hbCByZWFkIG9mIGEgdm9sYXRpbGUgdmFy aWFibGUsIHNvIG5vIG5lZWQgdG8gYXVkaXQgYWxsIHRoZQo+Pj4+PiAgICByZWFkLXNpZGUgYWNj ZXNzIHNwZWNpZmljYWxseS4KPj4+Pj4KPj4+Pj4gLSBUaGUgYmFycmllcnMgZm9yIHRoZSB2Ymxh bmsgY291bnRlciBzZXFsb2NrIHdlcmVuJ3QgY29tcGxldGU6IFRoZQo+Pj4+PiAgICByZWFkLXNp ZGUgd2FzIG1pc3NpbmcgdGhlIGZpcnN0IGJhcnJpZXIgYmV0d2VlbiB0aGUgY291bnRlciByZWFk IGFuZAo+Pj4+PiAgICB0aGUgdGltZXN0YW1wIHJlYWQsIGl0IG9ubHkgaGFkIGEgYmFycmllciBi ZXR3ZWVuIHRoZSB0cyBhbmQgdGhlCj4+Pj4+ICAgIGNvdW50ZXIgcmVhZC4gV2UgbmVlZCBib3Ro Lgo+Pj4+Pgo+Pj4+PiAtIEJhcnJpZXJzIHdlcmVuJ3QgcHJvcGVybHkgZG9jdW1lbnRlZC4gU2lu Y2UgYmFycmllcnMgb25seSB3b3JrIGlmCj4+Pj4+ICAgIHlvdSBoYXZlIHRoZW0gb24gYm90aHMg c2lkZXMgb2YgdGhlIHRyYW5zYWN0aW9uIGl0J3MgcHJ1ZGVudCB0bwo+Pj4+PiAgICByZWZlcmVu Y2Ugd2hlcmUgdGhlIG90aGVyIHNpZGUgaXMuIFRvIGF2b2lkIGR1cGxpY2F0aW5nIHRoZQo+Pj4+ PiAgICB3cml0ZS1zaWRlIGNvbW1lbnQgMyB0aW1lcyBleHRyYWN0IGEgbGl0dGxlIHN0b3JlX3Zi bGFuaygpIGhlbHBlci4KPj4+Pj4gICAgSW4gdGhhdCBoZWxwZXIgYWxzbyBhc3NlcnQgdGhhdCB3 ZSBkbyBpbmRlZWQgaG9sZAo+Pj4+PiAgICBkZXYtPnZibGFua190aW1lX2xvY2ssIHNpbmNlIGlu IHNvbWUgY2FzZXMgdGhlIGxvY2sgaXMgYWNxdWlyZWQgYQo+Pj4+PiAgICBmZXcgZnVuY3Rpb25z IHVwIGluIHRoZSBjYWxsY2hhaW4uCj4+Pj4+Cj4+Pj4+IFNwb3R0ZWQgd2hpbGUgcmV2aWV3aW5n IGEgcGF0Y2ggZnJvbSBDaHJpcyBXaWxzb24gdG8gYWRkIGEgZmFzdHBhdGggdG8KPj4+Pj4gdGhl IHZibGFua193YWl0IGlvY3RsLgo+Pj4+Pgo+Pj4+PiBDYzogQ2hyaXMgV2lsc29uIDxjaHJpc0Bj aHJpcy13aWxzb24uY28udWs+Cj4+Pj4+IENjOiBNYXJpbyBLbGVpbmVyIDxtYXJpby5rbGVpbmVy LmRlQGdtYWlsLmNvbT4KPj4+Pj4gQ2M6IFZpbGxlIFN5cmrDpGzDpCA8dmlsbGUuc3lyamFsYUBs aW51eC5pbnRlbC5jb20+Cj4+Pj4+IENjOiBNaWNoZWwgRMOkbnplciA8bWljaGVsQGRhZW56ZXIu bmV0Pgo+Pj4+PiBTaWduZWQtb2ZmLWJ5OiBEYW5pZWwgVmV0dGVyIDxkYW5pZWwudmV0dGVyQGlu dGVsLmNvbT4KPj4+Pj4gLS0tCj4+Pj4+ICAgZHJpdmVycy9ncHUvZHJtL2RybV9pcnEuYyB8IDky ICsrKysrKysrKysrKysrKysrKysrKysrKy0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tCj4+Pj4+ICAg aW5jbHVkZS9kcm0vZHJtUC5oICAgICAgICB8ICA4ICsrKy0tCj4+Pj4+ICAgMiBmaWxlcyBjaGFu Z2VkLCA1NCBpbnNlcnRpb25zKCspLCA0NiBkZWxldGlvbnMoLSkKPj4+Pj4KPj4+Pj4gZGlmZiAt LWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9kcm1faXJxLmMgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX2ly cS5jCj4+Pj4+IGluZGV4IGM4YTM0NDc2NTcwYS4uMjNiZmJjNjFhNDk0IDEwMDY0NAo+Pj4+PiAt LS0gYS9kcml2ZXJzL2dwdS9kcm0vZHJtX2lycS5jCj4+Pj4+ICsrKyBiL2RyaXZlcnMvZ3B1L2Ry bS9kcm1faXJxLmMKPj4+Pj4gQEAgLTc0LDYgKzc0LDMzIEBAIG1vZHVsZV9wYXJhbV9uYW1lZCh2 YmxhbmtvZmZkZWxheSwgZHJtX3ZibGFua19vZmZkZWxheSwgaW50LCAwNjAwKTsKPj4+Pj4gICBt b2R1bGVfcGFyYW1fbmFtZWQodGltZXN0YW1wX3ByZWNpc2lvbl91c2VjLCBkcm1fdGltZXN0YW1w X3ByZWNpc2lvbiwgaW50LCAwNjAwKTsKPj4+Pj4gICBtb2R1bGVfcGFyYW1fbmFtZWQodGltZXN0 YW1wX21vbm90b25pYywgZHJtX3RpbWVzdGFtcF9tb25vdG9uaWMsIGludCwgMDYwMCk7Cj4+Pj4+ Cj4+Pj4+ICtzdGF0aWMgdm9pZCBzdG9yZV92Ymxhbmsoc3RydWN0IGRybV9kZXZpY2UgKmRldiwg aW50IGNydGMsCj4+Pj4+ICsJCQkgdW5zaWduZWQgdmJsYW5rX2NvdW50X2luYywKPj4+Pj4gKwkJ CSBzdHJ1Y3QgdGltZXZhbCAqdF92YmxhbmspCj4+Pj4+ICt7Cj4+Pj4+ICsJc3RydWN0IGRybV92 YmxhbmtfY3J0YyAqdmJsYW5rID0gJmRldi0+dmJsYW5rW2NydGNdOwo+Pj4+PiArCXUzMiB0c2xv dDsKPj4+Pj4gKwo+Pj4+PiArCWFzc2VydF9zcGluX2xvY2tlZCgmZGV2LT52YmxhbmtfdGltZV9s b2NrKTsKPj4+Pj4gKwo+Pj4+PiArCWlmICh0X3ZibGFuaykgewo+Pj4+PiArCQl0c2xvdCA9IHZi bGFuay0+Y291bnQgKyB2YmxhbmtfY291bnRfaW5jOwo+Pj4+PiArCQl2Ymxhbmt0aW1lc3RhbXAo ZGV2LCBjcnRjLCB0c2xvdCkgPSAqdF92Ymxhbms7Cj4+Pj4+ICsJfQo+Pj4+PiArCj4+Pj4+ICsJ LyoKPj4+Pj4gKwkgKiB2YmxhbmsgdGltZXN0YW1wIHVwZGF0ZXMgYXJlIHByb3RlY3RlZCBvbiB0 aGUgd3JpdGUgc2lkZSB3aXRoCj4+Pj4+ICsJICogdmJsYW5rX3RpbWVfbG9jaywgYnV0IG9uIHRo ZSByZWFkIHNpZGUgZG9uZSBsb2NrbGVzc2x5IHVzaW5nIGEKPj4+Pj4gKwkgKiBzZXF1ZW5jZS1s b2NrIG9uIHRoZSB2YmxhbmsgY291bnRlci4gRW5zdXJlIGNvcnJlY3Qgb3JkZXJpbmcgdXNpbmcK Pj4+Pj4gKwkgKiBtZW1vcnkgYmFycnJpZXJzLiBXZSBuZWVkIHRoZSBiYXJyaWVyIGJvdGggYmVm b3JlIGFuZCBhbHNvIGFmdGVyIHRoZQo+Pj4+PiArCSAqIGNvdW50ZXIgdXBkYXRlIHRvIHN5bmNo cm9uaXplIHdpdGggdGhlIG5leHQgdGltZXN0YW1wIHdyaXRlLgo+Pj4+PiArCSAqIFRoZSByZWFk LXNpZGUgYmFycmllcnMgZm9yIHRoaXMgYXJlIGluIGRybV92YmxhbmtfY291bnRfYW5kX3RpbWUu Cj4+Pj4+ICsJICovCj4+Pj4+ICsJc21wX3dtYigpOwo+Pj4+PiArCXZibGFuay0+Y291bnQgKz0g dmJsYW5rX2NvdW50X2luYzsKPj4+Pj4gKwlzbXBfd21iKCk7Cj4+Pj4KPj4+PiBUaGUgY29tbWVu dCBhbmQgdGhlIGNvZGUgYXJlIGVhY2ggc2VsZi1jb250cmFkaWN0b3J5Lgo+Pj4+Cj4+Pj4gSWYg dmJsYW5rLT5jb3VudCB3cml0ZXMgYXJlIGFsd2F5cyBwcm90ZWN0ZWQgYnkgdmJsYW5rX3RpbWVf bG9jayAoc29tZXRoaW5nIEkKPj4+PiBkaWQgbm90IHZlcmlmeSBidXQgdGhhdCB0aGUgY29tbWVu dCBhYm92ZSBhc3NlcnRzKSwgdGhlbiB0aGUgdHJhaWxpbmcgd3JpdGUKPj4+PiBiYXJyaWVyIGlz IG5vdCByZXF1aXJlZCAoYW5kIHRoZSBhc3NlcnRpb24gdGhhdCBpdCBpcyBpbiB0aGUgY29tbWVu dCBpcyBpbmNvcnJlY3QpLgo+Pj4+Cj4+Pj4gQSBzcGluIHVubG9jayBvcGVyYXRpb24gaXMgYWx3 YXlzIGEgd3JpdGUgYmFycmllci4KPj4+Cj4+PiBIbSB5ZWFoLiBPdG9oIHRvIG1lIHRoYXQncyBi b3JkZXJpbmcgb24gImNvZGUgdG9vIGNsZXZlciBmb3IgbXkgb3duIGdvb2QiLgo+Pj4gVGhhdCB0 aGUgc3BpbmxvY2sgaXMgaGVsZCBJIGNhbiBhc3N1cmUuIFRoYXQgbm8gb25lIGdvZXMgYXJvdW5k IGFuZCBkb2VzCj4+PiBtdWx0aXBsZSB2YmxhbmsgdXBkYXRlcyAoYmVjYXVzZSBzb21laG93IHRo YXQgY29kZSByYWNlZCB3aXRoIHRoZSBodwo+Pj4gaXRzZWxmKSBJIGNhbid0IGVhc2lseSBhc3N1 cmUgd2l0aCBhIHNpbXBsZSBhc3NlcnQgb3Igc29tZXRoaW5nIHNpbWlsYXIuCj4+PiBJdCdzIG5v dCB0aGUgY2FzZSByaWdodCBub3csIGJ1dCB0aGF0IGNhbiBjaGFuZ2VzLgo+Pgo+PiBUaGUgYWxn b3JpdGhtIHdvdWxkIGJlIGJyb2tlbiBpZiBtdWx0aXBsZSB1cGRhdGVzIGZvciB0aGUgc2FtZSB2 YmxhbmsKPj4gY291bnQgd2VyZSBhbGxvd2VkOyB0aGF0J3Mgd2h5IGl0IGNoZWNrcyB0byBzZWUg aWYgdGhlIHZibGFuayBjb3VudCBoYXMKPj4gbm90IGFkdmFuY2VkIGJlZm9yZSBzdG9yaW5nIGEg bmV3IHRpbWVzdGFtcC4KPj4KPj4gT3RoZXJ3aXNlLCB0aGUgcmVhZCBzaWRlIHdvdWxkIG5vdCBi ZSBhYmxlIHRvIGRldGVybWluZSB0aGF0IHRoZQo+PiB0aW1lc3RhbXAgaXMgdmFsaWQgYnkgZG91 YmxlLWNoZWNraW5nIHRoYXQgdGhlIHZibGFuayBjb3VudCBoYXMgbm90Cj4+IGNoYW5nZWQuCj4+ Cj4+IEFuZCBiZXNpZGVzLCBldmVuIGlmIHRoZSBjb2RlIGxvb3BlZCB3aXRob3V0IGRyb3BwaW5n IHRoZSBzcGlubG9jaywKPj4gdGhlIGNvcnJlY3Qgd3JpdGUgb3JkZXIgd291bGQgc3RpbGwgYmUg b2JzZXJ2ZWQgYmVjYXVzZSBpdCB3b3VsZCBzdGlsbAo+PiBiZSBleGVjdXRpbmcgb24gdGhlIHNh bWUgY3B1Lgo+Pgo+PiBNeSBvYmplY3Rpb24gdG8gdGhlIHdyaXRlIG1lbW9yeSBiYXJyaWVyIGlz IG5vdCBhYm91dCBvcHRpbWl6YXRpb247Cj4+IGl0J3MgYWJvdXQgY29ycmVjdCBjb2RlLgo+Cj4g V2VsbCBkaWZmPTAgaXMgbm90IGFsbG93ZWQsIEkgZ3Vlc3MgSSBjb3VsZCBlbmZvcmNlIHRoaXMg d2l0aCBzb21lCj4gV0FSTl9PTi4gQW5kIEkgc3RpbGwgdGhpbmsgbXkgcG9pbnQgb2Ygbm9uLWxv Y2FsIGNvcnJlY3RuZXNzIGlzIHNvbGlkLgo+IFdpdGggdGhlIHNtcF93bWIoKSByZW1vdmVkIHRo ZSBmb2xsb3dpbmcgc3RpbGwgd29ya3MgY29ycmVjdGx5Ogo+Cj4gc3Bpbl9sb2NrKHZibGFua190 aW1lX2xvY2spOwo+IHN0b3JlX3ZibGFuayhkZXYsIGNydGMsIDEsIHRzMSk7Cj4gc3Bpbl91bmxv Y2sodmJsYW5rX3RpbWVfbG9jayk7Cj4KPiBzcGluX2xvY2sodmJsYW5rX3RpbWVfbG9jayk7Cj4g c3RvcmVfdmJsYW5rKGRldiwgY3J0YywgMSwgdHMyKTsKPiBzcGluX3VubG9jayh2YmxhbmtfdGlt ZV9sb2NrKTsKPgo+IEJ1dCB3aXRoIHRoZSBzbXBfd21iKCk7IHJlbW92ZWQgdGhlIGZvbGxvd2lu ZyB3b3VsZCBiZSBicm9rZW46Cj4KPiBzcGluX2xvY2sodmJsYW5rX3RpbWVfbG9jayk7Cj4gc3Rv cmVfdmJsYW5rKGRldiwgY3J0YywgMSwgdHMxKTsKPiBzdG9yZV92YmxhbmsoZGV2LCBjcnRjLCAx LCB0czIpOwo+IHNwaW5fdW5sb2NrKHZibGFua190aW1lX2xvY2spOwo+Cj4gYmVjYXVzZSB0aGUg Y29tcGlsZXIvY3B1IGlzIGZyZWUgdG8gcmVvcmRlciB0aGUgc3RvcmUgZm9yIHZibGFuay0+Y291 bnQKPiBfYWhlYWRfIG9mIHRoZSBzdG9yZSBmb3IgdGhlIHRpbWVzdGFtcC4gQW5kIHRoYXQgd291 bGQgdHJpY2sgcmVhZGVycyBpbnRvCj4gYmVsaWV2aW5nIHRoYXQgdGhleSBoYXZlIGEgdmFsaWQg dGltZXN0YW1wIHdoZW4gdGhleSBwb3RlbnRpYWxseSByYWNlZC4KPgo+IE5vdyB5b3UncmUgY29y cmVjdCB0aGF0IHJpZ2h0IG5vdyB0aGVyZSdzIG5vIHN1Y2ggdGhpbmcgZ29pbmcgb24sIGFuZCBp dCdzCj4gdW5saWtlbHkgdG8gaGFwcGVuIChnaXZlbiB0aGUgbmF0dXJlIG9mIHZibGFuayB1cGRh dGVzKS4gQnV0IG15IHBvaW50IGlzCj4gdGhhdCBpZiB3ZSBvcHRpbWl6ZSB0aGlzIHRoZW4gdGhl IGNvcnJlY3RuZXNzIGNhbid0IGJlIHByb3ZlbiBsb2NhbGx5Cj4gYW55bW9yZSBieSBqdXN0IGxv b2tpbmcgYXQgc3RvcmVfdmJsYW5rLCBidXQgaW5zdGVhZCB5b3UgbXVzdCBhdWRpdCBhbGwKPiB0 aGUgY2FsbGVycy4gQW5kIGxlYWtpbmcgbG9ja2luZy9iYXJyaWVycyBsaWtlIHRoYXQgaXMgdG9v IGZyYWdpbGUgZGVzaWduCj4gZm9yIG15IHRhc3RlLgo+Cj4gQnV0IHlvdSBpbnNpc3QgdGhhdCBt eSBhcHByb2FjaCBpcyBicm9rZW4gc29tZWhvdyBhbmQgZHJvcHBpbmcgdGhlIHNtcF93bWIKPiBp cyBuZWVkZWQgZm9yIGNvcnJlY3RuZXNzLiBJIGRvbid0IHNlZSBob3cgdGhhdCdzIHRoZSBjYXNl IGF0IGFsbC4KPiAtRGFuaWVsCj4KCkZ3aXcsIGkgc3BlbnQgc29tZSB0aW1lIHJlZWR1Y2F0aW5n IG15c2VsZiBhYm91dCBtZW1vcnkgYmFycmllcnMgKHRoYW5rcyAKZm9yIHlvdXIgZXhwbGFuYXRp b25zKSBhbmQgdGhpbmtpbmcgYWJvdXQgdGhpcywgYW5kIHRoZSBsYXN0IHZlcnNpb24gb2YgCnlv dXIgcGF0Y2ggbG9va3MgZ29vZCB0byBtZS4gSXQgYWxzbyBtYWtlcyBzZW5zZSB0byBtZSB0byBs ZWF2ZSB0aGF0IApsYXN0IHNtYl93bWIoKSBpbiBwbGFjZSB0byBtYWtlIGZ1dHVyZSB1c2Ugb2Yg dGhlIGhlbHBlciByb2J1c3QgLSBmb3IgCm5vbi1sb2NhbCBjb3JyZWN0bmVzcywgdG8gYXZvaWQg aGF2aW5nIHRvIGF1ZGl0IGFsbCBmdXR1cmUgY2FsbGVycyBvZiAKdGhhdCBoZWxwZXIuCgpJIGFs c28gdGVzdGVkIHlvdXIgcGF0Y2ggKyBhIHNsaWdodGx5IG1vZGlmaWVkIHZlcnNpb24gb2YgQ2hy aXMgdmJsYW5rIApkZWxheWVkIGRpc2FibGUgLyBpbnN0YW50IHF1ZXJ5IHBhdGNoZXMgKyBteSBm aXhlcyB1c2luZyBteSBvd24gc3RyZXNzIAp0ZXN0cyBhbmQgaGFyZHdhcmUgdGltaW5nIHRlc3Qg ZXF1aXBtZW50IG9uIGJvdGggaW50ZWwgYW5kIG5vdXZlYXUsIGFuZCAKZXZlcnl0aGluZyBzZWVt cyB0byB3b3JrIGZpbmUuCgpTbyBpJ20gYWxsIGZvciBpbmNsdWRpbmcgdGhpcyBwYXRjaCBhbmQg aXQgaGFzIG15CgpSZXZpZXdlZC1hbmQtdGVzdGVkLWJ5OiBNYXJpbyBLbGVpbmVyIDxtYXJpby5r bGVpbmVyLmRlQGdtYWlsLmNvbT4KCkkganVzdCBzZW50IG91dCBhbiB1cGRhdGVkIHZlcnNpb24g b2YgbXkgcGF0Y2hlcywgc28gdGhleSBkb24ndCBjb25mbGljdCAKd2l0aCB0aGlzIG9uZSBhbmQg YWxzbyBmaXggYSBjb21waWxlIGZhaWx1cmUgb2YgZHJtL3F4bCB3aXRoIHlvdXJzLgoKVGhhbmtz LAotbWFyaW8KX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18K ZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0 dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK