From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Durrant Subject: Re: [PATCH 1/2] x86/mm/p2m: don't needlessly limit MMIO mapping order to 4k Date: Mon, 29 Oct 2018 18:03:12 +0000 Message-ID: <2ff1b635f7224a57bc75baaf79e054f4@AMSPEX02CL03.citrite.net> References: <20181029180211.2155-1-paul.durrant@citrix.com> <20181029180211.2155-3-paul.durrant@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from us1-rack-dfw2.inumbo.com ([104.130.134.6]) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1gHBsh-0000tr-I5 for xen-devel@lists.xenproject.org; Mon, 29 Oct 2018 18:03:15 +0000 In-Reply-To: <20181029180211.2155-3-paul.durrant@citrix.com> Content-Language: en-US List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Sender: "Xen-devel" To: Paul Durrant , "xen-devel@lists.xenproject.org" Cc: Andrew Cooper , Wei Liu , Jan Beulich List-Id: xen-devel@lists.xenproject.org QXBvbG9naWVzLiBJZ25vcmUgdGhpcyBhcyBpdCBpcyBqdXN0IGEgcmUtcG9zdCBvZiBhIHN0YWxl IHBhdGNoLgoKICBQYXVsCgo+IC0tLS0tT3JpZ2luYWwgTWVzc2FnZS0tLS0tCj4gRnJvbTogUGF1 bCBEdXJyYW50IFttYWlsdG86cGF1bC5kdXJyYW50QGNpdHJpeC5jb21dCj4gU2VudDogMjkgT2N0 b2JlciAyMDE4IDE4OjAyCj4gVG86IHhlbi1kZXZlbEBsaXN0cy54ZW5wcm9qZWN0Lm9yZwo+IENj OiBQYXVsIER1cnJhbnQgPFBhdWwuRHVycmFudEBjaXRyaXguY29tPjsgSmFuIEJldWxpY2gKPiA8 amJldWxpY2hAc3VzZS5jb20+OyBBbmRyZXcgQ29vcGVyIDxBbmRyZXcuQ29vcGVyM0BjaXRyaXgu Y29tPjsgV2VpIExpdQo+IDx3ZWkubGl1MkBjaXRyaXguY29tPgo+IFN1YmplY3Q6IFtQQVRDSCAx LzJdIHg4Ni9tbS9wMm06IGRvbid0IG5lZWRsZXNzbHkgbGltaXQgTU1JTyBtYXBwaW5nIG9yZGVy Cj4gdG8gNGsKPiAKPiBUaGUgUDJNIGNvbW1vbiBjb2RlIGN1cnJlbnRseSByZXN0cmljdHMgdGhl IE1NSU8gbWFwcGluZyBvcmRlciBvZiBhbnkKPiBkb21haW4gd2l0aCBJT01NVSBtYXBwaW5ncywg YnV0IHRoYXQgaXMgbm90IHVzaW5nIHNoYXJlZCB0YWJsZXMsIHRvIDRrLgo+IFRoaXMgaGFzIGJl ZW4gc2hvd24gdG8gaGF2ZSBhIGh1Z2UgcGVyZm9ybWFuY2UgY29zdCB3aGVuIHBhc3NpbmcgdGhy b3VnaAo+IGEgUENJIGRldmljZSB3aXRoIGEgdmVyeSBsYXJnZSBCQVIgKGUuZy4gTlZJRElBIFA0 MCksIGluY3JlYXNpbmcgdGhlIGd1ZXN0Cj4gYm9vdCB0aW1lIGZyb20gfjIwcyB0byBzZXZlcmFs IG1pbnV0ZXMgd2hlbiBpb21tdT1uby1zaGFyZXB0IGlzIHNwZWNpZmllZAo+IG9uIHRoZSBYZW4g Y29tbWFuZCBsaW5lLgo+IAo+IFRoZSBsaW1pdGF0aW9uIHdhcyBhZGRlZCBieSBjb21taXQgYzNj NzU2YmQgIng4Ni9wMm06IHVzZSBsYXJnZSBwYWdlcwo+IGZvciBNTUlPIG1hcHBpbmdzIiBob3dl dmVyIHRoZSB1bmRlcmx5aW5nIGltcGxlbWVudGF0aW9ucyBvZiBwMm0tCj4gPnNldF9lbnRyeQo+ IGZvciBJbnRlbCBhbmQgQU1EIGFyZSBjb2RlZCB0byBjb3BlIHdpdGggbWFwcGluZyBvcmRlcnMg aGlnaGVyIHRoYW4gNGssCj4gZXZlbiB0aG91Z2ggdGhlIElPTU1VIG1hcHBpbmcgZnVuY3Rpb24g aXMgaXRzZWxmIGN1cnJlbnRseSBsaW1pdGVkIHRvIDRrLAo+IHNvIHRoZXJlIGlzIG5vIHJlYWwg bmVlZCB0byBsaW1pdCB0aGUgb3JkZXIgcGFzc2VkIGludG8gdGhlIG1ldGhvZCwgb3RoZXIKPiB0 aGFuIHRvIGF2b2lkIGEgcG90ZW50aWFsIERvUyBjYXVzZWQgYnkgYSBsb25nIHJ1bm5pbmcgaHlw ZXJjYWxsLgo+IAo+IEluIHByYWN0aWNlLCBtbWlvX29yZGVyKCkgYWxyZWFkeSBzdHJpY3RseSBk aXNhbGxvd3MgMUcgbWFwcGluZ3Mgc2luY2UgdGhlCj4gaWYgY2xhdXNlIGluIHF1ZXN0aW9uIHN0 YXJ0cyB3aXRoOgo+IAo+ICAgICBpZiAoIDAgLyoKPiAgICAgICAgICAgICAqIERvbid0IHVzZSAx R2IgcGFnZXMsIHRvIGxpbWl0IHRoZSBpdGVyYXRpb24gY291bnQgaW4KPiAgICAgICAgICAgICAq IHNldF90eXBlZF9wMm1fZW50cnkoKSB3aGVuIGl0IG5lZWRzIHRvIHphcCBNMlAgZW50cmllcwo+ ICAgICAgICAgICAgICogZm9yIGEgUkFNIHJhbmdlLgo+ICAgICAgICAgICAgICovICYmCj4gCj4g V2l0aCB0aGlzIHBhdGNoIGFwcGxpZWQgKGFuZCBoZW5jZSAyTSBtYXBwaW5ncyBpbiB1c2UpIHRo ZSBWTSBib290IHRpbWUgaXMKPiByZXN0b3JlZCB0byBzb21ldGhpbmcgcmVhc29uYWJsZS4gTm90 IGFzIGZhc3QgYXMgd2l0aG91dCBpb21tdT1uby1zaGFyZXB0LAo+IGJ1dCB3aXRoaW4gYSBmZXcg c2Vjb25kcyBvZiBpdC4KPiAKPiBOT1RFOiBUaGlzIHBhdGNoIHRha2VzIHRoZSBvcHBvcnR1bml0 eSB0byBzaG9ydGVuIGEgY291cGxlIG9mID4gODAKPiAgICAgICBjaGFyYWN0ZXIgbGluZXMgaW4g dGhlIGNvZGUuCj4gCj4gU2lnbmVkLW9mZi1ieTogUGF1bCBEdXJyYW50IDxwYXVsLmR1cnJhbnRA Y2l0cml4LmNvbT4KPiBBY2tlZC1ieTogR2VvcmdlIER1bmxhcCA8Z2VvcmdlLmR1bmxhcEBldS5j aXRyaXguY29tPgo+IC0tLQo+IENjOiBKYW4gQmV1bGljaCA8amJldWxpY2hAc3VzZS5jb20+Cj4g Q2M6IEFuZHJldyBDb29wZXIgPGFuZHJldy5jb29wZXIzQGNpdHJpeC5jb20+Cj4gQ2M6IFdlaSBM aXUgPHdlaS5saXUyQGNpdHJpeC5jb20+Cj4gCj4gdjI6Cj4gIC0gQWRkIGFuIGV4dHJhIHRvIHRo ZSBpZiBjbGF1c2UgZGlzYWxsb3dpbmcgMUcgbWFwcGluZ3MgdG8gbWFrZSBzdXJlCj4gICAgdGhl eSBhcmUgbm90IHVzZWQgaWYgbmVlZF9pb21tdV9wdF9zeW5jKCkgaXMgdHJ1ZSwgZXZlbiB0aG91 Z2ggdGhlCj4gICAgY2hlY2sgaXMgY3VycmVudGx5IG1vb3QgKHNlZSBtYWluIGNvbW1lbnQpLgo+ IC0tLQo+ICB4ZW4vYXJjaC94ODYvbW0vcDJtLmMgfCAxOSArKysrKysrKysrLS0tLS0tLS0tCj4g IDEgZmlsZSBjaGFuZ2VkLCAxMCBpbnNlcnRpb25zKCspLCA5IGRlbGV0aW9ucygtKQo+IAo+IGRp ZmYgLS1naXQgYS94ZW4vYXJjaC94ODYvbW0vcDJtLmMgYi94ZW4vYXJjaC94ODYvbW0vcDJtLmMK PiBpbmRleCBhMDBhM2MxYmZmLi5mOTcyYjQ4MTlkIDEwMDY0NAo+IC0tLSBhL3hlbi9hcmNoL3g4 Ni9tbS9wMm0uYwo+ICsrKyBiL3hlbi9hcmNoL3g4Ni9tbS9wMm0uYwo+IEBAIC0yMDgxLDE0ICsy MDgxLDExIEBAIHN0YXRpYyB1bnNpZ25lZCBpbnQgbW1pb19vcmRlcihjb25zdCBzdHJ1Y3QgZG9t YWluCj4gKmQsCj4gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICB1bnNpZ25lZCBsb25n IHN0YXJ0X2ZuLCB1bnNpZ25lZCBsb25nIG5yKQo+ICB7Cj4gICAgICAvKgo+IC0gICAgICogTm90 ZSB0aGF0IHRoZSAhaW9tbXVfdXNlX2hhcF9wdCgpIGhlcmUgaGFzIHRocmVlIGVmZmVjdHM6Cj4g LSAgICAgKiAtIGNvdmVyIGlvbW11X3ssdW59bWFwX3BhZ2UoKSBub3QgaGF2aW5nIGFuICJvcmRl ciIgaW5wdXQgeWV0LAo+IC0gICAgICogLSBleGNsdWRlIHNoYWRvdyBtb2RlICh3aGljaCBkb2Vz bid0IHN1cHBvcnQgbGFyZ2UgTU1JTyBtYXBwaW5ncyksCj4gLSAgICAgKiAtIGV4Y2x1ZGUgUFYg Z3Vlc3RzLCBzaG91bGQgZXhlY3V0aW9uIHJlYWNoIHRoaXMgY29kZSBmb3Igc3VjaC4KPiAtICAg ICAqIFNvIGJlIGNhcmVmdWwgd2hlbiBhbHRlcmluZyB0aGlzLgo+ICsgICAgICogUFYgZ3Vlc3Rz IG9yIHNoYWRvdy1tb2RlIEhWTSBndWVzdHMgbXVzdCBiZSByZXN0cmljdGVkIHRvIDRrCj4gKyAg ICAgKiBtYXBwaW5ncy4KPiAgICAgICAqLwo+IC0gICAgaWYgKCAhaW9tbXVfdXNlX2hhcF9wdChk KSB8fAo+IC0gICAgICAgICAoc3RhcnRfZm4gJiAoKDFVTCA8PCBQQUdFX09SREVSXzJNKSAtIDEp KSB8fCAhKG5yID4+Cj4gUEFHRV9PUkRFUl8yTSkgKQo+ICsgICAgaWYgKCAhaGFwX2VuYWJsZWQo ZCkgfHwgKHN0YXJ0X2ZuICYgKCgxVUwgPDwgUEFHRV9PUkRFUl8yTSkgLSAxKSkgfHwKPiArICAg ICAgICAgIShuciA+PiBQQUdFX09SREVSXzJNKSApCj4gICAgICAgICAgcmV0dXJuIFBBR0VfT1JE RVJfNEs7Cj4gCj4gICAgICBpZiAoIDAgLyoKPiBAQCAtMjA5Niw4ICsyMDkzLDEyIEBAIHN0YXRp YyB1bnNpZ25lZCBpbnQgbW1pb19vcmRlcihjb25zdCBzdHJ1Y3QgZG9tYWluCj4gKmQsCj4gICAg ICAgICAgICAgICogc2V0X3R5cGVkX3AybV9lbnRyeSgpIHdoZW4gaXQgbmVlZHMgdG8gemFwIE0y UCBlbnRyaWVzCj4gICAgICAgICAgICAgICogZm9yIGEgUkFNIHJhbmdlLgo+ICAgICAgICAgICAg ICAqLyAmJgo+IC0gICAgICAgICAhKHN0YXJ0X2ZuICYgKCgxVUwgPDwgUEFHRV9PUkRFUl8xRykg LSAxKSkgJiYgKG5yID4+Cj4gUEFHRV9PUkRFUl8xRykgJiYKPiAtICAgICAgICAgaGFwX2hhc18x Z2IgKQo+ICsgICAgICAgICAhKHN0YXJ0X2ZuICYgKCgxVUwgPDwgUEFHRV9PUkRFUl8xRykgLSAx KSkgJiYKPiArICAgICAgICAgKG5yID4+IFBBR0VfT1JERVJfMUcpICYmCj4gKyAgICAgICAgIGhh cF9oYXNfMWdiICYmCj4gKyAgICAgICAgIC8qIGRpc2FibGUgMUcgbWFwcGluZ3MgaWYgd2UgbmVl ZCB0byBrZWVwIHRoZSBJT01NVSBpbiBzeW5jICovCj4gKyAgICAgICAgICFuZWVkX2lvbW11X3B0 X3N5bmMoZCkKPiArICAgICAgICApCj4gICAgICAgICAgcmV0dXJuIFBBR0VfT1JERVJfMUc7Cj4g Cj4gICAgICBpZiAoIGhhcF9oYXNfMm1iICkKPiAtLQo+IDIuMTEuMAoKCl9fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fClhlbi1kZXZlbCBtYWlsaW5nIGxpc3QK WGVuLWRldmVsQGxpc3RzLnhlbnByb2plY3Qub3JnCmh0dHBzOi8vbGlzdHMueGVucHJvamVjdC5v cmcvbWFpbG1hbi9saXN0aW5mby94ZW4tZGV2ZWw=