From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexey G Subject: Re: [RFC PATCH 07/12] hvmloader: allocate MMCONFIG area in the MMIO hole + minor code refactoring Date: Thu, 22 Mar 2018 22:29:22 +1000 Message-ID: <20180322222922.00006f7c@gmail.com> 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> <20180322095716.5y2hdyh6xp7nto3g@MacBook-Pro-de-Roger.local> 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 1eyzLf-0001T5-Sr for xen-devel@lists.xenproject.org; Thu, 22 Mar 2018 12:29:39 +0000 Received: by mail-lf0-x22b.google.com with SMTP id l4-v6so8515981lfg.12 for ; Thu, 22 Mar 2018 05:29:37 -0700 (PDT) In-Reply-To: <20180322095716.5y2hdyh6xp7nto3g@MacBook-Pro-de-Roger.local> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Sender: "Xen-devel" To: Roger Pau =?iso-8859-1?Q?Monn=E9?= 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 T24gVGh1LCAyMiBNYXIgMjAxOCAwOTo1NzoxNiArMDAwMApSb2dlciBQYXUgTW9ubsOpIDxyb2dl ci5wYXVAY2l0cml4LmNvbT4gd3JvdGU6ClsuLi5dCj4+IFllcywgYW5kIGl0IGlzIHN0aWxsIG5l ZWRlZCBhcyB3ZSBoYXZlIHR3byBkaXN0aW5jdCAoYW5kIG5vdCBlcXVhbCkKPj4gaW50ZXJmYWNl cyB0byBQQ0kgY29uZiBzcGFjZS4gQXBhcnQgZnJvbSAwLi5GRmggcmFuZ2Ugb3ZlcmxhcHBpbmcK Pj4gdGhleSBjYW4gYmUgY29uc2lkZXJlZCB2ZXJ5IGRpZmZlcmVudCBpbnRlcmZhY2VzLiBBbmQg d2hldGhlciBpdCBpcwo+PiBhIHJlYWwgc3lzdGVtIG9yIGVtdWxhdGVkIC0tIHdlIGNhbiB1c2Ug ZWl0aGVyIG9uZSBvZiB0aGVzZSB0d28KPj4gaW50ZXJmYWNlcyBvciBib3RoLiAgCj4KPlRoZSBs ZWdhY3kgUENJIGNvbmZpZyBzcGFjZSBhY2Nlc3NlcyBhbmQgdGhlIE1DRkcgY29uZmlnIHNwYWNl IGFjY2Vzcwo+YXJlIGp1c3QgZGlmZmVyZW50IG1ldGhvZHMgb2YgYWNjZXNzaW5nIHRoZSBQQ0kg Y29uZmlndXJhdGlvbiBzcGFjZSwKPmJ1dCB0aGUgZGF0YSBfbXVzdF8gYmUgZXhhY3RseSB0aGUg c2FtZS4gSSBkb24ndCBzZWUgaG93IGEgZGV2aWNlCj53b3VsZCBjYXJlIGFib3V0IHdoZXJlIHRo ZSBhY2Nlc3MgdG8gdGhlIGNvbmZpZyBzcGFjZSBvcmlnaW5hdGVkLgoKSWYgdGhleSB3ZXJlIGRp ZmZlcmVudCBtZXRob2RzIG9mIGFjY2Vzc2luZyB0aGUgc2FtZSB0aGluZywgdGhleQpjb3VsZCd2 ZSBiZWVuIHVzZWQgaW50ZXJjaGFuZ2VhYmx5LiBXaGVuIHdlJ3ZlIGdvdCBhIFBDSSBjb25mIGlv cmVxCndoaWNoIGhhcyBvZmZzZXQ+MTAwaCB3ZSBrbm93IHdlIGNhbm5vdCBqdXN0IHBhc3MgaXQg dG8gZW11bGF0ZWQKQ0Y4L0NGQyBidXQgaGF2ZSB0byBlbXVsYXRlIHRoaXMgc3BlY2lmaWNhbGx5 LgoKPj4gRm9yIFFFTVUgemVybyBjaGFuZ2VzIGFyZSBuZWVkZWQgdG8gc3VwcG9ydCBNTUNPTkZJ RyBNTUlPIGFjY2Vzc2VzIGlmCj4+IHRoZXkgY29tZSBhcyBNTUlPIGlvcmVxcy4gSXQncyBqdXN0 IHdoYXQgaXRzIE1NQ09ORklHIGVtdWxhdGlvbiBjb2RlCj4+IGV4cGVjdHMuICAKPgo+QXMgSSBz YWlkIG1hbnkgdGltZXMgaW4gdGhpcyB0aHJlYWQsIHlvdSBzZWVtIHRvIGJlIGZvY3VzZWQgYXJv dW5kCj53aGF0J3MgYmVzdCBmb3IgUUVNVSBvbmx5LCBhbmQgdGhpcyBpcyB3cm9uZy4gVGhlIElP UkVRIGludGVyZmFjZSBpcwo+dXNlZCBieSBRRU1VLCBidXQgaXQncyBhbHNvIHVzZWQgYnkgb3Ro ZXIgZGV2aWNlIGVtdWxhdG9ycy4KPgo+SSBnZXQgdGhlIGZlZWxpbmcgdGhhdCB5b3UgYXNzdW1l IHRoYXQgdGhlIGNvcnJlY3Qgc29sdXRpb24gaXMgdGhlIG9uZQo+dGhhdCBpbnZvbHZlcyBsZXNz IGNoYW5nZXMgdG8gWGVuIGFuZCBRRU1VLiBUaGlzIGlzIHNpbXBseSBub3QgdHJ1ZS4KPgo+PiBB bnl3YXksIGZvciAoa2luZCBvZiB2YWd1ZSkgdXNlcnMgb2YgdGhlIG11bHRpcGxlIGlvcmVxIHNl cnZlcnMKPj4gY2FwYWJpbGl0eSB3ZSBjYW4gZW5hYmxlIE1NSU8gdHJhbnNsYXRpb24gdG8gUENJ IGNvbmYgaW9yZXFzLiBOb3RlCj4+IHRoYXQgYWN0dWFsbHkgdGhpcyBpcyBhbiBleHRyYSBzdGVw LCBub3QgZm9yd2FyZGluZyB0cmFwcGVkIE1NQ09ORklHCj4+IE1NSU8gYWNjZXNzZXMgdG8gdGhl IHNlbGVjdGVkIGRldmljZSBtb2RlbCBhcyBpcy4KPj4gIAo+PiA+R2V0dGluZyBib3RoIElPUkVR X1RZUEVfUENJX0NPTkZJRyBhbmQgSU9SRVFfVFlQRV9DT1BZIGZvciBQQ0kKPj4gPmNvbmZpZyBz cGFjZSBhY2Nlc3MgaXMgbWlzbGVhZGluZy4gIAo+PiAKPj4gVGhlc2UgYXJlIHZlcnkgZGlmZmVy ZW50IGFjY2Vzc2VzLCBib3RoIGluIHRyYW5zcG9ydCBhbmQKPj4gY2FwYWJpbGl0aWVzLiAKPj4g PkluIGJvdGggY2FzZXMgWGVuIHdvdWxkIGhhdmUgdG8gZG8gdGhlIE1DRkcgYWNjZXNzIGRlY29k aW5nIGluIG9yZGVyCj4+ID50byBmaWd1cmUgb3V0IHdoaWNoIElPUkVRIHNlcnZlciB3aWxsIGhh bmRsZSB0aGUgcmVxdWVzdC4gQXQgd2hpY2gKPj4gPnBvaW50IHRoZSBvbmx5IHN0ZXAgdGhhdCB5 b3UgYXZvaWQgaXMgdGhlIHJlY29uc3RydWN0aW9uIG9mIHRoZQo+PiA+bWVtb3J5IGFjY2VzcyBm cm9tIHRoZSBJT1JFUV9UWVBFX1BDSV9DT05GSUcgd2hpY2ggaXMgdHJpdmlhbC4gIAo+PiAKPj4g VGhlICJyZWNvbnN0cnVjdGlvbiBvZiB0aGUgbWVtb3J5IGFjY2VzcyIgeW91IG1lbnRpb25lZCB3 b24ndCBiZSBlYXN5Cj4+IGFjdHVhbGx5LiBUaGUgdGhpbmcgaXMsIGFkZHJlc3Nfc3BhY2VfcmVh ZC93cml0ZSBpcyBub3QgYWxsIHdoYXQgd2UKPj4gbmVlZC4KPj4gCj4+IEluIG9yZGVyIHRvIHRy YW5zbGF0ZSBQQ0kgY29uZiBpb3JlcXMgYmFjayB0byBlbXVsYXRlZCBNTUlPIG9wcywgd2UKPj4g bmVlZCB0byBiZSBhbiBpbnZvbHZlZCBwYXJ0eSwgbWFpbmx5IHRvIGtub3cgd2hlcmUgTU1DT05G SUcgYXJlYSBpcwo+PiBsb2NhdGVkIHNvIHdlIGNhbiBjb25zdHJ1Y3QgdGhlIGFkZHJlc3Mgd2l0 aGluIGl0cyByYW5nZSBmcm9tIEJERi4KPj4gVGhpcyBwaWVjZSBvZiBpbmZvcm1hdGlvbiBpcyBk ZXN0cm95ZWQgaW4gdGhlIHByb2Nlc3Mgb2YgTU1JTyBpb3JlcQo+PiB0cmFuc2xhdGlvbiB0byBQ Q0kgY29uZiB0eXBlLiAgCj4KPlFFTVUgY2VydGFpbmx5IGtub3dzIHRoZSBwb3NpdGlvbiBvZiB0 aGUgTUNGRyBhcmVhIChiZWNhdXNlIGl0J3MgdGhlCj5vbmUgdGhhdCB0ZWxscyBYZW4gYWJvdXQg aXQpLCBzbyBJIGRvbid0IHVuZGVyc3RhbmQgeW91ciBjb25jZXJucwo+YWJvdmUuCj4+IFRoZSBj b2RlIHdoaWNoIHBhcnNlIFBDSSBjb25mIGlvcmVxcyBpbiB4ZW4taHZtLmMgZG9lc24ndCBrbm93 Cj4+IGFueXRoaW5nIGFib3V0IHRoZSBjdXJyZW50IGVtdWxhdGVkIE1NQ09ORklHIHN0YXRlLiBU aGUgY29ycmVjdCB3YXkKPj4gdG8gaGF2ZSB0aGlzIGluZm8gaXMgdG8gcGFydGljaXBhdGUgaW4g aXRzIGVtdWxhdGlvbi4gQXMgd2UgZG9uJ3QKPj4gcGFydGljaXBhdGUsIHdlIGhhdmUgbm8gb3Ro ZXIgd2F5IHRoYW4gdHJ5aW5nIHRvIGdhaW4gYmFja2Rvb3IKPj4gYWNjZXNzIHRvIFBDSUhvc3Qg ZmllbGRzIHZpYSB0aGluZ3MgbGlrZSBvYmplY3RfcmVzb2x2ZV8qKCkuIFRoaXMKPj4gc29sdXRp b24gaXMgY3VtYmVyc29tZSBhbmQgdWdseSBidXQgd2lsbCB3b3JrLi4uIGFuZCBtYXkgYnJlYWsK Pj4gYW55dGltZSBkdWUgdG8gY2hhbmdlcyBpbiBRRU1VLiAgIAo+Cj5PSywgc28geW91IGRvbid0 IHdhbnQgdG8gcmVjb25zdHJ1Y3QgdGhlIGFjY2VzcywgZmluZS4KPgo+VGhlbiBqdXN0IGluamVj dCBpdCB1c2luZyBwY2llX21tY2ZnX2RhdGFfe3JlYWQvd3JpdGV9IG9yIHNvbWUgc2ltaWxhcgo+ d3JhcHBlci4gTXkgc3VnZ2VzdGlvbiB3YXMganVzdCB0byB0cnkgdG8gdXNlIHRoZSBlYXNpZXIg d2F5IHRvIGdldAo+dGhpcyBpbmplY3RlZCBpbnRvIFFFTVUuCgpRRU1VIGtub3dzIGl0cyBwb3Np dGlvbiwgdGhlIHByb2JsZW0gaXQgdGhhdCB4ZW4taHZtLmMgKGlvcmVxCnByb2Nlc3NvcikgaXMg cmF0aGVyIGlzb2xhdGVkIGZyb20gTU1DT05GSUcgZW11bGF0aW9uLgoKSWYgeW91IGNoZWNrIHRo ZSBwY2llX21tY2ZnX2RhdGFfcmVhZC93cml0ZSBNTUNPTkZJRyBoYW5kbGVycyBpbiBRRU1VLAp5 b3UgY2FuIHNlZSB0aGlzOgoKc3RhdGljIHVpbnQ2NF90IHBjaWVfbW1jZmdfZGF0YV9yZWFkKHZv aWQgKm9wYXF1ZSwgPC4uLj4KewogICAgUENJRXhwcmVzc0hvc3QgKmUgPSBvcGFxdWU7Ci4uLgoK V2Uga25vdyB0aGlzICdvcGFxdWUnIHdoZW4gd2UgZG8gTU1JTy1zdHlsZSBNTUNPTkZJRyBoYW5k bGluZyBhcwpwY2llX21tY2ZnX2RhdGFfcmVhZC93cml0ZSBhcmUgYWN0dWFsIGhhbmRsZXJzLgoK QnV0IHhlbi1odm0uYyBuZWVkcyB0byBnYWluIGFjY2VzcyB0byBQQ0lFeHByZXNzSG9zdCBvdXQg b2Ygbm93aGVyZSwKd2hpY2ggaXMgcG9zc2libGUgYnV0IGNvbnNpZGVyZWQgYSBoYWNrIGJ5IFFF TVUuIFdlIGNhbiBhbHNvIGluc2VydApzb21lIGNvZGUgdG8gTU1DT05GSUcgZW11bGF0aW9uIHdo aWNoIHdpbGwgc3RvcmUgaW5mbyB3ZSBuZWVkIHRvIHNvbWUKZ2xvYmFsIHZhcmlhYmxlcyB0byBi ZSB1c2VkIGFjcm9zcyB3aWxkbHkgZGlmZmVyZW50IGFuZCB1bnJlbGF0ZWQKbW9kdWxlcy4gSXQg d2lsbCB3b3JrLCBidXQgYW55b25lIHdobyBzZWUgaXQgd2lsbCBoYXZlIGJhZCB0aG91Z2h0cyBv bgpoaXMgbWluZC4KCj4+IFFFTVUgbWFpbnRhaW5lcnMgd2lsbCBncmluIHdoaWxlIGxvb2tpbmcg YXQgYWxsIHRoaXMgSSdtIGFmcmFpZCAtLQo+PiB0cmFwcGVkIE1NSU8gYWNjZXNzZXMgd2hpY2gg YXJlIHRyYW5zbGF0ZWQgdG8gUENJIGNvbmYgYWNjZXNzZXMgd2hpY2gKPj4gaW4gdHVybiB0cmFu c2xhdGVkIGJhY2sgdG8gZW11bGF0ZWQgTU1JTyBhY2Nlc3NlcyB1cG9uIHJlY2VpdmluZywKPj4g YWxvbmcgd2l0aCB0ZWRpb3VzIGF0dGVtcHRzIHRvIGdhaW4gYWNjZXNzIHRvIE1NQ09ORklHLXJl bGF0ZWQgaW5mbwo+PiBhcyB3ZSdyZSBub3QgaW52aXRlZCB0byB0aGUgTU1DT05GSUcgZW11bGF0 aW9uIHBhcnR5Lgo+Pgo+PiBUaGUgbW9yZSBJIHRoaW5rIGFib3V0IGl0LCB0aGUgbW9yZSBJIGxp a2UgdGhlIGV4aXN0aW5nCj4+IG1hcF9pb19yYW5nZV90b19pb3JlcV9zZXJ2ZXIoKSBhcHByb2Fj aC4gOiggSXQgd29ya3Mgd2l0aG91dCBkb2luZwo+PiBhbnl0aGluZywgbm8gaGFja3MsIG5vIG5l dyBpbnRlcmZhY2VzLCBib3RoIE1NQ09ORklHIGFuZCBDRjgvQ0ZDIGFyZQo+PiB3b3JraW5nIGFz IGV4cGVjdGVkLiBUaGVyZSBpcyBhIHByb2JsZW0gdG8gbWFrZSBpdCBjb21wYXRpYmxlIHdpdGgK Pj4gdGhlIHNwZWNpZmljIG11bHRpcGxlIGlvcmVxIHNlcnZlcnMgZmVhdHVyZSwgYnV0IHByb3Zp ZGluZyBhIG5ldwo+PiBkbW9wL2h5cGVyY2FsbCAod2hpY2ggeW91IHN1Z2dlc3QgaXMgYSBtdXN0 IGhhdmUgdGhpbmcgdG8gdHJhcAo+PiBNTUNPTkZJRyBNTUlPIHRvIGdpdmUgUUVNVSBvbmx5IHRo ZSBmcmVlZG9tIHRvIHRlbGwgd2hlcmUgaXQgaXMKPj4gbG9jYXRlZCkgYWxsb3dzIHRvIHNvbHZl IHRoaXMgcHJvYmxlbSBpbiBhbnkgcG9zc2libGUgd2F5LCBlaXRoZXIKPj4gTU1JTyAtPiBQQ0kg Y29uZiB0cmFuc2xhdGlvbiBvciBhbnl0aGluZyBlbHNlLiAgCj4KPkknbSBzb3JyeSwgYnV0IEkn bSBnZXR0aW5nIGxvc3QuCj4KPllvdSBjb21wbGFpbiB0aGF0IHVzaW5nIElPUkVRX1RZUEVfUENJ X0NPTkZJRyBpcyBub3QgYSBnb29kIGFwcHJvYWNoCj5iZWNhdXNlIFFFTVUgbmVlZHMgdG8ga25v dyB0aGUgcG9zaXRpb24gb2YgdGhlIE1DRkcgYXJlYSBpZiB3ZSB3YW50IHRvCj5yZWNvbnN0cnVj dCBhbmQgZm9yd2FyZCB0aGUgTU1JTyBhY2Nlc3MuIEFuZCB0aGVuIHlvdSBhcmUgcHJvcG9zaW5n IHRvCj51c2UgSU9SRVFfVFlQRV9DT1BZIHdoaWNoIF9yZXF1aXJlc18gUUVNVSB0byBrbm93IHRo ZSBwb3NpdGlvbiBvZiB0aGUKPk1DRkcgYXJlYSBpbiBvcmRlciB0byBkbyB0aGUgZGVjb2Rpbmcg b2YgdGhlIFBDSSBjb25maWcgc3BhY2UgYWNjZXNzLgo+PiA+PiBXZSBjYW4gc3RpbGwgcm91dGUg ZWl0aGVyIGlvcmVxCj4+ID4+IHR5cGUgdG8gbXVsdGlwbGUgZGV2aWNlIGVtdWxhdG9ycyBhY2Nv cmRpbmdseS4gICAgCj4+ID4KPj4gPkl0J3MgZXhhY3RseSB0aGUgc2FtZSB0aGF0J3MgZG9uZSBm b3IgSU8gc3BhY2UgUENJIGNvbmZpZyBzcGFjZQo+PiA+YWRkcmVzc2VzLiBRRU1VIGdldHMgYW4g SU9SRVFfVFlQRV9QQ0lfQ09ORklHIGFuZCBpdCByZXBsYXlzIHRoZSBJTwo+PiA+c3BhY2UgYWNj ZXNzIHVzaW5nIGRvX291dHAgYW5kIGNwdV9pb3JlcV9waW8uICAKPj4gCj4+IC4uLkFuZCBpdCBp cyBjb21wbGV0ZWx5IGxpbWl0ZWQgdG8gYmFzaWMgUENJIGNvbmYgc3BhY2UuIEkgZG9uJ3Qga25v dwo+PiB0aGUgY29udGV4dCBvZiB0aGlzIGxpbmUgaW4geGVuLWh2bS5jOgo+PiAKPj4gdmFsID0g KDF1IDw8IDMxKSB8ICgocmVxLT5hZGRyICYgMHgwZjAwKSA8PCAxNikgfCAoKHNiZGYgJiAweGZm ZmYpCj4+IDw8IDgpIHwgKHJlcS0+YWRkciAmIDB4ZmMpOwo+PiAKPj4gYnV0IHNlZW1zIGxpa2Ug Y3VycmVudCBRRU1VIHZlcnNpb25zIGRvIG5vdCBleHBlY3QgYW55dGhpbmcgc2ltaWxhcgo+PiB0 byBBTUQgRUNTLXN0eWxlIGFjY2Vzc2VzIGZvciAwQ0Y4aC4gSXQgaXMgbGltaXRlZCB0byBiYXNp YyBQQ0kgY29uZgo+PiBvbmx5LiAKPj4gPklmIHlvdSB0aGluayB1c2luZyBJT1JFUV9UWVBFX0NP UFkgZm9yIE1DRkcgYWNjZXNzZXMgaXMgc3VjaCBhCj4+ID5iZW5lZml0IGZvciBRRU1VLCB3aHkg bm90IGp1c3QgdHJhbnNsYXRlIHRoZSBJT1JFUV9UWVBFX1BDSV9DT05GSUcKPj4gPmludG8gSU9S RVFfVFlQRV9DT1BZIGluIGhhbmRsZV9pb3JlcSBhbmQgZGlzcGF0Y2ggaXQgdXNpbmcKPj4gPmNw dV9pb3JlcV9tb3ZlPyAgCj4+IAo+PiBBbnN3ZXJlZCBhYm92ZSwgd2UgbmVlZCB0byBzb21laG93 IGhhdmUgYWNjZXNzIHRvIHRoZSBpbmZvIHdoaWNoCj4+IGRvbid0IGJlbG9uZyB0byB1cyBmb3Ig dGhpcyBzdGVwLiAgCj4KPldoeSBub3Q/IFFFTVUgdGVsbHMgWGVuIHRoZSBwb3NpdGlvbiBvZiB0 aGUgTUNGRyBhcmVhIGJ1dCB0aGVuIHlvdQo+Y29tcGxhaW4gdGhhdCBRRU1VIGRvZXNuJ3Qga25v dyB0aGUgcG9zaXRpb24gb2YgdGhlIE1DRkcgYXJlYT8KCkFuc3dlcmVkIGFib3ZlLgoKX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWGVuLWRldmVsIG1haWxp bmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVucHJvamVjdC5vcmcKaHR0cHM6Ly9saXN0cy54ZW5w cm9qZWN0Lm9yZy9tYWlsbWFuL2xpc3RpbmZvL3hlbi1kZXZlbA==