From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Raslan, KarimAllah" Subject: Re: [PATCH v5 13/13] KVM/nVMX: Use page_address_valid in a few more locations Date: Fri, 25 Jan 2019 18:17:12 +0000 Message-ID: <1548440232.17444.23.camel@amazon.de> References: <1547026933-31226-1-git-send-email-karahmed@amazon.de> <1547026933-31226-14-git-send-email-karahmed@amazon.de> <20190123181807.GS19289@Konrads-MacBook-Pro.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Cc: "linux-kernel@vger.kernel.org" , "kvm@vger.kernel.org" , "rkrcmar@redhat.com" , "pbonzini@redhat.com" To: "konrad.wilk@oracle.com" Return-path: In-Reply-To: <20190123181807.GS19289@Konrads-MacBook-Pro.local> Content-Language: en-US Content-ID: Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org T24gV2VkLCAyMDE5LTAxLTIzIGF0IDEzOjE4IC0wNTAwLCBLb25yYWQgUnplc3p1dGVrIFdpbGsg d3JvdGU6DQo+IE9uIFdlZCwgSmFuIDA5LCAyMDE5IGF0IDEwOjQyOjEzQU0gKzAxMDAsIEthcmlt QWxsYWggQWhtZWQgd3JvdGU6DQo+ID4gDQo+ID4gVXNlIHBhZ2VfYWRkcmVzc192YWxpZCBpbiBh IGZldyBtb3JlIGxvY2F0aW9ucyB0aGF0IGlzIGFscmVhZHkgY2hlY2tpbmcgZm9yDQo+ID4gYSBw YWdlIGFsaWduZWQgYWRkcmVzcyB0aGF0IGRvZXMgbm90IGNyb3NzIHRoZSBtYXhpbXVtIHBoeXNp Y2FsIGFkZHJlc3MuDQo+IA0KPiBXaGVyZSBpcyB0aGlzIHBhZ2VfYWRkcmVzc192YWxpZCBkZWNs YXJlZD8gVGhlIGxhdGVzdCBsaW51cydzIHRyZWUgZG9lcw0KPiBub3QgaGF2ZSBpdCwgbm9yIGRv ZXMgeW91ciBwYXRjaHNldD8NCg0KSXQgaXMgYWxyZWFkeSBkZWZpbmVkIGluIHRoZSBjb2RlLCBJ IGNhbiBub3Qgc2VlIGFueSBjb21taXRzIHRoYXQgcmVtb3ZlZCBpdDoNCg0KJCBnaXQgZ3JlcCBw YWdlX2FkZHJlc3NfdmFsaWQNCmFyY2gveDg2L2t2bS92bXgvbmVzdGVkLmM6c3RhdGljIGJvb2wg cGFnZV9hZGRyZXNzX3ZhbGlkKHN0cnVjdCBrdm1fdmNwdSAqdmNwdSwNCmdwYV90IGdwYSkNCmFy Y2gveDg2L2t2bS92bXgvbmVzdGVkLmM6wqDCoMKgwqDCoMKgaWYgKCFwYWdlX2FkZHJlc3NfdmFs aWQodmNwdSwgdm1jczEyLQ0KPmlvX2JpdG1hcF9hKSB8fA0KYXJjaC94ODYva3ZtL3ZteC9uZXN0 ZWQuYzrCoMKgwqDCoMKgwqDCoMKgwqDCoCFwYWdlX2FkZHJlc3NfdmFsaWQodmNwdSwgdm1jczEy LT5pb19iaXRtYXBfDQpiKSkNCmFyY2gveDg2L2t2bS92bXgvbmVzdGVkLmM6wqDCoMKgwqDCoMKg aWYgKCFwYWdlX2FkZHJlc3NfdmFsaWQodmNwdSwgdm1jczEyLQ0KPm1zcl9iaXRtYXApKQ0KYXJj aC94ODYva3ZtL3ZteC9uZXN0ZWQuYzrCoMKgwqDCoMKgwqBpZiAoIXBhZ2VfYWRkcmVzc192YWxp ZCh2Y3B1LCB2bWNzMTItDQo+dmlydHVhbF9hcGljX3BhZ2VfYWRkcikpDQphcmNoL3g4Ni9rdm0v dm14L25lc3RlZC5jOsKgwqDCoMKgwqDCoMKgwqDCoMKgIXBhZ2VfYWRkcmVzc192YWxpZCh2Y3B1 LCB2bWNzMTItDQo+YXBpY19hY2Nlc3NfYWRkcikpDQphcmNoL3g4Ni9rdm0vdm14L25lc3RlZC5j OsKgwqDCoMKgwqDCoMKgwqDCoMKgIXBhZ2VfYWRkcmVzc192YWxpZCh2Y3B1LCB2bWNzMTItDQo+ cG1sX2FkZHJlc3MpKQ0KYXJjaC94ODYva3ZtL3ZteC9uZXN0ZWQuYzrCoMKgwqDCoMKgwqBpZiAo IXBhZ2VfYWRkcmVzc192YWxpZCh2Y3B1LCB2bWNzMTItDQo+dm1yZWFkX2JpdG1hcCkgfHwNCmFy Y2gveDg2L2t2bS92bXgvbmVzdGVkLmM6wqDCoMKgwqDCoMKgwqDCoMKgwqAhcGFnZV9hZGRyZXNz X3ZhbGlkKHZjcHUsIHZtY3MxMi0NCj52bXdyaXRlX2JpdG1hcCkpDQphcmNoL3g4Ni9rdm0vdm14 L25lc3RlZC5jOsKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC oMKgwqAhcGFnZV9hZGRyZXNzX3ZhbGlkKHZjcHUsDQp2bWNzMTItPmVwdHBfbGlzdF9hZGRyZXNz KSkNCmFyY2gveDg2L2t2bS92bXgvbmVzdGVkLmM6wqDCoMKgwqDCoMKgaWYgKCFwYWdlX2FkZHJl c3NfdmFsaWQodmNwdSwgdm1jczEyLQ0KPnZtY3NfbGlua19wb2ludGVyKSkNCmFyY2gveDg2L2t2 bS92bXgvbmVzdGVkLmM6wqDCoMKgwqDCoMKgaWYgKCFwYWdlX2FkZHJlc3NfdmFsaWQodmNwdSwg a3ZtX3N0YXRlLQ0KPnZteC52bXhvbl9wYSkpDQphcmNoL3g4Ni9rdm0vdm14L25lc3RlZC5jOsKg wqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCFwYWdlX2FkZHJlc3NfdmFsaWQodmNw dSwga3ZtX3N0YXRlLQ0KPnZteC52bWNzX3BhKSkNCg0KPiA+IA0KPiA+IA0KPiA+IFNpZ25lZC1v ZmYtYnk6IEthcmltQWxsYWggQWhtZWQgPGthcmFobWVkQGFtYXpvbi5kZT4NCj4gPiAtLS0NCj4g PiAgYXJjaC94ODYva3ZtL3ZteC9uZXN0ZWQuYyB8IDYgKysrLS0tDQo+ID4gIDEgZmlsZSBjaGFu Z2VkLCAzIGluc2VydGlvbnMoKyksIDMgZGVsZXRpb25zKC0pDQo+ID4gDQo+ID4gZGlmZiAtLWdp dCBhL2FyY2gveDg2L2t2bS92bXgvbmVzdGVkLmMgYi9hcmNoL3g4Ni9rdm0vdm14L25lc3RlZC5j DQo+ID4gaW5kZXggY2NiM2I2My4uNzdhYWQ0NiAxMDA2NDQNCj4gPiAtLS0gYS9hcmNoL3g4Ni9r dm0vdm14L25lc3RlZC5jDQo+ID4gKysrIGIvYXJjaC94ODYva3ZtL3ZteC9uZXN0ZWQuYw0KPiA+ IEBAIC00MjAzLDcgKzQyMDMsNyBAQCBzdGF0aWMgaW50IGhhbmRsZV92bW9uKHN0cnVjdCBrdm1f dmNwdSAqdmNwdSkNCj4gPiAgCSAqIE5vdGUgLSBJQTMyX1ZNWF9CQVNJQ1s0OF0gd2lsbCBuZXZl ciBiZSAxIGZvciB0aGUgbmVzdGVkIGNhc2U7DQo+ID4gIAkgKiB3aGljaCByZXBsYWNlcyBwaHlz aWNhbCBhZGRyZXNzIHdpZHRoIHdpdGggMzINCj4gPiAgCSAqLw0KPiA+IC0JaWYgKCFQQUdFX0FM SUdORUQodm1wdHIpIHx8ICh2bXB0ciA+PiBjcHVpZF9tYXhwaHlhZGRyKHZjcHUpKSkNCj4gPiAr CWlmICghcGFnZV9hZGRyZXNzX3ZhbGlkKHZjcHUsIHZtcHRyKSkNCj4gPiAgCQlyZXR1cm4gbmVz dGVkX3ZteF9mYWlsSW52YWxpZCh2Y3B1KTsNCj4gPiAgDQo+ID4gIAlpZiAoa3ZtX3JlYWRfZ3Vl c3QodmNwdS0+a3ZtLCB2bXB0ciwgJnJldmlzaW9uLCBzaXplb2YocmV2aXNpb24pKSB8fA0KPiA+ IEBAIC00MjY2LDcgKzQyNjYsNyBAQCBzdGF0aWMgaW50IGhhbmRsZV92bWNsZWFyKHN0cnVjdCBr dm1fdmNwdSAqdmNwdSkNCj4gPiAgCWlmIChuZXN0ZWRfdm14X2dldF92bXB0cih2Y3B1LCAmdm1w dHIpKQ0KPiA+ICAJCXJldHVybiAxOw0KPiA+ICANCj4gPiAtCWlmICghUEFHRV9BTElHTkVEKHZt cHRyKSB8fCAodm1wdHIgPj4gY3B1aWRfbWF4cGh5YWRkcih2Y3B1KSkpDQo+ID4gKwlpZiAoIXBh Z2VfYWRkcmVzc192YWxpZCh2Y3B1LCB2bXB0cikpDQo+ID4gIAkJcmV0dXJuIG5lc3RlZF92bXhf ZmFpbFZhbGlkKHZjcHUsDQo+ID4gIAkJCVZNWEVSUl9WTUNMRUFSX0lOVkFMSURfQUREUkVTUyk7 DQo+ID4gIA0KPiA+IEBAIC00NDczLDcgKzQ0NzMsNyBAQCBzdGF0aWMgaW50IGhhbmRsZV92bXB0 cmxkKHN0cnVjdCBrdm1fdmNwdSAqdmNwdSkNCj4gPiAgCWlmIChuZXN0ZWRfdm14X2dldF92bXB0 cih2Y3B1LCAmdm1wdHIpKQ0KPiA+ICAJCXJldHVybiAxOw0KPiA+ICANCj4gPiAtCWlmICghUEFH RV9BTElHTkVEKHZtcHRyKSB8fCAodm1wdHIgPj4gY3B1aWRfbWF4cGh5YWRkcih2Y3B1KSkpDQo+ ID4gKwlpZiAoIXBhZ2VfYWRkcmVzc192YWxpZCh2Y3B1LCB2bXB0cikpDQo+ID4gIAkJcmV0dXJu IG5lc3RlZF92bXhfZmFpbFZhbGlkKHZjcHUsDQo+ID4gIAkJCVZNWEVSUl9WTVBUUkxEX0lOVkFM SURfQUREUkVTUyk7DQo+ID4gIA0KPiA+IC0tIA0KPiA+IDIuNy40DQo+ID4gDQoKCgpBbWF6b24g RGV2ZWxvcG1lbnQgQ2VudGVyIEdlcm1hbnkgR21iSApLcmF1c2Vuc3RyLiAzOAoxMDExNyBCZXJs aW4KR2VzY2hhZWZ0c2Z1ZWhyZXI6IENocmlzdGlhbiBTY2hsYWVnZXIsIFJhbGYgSGVyYnJpY2gK VXN0LUlEOiBERSAyODkgMjM3IDg3OQpFaW5nZXRyYWdlbiBhbSBBbXRzZ2VyaWNodCBDaGFybG90 dGVuYnVyZyBIUkIgMTQ5MTczIEIKCg==