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 08:49:58 +1000 Message-ID: <20180322084958.0000571c@gmail.com> 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> 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 1eymYh-0003Kw-Ey for xen-devel@lists.xenproject.org; Wed, 21 Mar 2018 22:50:15 +0000 Received: by mail-lf0-x233.google.com with SMTP id c78-v6so5605412lfh.1 for ; Wed, 21 Mar 2018 15:50:13 -0700 (PDT) In-Reply-To: <20180321171504.afl75q77kji34ybo@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 T24gV2VkLCAyMSBNYXIgMjAxOCAxNzoxNTowNCArMDAwMApSb2dlciBQYXUgTW9ubsOpIDxyb2dl ci5wYXVAY2l0cml4LmNvbT4gd3JvdGU6ClsuLi5dCj4+IEFib3ZlIHNjZW5hcmlvIG1ha2VzIGl0 IG9idmlvdXMgdGhhdCBhdCBsZWFzdCBmb3IgUUVNVSB0aGUgTU1JTy0+UENJCj4+IGNvbmYgdHJh bnNsYXRpb24gaXMgYSByZWR1bmRhbnQgc3RlcC4gV2h5IG5vdCB0byBhbGxvdyBzcGVjaWZ5aW5n Cj4+IGZvciBETSB3aGV0aGVyIGl0IHByZWZlcnMgdG8gcmVjZWl2ZSBNTUNPTkZJRyBhY2Nlc3Nl cyBhcyBuYXRpdmUKPj4gKE1NSU8gb25lcykgb3IgYXMgdHJhbnNsYXRlZCBQQ0kgY29uZiBpb3Jl cXM/ICAKPgo+WW91IGFyZSBqdXN0IGFkZGluZyBhbiBleHRyYSBsZXZlbCBvZiBjb21wbGV4aXR5 IHRvIGFuIGludGVyZmFjZQo+dGhhdCdzIGZhaXJseSBzaW1wbGUuIFlvdSByZWdpc3RlciBhIFBD SSBkZXZpY2UgdXNpbmcKPlhFTl9ETU9QX0lPX1JBTkdFX1BDSSBhbmQgeW91IGdldCBJT1JFUV9U WVBFX1BDSV9DT05GSUcgaW9yZXFzLgoKWWVzLCBhbmQgaXQgaXMgc3RpbGwgbmVlZGVkIGFzIHdl IGhhdmUgdHdvIGRpc3RpbmN0IChhbmQgbm90IGVxdWFsKQppbnRlcmZhY2VzIHRvIFBDSSBjb25m IHNwYWNlLiBBcGFydCBmcm9tIDAuLkZGaCByYW5nZSBvdmVybGFwcGluZyB0aGV5CmNhbiBiZSBj b25zaWRlcmVkIHZlcnkgZGlmZmVyZW50IGludGVyZmFjZXMuIEFuZCB3aGV0aGVyIGl0IGlzIGEg cmVhbApzeXN0ZW0gb3IgZW11bGF0ZWQgLS0gd2UgY2FuIHVzZSBlaXRoZXIgb25lIG9mIHRoZXNl IHR3byBpbnRlcmZhY2VzIG9yCmJvdGguCgpGb3IgUUVNVSB6ZXJvIGNoYW5nZXMgYXJlIG5lZWRl ZCB0byBzdXBwb3J0IE1NQ09ORklHIE1NSU8gYWNjZXNzZXMgaWYKdGhleSBjb21lIGFzIE1NSU8g aW9yZXFzLiBJdCdzIGp1c3Qgd2hhdCBpdHMgTU1DT05GSUcgZW11bGF0aW9uIGNvZGUKZXhwZWN0 cy4KQW55d2F5LCBmb3IgKGtpbmQgb2YgdmFndWUpIHVzZXJzIG9mIHRoZSBtdWx0aXBsZSBpb3Jl cSBzZXJ2ZXJzCmNhcGFiaWxpdHkgd2UgY2FuIGVuYWJsZSBNTUlPIHRyYW5zbGF0aW9uIHRvIFBD SSBjb25mIGlvcmVxcy4gTm90ZSB0aGF0CmFjdHVhbGx5IHRoaXMgaXMgYW4gZXh0cmEgc3RlcCwg bm90IGZvcndhcmRpbmcgdHJhcHBlZCBNTUNPTkZJRyBNTUlPCmFjY2Vzc2VzIHRvIHRoZSBzZWxl Y3RlZCBkZXZpY2UgbW9kZWwgYXMgaXMuCgo+R2V0dGluZyBib3RoIElPUkVRX1RZUEVfUENJX0NP TkZJRyBhbmQgSU9SRVFfVFlQRV9DT1BZIGZvciBQQ0kgY29uZmlnCj5zcGFjZSBhY2Nlc3MgaXMg bWlzbGVhZGluZy4KClRoZXNlIGFyZSB2ZXJ5IGRpZmZlcmVudCBhY2Nlc3NlcywgYm90aCBpbiB0 cmFuc3BvcnQgYW5kIGNhcGFiaWxpdGllcy4KCj5JbiBib3RoIGNhc2VzIFhlbiB3b3VsZCBoYXZl IHRvIGRvIHRoZSBNQ0ZHIGFjY2VzcyBkZWNvZGluZyBpbiBvcmRlcgo+dG8gZmlndXJlIG91dCB3 aGljaCBJT1JFUSBzZXJ2ZXIgd2lsbCBoYW5kbGUgdGhlIHJlcXVlc3QuIEF0IHdoaWNoCj5wb2lu dCB0aGUgb25seSBzdGVwIHRoYXQgeW91IGF2b2lkIGlzIHRoZSByZWNvbnN0cnVjdGlvbiBvZiB0 aGUgbWVtb3J5Cj5hY2Nlc3MgZnJvbSB0aGUgSU9SRVFfVFlQRV9QQ0lfQ09ORklHIHdoaWNoIGlz IHRyaXZpYWwuCgpUaGUgInJlY29uc3RydWN0aW9uIG9mIHRoZSBtZW1vcnkgYWNjZXNzIiB5b3Ug bWVudGlvbmVkIHdvbid0IGJlIGVhc3kKYWN0dWFsbHkuIFRoZSB0aGluZyBpcywgYWRkcmVzc19z cGFjZV9yZWFkL3dyaXRlIGlzIG5vdCBhbGwgd2hhdCB3ZQpuZWVkLgoKSW4gb3JkZXIgdG8gdHJh bnNsYXRlIFBDSSBjb25mIGlvcmVxcyBiYWNrIHRvIGVtdWxhdGVkIE1NSU8gb3BzLCB3ZQpuZWVk IHRvIGJlIGFuIGludm9sdmVkIHBhcnR5LCBtYWlubHkgdG8ga25vdyB3aGVyZSBNTUNPTkZJRyBh cmVhIGlzCmxvY2F0ZWQgc28gd2UgY2FuIGNvbnN0cnVjdCB0aGUgYWRkcmVzcyB3aXRoaW4gaXRz IHJhbmdlIGZyb20gQkRGLgpUaGlzIHBpZWNlIG9mIGluZm9ybWF0aW9uIGlzIGRlc3Ryb3llZCBp biB0aGUgcHJvY2VzcyBvZiBNTUlPIGlvcmVxCnRyYW5zbGF0aW9uIHRvIFBDSSBjb25mIHR5cGUu CgpUaGUgY29kZSB3aGljaCBwYXJzZSBQQ0kgY29uZiBpb3JlcXMgaW4geGVuLWh2bS5jIGRvZXNu J3Qga25vdyBhbnl0aGluZwphYm91dCB0aGUgY3VycmVudCBlbXVsYXRlZCBNTUNPTkZJRyBzdGF0 ZS4gVGhlIGNvcnJlY3Qgd2F5IHRvIGhhdmUgdGhpcwppbmZvIGlzIHRvIHBhcnRpY2lwYXRlIGlu IGl0cyBlbXVsYXRpb24uIEFzIHdlIGRvbid0IHBhcnRpY2lwYXRlLCB3ZQpoYXZlIG5vIG90aGVy IHdheSB0aGFuIHRyeWluZyB0byBnYWluIGJhY2tkb29yIGFjY2VzcyB0byBQQ0lIb3N0IGZpZWxk cwp2aWEgdGhpbmdzIGxpa2Ugb2JqZWN0X3Jlc29sdmVfKigpLiBUaGlzIHNvbHV0aW9uIGlzIGN1 bWJlcnNvbWUgYW5kCnVnbHkgYnV0IHdpbGwgd29yay4uLiBhbmQgbWF5IGJyZWFrIGFueXRpbWUg ZHVlIHRvIGNoYW5nZXMgaW4gUUVNVS4gCgpRRU1VIG1haW50YWluZXJzIHdpbGwgZ3JpbiB3aGls ZSBsb29raW5nIGF0IGFsbCB0aGlzIEknbSBhZnJhaWQgLS0KdHJhcHBlZCBNTUlPIGFjY2Vzc2Vz IHdoaWNoIGFyZSB0cmFuc2xhdGVkIHRvIFBDSSBjb25mIGFjY2Vzc2VzIHdoaWNoCmluIHR1cm4g dHJhbnNsYXRlZCBiYWNrIHRvIGVtdWxhdGVkIE1NSU8gYWNjZXNzZXMgdXBvbiByZWNlaXZpbmcs IGFsb25nCndpdGggdGVkaW91cyBhdHRlbXB0cyB0byBnYWluIGFjY2VzcyB0byBNTUNPTkZJRy1y ZWxhdGVkIGluZm8gYXMgd2UncmUKbm90IGludml0ZWQgdG8gdGhlIE1NQ09ORklHIGVtdWxhdGlv biBwYXJ0eS4KClRoZSBtb3JlIEkgdGhpbmsgYWJvdXQgaXQsIHRoZSBtb3JlIEkgbGlrZSB0aGUg ZXhpc3RpbmcKbWFwX2lvX3JhbmdlX3RvX2lvcmVxX3NlcnZlcigpIGFwcHJvYWNoLiA6KCBJdCB3 b3JrcyB3aXRob3V0IGRvaW5nCmFueXRoaW5nLCBubyBoYWNrcywgbm8gbmV3IGludGVyZmFjZXMs IGJvdGggTU1DT05GSUcgYW5kIENGOC9DRkMgYXJlCndvcmtpbmcgYXMgZXhwZWN0ZWQuIFRoZXJl IGlzIGEgcHJvYmxlbSB0byBtYWtlIGl0IGNvbXBhdGlibGUgd2l0aAp0aGUgc3BlY2lmaWMgbXVs dGlwbGUgaW9yZXEgc2VydmVycyBmZWF0dXJlLCBidXQgcHJvdmlkaW5nIGEgbmV3CmRtb3AvaHlw ZXJjYWxsICh3aGljaCB5b3Ugc3VnZ2VzdCBpcyBhIG11c3QgaGF2ZSB0aGluZyB0byB0cmFwIE1N Q09ORklHCk1NSU8gdG8gZ2l2ZSBRRU1VIG9ubHkgdGhlIGZyZWVkb20gdG8gdGVsbCB3aGVyZSBp dCBpcyBsb2NhdGVkKSBhbGxvd3MKdG8gc29sdmUgdGhpcyBwcm9ibGVtIGluIGFueSBwb3NzaWJs ZSB3YXksIGVpdGhlciBNTUlPIC0+IFBDSSBjb25mCnRyYW5zbGF0aW9uIG9yIGFueXRoaW5nIGVs c2UuCgo+PiBXZSBjYW4gc3RpbGwgcm91dGUgZWl0aGVyIGlvcmVxCj4+IHR5cGUgdG8gbXVsdGlw bGUgZGV2aWNlIGVtdWxhdG9ycyBhY2NvcmRpbmdseS4gIAo+Cj5JdCdzIGV4YWN0bHkgdGhlIHNh bWUgdGhhdCdzIGRvbmUgZm9yIElPIHNwYWNlIFBDSSBjb25maWcgc3BhY2UKPmFkZHJlc3Nlcy4g UUVNVSBnZXRzIGFuIElPUkVRX1RZUEVfUENJX0NPTkZJRyBhbmQgaXQgcmVwbGF5cyB0aGUgSU8K PnNwYWNlIGFjY2VzcyB1c2luZyBkb19vdXRwIGFuZCBjcHVfaW9yZXFfcGlvLgoKLi4uQW5kIGl0 IGlzIGNvbXBsZXRlbHkgbGltaXRlZCB0byBiYXNpYyBQQ0kgY29uZiBzcGFjZS4gSSBkb24ndCBr bm93CnRoZSBjb250ZXh0IG9mIHRoaXMgbGluZSBpbiB4ZW4taHZtLmM6Cgp2YWwgPSAoMXUgPDwg MzEpIHwgKChyZXEtPmFkZHIgJiAweDBmMDApIDw8IDE2KSB8ICgoc2JkZiAmIDB4ZmZmZikgPDwg OCkKICAgICAgIHwgKHJlcS0+YWRkciAmIDB4ZmMpOwoKYnV0IHNlZW1zIGxpa2UgY3VycmVudCBR RU1VIHZlcnNpb25zIGRvIG5vdCBleHBlY3QgYW55dGhpbmcgc2ltaWxhciB0bwpBTUQgRUNTLXN0 eWxlIGFjY2Vzc2VzIGZvciAwQ0Y4aC4gSXQgaXMgbGltaXRlZCB0byBiYXNpYyBQQ0kgY29uZiBv bmx5LgoKPklmIHlvdSB0aGluayB1c2luZyBJT1JFUV9UWVBFX0NPUFkgZm9yIE1DRkcgYWNjZXNz ZXMgaXMgc3VjaCBhIGJlbmVmaXQKPmZvciBRRU1VLCB3aHkgbm90IGp1c3QgdHJhbnNsYXRlIHRo ZSBJT1JFUV9UWVBFX1BDSV9DT05GSUcgaW50bwo+SU9SRVFfVFlQRV9DT1BZIGluIGhhbmRsZV9p b3JlcSBhbmQgZGlzcGF0Y2ggaXQgdXNpbmcKPmNwdV9pb3JlcV9tb3ZlPwoKQW5zd2VyZWQgYWJv dmUsIHdlIG5lZWQgdG8gc29tZWhvdyBoYXZlIGFjY2VzcyB0byB0aGUgaW5mbyB3aGljaCBkb24n dApiZWxvbmcgdG8gdXMgZm9yIHRoaXMgc3RlcC4KCj5UaGFua3MsIFJvZ2VyLgoKX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWGVuLWRldmVsIG1haWxpbmcg bGlzdApYZW4tZGV2ZWxAbGlzdHMueGVucHJvamVjdC5vcmcKaHR0cHM6Ly9saXN0cy54ZW5wcm9q ZWN0Lm9yZy9tYWlsbWFuL2xpc3RpbmZvL3hlbi1kZXZlbA==