[Devel] [PATCH 0/6] backporting async_pf injection functionality

[Denis V. Lunev]

On 09/25/2017 03:47 PM, Kirill Tkhai wrote:
> Hi, Denis,
>
> your patches in base64 format. Could you resend them as plain-text like we
> usually do?
>
> Here is body of the patch 1/6:
>
> "X-MS-Exchange-Processed-By-BccFoldering: 15.20.0056.000
> X-Microsoft-Exchange-Diagnostics:
> 	1;HE1PR0801MB1338;27:2tKlmOLYPsUB4+tCQ+/VLLnyTS38TwWCV2ZWj5N2k5yUKz/VyQQs2QuWlAhcnCHJG3wiwYtk59+WvZbGgAc5/5I8Nb/S5hdsFssBCa4bX6OWopDBrSyQq5/K2D/CsTmJ
> MIME-Version: 1.0
>
> RnJvbTogV2FucGVuZyBMaSA8d2FucGVuZy5saUBob3RtYWlsLmNvbT4KCiBXQVJOSU5HOiBDUFU6
> IDMgUElEOiAyODQwIGF0IGFyY2gveDg2L2t2bS92bXguYzoxMDk2NiBuZXN0ZWRfdm14X3ZtZXhp
> dCsweGRjZC8weGRlMCBba3ZtX2ludGVsXQogQ1BVOiAzIFBJRDogMjg0MCBDb21tOiBxZW11LXN5
> c3RlbS14ODYgVGFpbnRlZDogRyAgICAgICAgICAgT0UgICA0LjEyLjAtcmMzKyAjMjMKIFJJUDog
> MDAxMDpuZXN0ZWRfdm14X3ZtZXhpdCsweGRjZC8weGRlMCBba3ZtX2ludGVsXQogQ2FsbCBUcmFj
> ZToKICA/IGt2bV9jaGVja19hc3luY19wZl9jb21wbGV0aW9uKzB4ZWYvMHgxMjAgW2t2bV0KICA/
> IHJjdV9yZWFkX2xvY2tfc2NoZWRfaGVsZCsweDc5LzB4ODAKICB2bXhfcXVldWVfZXhjZXB0aW9u
> KzB4MTA0LzB4MTYwIFtrdm1faW50ZWxdCiAgPyB2bXhfcXVldWVfZXhjZXB0aW9uKzB4MTA0LzB4
> MTYwIFtrdm1faW50ZWxdCiAga3ZtX2FyY2hfdmNwdV9pb2N0bF9ydW4rMHgxMTcxLzB4MWNlMCBb
> a3ZtXQogID8ga3ZtX2FyY2hfdmNwdV9sb2FkKzB4NDcvMHgyNDAgW2t2bV0KICA/IGt2bV9hcmNo
> X3ZjcHVfbG9hZCsweDYyLzB4MjQwIFtrdm1dCiAga3ZtX3ZjcHVfaW9jdGwrMHgzODQvMHg3YjAg
> W2t2bV0KICA/IGt2bV92Y3B1X2lvY3RsKzB4Mzg0LzB4N2IwIFtrdm1dCiAgPyBfX2ZnZXQrMHhm
> My8weDIxMAogIGRvX3Zmc19pb2N0bCsweGE0LzB4NzAwCiAgPyBfX2ZnZXQrMHgxMTQvMHgyMTAK
> ICBTeVNfaW9jdGwrMHg3OS8weDkwCiAgZG9fc3lzY2FsbF82NCsweDgxLzB4MjIwCiAgZW50cnlf
> U1lTQ0FMTDY0X3Nsb3dfcGF0aCsweDI1LzB4MjUKClRoaXMgaXMgdHJpZ2dlcmVkIG9jY2FzaW9u
> YWxseSBieSBydW5uaW5nIGJvdGggd2luNyBhbmQgd2luMjAxNiBpbiBMMiwgaW4KYWRkaXRpb24s
> IEVQVCBpcyBkaXNhYmxlZCBvbiBib3RoIEwxIGFuZCBMMi4gSXQgY2FuJ3QgYmUgcmVwcm9kdWNl
> ZCBlYXNpbHkuCgpDb21taXQgMGI2YWMzNDNmYyAoS1ZNOiBuVk1YOiBDb3JyZWN0IGhhbmRsaW5n
> IG9mIGV4Y2VwdGlvbiBpbmplY3Rpb24pIG1lbnRpb25lZAp0aGF0ICJLVk0gd2FudHMgdG8gaW5q
> ZWN0IHBhZ2UtZmF1bHRzIHdoaWNoIGl0IGdvdCB0byB0aGUgZ3Vlc3QuIFRoaXMgZnVuY3Rpb24K
> YXNzdW1lcyBpdCBpcyBjYWxsZWQgd2l0aCB0aGUgZXhpdCByZWFzb24gaW4gdm1jczAyIGJlaW5n
> IGEgI1BGIGV4Y2VwdGlvbiIuCkNvbW1pdCBlMDExYzY2MyAoS1ZNOiBuVk1YOiBDaGVjayBhbGwg
> ZXhjZXB0aW9ucyBmb3IgaW50ZXJjZXB0IGR1cmluZyBkZWxpdmVyeSB0bwpMMikgYWxsb3dzIHRv
> IGNoZWNrIGFsbCBleGNlcHRpb25zIGZvciBpbnRlcmNlcHQgZHVyaW5nIGRlbGl2ZXJ5IHRvIEwy
> LiBIb3dldmVyLAp0aGVyZSBpcyBubyBndWFyYW50ZWUgdGhlIGV4aXQgcmVhc29uIGlzIGV4Y2Vw
> dGlvbiBjdXJyZW50bHksIHdoZW4gdGhlcmUgaXMgYW4KZXh0ZXJuYWwgaW50ZXJydXB0IG9jY3Vy
> cmVkIG9uIGhvc3QsIG1heWJlIGEgdGltZSBpbnRlcnJ1cHQgZm9yIGhvc3Qgd2hpY2ggc2hvdWxk
> Cm5vdCBiZSBpbmplY3RlZCB0byBndWVzdCwgYW5kIHNvbWV3aGVyZSBxdWV1ZXMgYW4gZXhjZXB0
> aW9uLCB0aGVuIHRoZSBmdW5jdGlvbgpuZXN0ZWRfdm14X2NoZWNrX2V4Y2VwdGlvbigpIHdpbGwg
> YmUgY2FsbGVkIGFuZCB0aGUgdm1leGl0IGVtdWxhdGlvbiBjb2RlcyB3aWxsCnRyeSB0byBlbXVs
> YXRlIHRoZSAiQWNrbm93bGVkZ2UgaW50ZXJydXB0IG9uIGV4aXQiIGJlaGF2aW9yLCB0aGUgd2Fy
> bmluZyBpcwp0cmlnZ2VyZWQuCgpSZXVzaW5nIHRoZSBleGl0IHJlYXNvbiBmcm9tIHRoZSBMMi0+
> TDAgdm1leGl0IGlzIHdyb25nIGluIHRoaXMgY2FzZSwKdGhlIHJlYXNvbiBtdXN0IGFsd2F5cyBi
> ZSBFWENFUFRJT05fTk1JIHdoZW4gaW5qZWN0aW5nIGFuIGV4Y2VwdGlvbiBpbnRvCkwxIGFzIGEg
> bmVzdGVkIHZtZXhpdC4KCkNjOiBQYW9sbyBCb256aW5pIDxwYm9uemluaUByZWRoYXQuY29tPgpD
> YzogUmFkaW0gS3LEjW3DocWZIDxya3JjbWFyQHJlZGhhdC5jb20+ClNpZ25lZC1vZmYtYnk6IFdh
> bnBlbmcgTGkgPHdhbnBlbmcubGlAaG90bWFpbC5jb20+CkZpeGVzOiBlMDExYzY2M2I5YzcgKCJL
> Vk06IG5WTVg6IENoZWNrIGFsbCBleGNlcHRpb25zIGZvciBpbnRlcmNlcHQgZHVyaW5nIGRlbGl2
> ZXJ5IHRvIEwyIikKU2lnbmVkLW9mZi1ieTogUmFkaW0gS3LEjW3DocWZIDxya3JjbWFyQHJlZGhh
> dC5jb20+CihjaGVycnkgcGlja2VkIGZyb20gY29tbWl0IGQ0OTEyMjE1ZDEwMzFlNGZiM2QxMDM4
> ZDJlMTg1NzIxOGRiYTBkMGEpCmZpeCAjUFNCTS01NjQ5OApTaWduZWQtb2ZmLWJ5OiBEZW5pcyBQ
> bG90bmlrb3YgPGRwbG90bmlrb3ZAdmlydHVvenpvLmNvbT4KLS0tCiBhcmNoL3g4Ni9rdm0vdm14
> LmMgfCAyICstCiAxIGZpbGUgY2hhbmdlZCwgMSBpbnNlcnRpb24oKyksIDEgZGVsZXRpb24oLSkK
> CmRpZmYgLS1naXQgYS9hcmNoL3g4Ni9rdm0vdm14LmMgYi9hcmNoL3g4Ni9rdm0vdm14LmMKaW5k
> ZXggMDliMTg1MS4uYjUyYmExOCAxMDA2NDQKLS0tIGEvYXJjaC94ODYva3ZtL3ZteC5jCisrKyBi
> L2FyY2gveDg2L2t2bS92bXguYwpAQCAtMjI4OCw3ICsyMjg4LDcgQEAgc3RhdGljIGludCBuZXN0
> ZWRfdm14X2NoZWNrX2V4Y2VwdGlvbihzdHJ1Y3Qga3ZtX3ZjcHUgKnZjcHUsIHVuc2lnbmVkIG5y
> KQogCWlmICghKHZtY3MxMi0+ZXhjZXB0aW9uX2JpdG1hcCAmICgxdSA8PCBucikpKQogCQlyZXR1
> cm4gMDsKIAotCW5lc3RlZF92bXhfdm1leGl0KHZjcHUsIHRvX3ZteCh2Y3B1KS0+ZXhpdF9yZWFz
> b24sCisJbmVzdGVkX3ZteF92bWV4aXQodmNwdSwgRVhJVF9SRUFTT05fRVhDRVBUSU9OX05NSSwK
> IAkJCSAgdm1jc19yZWFkMzIoVk1fRVhJVF9JTlRSX0lORk8pLAogCQkJICB2bWNzX3JlYWRsKEVY
> SVRfUVVBTElGSUNBVElPTikpOwogCXJldHVybiAxOwotLSAKMi43LjQKCl9fX19fX19fX19fX19f
> X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkRldmVsIG1haWxpbmcgbGlzdApEZXZl
> bEBvcGVudnoub3JnCmh0dHBzOi8vbGlzdHMub3BlbnZ6Lm9yZy9tYWlsbWFuL2xpc3RpbmZvL2Rl
> dmVsCg=="
>
>
> On 20.09.2017 17:30, Denis Plotnikov wrote:
>> the patch set is for
>> 1. Replace Roman's patch avoiding async_pf injection while in the guest mode with
>>    the similar patch from the mainstream for kernel code consistency
>> 2. Force a nested vmexit if the injected #PF is async_pf
>> 3. Let guest support delivery of async_pf from guest mode
>>
>> Denis Plotnikov (1):
>>   Revert "kvm/x86: skip async_pf when in guest mode"
>>
>> Wanpeng Li (5):
>>   KVM: nVMX: Fix exception injection
>>   KVM: async_pf: Add L1 guest async_pf #PF vmexit handler
>>   KVM: async_pf: Force a nested vmexit if the injected #PF is async_pf
>>   KVM: async_pf: avoid async pf injection when in guest mode
>>   KVM: async_pf: Let guest support delivery of async_pf from guest mode
>>
>>  Documentation/virtual/kvm/msr.txt    |  5 ++--
>>  arch/x86/include/asm/kvm_emulate.h   |  1 +
>>  arch/x86/include/asm/kvm_host.h      |  4 +++
>>  arch/x86/include/uapi/asm/kvm_para.h |  1 +
>>  arch/x86/kernel/kvm.c                |  7 ++++-
>>  arch/x86/kvm/mmu.c                   | 40 +++++++++++++++++++++++++++--
>>  arch/x86/kvm/mmu.h                   |  4 +++
>>  arch/x86/kvm/svm.c                   | 50 +++++++++++-------------------------
>>  arch/x86/kvm/vmx.c                   | 36 +++++++++++++++++---------
>>  arch/x86/kvm/x86.c                   | 20 ++++++++++-----
>>  10 files changed, 109 insertions(+), 59 deletions(-)
this is somehow broken on your side.
I have received these thread in plain text, as usual.

Den