From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Cooper Subject: Re: [PATCH for 4.9 3/6] x86/hvm: Fix segmentation logic for system segments Date: Mon, 3 Apr 2017 16:42:41 +0100 Message-ID: References: <1490989853-21879-1-git-send-email-andrew.cooper3@citrix.com> <1490989853-21879-4-git-send-email-andrew.cooper3@citrix.com> <58E22E6B020000780014BF66@prv-mh.provo.novell.com> <26f9868f-ad04-7b82-821d-82d93abfb956@citrix.com> <58E28155020000780014C2CE@prv-mh.provo.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <58E28155020000780014C2CE@prv-mh.provo.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Jan Beulich Cc: Julien Grall , Paul Durrant , Tim Deegan , Xen-devel List-Id: xen-devel@lists.xenproject.org T24gMDMvMDQvMTcgMTY6MDcsIEphbiBCZXVsaWNoIHdyb3RlOgo+Pj4+IE9uIDAzLjA0LjE3IGF0 IDE2OjI3LCA8YW5kcmV3LmNvb3BlcjNAY2l0cml4LmNvbT4gd3JvdGU6Cj4+IE9uIDAzLzA0LzE3 IDEwOjEzLCBKYW4gQmV1bGljaCB3cm90ZToKPj4+Pj4+IE9uIDMxLjAzLjE3IGF0IDIxOjUwLCA8 YW5kcmV3LmNvb3BlcjNAY2l0cml4LmNvbT4gd3JvdGU6Cj4+Pj4gLS0tIGEveGVuL2FyY2gveDg2 L2h2bS9odm0uYwo+Pj4+ICsrKyBiL3hlbi9hcmNoL3g4Ni9odm0vaHZtLmMKPj4+PiBAQCAtMjM3 NCwxMyArMjM3NCwyNyBAQCBpbnQgaHZtX3NldF9jcjQodW5zaWduZWQgbG9uZyB2YWx1ZSwgYm9v bF90IAo+PiBtYXlfZGVmZXIpCj4+Pj4gICAgICByZXR1cm4gWDg2RU1VTF9PS0FZOwo+Pj4+ICB9 Cj4+Pj4gIAo+Pj4+ICtlbnVtIGh2bV9zZWdtZW50YXRpb25fbW9kZSBodm1fc2VnX21vZGUoCj4+ Pj4gKyAgICBjb25zdCBzdHJ1Y3QgdmNwdSAqdiwgZW51bSB4ODZfc2VnbWVudCBzZWcsCj4+Pj4g KyAgICBjb25zdCBzdHJ1Y3Qgc2VnbWVudF9yZWdpc3RlciAqY3MpCj4+PiBUaGUgaW5wdXRzIGhl cmUgYXJlIGF0IGxlYXN0IHNvbWV3aGF0IGNvdW50ZXJpbnR1aXRpdmUgKGZvciBleGFtcGxlLAo+ Pj4gZnJvbSBhbiBhYnN0cmFjdCBwb3YgaXQgaXMgdW5leHBlY3RlZCB0aGF0IHRoZSByZXN1bHQg ZGVwZW5kcyBvbiBzZWcKPj4+IGFuZCBjcykuIEF0IHRoZSB2ZXJ5IGxlYXN0IEkgdGhpbmsgdGhl IG5hbWluZyBzaG91bGQgbWFrZSBjbGVhciB0aGF0Cj4+PiBjcyBpcyBub3QganVzdCBzb21lIGNv ZGUgc2VnbWVudCwgYnV0IHRoZSBDUyB2IGhhcyBjdXJyZW50bHkgaW4gdXNlCj4+PiAoZS5nLiBh Y3RpdmVfY3MpLiBHb2luZyBmdXJ0aGVyIHRoZSBxdWVzdGlvbiBpcyB3aGV0aGVyIGhhdmluZyB0 aGlzCj4+PiBoZWxwZXIgaXMgcmVhbGx5IHVzZWZ1bCAoYW5kIG5vdCBwZXJoYXBzIGludml0aW5n IG1pcy11c2UpLCBhbmQgaGVuY2UKPj4+IHdoZXRoZXIgdGhlIGlucHV0cyBwYXNzZWQgaGVyZSB3 b3VsZG4ndCBiZXR0ZXIgYmUgcGFzc2VkIGRpcmVjdGx5Cj4+PiB0byBodm1fdmlydHVhbF90b19s aW5lYXJfYWRkcigpLCB0aGUgbW9yZSB0aGF0IHRoZSAic2VnIiBhcmd1bWVudAo+Pj4gaXMgcmVx dWlyZWQgdG8gbWF0Y2ggdXAgYmV0d2VlbiB0aGUgdHdvIGNhbGxzLgo+PiBJIHB1cnBvc2VmdWxs eSB3YW50ZWQgdG8gYXZvaWQgcGVvcGxlIG9wZW5jb2RpbmcgdGhlIGxvZ2ljIGFuZCBnZXR0aW5n Cj4+IGl0IHdyb25nIChsb29rcyBsaWtlIGV2ZW4gSSBnb3QgaXQgd3JvbmcpLgo+Pgo+PiBJJ20g bm90IGNvbnZpbmNlZCB0aGF0IHBhc3NpbmcgdGhlIHBhcmFtZXRlcnMgaW5kaXZpZHVhbGx5IGlz IGJldHRlci4KPj4KPj4+PiArewo+Pj4+ICsgICAgaWYgKCAhKHYtPmFyY2guaHZtX3ZjcHUuZ3Vl c3RfY3JbMF0gJiBYODZfQ1IwX1BFKSApCj4+Pj4gKyAgICAgICAgcmV0dXJuIGh2bV9zZWdfbW9k ZV9yZWFsOwo+Pj4gV2hhdCBhYm91dCBWTTg2IG1vZGU/Cj4+IFZlcnkgZ29vZCBwb2ludC4KPj4K Pj4+PiArICAgIGlmICggaHZtX2xvbmdfbW9kZV9hY3RpdmUodikgJiYKPj4+PiArICAgICAgICAg KGlzX3g4Nl9zeXN0ZW1fc2VnbWVudChzZWcpIHx8IGNzLT5hdHRyLmZpZWxkcy5sKSApCj4+Pj4g KyAgICAgICAgcmV0dXJuIGh2bV9zZWdfbW9kZV9sb25nOwo+Pj4+ICsKPj4+PiArICAgIHJldHVy biBodm1fc2VnX21vZGVfcHJvdDsKPj4+IERpZCB5b3UgdmVyaWZ5IHRoaXMgYWN0dWFsbHkgbWF0 Y2hpbmcgcmVhbCBoYXJkd2FyZSBiZWhhdmlvcj8gVGhlcmUKPj4+IGJlaW5nIG9idmlvdXMgYW5v bWFsaWVzIHdoZW4gY29tcGF0IHJpbmctMCBjb2RlIGV4ZWN1dGVzCj4+PiBMR0RUL0xJRFQvU0dE VC9TSURUIChpbiBsb25nIG1vZGUgdGhlc2Ugb3VnaHQgdG8gaGF2ZSAxMC1ieXRlCj4+PiBvcGVy YW5kcywgeWV0IDMyLWJpdC9jb21wYXQgY29kZSB3b3VsZCBleHBlY3QgNi1ieXRlIG9uZXMsIHNv Cj4+PiBvbmUgb2YgdGhlIHR3byB2aWV3cyBpcyBuZWNlc3NhcmlseSB3cm9uZywgYW5kIHdoaWNo ZXZlciBpdCBpcywgaXQKPj4+IGludHJvZHVjZXMgYW4gaW5jb25zaXN0ZW5jeSksIEkgd291bGRu J3QgdGFrZSBpdCBmb3IgZ2l2ZW4gdGhhdCBfYWxsXwo+Pj4gZGVzY3JpcHRvciB0YWJsZSBhY2Nl c3NpbmcgaW5zbnMgYmVoYXZlIGxpa2UgdGhleSB3b3VsZCBmcm9tIGEKPj4+IDY0LWJpdCBjb2Rl IHNlZ21lbnQgKEkgbmV2ZXJ0aGVsZXNzIGFzc3VtZSB0aGV5IGRvLCBidXQgYXMgSQo+Pj4gY2Fu J3Qgc2VlIGl0IHdyaXR0ZW4gZG93biBhbnl3aGVyZSwgd2Ugc2hvdWxkbid0IGFzc3VtZSBzbywK Pj4+IGNvbnNpZGVyaW5nIGhvdyBtYW55IG90aGVyIG9kZGl0aWVzIHRoZXJlIGFyZSBpbiB4ODYp Lgo+Pj4KPj4+IFRoaXMgcXVlc3Rpb24gaXMgYWxzbyBiZWluZyBzdXBwb3J0ZWQgYnkgdGhlIFNE TSB1c2luZyB0aGUgc2FtZQo+Pj4gc3RhbmRhcmQgIlNhbWUgZXhjZXB0aW9ucyBhcyBpbiBwcm90 ZWN0ZWQgbW9kZSIgaW4gdGhlCj4+PiByZXNwZWN0aXZlIGluc25zJyAiQ29tcGF0aWJpbGl0eSBN b2RlIEV4Y2VwdGlvbnMiIHNlY3Rpb25zLCB5ZXQKPj4+IHRoZSBiZWhhdmlvciBhYm92ZSBpbXBs aWVzIHRoYXQgI0dQKDApIG1pZ2h0IGFsc28gcmVzdWx0IGZvcgo+Pj4gY29tcGF0IG1vZGUgZGVz Y3JpcHRvciB0YWJsZSBhY2Nlc3NlcyBpZiB0aGUgZGVzY3JpcHRvciBhZGRyZXNzCj4+PiBlbmRz IHVwIGJlaW5nIG5vbi1jYW5vbmljYWwuIEludGVyZXN0aW5nbHkgZW5vdWdoIHRoZSBQTQo+Pj4g ZG9lc24ndCBzZXBhcmF0ZSBleGNlcHRpb24gc3BlY2lmaWNhdGlvbnMgZm9yIDMyLWJpdCBwcm90 ZWN0ZWQsCj4+PiBjb21wYXQsIGFuZCA2NC1iaXQgbW9kZXMuCj4+IFlvdSBhcmUgbWlzdGFrZW4u Cj4+Cj4+IHtMLFN9e0ksR31EVCBhcmUge3JlYWQsd3JpdGV9X3NlZ21lbnRfcmVnaXN0ZXIoKSBv cGVyYXRpb25zLCB1c2luZyBhCj4+IHBsYWluIG1lbW9yeSBvcGVyYW5kLgo+Pgo+PiBXaGVuIHdl IGNvbWUgdG8gdGhlIHNlZ21lbnRhdGlvbiBjaGVjaywgaXQgd2lsbCBiZSBieSBkZWZhdWx0Cj4+ ICVkcy1yZWxhdGl2ZSwgd2l0aCBzaXplIGFzIGNhbGN1bGF0ZWQgYnkgb3BfYnl0ZXMgaW4gdGhl IGluc3RydWN0aW9uCj4+IGVtdWxhdG9yLgo+IEkgdGhpbmsgSSBkaWRuJ3QgbWFrZSBteXNlbGYg Y2xlYXIgdGhlbjogSSdtIG5vdCBhdCBhbGwgdGFsa2luZyBhYm91dCBob3cKPiB0aGUgbWVtb3J5 IGFjY2VzcyBvZiB0aGVzZSBpbnNucyBnZXQgY2FycmllZCBvdXQsIEkgc29sZWx5IHRhbGsgYWJv dXQKPiB0aGUgc2l6ZSBvZiB0aGVpciBvcGVyYW5kczoKCkkgc3RpbGwgZmFpbCB0byBzZWUgd2hh dCB0aGUgc2l6ZSBvZiB0aGUgb3BlcmFuZHMgaGF2ZSB0byBkbyB3aXRoIHRoZQpjaG9pY2Ugb2Yg c2VnbWVudGF0aW9uIG1vZGUuCgo+IEluIGxvbmcgbW9kZSB0byBsb2FkIElEVFIgb3IgR0RUUiB5 b3UnZCBleHBlY3QgYSA2NC1iaXQgYmFzZSBhbmQgYSAxNi1iaXQgbGltaXQuCgpXaHk/ICBJJ2Qg ZXhwZWN0IG5vdGhpbmcgb2YgdGhlIHNvcnQsIGJlY2F1c2UgMzJiaXQgY29tcGF0IHNlZ21lbnRz IGFyZQpwdXJwb3NlZnVsbHkgZGVzaWduZWQgdG8gYmUgbm8gZnVuY3Rpb25hbCBkaWZmZXJlbmNl IGZyb20gcmVndWxhciAzMmJpdApwcm90ZWN0ZWQgbW9kZSBzZWdtZW50cy4gIFRoYXQgaW5jbHVk ZXMgbm90IGNoYW5naW5nIHRoZSBiZWhhdmlvdXIgb2YKaW5zdHJ1Y3Rpb25zIGxpa2UgdGhpcy4K Cj4gSGVuY2UgaWYgX2FsbF8gc3lzdGVtIHNlZ21lbnQKPiByZWdpc3RlciByZWxhdGVkIGluc25z IHdvcmtlZCBjb25zaXN0ZW50bHkgaW4gbG9uZyBtb2RlLCB0aGUgZm91cgo+IG5hbWVkIGluc25z IHdvdWxkIGhhdmUgdG8gaGF2ZSAxMC1ieXRlIG9wZXJhbmRzLgoKVGhpcyBpc24ndCBhIHZhbGlk IGV4cGVjdGF0aW9uIHRvIGRyYXcuCgo+ICBJJ20gcHJldHR5IHN1cmUKPiB0aGV5IGRvbid0IHRo b3VnaCwgc28gdGhlcmUgaXMgX29uZV8gYW5vbWFseSBhbHJlYWR5LgoKSW5kZWVkIHRoZXkgZG9u J3QuICBJbiBhIGNvbXBhdGliaWxpdHkgbW9kZSBzZWdtZW50LCB0aGV5IGhhdmUgdGFrZSBhCjYt Ynl0ZSBvcGVyYW5kLCBpZGVudGljYWxseSB0byAzMmJpdCBtb2RlLgoKPiBXaXRoIHRoYXQgSSBk b24ndCB0aGluayB3ZSBjYW4gcnVsZSBvdXQgdGhlcmUgYmVpbmcgb3RoZXIgYW5vbWFsaWVzLCB3 aXRoIHRoaXMKPiBub3QgYmVpbmcgdGFsa2VkIGFib3V0IGV4cGxpY2l0bHkgYW55d2hlcmUgaW4g dGhlIGRvYy4KCkkgZG9uJ3QgdGhpbmsgYW55IG9mIHRoaXMgaXMgcmVsZXZhbnQgdG8gdGhlIGNv cnJlY3RuZXNzIG9mIHRoaXMgcGF0Y2guCgpXaXRob3V0IHRoaXMgZml4LCBpbXBsaWNpdCBhY2Nl c3NlcyB0byBzeXN0ZW0gc2VnbWVudHMgaW4gYQpjb21wYXRpYmlsaXR5IG1vZGUgc2VnbWVudCB3 aWxsIHRydW5jYXRlIHRoZSByZXN1bHRpbmcgbGluZWFyIGFkZHJlc3MgYXMKcGFydCBvZiBwZXJm b3JtaW5nIHRoZSBzZWdtZW50YXRpb24gY2FsY3VsYXRpb25zLCB3aGljaCBpcyBvYnZpb3VzbHkg bm90CmhvdyByZWFsIGhhcmR3YXJlIGJlaGF2ZXMuCgp+QW5kcmV3CgpfX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpYZW4tZGV2ZWwgbWFpbGluZyBsaXN0Clhl bi1kZXZlbEBsaXN0cy54ZW4ub3JnCmh0dHBzOi8vbGlzdHMueGVuLm9yZy94ZW4tZGV2ZWwK