From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maarten Lankhorst Subject: Re: [PATCH 2/3] drm/i915: Protect dev_priv->atomic_cdclk_freq with all the crtc locks Date: Tue, 15 Nov 2016 11:14:29 +0100 Message-ID: References: <1479141311-11904-1-git-send-email-ville.syrjala@linux.intel.com> <1479141311-11904-3-git-send-email-ville.syrjala@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTPS id 0A7786E158 for ; Tue, 15 Nov 2016 10:14:32 +0000 (UTC) In-Reply-To: <1479141311-11904-3-git-send-email-ville.syrjala@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: ville.syrjala@linux.intel.com, intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T3AgMTQtMTEtMTYgb20gMTc6MzUgc2NocmVlZiB2aWxsZS5zeXJqYWxhQGxpbnV4LmludGVsLmNv bToKPiBGcm9tOiBWaWxsZSBTeXJqw6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29t Pgo+Cj4gQSBtb2Rlc2V0IG9uIG9uZSBwaXBlIGNhbiB1cGRhdGUgZGV2X3ByaXYtPmF0b21pY19j ZGNsa19mcmVxIHdpdGhvdXQKPiBhY3R1YWxseSB0b3VjaGluZyB0aGUgaGFyZHdhcmUsIGluIHdo aWNoIGNhc2Ugd2Ugd29uJ3QgZm9yY2UgYSBtb2Rlc2V0Cj4gb24gYWxsIHRoZSBwaXBlcywgYW5k IHRodXMgd29uJ3QgbG9jayBhbnkgb2YgdGhlIG90aGVyIHBpcGVzIGVpdGhlci4KPiBUaGF0IG1l YW5zIGEgcGFyYWxsZWwgcGxhbmUgdXBkYXRlIG9uIGFub3RoZXIgcGlwZSBjb3VsZCBiZSBsb29r aW5nIGF0Cj4gYSBzdGFsZSBkZXZfcHJpdi0+YXRvbWljX2NkY2Rsa19mcmVxIGFuZCB0aHVzIGZh aWwgdG8gbm90aWNlIHdoZW4gdGhlCj4gcGxhbmUgY29uZmlndXJhdGlvbiBpcyBpbnZhbGlkLCBv ciBwb3RlbnRpYWxseSByZWplY3QgYSB2YWxpZCB1cGRhdGUuCj4KPiBUbyBvdmVyY29tZSB0aGlz IHdlIG11c3QgcHJvdGVjdCB3cml0ZXMgdG8gYXRvbWljX2NkY2xrX2ZyZXEgd2l0aAo+IGFsbCB0 aGUgY3J0YyBsb2NrcywgYW5kIHRodXMgZm9yIHJlYWRzIGFueSBzaW5nbGUgY3J0YyBsb2NrIHdp bGwKPiBiZSBzdWZmaWNpZW50IHByb3RlY3Rpb24uCj4KPiBDYzogTWFhcnRlbiBMYW5raG9yc3Qg PG1hYXJ0ZW4ubGFua2hvcnN0QGxpbnV4LmludGVsLmNvbT4KPiBTaWduZWQtb2ZmLWJ5OiBWaWxs ZSBTeXJqw6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29tPgo+IC0tLQo+ICBkcml2 ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2Rydi5oICAgICAgfCAgOSArKysrKysrLQo+ICBkcml2ZXJz L2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMgfCA0MSArKysrKysrKysrKysrKysrKysrKysr KysrKysrKysrLS0tLS0KPiAgMiBmaWxlcyBjaGFuZ2VkLCA0NCBpbnNlcnRpb25zKCspLCA2IGRl bGV0aW9ucygtKQo+Cj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZHJ2 LmggYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2Rydi5oCj4gaW5kZXggYzBmMWRmYzcxMTll Li42NmQyOTUwZGM2NTcgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9k cnYuaAo+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZHJ2LmgKPiBAQCAtMTg3NCw3 ICsxODc0LDE0IEBAIHN0cnVjdCBkcm1faTkxNV9wcml2YXRlIHsKPiAgCj4gIAl1bnNpZ25lZCBp bnQgZnNiX2ZyZXEsIG1lbV9mcmVxLCBpc19kZHIzOwo+ICAJdW5zaWduZWQgaW50IHNrbF9wcmVm ZXJyZWRfdmNvX2ZyZXE7Cj4gLQl1bnNpZ25lZCBpbnQgY2RjbGtfZnJlcSwgbWF4X2NkY2xrX2Zy ZXEsIGF0b21pY19jZGNsa19mcmVxOwo+ICsJdW5zaWduZWQgaW50IGNkY2xrX2ZyZXEsIG1heF9j ZGNsa19mcmVxOwo+ICsKPiArCS8qCj4gKwkgKiBGb3IgcmVhZGluZyBob2xkaW5nIGFueSBjcnRj IGxvY2sgaXMgc3VmZmljaWVudCwKPiArCSAqIGZvciB3cml0aW5nIG11c3QgaG9sZCBhbGwgb2Yg dGhlbS4KPiArCSAqLwo+ICsJdW5zaWduZWQgaW50IGF0b21pY19jZGNsa19mcmVxOwo+ICsKPiAg CXVuc2lnbmVkIGludCBtYXhfZG90Y2xrX2ZyZXE7Cj4gIAl1bnNpZ25lZCBpbnQgcmF3Y2xrX2Zy ZXE7Cj4gIAl1bnNpZ25lZCBpbnQgaHBsbF9mcmVxOwo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dw dS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9k aXNwbGF5LmMKPiBpbmRleCA3MGYzZjBiNzAyNjMuLmQ3YTRiYzYzYjA1YiAxMDA2NDQKPiAtLS0g YS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMKPiArKysgYi9kcml2ZXJzL2dw dS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMKPiBAQCAtMTM5NDYsMTMgKzEzOTQ2LDMyIEBAIHN0 YXRpYyBpbnQgaGFzd2VsbF9tb2RlX3NldF9wbGFuZXNfd29ya2Fyb3VuZChzdHJ1Y3QgZHJtX2F0 b21pY19zdGF0ZSAqc3RhdGUpCj4gIAlyZXR1cm4gMDsKPiAgfQo+ICAKPiArc3RhdGljIGludCBp bnRlbF9sb2NrX2FsbF9waXBlcyhzdHJ1Y3QgZHJtX2F0b21pY19zdGF0ZSAqc3RhdGUpCj4gK3sK PiArCXN0cnVjdCBkcm1fY3J0YyAqY3J0YzsKPiArCj4gKwkvKiBBZGQgYWxsIHBpcGVzIHRvIHRo ZSBzdGF0ZSAqLwo+ICsJZm9yX2VhY2hfY3J0YyhzdGF0ZS0+ZGV2LCBjcnRjKSB7Cj4gKwkJc3Ry dWN0IGRybV9jcnRjX3N0YXRlICpjcnRjX3N0YXRlOwo+ICsKPiArCQljcnRjX3N0YXRlID0gZHJt X2F0b21pY19nZXRfY3J0Y19zdGF0ZShzdGF0ZSwgY3J0Yyk7Cj4gKwkJaWYgKElTX0VSUihjcnRj X3N0YXRlKSkKPiArCQkJcmV0dXJuIFBUUl9FUlIoY3J0Y19zdGF0ZSk7Cj4gKwl9Cj4gKwo+ICsJ cmV0dXJuIDA7Cj4gK30KPiArCj4gIHN0YXRpYyBpbnQgaW50ZWxfbW9kZXNldF9hbGxfcGlwZXMo c3RydWN0IGRybV9hdG9taWNfc3RhdGUgKnN0YXRlKQo+ICB7Cj4gIAlzdHJ1Y3QgZHJtX2NydGMg KmNydGM7Cj4gIAlzdHJ1Y3QgZHJtX2NydGNfc3RhdGUgKmNydGNfc3RhdGU7Cj4gIAlpbnQgcmV0 ID0gMDsKPiAgCj4gLQkvKiBhZGQgYWxsIGFjdGl2ZSBwaXBlcyB0byB0aGUgc3RhdGUgKi8KPiAr CS8qCj4gKwkgKiBBZGQgYWxsIHBpcGVzIHRvIHRoZSBzdGF0ZSwgYW5kIGZvcmNlCj4gKwkgKiBh IG1vZGVzZXQgb24gYWxsIHRoZSBhY3RpdmUgb25lcy4KPiArCSAqLwo+ICAJZm9yX2VhY2hfY3J0 YyhzdGF0ZS0+ZGV2LCBjcnRjKSB7Cj4gIAkJY3J0Y19zdGF0ZSA9IGRybV9hdG9taWNfZ2V0X2Ny dGNfc3RhdGUoc3RhdGUsIGNydGMpOwo+ICAJCWlmIChJU19FUlIoY3J0Y19zdGF0ZSkpCj4gQEAg LTE0MDE4LDEyICsxNDAzNywyNCBAQCBzdGF0aWMgaW50IGludGVsX21vZGVzZXRfY2hlY2tzKHN0 cnVjdCBkcm1fYXRvbWljX3N0YXRlICpzdGF0ZSkKPiAgCQlpZiAocmV0IDwgMCkKPiAgCQkJcmV0 dXJuIHJldDsKPiAgCj4gKwkJLyoKPiArCQkgKiBXcml0ZXMgdG8gZGV2X3ByaXYtPmF0b21pY19j ZGNsa19mcmVxIG11c3QgcHJvdGVjdGVkIGJ5Cj4gKwkJICogaG9sZGluZyBhbGwgdGhlIGNydGMg bG9ja3MsIGV2ZW4gaWYgd2UgZG9uJ3QgZW5kIHVwCj4gKwkJICogdG91Y2hpbmcgdGhlIGhhcmR3 YXJlCj4gKwkJICovCj4gKwkJaWYgKGludGVsX3N0YXRlLT5jZGNsayAhPSBkZXZfcHJpdi0+YXRv bWljX2NkY2xrX2ZyZXEpIHsKPiArCQkJcmV0ID0gaW50ZWxfbG9ja19hbGxfcGlwZXMoc3RhdGUp Owo+ICsJCQlpZiAocmV0IDwgMCkKPiArCQkJCXJldHVybiByZXQ7Cj4gKwkJfQo+ICsKV291bGQg aXQgYmUgdGVycmlibGUgdG8ganVzdCB1c2UgaW50ZWxfbW9kZXNldF9hbGxfcGlwZXMgaGVyZT8g U2luY2UgdGhpcyBjYW4gb25seSBiZSBkaWZmZXJlbnQgaW4gdGhlIGFsbCBjcnRjJ3MgZGlzYWJs ZWQgY2FzZQppdCB3b24ndCBtYXR0ZXIgbXVjaC4KCn5NYWFydGVuCl9fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50 ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9y Zy9tYWlsbWFuL2xpc3RpbmZvL2ludGVsLWdmeAo=