From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Durrant Subject: Re: [RFC PATCH 07/12] hvmloader: allocate MMCONFIG area in the MMIO hole + minor code refactoring Date: Thu, 22 Mar 2018 09:29:44 +0000 Message-ID: References: <62c7a3037bb9f5631542f635dc36ae3338c7bf47.1520867741.git.x1917x@gmail.com> <20180319155802.immvsgqttrs3yezg@MacBook-Pro-de-Roger.local> <20180320054922.00005669@gmail.com> <20180320085048.orjv67qtwhe35ntl@MacBook-Pro-de-Roger.local> <20180321105840.00004698@gmail.com> <20180321090911.qd6epofdaywc4blb@MacBook-Pro-de-Roger.local> <20180322002540.000032f1@gmail.com> <20180321152017.bv32mt4bpxaa7zfr@MacBook-Pro-de-Roger.local> <20180322025656.00005281@gmail.com> <20180321171504.afl75q77kji34ybo@MacBook-Pro-de-Roger.local> <20180322084958.0000571c@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from all-amaz-eas1.inumbo.com ([34.197.232.57]) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1eywXn-0008Mx-FS for xen-devel@lists.xenproject.org; Thu, 22 Mar 2018 09:29:59 +0000 In-Reply-To: <20180322084958.0000571c@gmail.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: 'Alexey G' , Roger Pau Monne Cc: Stefano Stabellini , Wei Liu , Andrew Cooper , Jan Beulich , Ian Jackson , Anthony Perard , "xen-devel@lists.xenproject.org" List-Id: xen-devel@lists.xenproject.org PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQo+IEZyb206IEFsZXhleSBHIFttYWlsdG86eDE5 MTd4QGdtYWlsLmNvbV0KPiBTZW50OiAyMSBNYXJjaCAyMDE4IDIyOjUwCj4gVG86IFJvZ2VyIFBh dSBNb25uZSA8cm9nZXIucGF1QGNpdHJpeC5jb20+Cj4gQ2M6IHhlbi1kZXZlbEBsaXN0cy54ZW5w cm9qZWN0Lm9yZzsgQW5kcmV3IENvb3Blcgo+IDxBbmRyZXcuQ29vcGVyM0BjaXRyaXguY29tPjsg SWFuIEphY2tzb24gPElhbi5KYWNrc29uQGNpdHJpeC5jb20+OyBKYW4KPiBCZXVsaWNoIDxqYmV1 bGljaEBzdXNlLmNvbT47IFdlaSBMaXUgPHdlaS5saXUyQGNpdHJpeC5jb20+OyBQYXVsIER1cnJh bnQKPiA8UGF1bC5EdXJyYW50QGNpdHJpeC5jb20+OyBBbnRob255IFBlcmFyZCA8YW50aG9ueS5w ZXJhcmRAY2l0cml4LmNvbT47Cj4gU3RlZmFubyBTdGFiZWxsaW5pIDxzc3RhYmVsbGluaUBrZXJu ZWwub3JnPgo+IFN1YmplY3Q6IFJlOiBbWGVuLWRldmVsXSBbUkZDIFBBVENIIDA3LzEyXSBodm1s b2FkZXI6IGFsbG9jYXRlIE1NQ09ORklHCj4gYXJlYSBpbiB0aGUgTU1JTyBob2xlICsgbWlub3Ig Y29kZSByZWZhY3RvcmluZwo+IAo+IE9uIFdlZCwgMjEgTWFyIDIwMTggMTc6MTU6MDQgKzAwMDAK PiBSb2dlciBQYXUgTW9ubsOpIDxyb2dlci5wYXVAY2l0cml4LmNvbT4gd3JvdGU6Cj4gWy4uLl0K PiA+PiBBYm92ZSBzY2VuYXJpbyBtYWtlcyBpdCBvYnZpb3VzIHRoYXQgYXQgbGVhc3QgZm9yIFFF TVUgdGhlIE1NSU8tPlBDSQo+ID4+IGNvbmYgdHJhbnNsYXRpb24gaXMgYSByZWR1bmRhbnQgc3Rl cC4gV2h5IG5vdCB0byBhbGxvdyBzcGVjaWZ5aW5nCj4gPj4gZm9yIERNIHdoZXRoZXIgaXQgcHJl ZmVycyB0byByZWNlaXZlIE1NQ09ORklHIGFjY2Vzc2VzIGFzIG5hdGl2ZQo+ID4+IChNTUlPIG9u ZXMpIG9yIGFzIHRyYW5zbGF0ZWQgUENJIGNvbmYgaW9yZXFzPwo+ID4KPiA+WW91IGFyZSBqdXN0 IGFkZGluZyBhbiBleHRyYSBsZXZlbCBvZiBjb21wbGV4aXR5IHRvIGFuIGludGVyZmFjZQo+ID50 aGF0J3MgZmFpcmx5IHNpbXBsZS4gWW91IHJlZ2lzdGVyIGEgUENJIGRldmljZSB1c2luZwo+ID5Y RU5fRE1PUF9JT19SQU5HRV9QQ0kgYW5kIHlvdSBnZXQgSU9SRVFfVFlQRV9QQ0lfQ09ORklHIGlv cmVxcy4KPiAKPiBZZXMsIGFuZCBpdCBpcyBzdGlsbCBuZWVkZWQgYXMgd2UgaGF2ZSB0d28gZGlz dGluY3QgKGFuZCBub3QgZXF1YWwpCj4gaW50ZXJmYWNlcyB0byBQQ0kgY29uZiBzcGFjZS4gQXBh cnQgZnJvbSAwLi5GRmggcmFuZ2Ugb3ZlcmxhcHBpbmcgdGhleQo+IGNhbiBiZSBjb25zaWRlcmVk IHZlcnkgZGlmZmVyZW50IGludGVyZmFjZXMuIEFuZCB3aGV0aGVyIGl0IGlzIGEgcmVhbAo+IHN5 c3RlbSBvciBlbXVsYXRlZCAtLSB3ZSBjYW4gdXNlIGVpdGhlciBvbmUgb2YgdGhlc2UgdHdvIGlu dGVyZmFjZXMgb3IKPiBib3RoLgo+IAo+IEZvciBRRU1VIHplcm8gY2hhbmdlcyBhcmUgbmVlZGVk IHRvIHN1cHBvcnQgTU1DT05GSUcgTU1JTyBhY2Nlc3Nlcwo+IGlmCj4gdGhleSBjb21lIGFzIE1N SU8gaW9yZXFzLiBJdCdzIGp1c3Qgd2hhdCBpdHMgTU1DT05GSUcgZW11bGF0aW9uIGNvZGUKPiBl eHBlY3RzLgo+IEFueXdheSwgZm9yIChraW5kIG9mIHZhZ3VlKSB1c2VycyBvZiB0aGUgbXVsdGlw bGUgaW9yZXEgc2VydmVycwo+IGNhcGFiaWxpdHkgd2UgY2FuIGVuYWJsZSBNTUlPIHRyYW5zbGF0 aW9uIHRvIFBDSSBjb25mIGlvcmVxcy4gTm90ZSB0aGF0Cj4gYWN0dWFsbHkgdGhpcyBpcyBhbiBl eHRyYSBzdGVwLCBub3QgZm9yd2FyZGluZyB0cmFwcGVkIE1NQ09ORklHIE1NSU8KPiBhY2Nlc3Nl cyB0byB0aGUgc2VsZWN0ZWQgZGV2aWNlIG1vZGVsIGFzIGlzLgo+IAo+ID5HZXR0aW5nIGJvdGgg SU9SRVFfVFlQRV9QQ0lfQ09ORklHIGFuZCBJT1JFUV9UWVBFX0NPUFkgZm9yIFBDSQo+IGNvbmZp Zwo+ID5zcGFjZSBhY2Nlc3MgaXMgbWlzbGVhZGluZy4KPiAKPiBUaGVzZSBhcmUgdmVyeSBkaWZm ZXJlbnQgYWNjZXNzZXMsIGJvdGggaW4gdHJhbnNwb3J0IGFuZCBjYXBhYmlsaXRpZXMuCj4gCj4g PkluIGJvdGggY2FzZXMgWGVuIHdvdWxkIGhhdmUgdG8gZG8gdGhlIE1DRkcgYWNjZXNzIGRlY29k aW5nIGluIG9yZGVyCj4gPnRvIGZpZ3VyZSBvdXQgd2hpY2ggSU9SRVEgc2VydmVyIHdpbGwgaGFu ZGxlIHRoZSByZXF1ZXN0LiBBdCB3aGljaAo+ID5wb2ludCB0aGUgb25seSBzdGVwIHRoYXQgeW91 IGF2b2lkIGlzIHRoZSByZWNvbnN0cnVjdGlvbiBvZiB0aGUgbWVtb3J5Cj4gPmFjY2VzcyBmcm9t IHRoZSBJT1JFUV9UWVBFX1BDSV9DT05GSUcgd2hpY2ggaXMgdHJpdmlhbC4KPiAKPiBUaGUgInJl Y29uc3RydWN0aW9uIG9mIHRoZSBtZW1vcnkgYWNjZXNzIiB5b3UgbWVudGlvbmVkIHdvbid0IGJl IGVhc3kKPiBhY3R1YWxseS4gVGhlIHRoaW5nIGlzLCBhZGRyZXNzX3NwYWNlX3JlYWQvd3JpdGUg aXMgbm90IGFsbCB3aGF0IHdlCj4gbmVlZC4KPiAKPiBJbiBvcmRlciB0byB0cmFuc2xhdGUgUENJ IGNvbmYgaW9yZXFzIGJhY2sgdG8gZW11bGF0ZWQgTU1JTyBvcHMsIHdlCj4gbmVlZCB0byBiZSBh biBpbnZvbHZlZCBwYXJ0eSwgbWFpbmx5IHRvIGtub3cgd2hlcmUgTU1DT05GSUcgYXJlYSBpcwo+ IGxvY2F0ZWQgc28gd2UgY2FuIGNvbnN0cnVjdCB0aGUgYWRkcmVzcyB3aXRoaW4gaXRzIHJhbmdl IGZyb20gQkRGLgo+IFRoaXMgcGllY2Ugb2YgaW5mb3JtYXRpb24gaXMgZGVzdHJveWVkIGluIHRo ZSBwcm9jZXNzIG9mIE1NSU8gaW9yZXEKPiB0cmFuc2xhdGlvbiB0byBQQ0kgY29uZiB0eXBlLgo+ IAo+IFRoZSBjb2RlIHdoaWNoIHBhcnNlIFBDSSBjb25mIGlvcmVxcyBpbiB4ZW4taHZtLmMgZG9l c24ndCBrbm93IGFueXRoaW5nCj4gYWJvdXQgdGhlIGN1cnJlbnQgZW11bGF0ZWQgTU1DT05GSUcg c3RhdGUuIFRoZSBjb3JyZWN0IHdheSB0byBoYXZlIHRoaXMKPiBpbmZvIGlzIHRvIHBhcnRpY2lw YXRlIGluIGl0cyBlbXVsYXRpb24uIEFzIHdlIGRvbid0IHBhcnRpY2lwYXRlLCB3ZQo+IGhhdmUg bm8gb3RoZXIgd2F5IHRoYW4gdHJ5aW5nIHRvIGdhaW4gYmFja2Rvb3IgYWNjZXNzIHRvIFBDSUhv c3QgZmllbGRzCj4gdmlhIHRoaW5ncyBsaWtlIG9iamVjdF9yZXNvbHZlXyooKS4gVGhpcyBzb2x1 dGlvbiBpcyBjdW1iZXJzb21lIGFuZAo+IHVnbHkgYnV0IHdpbGwgd29yay4uLiBhbmQgbWF5IGJy ZWFrIGFueXRpbWUgZHVlIHRvIGNoYW5nZXMgaW4gUUVNVS4KPiAKPiBRRU1VIG1haW50YWluZXJz IHdpbGwgZ3JpbiB3aGlsZSBsb29raW5nIGF0IGFsbCB0aGlzIEknbSBhZnJhaWQgLS0KPiB0cmFw cGVkIE1NSU8gYWNjZXNzZXMgd2hpY2ggYXJlIHRyYW5zbGF0ZWQgdG8gUENJIGNvbmYgYWNjZXNz ZXMgd2hpY2gKPiBpbiB0dXJuIHRyYW5zbGF0ZWQgYmFjayB0byBlbXVsYXRlZCBNTUlPIGFjY2Vz c2VzIHVwb24gcmVjZWl2aW5nLCBhbG9uZwo+IHdpdGggdGVkaW91cyBhdHRlbXB0cyB0byBnYWlu IGFjY2VzcyB0byBNTUNPTkZJRy1yZWxhdGVkIGluZm8gYXMgd2UncmUKPiBub3QgaW52aXRlZCB0 byB0aGUgTU1DT05GSUcgZW11bGF0aW9uIHBhcnR5Lgo+IAo+IFRoZSBtb3JlIEkgdGhpbmsgYWJv dXQgaXQsIHRoZSBtb3JlIEkgbGlrZSB0aGUgZXhpc3RpbmcKPiBtYXBfaW9fcmFuZ2VfdG9faW9y ZXFfc2VydmVyKCkgYXBwcm9hY2guIDooIEl0IHdvcmtzIHdpdGhvdXQgZG9pbmcKPiBhbnl0aGlu Zywgbm8gaGFja3MsIG5vIG5ldyBpbnRlcmZhY2VzLCBib3RoIE1NQ09ORklHIGFuZCBDRjgvQ0ZD IGFyZQo+IHdvcmtpbmcgYXMgZXhwZWN0ZWQuIFRoZXJlIGlzIGEgcHJvYmxlbSB0byBtYWtlIGl0 IGNvbXBhdGlibGUgd2l0aAo+IHRoZSBzcGVjaWZpYyBtdWx0aXBsZSBpb3JlcSBzZXJ2ZXJzIGZl YXR1cmUsIGJ1dCBwcm92aWRpbmcgYSBuZXcKPiBkbW9wL2h5cGVyY2FsbCAod2hpY2ggeW91IHN1 Z2dlc3QgaXMgYSBtdXN0IGhhdmUgdGhpbmcgdG8gdHJhcCBNTUNPTkZJRwo+IE1NSU8gdG8gZ2l2 ZSBRRU1VIG9ubHkgdGhlIGZyZWVkb20gdG8gdGVsbCB3aGVyZSBpdCBpcyBsb2NhdGVkKSBhbGxv d3MKPiB0byBzb2x2ZSB0aGlzIHByb2JsZW0gaW4gYW55IHBvc3NpYmxlIHdheSwgZWl0aGVyIE1N SU8gLT4gUENJIGNvbmYKPiB0cmFuc2xhdGlvbiBvciBhbnl0aGluZyBlbHNlLgo+IAoKSSBkb24n dCB0aGluayB3ZSBldmVuIHdhbnQgUUVNVSB0byBoYXZlIHRoZSBmcmVlZG9tIHRvIHNheSB3aGVy ZSB0aGUgTU1DT05GSUcgYXJlYXMgYXJlIGxvY2F0ZWQsIGRvIHdlPyBRRU1VIGlzIG5vdCBpbiBj aGFyZ2Ugb2YgdGhlIGd1ZXN0IG1lbW9yeSBtYXAgYW5kIGl0IGlzIG5vdCByZXNwb25zaWJsZSBm b3IgdGhlIGJ1aWxkaW5nIHRoZSBNQ0ZHIHRhYmxlLCBYZW4gaXMuIFNvIGl0IHNob3VsZCBiZSBY ZW4gdGhhdCBkZWNpZGVzIHdoZXJlIHRoZSBNTUNPTkZJRyBhcmVhIGdvZXMgZm9yIGVhY2ggcmVn aXN0ZXJlZCBQQ0kgZGV2aWNlIGFuZCBpdCBzaG91bGQgYmUgWGVuIHRoYXQgYWRkcyB0aGF0IHRv IHRoZSBNQ0ZHIHRhYmxlLiBJdCBzaG91bGQgYmUgWGVuIHRoYXQgaGFuZGxlcyB0aGUgTU1DT05G SUcgTU1JTyBhY2Nlc3NlcyBhbmQgdGhlc2Ugc2hvdWxkIGJlIGZvcndhcmRlZCB0byBRRU1VIGFz IFBDSSBjb25maWcgSU9SRVFzLgpOb3csIGl0IG1heSBiZSB0aGF0IHdlIG5lZWQgdG8gaW50cm9k dWNlIGEgWGVuIHNwZWNpZmljIG1lY2hhbmlzbSBpbnRvIFFFTVUgdG8gdGhlbiByb3V0ZSB0aG9z ZSBjb25maWcgc3BhY2UgdHJhbnNhY3Rpb25zIHRvIHRoZSBkZXZpY2UgbW9kZWxzIGJ1dCB0aGF0 IHdvdWxkIGJlIGFuIGltcHJvdmVtZW50IG92ZXIgdGhlIGN1cnJlbnQgY2Y4L2NmYyBoYWNrZXJ5 IGFueXdheS4KCiAgUGF1bAoKPiA+PiBXZSBjYW4gc3RpbGwgcm91dGUgZWl0aGVyIGlvcmVxCj4g Pj4gdHlwZSB0byBtdWx0aXBsZSBkZXZpY2UgZW11bGF0b3JzIGFjY29yZGluZ2x5Lgo+ID4KPiA+ SXQncyBleGFjdGx5IHRoZSBzYW1lIHRoYXQncyBkb25lIGZvciBJTyBzcGFjZSBQQ0kgY29uZmln IHNwYWNlCj4gPmFkZHJlc3Nlcy4gUUVNVSBnZXRzIGFuIElPUkVRX1RZUEVfUENJX0NPTkZJRyBh bmQgaXQgcmVwbGF5cyB0aGUgSU8KPiA+c3BhY2UgYWNjZXNzIHVzaW5nIGRvX291dHAgYW5kIGNw dV9pb3JlcV9waW8uCj4gCj4gLi4uQW5kIGl0IGlzIGNvbXBsZXRlbHkgbGltaXRlZCB0byBiYXNp YyBQQ0kgY29uZiBzcGFjZS4gSSBkb24ndCBrbm93Cj4gdGhlIGNvbnRleHQgb2YgdGhpcyBsaW5l IGluIHhlbi1odm0uYzoKPiAKPiB2YWwgPSAoMXUgPDwgMzEpIHwgKChyZXEtPmFkZHIgJiAweDBm MDApIDw8IDE2KSB8ICgoc2JkZiAmIDB4ZmZmZikgPDwgOCkKPiAgICAgICAgfCAocmVxLT5hZGRy ICYgMHhmYyk7Cj4gCj4gYnV0IHNlZW1zIGxpa2UgY3VycmVudCBRRU1VIHZlcnNpb25zIGRvIG5v dCBleHBlY3QgYW55dGhpbmcgc2ltaWxhciB0bwo+IEFNRCBFQ1Mtc3R5bGUgYWNjZXNzZXMgZm9y IDBDRjhoLiBJdCBpcyBsaW1pdGVkIHRvIGJhc2ljIFBDSSBjb25mIG9ubHkuCj4gCj4gPklmIHlv dSB0aGluayB1c2luZyBJT1JFUV9UWVBFX0NPUFkgZm9yIE1DRkcgYWNjZXNzZXMgaXMgc3VjaCBh IGJlbmVmaXQKPiA+Zm9yIFFFTVUsIHdoeSBub3QganVzdCB0cmFuc2xhdGUgdGhlIElPUkVRX1RZ UEVfUENJX0NPTkZJRyBpbnRvCj4gPklPUkVRX1RZUEVfQ09QWSBpbiBoYW5kbGVfaW9yZXEgYW5k IGRpc3BhdGNoIGl0IHVzaW5nCj4gPmNwdV9pb3JlcV9tb3ZlPwo+IAo+IEFuc3dlcmVkIGFib3Zl LCB3ZSBuZWVkIHRvIHNvbWVob3cgaGF2ZSBhY2Nlc3MgdG8gdGhlIGluZm8gd2hpY2ggZG9uJ3QK PiBiZWxvbmcgdG8gdXMgZm9yIHRoaXMgc3RlcC4KPiAKPiA+VGhhbmtzLCBSb2dlci4KCl9fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fClhlbi1kZXZlbCBtYWls aW5nIGxpc3QKWGVuLWRldmVsQGxpc3RzLnhlbnByb2plY3Qub3JnCmh0dHBzOi8vbGlzdHMueGVu cHJvamVjdC5vcmcvbWFpbG1hbi9saXN0aW5mby94ZW4tZGV2ZWw=