From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tvrtko Ursulin Subject: Re: [PATCH 2/2] drm/i915: Don't assume primary & cursor are always on for wm calculation Date: Wed, 04 Mar 2015 17:29:42 +0000 Message-ID: <54F74106.7040309@linux.intel.com> References: <1425435313-31711-1-git-send-email-matthew.d.roper@intel.com> <1425435313-31711-2-git-send-email-matthew.d.roper@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTP id DC15E89B99 for ; Wed, 4 Mar 2015 09:29:45 -0800 (PST) In-Reply-To: <1425435313-31711-2-git-send-email-matthew.d.roper@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Matt Roper , intel-gfx@lists.freedesktop.org Cc: Michael Leuchtenburg List-Id: intel-gfx@lists.freedesktop.org Ck9uIDAzLzA0LzIwMTUgMDI6MTUgQU0sIE1hdHQgUm9wZXIgd3JvdGU6Cj4gQ3VycmVudCBJTEst c3R5bGUgd2F0ZXJtYXJrIGNvZGUgYXNzdW1lcyB0aGUgcHJpbWFyeSBwbGFuZSBhbmQgY3Vyc29y Cj4gcGxhbmUgYXJlIGFsd2F5cyBlbmFibGVkLiAgVGhpcyBhc3N1bXB0aW9uLCBhbG9uZyB3aXRo IHRoZSBjb21iaW5hdGlvbgo+IG9mIHR3byBpbmRlcGVuZGVudCBjb21taXRzIHRoYXQgZ290IG1l cmdlZCBhdCB0aGUgc2FtZSB0aW1lLCByZXN1bHRzIGluCj4gYSBOVUxMIGRlcmVmZXJlbmNlLiAg VGhlIG9mZmVuZGluZyBjb21taXRzIGFyZToKPgo+ICAgICAgICAgIGNvbW1pdCBmZDJkNjEzNDFi ZjM5ZDEwNTQyNTZjMDdkNmVkZGQ2MjRlYmM0MjQxCj4gICAgICAgICAgQXV0aG9yOiBNYXR0IFJv cGVyIDxtYXR0aGV3LmQucm9wZXJAaW50ZWwuY29tPgo+ICAgICAgICAgIERhdGU6ICAgRnJpIEZl YiAyNyAxMDoxMjowMSAyMDE1IC0wODAwCj4KPiAgICAgICAgICAgICAgZHJtL2k5MTU6IFVzZSBw bGFuZS0+c3RhdGUtPmZiIGluIHdhdGVybWFyayBjb2RlICh2MikKPgo+IGFuZAo+Cj4gICAgICAg ICAgY29tbWl0IDBmZGE2NTY4MGU5MjU0NWNhZWE1YmU3ODA1YTdmMGE2MTdmYjZjMjAKPiAgICAg ICAgICBBdXRob3I6IFR2cnRrbyBVcnN1bGluIDx0dnJ0a28udXJzdWxpbkBpbnRlbC5jb20+Cj4g ICAgICAgICAgRGF0ZTogICBGcmkgRmViIDI3IDE1OjEyOjM1IDIwMTUgKzAwMDAKPgo+ICAgICAg ICAgICAgICBkcm0vaTkxNS9za2w6IFVwZGF0ZSB3YXRlcm1hcmtzIGZvciBZIHRpbGluZwo+Cj4g VGhlIGZpcnN0IGNvbW1pdCBjYXVzZXMgdXMgdG8gdXNlIHRoZSBGQiBmcm9tIHBsYW5lLT5zdGF0 ZS0+ZmIgcmF0aGVyCj4gdGhhbiB0aGUgbGVnYWN5IHBsYW5lLT5mYiwgd2hpY2ggaXMgdXBkYXRl ZCBhIGJpdCBsYXRlciBpbiB0aGUgcHJvY2Vzcy4KPgo+IFRoZSBzZWNvbmQgY29tbWl0IGluY2x1 ZGVzIGEgY2hhbmdlIHRoYXQgbm93IHRyaWdnZXJzIHdhdGVybWFyawo+IHJlcHJvZ3JhbW1pbmcg b24gcHJpbWFyeSBwbGFuZSBlbmFibGUvZGlzYWJsZSB3aGVyZSB3ZSBkaWRuJ3QgaGF2ZSBvbmUK PiBiZWZvcmUgKHdoaWNoIHdhc24ndCByZWFsbHkgY29ycmVjdCwgYnV0IHdlIGhhZCBiZWVuIGdl dHRpbmcgbHVja3kKPiBiZWNhdXNlIHdlIGFsd2F5cyBjYWxjdWxhdGVkIGFzIGlmIHRoZSBwcmlt YXJ5IHBsYW5lIHdhcyBvbikuCj4KPiBUb2dldGhlciwgdGhlc2UgdHdvIGNvbW1pdHMgY2F1c2Ug dGhlIHdhdGVybWFyayBjYWxjdWxhdGlvbiB0bwo+IChwcm9wZXJseSkgc2VlIHBsYW5lLT5zdGF0 ZS0+ZmIgPSBOVUxMIHdoZW4gd2UncmUgaW4gdGhlIHByb2Nlc3Mgb2YKPiBkaXNhYmxpbmcgdGhl IHByaW1hcnkgcGxhbmUuICBIb3dldmVyIHRoZSBleGlzdGluZyB3YXRlcm1hcmsgY29kZQo+IGFz c3VtZXMgdGhlcmUncyBhbHdheXMgYSBwcmltYXJ5IGZiIGFuZCB0cmllcyB0byBkZXJlZmVyZW5j ZSBpdCB0byBmaW5kCj4gb3V0IHBpeGVsIGZvcm1hdCAvIGJwcCBpbmZvcm1hdGlvbi4KPgo+IFRo ZSBmaXggaXMgdG8gbWFrZSBJTEstc3R5bGUgd2F0ZXJtYXJrIGNhbGN1bGF0aW9uIGFjdHVhbGx5 IGNoZWNrIHRoZQo+IHRydWUgc3RhdHVzIG9mIHByaW1hcnkgJiBjdXJzb3IgcGxhbmVzIGFuZCBh ZGp1c3Qgb3VyIHdhdGVybWFyayBsb2dpYwo+IGFjY29yZGluZ2x5Lgo+Cj4gQ2M6IFR2cnRrbyBV cnN1bGluIDx0dnJ0a28udXJzdWxpbkBpbnRlbC5jb20+Cj4gQ2M6IE1pY2hhZWwgTGV1Y2h0ZW5i dXJnIDxtaWNoYWVsQHNsYXNoaG9tZS5vcmc+Cj4gQnVnemlsbGE6IGh0dHBzOi8vYnVncy5mcmVl ZGVza3RvcC5vcmcvc2hvd19idWcuY2dpP2lkPTg5Mzg4Cj4gU2lnbmVkLW9mZi1ieTogTWF0dCBS b3BlciA8bWF0dGhldy5kLnJvcGVyQGludGVsLmNvbT4KPiAtLS0KPiAgIGRyaXZlcnMvZ3B1L2Ry bS9pOTE1L2ludGVsX3BtLmMgfCAyMiArKysrKysrKysrKysrKysrKy0tLS0tCj4gICAxIGZpbGUg Y2hhbmdlZCwgMTcgaW5zZXJ0aW9ucygrKSwgNSBkZWxldGlvbnMoLSkKPgo+IGRpZmYgLS1naXQg YS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9wbS5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUv aW50ZWxfcG0uYwo+IGluZGV4IGU3MTBiNDMuLjkzZmQxNWYgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVy cy9ncHUvZHJtL2k5MTUvaW50ZWxfcG0uYwo+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2lu dGVsX3BtLmMKPiBAQCAtMTkyNCwxMyArMTkyNCwyNSBAQCBzdGF0aWMgdm9pZCBpbGtfY29tcHV0 ZV93bV9wYXJhbWV0ZXJzKHN0cnVjdCBkcm1fY3J0YyAqY3J0YywKPiAgIAlwLT5hY3RpdmUgPSB0 cnVlOwo+ICAgCXAtPnBpcGVfaHRvdGFsID0gaW50ZWxfY3J0Yy0+Y29uZmlnLT5iYXNlLmFkanVz dGVkX21vZGUuY3J0Y19odG90YWw7Cj4gICAJcC0+cGl4ZWxfcmF0ZSA9IGlsa19waXBlX3BpeGVs X3JhdGUoZGV2LCBjcnRjKTsKPiAtCXAtPnByaS5ieXRlc19wZXJfcGl4ZWwgPSBjcnRjLT5wcmlt YXJ5LT5zdGF0ZS0+ZmItPmJpdHNfcGVyX3BpeGVsIC8gODsKPiAtCXAtPmN1ci5ieXRlc19wZXJf cGl4ZWwgPSA0Owo+ICsKPiArCWlmIChjcnRjLT5wcmltYXJ5LT5zdGF0ZS0+ZmIpIHsKPiArCQlw LT5wcmkuZW5hYmxlZCA9IHRydWU7Cj4gKwkJcC0+cHJpLmJ5dGVzX3Blcl9waXhlbCA9Cj4gKwkJ CWNydGMtPnByaW1hcnktPnN0YXRlLT5mYi0+Yml0c19wZXJfcGl4ZWwgLyA4Owo+ICsJfSBlbHNl IHsKPiArCQlwLT5wcmkuZW5hYmxlZCA9IGZhbHNlOwo+ICsJCXAtPnByaS5ieXRlc19wZXJfcGl4 ZWwgPSAwOwo+ICsJfQo+ICsKPiArCWlmIChjcnRjLT5jdXJzb3ItPnN0YXRlLT5mYikgewo+ICsJ CXAtPmN1ci5lbmFibGVkID0gdHJ1ZTsKPiArCQlwLT5jdXIuYnl0ZXNfcGVyX3BpeGVsID0gNDsK PiArCX0gZWxzZSB7Cj4gKwkJcC0+Y3VyLmVuYWJsZWQgPSBmYWxzZTsKPiArCQlwLT5jdXIuYnl0 ZXNfcGVyX3BpeGVsID0gMDsKPiArCX0KPiAgIAlwLT5wcmkuaG9yaXpfcGl4ZWxzID0gaW50ZWxf Y3J0Yy0+Y29uZmlnLT5waXBlX3NyY193Owo+ICAgCXAtPmN1ci5ob3Jpel9waXhlbHMgPSBpbnRl bF9jcnRjLT5iYXNlLmN1cnNvci0+c3RhdGUtPmNydGNfdzsKPiAtCS8qIFRPRE86IGZvciBub3cs IGFzc3VtZSBwcmltYXJ5IGFuZCBjdXJzb3IgcGxhbmVzIGFyZSBhbHdheXMgZW5hYmxlZC4gKi8K PiAtCXAtPnByaS5lbmFibGVkID0gdHJ1ZTsKPiAtCXAtPmN1ci5lbmFibGVkID0gdHJ1ZTsKPgo+ ICAgCWRybV9mb3JfZWFjaF9sZWdhY3lfcGxhbmUocGxhbmUsICZkZXYtPm1vZGVfY29uZmlnLnBs YW5lX2xpc3QpIHsKPiAgIAkJc3RydWN0IGludGVsX3BsYW5lICppbnRlbF9wbGFuZSA9IHRvX2lu dGVsX3BsYW5lKHBsYW5lKTsKPgoKSG0gdGhlcmUgYXJlIG90aGVyIHBsYWNlIHdoaWNoIGRlcmVm ZXJlbmNlIGNydGMtPnByaW1hcnktPnN0YXRlLT5mYiwgCmxpa2UgaTl4eF91cGRhdGVfd20gYW5k IHNrbF9jb21wdXRlX3dtX3BpcGVfcGFyYW1ldGVycyAtIHRoZXkgd29uJ3QgCnN1ZmZlciB0aGUg c2FtZSBwcm9ibGVtPwoKUmVnYXJkcywKClR2cnRrbwoKX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhA bGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1h bi9saXN0aW5mby9pbnRlbC1nZngK