From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roger Pau =?utf-8?B?TW9ubsOp?= Subject: Re: [RFC PATCH 07/12] hvmloader: allocate MMCONFIG area in the MMIO hole + minor code refactoring Date: Thu, 22 Mar 2018 09:57:16 +0000 Message-ID: <20180322095716.5y2hdyh6xp7nto3g@MacBook-Pro-de-Roger.local> References: <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 us1-rack-dfw2.inumbo.com ([104.130.134.6]) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1eywyQ-0002GZ-Qw for xen-devel@lists.xenproject.org; Thu, 22 Mar 2018 09:57:30 +0000 Content-Disposition: inline In-Reply-To: <20180322084958.0000571c@gmail.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Sender: "Xen-devel" To: Alexey G Cc: Stefano Stabellini , Wei Liu , Andrew Cooper , Ian Jackson , Paul Durrant , Jan Beulich , Anthony Perard , xen-devel@lists.xenproject.org List-Id: xen-devel@lists.xenproject.org T24gVGh1LCBNYXIgMjIsIDIwMTggYXQgMDg6NDk6NThBTSArMTAwMCwgQWxleGV5IEcgd3JvdGU6 Cj4gT24gV2VkLCAyMSBNYXIgMjAxOCAxNzoxNTowNCArMDAwMAo+IFJvZ2VyIFBhdSBNb25uw6kg PHJvZ2VyLnBhdUBjaXRyaXguY29tPiB3cm90ZToKPiBbLi4uXQo+ID4+IEFib3ZlIHNjZW5hcmlv IG1ha2VzIGl0IG9idmlvdXMgdGhhdCBhdCBsZWFzdCBmb3IgUUVNVSB0aGUgTU1JTy0+UENJCj4g Pj4gY29uZiB0cmFuc2xhdGlvbiBpcyBhIHJlZHVuZGFudCBzdGVwLiBXaHkgbm90IHRvIGFsbG93 IHNwZWNpZnlpbmcKPiA+PiBmb3IgRE0gd2hldGhlciBpdCBwcmVmZXJzIHRvIHJlY2VpdmUgTU1D T05GSUcgYWNjZXNzZXMgYXMgbmF0aXZlCj4gPj4gKE1NSU8gb25lcykgb3IgYXMgdHJhbnNsYXRl ZCBQQ0kgY29uZiBpb3JlcXM/ICAKPiA+Cj4gPllvdSBhcmUganVzdCBhZGRpbmcgYW4gZXh0cmEg bGV2ZWwgb2YgY29tcGxleGl0eSB0byBhbiBpbnRlcmZhY2UKPiA+dGhhdCdzIGZhaXJseSBzaW1w bGUuIFlvdSByZWdpc3RlciBhIFBDSSBkZXZpY2UgdXNpbmcKPiA+WEVOX0RNT1BfSU9fUkFOR0Vf UENJIGFuZCB5b3UgZ2V0IElPUkVRX1RZUEVfUENJX0NPTkZJRyBpb3JlcXMuCj4gCj4gWWVzLCBh bmQgaXQgaXMgc3RpbGwgbmVlZGVkIGFzIHdlIGhhdmUgdHdvIGRpc3RpbmN0IChhbmQgbm90IGVx dWFsKQo+IGludGVyZmFjZXMgdG8gUENJIGNvbmYgc3BhY2UuIEFwYXJ0IGZyb20gMC4uRkZoIHJh bmdlIG92ZXJsYXBwaW5nIHRoZXkKPiBjYW4gYmUgY29uc2lkZXJlZCB2ZXJ5IGRpZmZlcmVudCBp bnRlcmZhY2VzLiBBbmQgd2hldGhlciBpdCBpcyBhIHJlYWwKPiBzeXN0ZW0gb3IgZW11bGF0ZWQg LS0gd2UgY2FuIHVzZSBlaXRoZXIgb25lIG9mIHRoZXNlIHR3byBpbnRlcmZhY2VzIG9yCj4gYm90 aC4KClRoZSBsZWdhY3kgUENJIGNvbmZpZyBzcGFjZSBhY2Nlc3NlcyBhbmQgdGhlIE1DRkcgY29u ZmlnIHNwYWNlIGFjY2VzcwphcmUganVzdCBkaWZmZXJlbnQgbWV0aG9kcyBvZiBhY2Nlc3Npbmcg dGhlIFBDSSBjb25maWd1cmF0aW9uIHNwYWNlLApidXQgdGhlIGRhdGEgX211c3RfIGJlIGV4YWN0 bHkgdGhlIHNhbWUuIEkgZG9uJ3Qgc2VlIGhvdyBhIGRldmljZQp3b3VsZCBjYXJlIGFib3V0IHdo ZXJlIHRoZSBhY2Nlc3MgdG8gdGhlIGNvbmZpZyBzcGFjZSBvcmlnaW5hdGVkLgoKPiBGb3IgUUVN VSB6ZXJvIGNoYW5nZXMgYXJlIG5lZWRlZCB0byBzdXBwb3J0IE1NQ09ORklHIE1NSU8gYWNjZXNz ZXMgaWYKPiB0aGV5IGNvbWUgYXMgTU1JTyBpb3JlcXMuIEl0J3MganVzdCB3aGF0IGl0cyBNTUNP TkZJRyBlbXVsYXRpb24gY29kZQo+IGV4cGVjdHMuCgpBcyBJIHNhaWQgbWFueSB0aW1lcyBpbiB0 aGlzIHRocmVhZCwgeW91IHNlZW0gdG8gYmUgZm9jdXNlZCBhcm91bmQKd2hhdCdzIGJlc3QgZm9y IFFFTVUgb25seSwgYW5kIHRoaXMgaXMgd3JvbmcuIFRoZSBJT1JFUSBpbnRlcmZhY2UgaXMKdXNl ZCBieSBRRU1VLCBidXQgaXQncyBhbHNvIHVzZWQgYnkgb3RoZXIgZGV2aWNlIGVtdWxhdG9ycy4K CkkgZ2V0IHRoZSBmZWVsaW5nIHRoYXQgeW91IGFzc3VtZSB0aGF0IHRoZSBjb3JyZWN0IHNvbHV0 aW9uIGlzIHRoZSBvbmUKdGhhdCBpbnZvbHZlcyBsZXNzIGNoYW5nZXMgdG8gWGVuIGFuZCBRRU1V LiBUaGlzIGlzIHNpbXBseSBub3QgdHJ1ZS4KCj4gQW55d2F5LCBmb3IgKGtpbmQgb2YgdmFndWUp IHVzZXJzIG9mIHRoZSBtdWx0aXBsZSBpb3JlcSBzZXJ2ZXJzCj4gY2FwYWJpbGl0eSB3ZSBjYW4g ZW5hYmxlIE1NSU8gdHJhbnNsYXRpb24gdG8gUENJIGNvbmYgaW9yZXFzLiBOb3RlIHRoYXQKPiBh Y3R1YWxseSB0aGlzIGlzIGFuIGV4dHJhIHN0ZXAsIG5vdCBmb3J3YXJkaW5nIHRyYXBwZWQgTU1D T05GSUcgTU1JTwo+IGFjY2Vzc2VzIHRvIHRoZSBzZWxlY3RlZCBkZXZpY2UgbW9kZWwgYXMgaXMu Cj4KPiA+R2V0dGluZyBib3RoIElPUkVRX1RZUEVfUENJX0NPTkZJRyBhbmQgSU9SRVFfVFlQRV9D T1BZIGZvciBQQ0kgY29uZmlnCj4gPnNwYWNlIGFjY2VzcyBpcyBtaXNsZWFkaW5nLgo+IAo+IFRo ZXNlIGFyZSB2ZXJ5IGRpZmZlcmVudCBhY2Nlc3NlcywgYm90aCBpbiB0cmFuc3BvcnQgYW5kIGNh cGFiaWxpdGllcy4KPiAKPiA+SW4gYm90aCBjYXNlcyBYZW4gd291bGQgaGF2ZSB0byBkbyB0aGUg TUNGRyBhY2Nlc3MgZGVjb2RpbmcgaW4gb3JkZXIKPiA+dG8gZmlndXJlIG91dCB3aGljaCBJT1JF USBzZXJ2ZXIgd2lsbCBoYW5kbGUgdGhlIHJlcXVlc3QuIEF0IHdoaWNoCj4gPnBvaW50IHRoZSBv bmx5IHN0ZXAgdGhhdCB5b3UgYXZvaWQgaXMgdGhlIHJlY29uc3RydWN0aW9uIG9mIHRoZSBtZW1v cnkKPiA+YWNjZXNzIGZyb20gdGhlIElPUkVRX1RZUEVfUENJX0NPTkZJRyB3aGljaCBpcyB0cml2 aWFsLgo+IAo+IFRoZSAicmVjb25zdHJ1Y3Rpb24gb2YgdGhlIG1lbW9yeSBhY2Nlc3MiIHlvdSBt ZW50aW9uZWQgd29uJ3QgYmUgZWFzeQo+IGFjdHVhbGx5LiBUaGUgdGhpbmcgaXMsIGFkZHJlc3Nf c3BhY2VfcmVhZC93cml0ZSBpcyBub3QgYWxsIHdoYXQgd2UKPiBuZWVkLgo+IAo+IEluIG9yZGVy IHRvIHRyYW5zbGF0ZSBQQ0kgY29uZiBpb3JlcXMgYmFjayB0byBlbXVsYXRlZCBNTUlPIG9wcywg d2UKPiBuZWVkIHRvIGJlIGFuIGludm9sdmVkIHBhcnR5LCBtYWlubHkgdG8ga25vdyB3aGVyZSBN TUNPTkZJRyBhcmVhIGlzCj4gbG9jYXRlZCBzbyB3ZSBjYW4gY29uc3RydWN0IHRoZSBhZGRyZXNz IHdpdGhpbiBpdHMgcmFuZ2UgZnJvbSBCREYuCj4gVGhpcyBwaWVjZSBvZiBpbmZvcm1hdGlvbiBp cyBkZXN0cm95ZWQgaW4gdGhlIHByb2Nlc3Mgb2YgTU1JTyBpb3JlcQo+IHRyYW5zbGF0aW9uIHRv IFBDSSBjb25mIHR5cGUuCgpRRU1VIGNlcnRhaW5seSBrbm93cyB0aGUgcG9zaXRpb24gb2YgdGhl IE1DRkcgYXJlYSAoYmVjYXVzZSBpdCdzIHRoZQpvbmUgdGhhdCB0ZWxscyBYZW4gYWJvdXQgaXQp LCBzbyBJIGRvbid0IHVuZGVyc3RhbmQgeW91ciBjb25jZXJucwphYm92ZS4KCj4gVGhlIGNvZGUg d2hpY2ggcGFyc2UgUENJIGNvbmYgaW9yZXFzIGluIHhlbi1odm0uYyBkb2Vzbid0IGtub3cgYW55 dGhpbmcKPiBhYm91dCB0aGUgY3VycmVudCBlbXVsYXRlZCBNTUNPTkZJRyBzdGF0ZS4gVGhlIGNv cnJlY3Qgd2F5IHRvIGhhdmUgdGhpcwo+IGluZm8gaXMgdG8gcGFydGljaXBhdGUgaW4gaXRzIGVt dWxhdGlvbi4gQXMgd2UgZG9uJ3QgcGFydGljaXBhdGUsIHdlCj4gaGF2ZSBubyBvdGhlciB3YXkg dGhhbiB0cnlpbmcgdG8gZ2FpbiBiYWNrZG9vciBhY2Nlc3MgdG8gUENJSG9zdCBmaWVsZHMKPiB2 aWEgdGhpbmdzIGxpa2Ugb2JqZWN0X3Jlc29sdmVfKigpLiBUaGlzIHNvbHV0aW9uIGlzIGN1bWJl cnNvbWUgYW5kCj4gdWdseSBidXQgd2lsbCB3b3JrLi4uIGFuZCBtYXkgYnJlYWsgYW55dGltZSBk dWUgdG8gY2hhbmdlcyBpbiBRRU1VLiAKCk9LLCBzbyB5b3UgZG9uJ3Qgd2FudCB0byByZWNvbnN0 cnVjdCB0aGUgYWNjZXNzLCBmaW5lLgoKVGhlbiBqdXN0IGluamVjdCBpdCB1c2luZyBwY2llX21t Y2ZnX2RhdGFfe3JlYWQvd3JpdGV9IG9yIHNvbWUgc2ltaWxhcgp3cmFwcGVyLiBNeSBzdWdnZXN0 aW9uIHdhcyBqdXN0IHRvIHRyeSB0byB1c2UgdGhlIGVhc2llciB3YXkgdG8gZ2V0CnRoaXMgaW5q ZWN0ZWQgaW50byBRRU1VLgoKPiBRRU1VIG1haW50YWluZXJzIHdpbGwgZ3JpbiB3aGlsZSBsb29r aW5nIGF0IGFsbCB0aGlzIEknbSBhZnJhaWQgLS0KPiB0cmFwcGVkIE1NSU8gYWNjZXNzZXMgd2hp Y2ggYXJlIHRyYW5zbGF0ZWQgdG8gUENJIGNvbmYgYWNjZXNzZXMgd2hpY2gKPiBpbiB0dXJuIHRy YW5zbGF0ZWQgYmFjayB0byBlbXVsYXRlZCBNTUlPIGFjY2Vzc2VzIHVwb24gcmVjZWl2aW5nLCBh bG9uZwo+IHdpdGggdGVkaW91cyBhdHRlbXB0cyB0byBnYWluIGFjY2VzcyB0byBNTUNPTkZJRy1y ZWxhdGVkIGluZm8gYXMgd2UncmUKPiBub3QgaW52aXRlZCB0byB0aGUgTU1DT05GSUcgZW11bGF0 aW9uIHBhcnR5Lgo+Cj4gVGhlIG1vcmUgSSB0aGluayBhYm91dCBpdCwgdGhlIG1vcmUgSSBsaWtl IHRoZSBleGlzdGluZwo+IG1hcF9pb19yYW5nZV90b19pb3JlcV9zZXJ2ZXIoKSBhcHByb2FjaC4g OiggSXQgd29ya3Mgd2l0aG91dCBkb2luZwo+IGFueXRoaW5nLCBubyBoYWNrcywgbm8gbmV3IGlu dGVyZmFjZXMsIGJvdGggTU1DT05GSUcgYW5kIENGOC9DRkMgYXJlCj4gd29ya2luZyBhcyBleHBl Y3RlZC4gVGhlcmUgaXMgYSBwcm9ibGVtIHRvIG1ha2UgaXQgY29tcGF0aWJsZSB3aXRoCj4gdGhl IHNwZWNpZmljIG11bHRpcGxlIGlvcmVxIHNlcnZlcnMgZmVhdHVyZSwgYnV0IHByb3ZpZGluZyBh IG5ldwo+IGRtb3AvaHlwZXJjYWxsICh3aGljaCB5b3Ugc3VnZ2VzdCBpcyBhIG11c3QgaGF2ZSB0 aGluZyB0byB0cmFwIE1NQ09ORklHCj4gTU1JTyB0byBnaXZlIFFFTVUgb25seSB0aGUgZnJlZWRv bSB0byB0ZWxsIHdoZXJlIGl0IGlzIGxvY2F0ZWQpIGFsbG93cwo+IHRvIHNvbHZlIHRoaXMgcHJv YmxlbSBpbiBhbnkgcG9zc2libGUgd2F5LCBlaXRoZXIgTU1JTyAtPiBQQ0kgY29uZgo+IHRyYW5z bGF0aW9uIG9yIGFueXRoaW5nIGVsc2UuCgpJJ20gc29ycnksIGJ1dCBJJ20gZ2V0dGluZyBsb3N0 LgoKWW91IGNvbXBsYWluIHRoYXQgdXNpbmcgSU9SRVFfVFlQRV9QQ0lfQ09ORklHIGlzIG5vdCBh IGdvb2QgYXBwcm9hY2gKYmVjYXVzZSBRRU1VIG5lZWRzIHRvIGtub3cgdGhlIHBvc2l0aW9uIG9m IHRoZSBNQ0ZHIGFyZWEgaWYgd2Ugd2FudCB0bwpyZWNvbnN0cnVjdCBhbmQgZm9yd2FyZCB0aGUg TU1JTyBhY2Nlc3MuIEFuZCB0aGVuIHlvdSBhcmUgcHJvcG9zaW5nIHRvCnVzZSBJT1JFUV9UWVBF X0NPUFkgd2hpY2ggX3JlcXVpcmVzXyBRRU1VIHRvIGtub3cgdGhlIHBvc2l0aW9uIG9mIHRoZQpN Q0ZHIGFyZWEgaW4gb3JkZXIgdG8gZG8gdGhlIGRlY29kaW5nIG9mIHRoZSBQQ0kgY29uZmlnIHNw YWNlIGFjY2Vzcy4KCj4gPj4gV2UgY2FuIHN0aWxsIHJvdXRlIGVpdGhlciBpb3JlcQo+ID4+IHR5 cGUgdG8gbXVsdGlwbGUgZGV2aWNlIGVtdWxhdG9ycyBhY2NvcmRpbmdseS4gIAo+ID4KPiA+SXQn cyBleGFjdGx5IHRoZSBzYW1lIHRoYXQncyBkb25lIGZvciBJTyBzcGFjZSBQQ0kgY29uZmlnIHNw YWNlCj4gPmFkZHJlc3Nlcy4gUUVNVSBnZXRzIGFuIElPUkVRX1RZUEVfUENJX0NPTkZJRyBhbmQg aXQgcmVwbGF5cyB0aGUgSU8KPiA+c3BhY2UgYWNjZXNzIHVzaW5nIGRvX291dHAgYW5kIGNwdV9p b3JlcV9waW8uCj4gCj4gLi4uQW5kIGl0IGlzIGNvbXBsZXRlbHkgbGltaXRlZCB0byBiYXNpYyBQ Q0kgY29uZiBzcGFjZS4gSSBkb24ndCBrbm93Cj4gdGhlIGNvbnRleHQgb2YgdGhpcyBsaW5lIGlu IHhlbi1odm0uYzoKPiAKPiB2YWwgPSAoMXUgPDwgMzEpIHwgKChyZXEtPmFkZHIgJiAweDBmMDAp IDw8IDE2KSB8ICgoc2JkZiAmIDB4ZmZmZikgPDwgOCkKPiAgICAgICAgfCAocmVxLT5hZGRyICYg MHhmYyk7Cj4gCj4gYnV0IHNlZW1zIGxpa2UgY3VycmVudCBRRU1VIHZlcnNpb25zIGRvIG5vdCBl eHBlY3QgYW55dGhpbmcgc2ltaWxhciB0bwo+IEFNRCBFQ1Mtc3R5bGUgYWNjZXNzZXMgZm9yIDBD RjhoLiBJdCBpcyBsaW1pdGVkIHRvIGJhc2ljIFBDSSBjb25mIG9ubHkuCj4gCj4gPklmIHlvdSB0 aGluayB1c2luZyBJT1JFUV9UWVBFX0NPUFkgZm9yIE1DRkcgYWNjZXNzZXMgaXMgc3VjaCBhIGJl bmVmaXQKPiA+Zm9yIFFFTVUsIHdoeSBub3QganVzdCB0cmFuc2xhdGUgdGhlIElPUkVRX1RZUEVf UENJX0NPTkZJRyBpbnRvCj4gPklPUkVRX1RZUEVfQ09QWSBpbiBoYW5kbGVfaW9yZXEgYW5kIGRp c3BhdGNoIGl0IHVzaW5nCj4gPmNwdV9pb3JlcV9tb3ZlPwo+IAo+IEFuc3dlcmVkIGFib3ZlLCB3 ZSBuZWVkIHRvIHNvbWVob3cgaGF2ZSBhY2Nlc3MgdG8gdGhlIGluZm8gd2hpY2ggZG9uJ3QKPiBi ZWxvbmcgdG8gdXMgZm9yIHRoaXMgc3RlcC4KCldoeSBub3Q/IFFFTVUgdGVsbHMgWGVuIHRoZSBw b3NpdGlvbiBvZiB0aGUgTUNGRyBhcmVhIGJ1dCB0aGVuIHlvdQpjb21wbGFpbiB0aGF0IFFFTVUg ZG9lc24ndCBrbm93IHRoZSBwb3NpdGlvbiBvZiB0aGUgTUNGRyBhcmVhPwoKUm9nZXIuCgpfX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpYZW4tZGV2ZWwgbWFp bGluZyBsaXN0Clhlbi1kZXZlbEBsaXN0cy54ZW5wcm9qZWN0Lm9yZwpodHRwczovL2xpc3RzLnhl bnByb2plY3Qub3JnL21haWxtYW4vbGlzdGluZm8veGVuLWRldmVs