From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= <ville.syrjala@linux.intel.com>
Subject: Re: [PATCH v3 1/2] drm/i915: Move SAGV block time to
 dev_priv
Date: Tue, 8 Oct 2019 14:29:59 +0300
Message-ID: <20191008112959.GO1208@intel.com>
References: <20190927222427.5491-1-james.ausmus@intel.com>
 <20191004221449.1317-1-james.ausmus@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
Return-path: <intel-gfx-bounces@lists.freedesktop.org>
Received: from mga18.intel.com (mga18.intel.com [134.134.136.126])
 by gabe.freedesktop.org (Postfix) with ESMTPS id 49A606E791
 for <intel-gfx@lists.freedesktop.org>; Tue,  8 Oct 2019 11:30:03 +0000 (UTC)
Content-Disposition: inline
In-Reply-To: <20191004221449.1317-1-james.ausmus@intel.com>
List-Unsubscribe: <https://lists.freedesktop.org/mailman/options/intel-gfx>,
 <mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <https://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <https://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
 <mailto:intel-gfx-request@lists.freedesktop.org?subject=subscribe>
Errors-To: intel-gfx-bounces@lists.freedesktop.org
Sender: "Intel-gfx" <intel-gfx-bounces@lists.freedesktop.org>
To: James Ausmus <james.ausmus@intel.com>
Cc: intel-gfx@lists.freedesktop.org, Lucas De Marchi <lucas.demarchi@intel.com>
List-Id: intel-gfx@lists.freedesktop.org

T24gRnJpLCBPY3QgMDQsIDIwMTkgYXQgMDM6MTQ6NDhQTSAtMDcwMCwgSmFtZXMgQXVzbXVzIHdy
b3RlOgo+IEluIHByZXAgZm9yIG5ld2VyIHBsYXRmb3JtcyBoYXZpbmcgbW9yZSBjb21wbGljYXRl
ZCB3YXlzIHRvIGRldGVybWluZQo+IHRoZSBTQUdWIGJsb2NrIHRpbWUsIG1vdmUgdGhlIHZhcmlh
YmxlIHRvIGRldl9wcml2LCBhbmQgZXh0cmFjdCB0aGUKPiBzZXR0aW5nIHRvIGFuIGluaXRpYWwg
c2V0dXAgZnVuY3Rpb24uIFdoaWxlIHdlJ3JlIGF0IGl0LCB1cGRhdGUgdGhlIGlmCj4gbGFkZGVy
IHRvIGZvbGxvdyB0aGUgbmV3IGdlbiAtPiBvbGQgZ2VuIG9yZGVyIHByZWZlcmVuY2UsIGFuZCB3
YXJuIG9uCj4gYW55IG5vbi1zcGVjaWZpZWQgZ2VuLgo+IAo+IHYyOiBTaG9ydGVuIHRoZSBmdW5j
dGlvbiBuYW1lIChWaWxsZSksIHJldHVybiBkaXJlY3RseSAoVmlsbGUpLCBtb3ZlCj4gc2Fndl9i
bG9ja190aW1lX3VzIHZhbHVlIHRvIGRldl9wcml2IChWaWxsZSkKPiAKPiB2MzogQ2hhbmdlIHNh
Z3ZfYmxvY2tfdGltZV91cyB0byB1MzIgKEx1Y2FzKSwgQ2hhbmdlIGZhbGxiYWNrIHZhbHVlIHRv
Cj4gLTEgKEx1Y2FzKSwgdXNlIGludGVsX2hhc19zYWd2IGZvciBzZXR1cCBjaGVjayByYXRoZXIg
dGhhbiBoYW5kLXJvbGxpbmcKPiAoTHVjYXMpCj4gCj4gQ2M6IFZpbGxlIFN5cmrDpGzDpCA8dmls
bGUuc3lyamFsYUBsaW51eC5pbnRlbC5jb20+Cj4gQ2M6IFN0YW5pc2xhdiBMaXNvdnNraXkgPHN0
YW5pc2xhdi5saXNvdnNraXlAaW50ZWwuY29tPgo+IENjOiBMdWNhcyBEZSBNYXJjaGkgPGx1Y2Fz
LmRlbWFyY2hpQGludGVsLmNvbT4KPiBTaWduZWQtb2ZmLWJ5OiBKYW1lcyBBdXNtdXMgPGphbWVz
LmF1c211c0BpbnRlbC5jb20+Cj4gLS0tCj4gIGRyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZHJ2
LmggfCAgMiArKwo+ICBkcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9wbS5jIHwgMzMgKysrKysr
KysrKysrKysrKysrKysrKysrLS0tLS0tLS0tCj4gIDIgZmlsZXMgY2hhbmdlZCwgMjYgaW5zZXJ0
aW9ucygrKSwgOSBkZWxldGlvbnMoLSkKPiAKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJt
L2k5MTUvaTkxNV9kcnYuaCBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZHJ2LmgKPiBpbmRl
eCBjZGU0YzdmYjU1NzAuLjFkOWE5ZTgyNzI2MSAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9k
cm0vaTkxNS9pOTE1X2Rydi5oCj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9kcnYu
aAo+IEBAIC0xNTYwLDYgKzE1NjAsOCBAQCBzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0ZSB7Cj4gIAkJ
STkxNV9TQUdWX05PVF9DT05UUk9MTEVECj4gIAl9IHNhZ3Zfc3RhdHVzOwo+ICAKPiArCXUzMiBz
YWd2X2Jsb2NrX3RpbWVfdXM7Cj4gKwoKdTMyIHNlZW1zIGEgYml0IGV4Y2Vzc2l2ZS4gQWx0aG91
Z2ggdGhlIHBjb2RlIGNvbW1hbmQgZG9lc24ndCBkb2N1bWVudAphbnkgdXBwZXIgYm91bmQgc28g
bWF5YmUgdTMyIGlzIHJlYWxseSB0aGUgY29ycmVjdCBjaG9pY2UgaGVyZS4gQW5kIHdlJ2QKbmVl
ZCB0byBwdXQgaXQgc29tZXdoZXJlIGVsc2UgdG8gYXZvaWQgdGhlIGhvbGUgYW55d2F5LgoKU2Vy
aWVzIGlzClJldmlld2VkLWJ5OiBWaWxsZSBTeXJqw6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXgu
aW50ZWwuY29tPgoKPiAgCXN0cnVjdCB7Cj4gIAkJLyoKPiAgCQkgKiBSYXcgd2F0ZXJtYXJrIGxh
dGVuY3kgdmFsdWVzOgo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9w
bS5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcG0uYwo+IGluZGV4IGJmY2YwM2FiNTI0
NS4uMGZmY2FmZTk3MjE2IDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVs
X3BtLmMKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9wbS5jCj4gQEAgLTM2NDIs
NiArMzY0MiwyNiBAQCBpbnRlbF9oYXNfc2FndihzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0ZSAqZGV2
X3ByaXYpCj4gIAkJZGV2X3ByaXYtPnNhZ3Zfc3RhdHVzICE9IEk5MTVfU0FHVl9OT1RfQ09OVFJP
TExFRDsKPiAgfQo+ICAKPiArc3RhdGljIHZvaWQKPiArc2tsX3NldHVwX3NhZ3ZfYmxvY2tfdGlt
ZShzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0ZSAqZGV2X3ByaXYpCj4gK3sKPiArCWlmIChJU19HRU4o
ZGV2X3ByaXYsIDExKSkgewo+ICsJCWRldl9wcml2LT5zYWd2X2Jsb2NrX3RpbWVfdXMgPSAxMDsK
PiArCQlyZXR1cm47Cj4gKwl9IGVsc2UgaWYgKElTX0dFTihkZXZfcHJpdiwgMTApKSB7Cj4gKwkJ
ZGV2X3ByaXYtPnNhZ3ZfYmxvY2tfdGltZV91cyA9IDIwOwo+ICsJCXJldHVybjsKPiArCX0gZWxz
ZSBpZiAoSVNfR0VOKGRldl9wcml2LCA5KSkgewo+ICsJCWRldl9wcml2LT5zYWd2X2Jsb2NrX3Rp
bWVfdXMgPSAzMDsKPiArCQlyZXR1cm47Cj4gKwl9IGVsc2Ugewo+ICsJCU1JU1NJTkdfQ0FTRShJ
TlRFTF9HRU4oZGV2X3ByaXYpKTsKPiArCX0KPiArCj4gKwkvKiBEZWZhdWx0IHRvIGFuIHVudXNh
YmxlIGJsb2NrIHRpbWUgKi8KPiArCWRldl9wcml2LT5zYWd2X2Jsb2NrX3RpbWVfdXMgPSAtMTsK
PiArfQo+ICsKPiAgLyoKPiAgICogU0FHViBkeW5hbWljYWxseSBhZGp1c3RzIHRoZSBzeXN0ZW0g
YWdlbnQgdm9sdGFnZSBhbmQgY2xvY2sgZnJlcXVlbmNpZXMKPiAgICogZGVwZW5kaW5nIG9uIHBv
d2VyIGFuZCBwZXJmb3JtYW5jZSByZXF1aXJlbWVudHMuIFRoZSBkaXNwbGF5IGVuZ2luZSBhY2Nl
c3MKPiBAQCAtMzczMCwxOCArMzc1MCwxMCBAQCBib29sIGludGVsX2Nhbl9lbmFibGVfc2Fndihz
dHJ1Y3QgaW50ZWxfYXRvbWljX3N0YXRlICpzdGF0ZSkKPiAgCXN0cnVjdCBpbnRlbF9jcnRjX3N0
YXRlICpjcnRjX3N0YXRlOwo+ICAJZW51bSBwaXBlIHBpcGU7Cj4gIAlpbnQgbGV2ZWwsIGxhdGVu
Y3k7Cj4gLQlpbnQgc2Fndl9ibG9ja190aW1lX3VzOwo+ICAKPiAgCWlmICghaW50ZWxfaGFzX3Nh
Z3YoZGV2X3ByaXYpKQo+ICAJCXJldHVybiBmYWxzZTsKPiAgCj4gLQlpZiAoSVNfR0VOKGRldl9w
cml2LCA5KSkKPiAtCQlzYWd2X2Jsb2NrX3RpbWVfdXMgPSAzMDsKPiAtCWVsc2UgaWYgKElTX0dF
TihkZXZfcHJpdiwgMTApKQo+IC0JCXNhZ3ZfYmxvY2tfdGltZV91cyA9IDIwOwo+IC0JZWxzZQo+
IC0JCXNhZ3ZfYmxvY2tfdGltZV91cyA9IDEwOwo+IC0KPiAgCS8qCj4gIAkgKiBJZiB0aGVyZSBh
cmUgbm8gYWN0aXZlIENSVENzLCBubyBhZGRpdGlvbmFsIGNoZWNrcyBuZWVkIGJlIHBlcmZvcm1l
ZAo+ICAJICovCj4gQEAgLTM3ODgsNyArMzgwMCw3IEBAIGJvb2wgaW50ZWxfY2FuX2VuYWJsZV9z
YWd2KHN0cnVjdCBpbnRlbF9hdG9taWNfc3RhdGUgKnN0YXRlKQo+ICAJCSAqIGluY3VyIG1lbW9y
eSBsYXRlbmNpZXMgaGlnaGVyIHRoYW4gc2Fndl9ibG9ja190aW1lX3VzIHdlCj4gIAkJICogY2Fu
J3QgZW5hYmxlIFNBR1YuCj4gIAkJICovCj4gLQkJaWYgKGxhdGVuY3kgPCBzYWd2X2Jsb2NrX3Rp
bWVfdXMpCj4gKwkJaWYgKGxhdGVuY3kgPCBkZXZfcHJpdi0+c2Fndl9ibG9ja190aW1lX3VzKQo+
ICAJCQlyZXR1cm4gZmFsc2U7Cj4gIAl9Cj4gIAo+IEBAIC05MDEzLDYgKzkwMjUsOSBAQCB2b2lk
IGludGVsX2luaXRfcG0oc3RydWN0IGRybV9pOTE1X3ByaXZhdGUgKmRldl9wcml2KQo+ICAJZWxz
ZSBpZiAoSVNfR0VOKGRldl9wcml2LCA1KSkKPiAgCQlpOTE1X2lyb25sYWtlX2dldF9tZW1fZnJl
cShkZXZfcHJpdik7Cj4gIAo+ICsJaWYgKGludGVsX2hhc19zYWd2KGRldl9wcml2KSkKPiArCQlz
a2xfc2V0dXBfc2Fndl9ibG9ja190aW1lKGRldl9wcml2KTsKPiArCj4gIAkvKiBGb3IgRklGTyB3
YXRlcm1hcmsgdXBkYXRlcyAqLwo+ICAJaWYgKElOVEVMX0dFTihkZXZfcHJpdikgPj0gOSkgewo+
ICAJCXNrbF9zZXR1cF93bV9sYXRlbmN5KGRldl9wcml2KTsKPiAtLSAKPiAyLjIyLjEKCi0tIApW
aWxsZSBTeXJqw6Rsw6QKSW50ZWwKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRl
c2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8v
aW50ZWwtZ2Z4